-
Notifications
You must be signed in to change notification settings - Fork 0
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
Feature: refactor service worker #41
Merged
JoshuaCWebDeveloper
merged 14 commits into
develop
from
features/refactor-service-worker
Nov 18, 2022
Merged
Feature: refactor service worker #41
JoshuaCWebDeveloper
merged 14 commits into
develop
from
features/refactor-service-worker
Nov 18, 2022
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
* Rename service-worker.ts to worker-messaging.ts * Exclude self from 'no-restricted-globals' eslint rule * Modularize worker/index.ts and replace with new root worker/service-worker.ts file * Add missing message handlers to new refactored code * Add trace logging for forwarding fetch requests * Format logger output * Create new DevTools util accessible via 'SamizdAppDevTools' * Return root logger from logging.getLoggers() * Write boilerplate tests for new worker
* Add localforage to SamizdAppDevTools * Add workbox logging comment to root service-worker.ts * Refactor p2p-client into high-level OOP architecture
* Don't use logger namespaces as logger names (makes name matching easier) * Prefix log with method name, not logLevel setting * Don't log using console.trace() * Persist loglevel using localforage and correctly set default levels * Move logging module to separate directory * Set prettier tabWidth to 2 for yaml and .prettierrc files * Create new logging.yaml file that sets default levels on all matched loggers * Handle possibility of no logging config
* Uninstall multiaddr package (use the same one libp2p uses instead) * Delete old worker/index.ts file (uninstalling old package caused import errors) * Refactor bootstrap-list to more cleanly manage addresses and configure libp2p discovery * Refactor stream-factory with cleaner logic and more robust restart * Adjust p2p logging * Import workbox-precaching for types (preivously was imported in index.ts) * Add new SamizdAppDevTools.addressBook property * Improve p2p logging * Disable autodial and restart() and replace with serve connection retry logic
Don't cache empty p2p bootstrap list
* Make JSON parsing more defensive in bootstrap-list loadCache() * Create new migrations layer and add first migration: libp2p.bootstrap -> p2p.bootstrap-list
Close stats websocket after collection
Disable static cache when running locally
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
No description provided.