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

fix(ecau): fix seed parameters not iterable error #784

Merged
merged 2 commits into from
Jul 27, 2024

Conversation

ROpdebee
Copy link
Owner

@ROpdebee ROpdebee commented Jul 27, 2024

When ECAU is injected in the content context in Firefox, it uses the URLSearchParams from the page context via an x-ray. However, a bug in Firefox prevents us from iterating through the result of the entries() call. Instead, we can iterate through the URLSearchParams directly.

Fixes #773.

ROpdebee added 2 commits July 27, 2024 13:13
It nags about this on every build
When ECAU is injected in the content context in Firefox, it uses the `URLSearchParams`
from the page context via an x-ray. However, a bug in Firefox [1] prevents us from
iterating through the result of the `entries()` call. Instead, we can iterate through
the `URLSearchParams` directly.

[1] https://bugzilla.mozilla.org/show_bug.cgi?id=1023984
@ROpdebee ROpdebee added bug Something isn't working mb_enhanced_cover_art_uploads labels Jul 27, 2024
@ROpdebee
Copy link
Owner Author

/deploy-preview

Copy link

This PR changes 1 built userscript(s):

See all changes

github-actions bot added a commit that referenced this pull request Jul 27, 2024
fix(ecau): fix seed parameters not iterable error (#784)
@ROpdebee ROpdebee merged commit 0ce8821 into main Jul 27, 2024
11 checks passed
@ROpdebee ROpdebee deleted the fix-urlparams-iterable branch July 27, 2024 11:45
Copy link

🚀 Released 1 new userscript version(s):

  • mb_enhanced_cover_art_uploads 2024.7.27 in cd33a9b

github-actions bot added a commit that referenced this pull request Jul 27, 2024
fix(ecau): fix seed parameters not iterable error (#784)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Failed to process seeded cover art parameters: seedParameters.entries() is not iterable
1 participant