Enable compat with service workers and v3 extensions #1056
Merged
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.
Description of the change
This PR adds a
fetch
transport for sending Rollbar payloads. This enables compatibility with Service Workers, and with manifest v3 extensions which now use a Service Worker for the background script.The PR introduces a new optional config option,
defaultTransport
. When set to "fetch" or "xhr", this transport will be used when available. The default when not set is "xhr". When only one transport is detected to be available, that one will be used regardless of thedefaultTransport
setting.Since we have two transports now, these are moved into a /transport/ subdir. This causes a large block diff in the PR for the existing xhr transport code. None of that code has been changed, just moved to a new file.
Type of change
Related issues
#1040
Development
Code review