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

feat: preload webui root CID #735

Merged
merged 1 commit into from
Jul 16, 2019
Merged

feat: preload webui root CID #735

merged 1 commit into from
Jul 16, 2019

Conversation

lidel
Copy link
Member

@lidel lidel commented Jul 16, 2019

This PR updated webui to v2.4.7 introduces a simple optimization: preloads the root CID after IPFS Client start to speed up the first time Web UI is opened. This is a best-effort version of #682 that respects user bandwidth and does not fetch entire Web UI when extension starts.

If embedded js-ipfs is used (eg. in Brave #716) it will trigger a remote (always recursive) preload of entire DAG to one of preload nodes. This way when embedded node wants to load resources related to webui it will get them fast from preload nodes.

Optimization: preload the root CID to speed up the first time
Web UI is opened. If embedded js-ipfs is used it will trigger
remote (always recursive) preload of entire DAG to one of preload nodes.
This way when embedded node wants to load resource related to webui
it will get it fast from preload nodes.

This is a best-effort version of #682
@lidel lidel merged commit db78df2 into master Jul 16, 2019
@lidel lidel deleted the feat/preload-webui branch July 16, 2019 21:57
@lidel lidel added area/brave Issues related to Brave Browser area/chromium Issues related to Chromium-based browsers area/firefox Issues related to Mozilla Firefox labels Oct 1, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/brave Issues related to Brave Browser area/chromium Issues related to Chromium-based browsers area/firefox Issues related to Mozilla Firefox
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant