Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Improve defaults and UX around local gateway redirect #1072

Open
Tracked by #118
lidel opened this issue Mar 30, 2022 · 0 comments
Open
Tracked by #118

Improve defaults and UX around local gateway redirect #1072

lidel opened this issue Mar 30, 2022 · 0 comments
Labels
area/MV2 Issues related to Manifest V2 version area/MV3 Issues related to Manifest V3 version need/analysis Needs further analysis before proceeding need/community-input Needs input from the wider community need/maintainer-input Needs input from the current maintainer(s) P1 High: Likely tackled by core team if no one steps up starmaps
Milestone

Comments

@lidel
Copy link
Member

lidel commented Mar 30, 2022

There is an argument that installing IPFS Companion should not degrade browsing performance,
but augment it: recover when HTTP server is down, cache for later offline use.
Avoiding HTTP connection to the original server comes with trade-offs and surprises, which means it should be a user choice.

Food for thought

  • make redirect to local gateway opt-in instead of opt-out when user has ipfs-companion installed
    (by default, only used as a recovery only when original server is down)
  • indicate to user when page has IPFS resources or is a DNSLink website, and suggest "opening via IPFS"
    • allow enabling redirect to local gateway (per Origin or globally)
    • remember the choice, provide UI for changing it later

Rationale

  • redirect to local gateway should be a conscious decision made by user, just like it is in Brave, and not a surprise, like it is in MV2 Companion
  • safer default would avoid breaking websites which think they support IPFS, but are not (misconfigured or outdated DNSLink, JS code not working correctly with redirected resources, browser privacy extensions clashing with Companion logic.. list is long)

Prior art

@lidel lidel added need/triage Needs initial labeling and prioritization need/analysis Needs further analysis before proceeding need/community-input Needs input from the wider community need/maintainer-input Needs input from the current maintainer(s) area/MV2 Issues related to Manifest V2 version area/MV3 Issues related to Manifest V3 version labels Mar 30, 2022
@lidel lidel added this to the v3.0 milestone Mar 30, 2022
@lidel lidel added P1 High: Likely tackled by core team if no one steps up and removed need/triage Needs initial labeling and prioritization labels Apr 11, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/MV2 Issues related to Manifest V2 version area/MV3 Issues related to Manifest V3 version need/analysis Needs further analysis before proceeding need/community-input Needs input from the wider community need/maintainer-input Needs input from the current maintainer(s) P1 High: Likely tackled by core team if no one steps up starmaps
Projects
No open projects
Status: Needs Grooming
Development

No branches or pull requests

2 participants