Skip to content
This repository has been archived by the owner on Feb 25, 2023. It is now read-only.

Dictionary error on Firefox Nightly #588

Closed
SimonPilkington opened this issue Jun 4, 2020 · 12 comments
Closed

Dictionary error on Firefox Nightly #588

SimonPilkington opened this issue Jun 4, 2020 · 12 comments

Comments

@SimonPilkington
Copy link

SimonPilkington commented Jun 4, 2020

Yomichan is currently unusable on Firefox nightly. Both testing and stable versions are affected. Listing dictionaries fails with this error:

Yomichan v20.5.22.0 has encountered a problem.
Originating URL: moz-extension://9699283a-4027-4ce4-81ec-26949dd5755a/bg/settings.html
Error: Unexpected response of type undefined ({"action":"getDictionaryInfo","params":{}})
_apiInvoke/</<@moz-extension://9699283a-4027-4ce4-81ec-26949dd5755a/mixed/js/api.js:295:28
Async*_apiInvoke/<@moz-extension://9699283a-4027-4ce4-81ec-26949dd5755a/mixed/js/api.js:285:28
_apiInvoke@moz-extension://9699283a-4027-4ce4-81ec-26949dd5755a/mixed/js/api.js:283:12
apiGetDictionaryInfo@moz-extension://9699283a-4027-4ce4-81ec-26949dd5755a/mixed/js/api.js:128:12
onDatabaseUpdated@moz-extension://9699283a-4027-4ce4-81ec-26949dd5755a/bg/js/settings/dictionaries.js:434:36
dictSettingsInitialize@moz-extension://9699283a-4027-4ce4-81ec-26949dd5755a/bg/js/settings/dictionaries.js:415:11
async*onReady@moz-extension://9699283a-4027-4ce4-81ec-26949dd5755a/bg/js/settings/main.js:318:11
async*@moz-extension://9699283a-4027-4ce4-81ec-26949dd5755a/bg/js/settings/main.js:328:25
j@moz-extension://9699283a-4027-4ce4-81ec-26949dd5755a/mixed/lib/jquery.min.js:2:29999
g/</k<@moz-extension://9699283a-4027-4ce4-81ec-26949dd5755a/mixed/lib/jquery.min.js:2:30313
setTimeout handler*g/<@moz-extension://9699283a-4027-4ce4-81ec-26949dd5755a/mixed/lib/jquery.min.js:2:30522
i@moz-extension://9699283a-4027-4ce4-81ec-26949dd5755a/mixed/lib/jquery.min.js:2:28017
fireWith@moz-extension://9699283a-4027-4ce4-81ec-26949dd5755a/mixed/lib/jquery.min.js:2:28783
fire@moz-extension://9699283a-4027-4ce4-81ec-26949dd5755a/mixed/lib/jquery.min.js:2:28819
i@moz-extension://9699283a-4027-4ce4-81ec-26949dd5755a/mixed/lib/jquery.min.js:2:28017
fireWith@moz-extension://9699283a-4027-4ce4-81ec-26949dd5755a/mixed/lib/jquery.min.js:2:28783
ready@moz-extension://9699283a-4027-4ce4-81ec-26949dd5755a/mixed/lib/jquery.min.js:2:31934
S@moz-extension://9699283a-4027-4ce4-81ec-26949dd5755a/mixed/lib/jquery.min.js:3:35
EventListener.handleEvent*@moz-extension://9699283a-4027-4ce4-81ec-26949dd5755a/mixed/lib/jquery.min.js:3:149
@moz-extension://9699283a-4027-4ce4-81ec-26949dd5755a/mixed/lib/jquery.min.js:2:220
@moz-extension://9699283a-4027-4ce4-81ec-26949dd5755a/mixed/lib/jquery.min.js:2:225

Importing a dictionary also fails:

Originating URL: moz-extension://9699283a-4027-4ce4-81ec-26949dd5755a/bg/settings.html
Error: Timeout
_apiInvokeWithProgress/</timer<@moz-extension://9699283a-4027-4ce4-81ec-26949dd5755a/mixed/js/api.js:261:20
setTimeout handler*_apiInvokeWithProgress/<@moz-extension://9699283a-4027-4ce4-81ec-26949dd5755a/mixed/js/api.js:259:17
_apiInvokeWithProgress@moz-extension://9699283a-4027-4ce4-81ec-26949dd5755a/mixed/js/api.js:206:12
apiImportDictionaryArchive@moz-extension://9699283a-4027-4ce4-81ec-26949dd5755a/mixed/js/api.js:152:12
onDictionaryImport@moz-extension://9699283a-4027-4ce4-81ec-26949dd5755a/bg/js/settings/dictionaries.js:683:44
async*dictSettingsInitialize@moz-extension://9699283a-4027-4ce4-81ec-26949dd5755a/bg/js/settings/dictionaries.js:410:42
onReady@moz-extension://9699283a-4027-4ce4-81ec-26949dd5755a/bg/js/settings/main.js:318:11
async*@moz-extension://9699283a-4027-4ce4-81ec-26949dd5755a/bg/js/settings/main.js:328:25
j@moz-extension://9699283a-4027-4ce4-81ec-26949dd5755a/mixed/lib/jquery.min.js:2:29999
g/</k<@moz-extension://9699283a-4027-4ce4-81ec-26949dd5755a/mixed/lib/jquery.min.js:2:30313
setTimeout handler*g/<@moz-extension://9699283a-4027-4ce4-81ec-26949dd5755a/mixed/lib/jquery.min.js:2:30522
i@moz-extension://9699283a-4027-4ce4-81ec-26949dd5755a/mixed/lib/jquery.min.js:2:28017
fireWith@moz-extension://9699283a-4027-4ce4-81ec-26949dd5755a/mixed/lib/jquery.min.js:2:28783
fire@moz-extension://9699283a-4027-4ce4-81ec-26949dd5755a/mixed/lib/jquery.min.js:2:28819
i@moz-extension://9699283a-4027-4ce4-81ec-26949dd5755a/mixed/lib/jquery.min.js:2:28017
fireWith@moz-extension://9699283a-4027-4ce4-81ec-26949dd5755a/mixed/lib/jquery.min.js:2:28783
ready@moz-extension://9699283a-4027-4ce4-81ec-26949dd5755a/mixed/lib/jquery.min.js:2:31934
S@moz-extension://9699283a-4027-4ce4-81ec-26949dd5755a/mixed/lib/jquery.min.js:3:35
EventListener.handleEvent*@moz-extension://9699283a-4027-4ce4-81ec-26949dd5755a/mixed/lib/jquery.min.js:3:149
@moz-extension://9699283a-4027-4ce4-81ec-26949dd5755a/mixed/lib/jquery.min.js:2:220
@moz-extension://9699283a-4027-4ce4-81ec-26949dd5755a/mixed/lib/jquery.min.js:2:225
@SimonPilkington
Copy link
Author

The offending commit is 51e17b3.

@siikamiika
Copy link
Collaborator

I can confirm this on Nightly 79 (Windows). Firefox 77 on Windows and Linux works. Extension messaging is broken while Yomichan Options page is open, but closing the page makes messaging work again.

@siikamiika
Copy link
Collaborator

Marked this as a browser bug for now. Could be worth it to report to Mozilla although they mention that Nightly can be unstable.

The offending commit is 51e17b3.

I can reproduce the issue on 51e17b3 parent commit too.

@SimonPilkington
Copy link
Author

Dictionaries are definitely broken on 51e17b3 but not on 3c48290 for me. I'm on Linux, but I don't know how that could be related. #589 is broken on both however.

@toasted-nutbread
Copy link
Collaborator

toasted-nutbread commented Jun 4, 2020

The timeout issue should be fixed with #587. I made a hotfix tag for this (20.5.22.1), but @FooSoft is in charge of releasing.

Edit: this may be an issue with the other timeout involved in that process.

@FooSoft
Copy link
Owner

FooSoft commented Jun 5, 2020

Fixed push out -- apologies for the delay!

@SimonPilkington
Copy link
Author

#587 does not fix importing on Nightly. I have reported this and #589 to Mozilla: https://bugzilla.mozilla.org/show_bug.cgi?id=1643649

@toasted-nutbread
Copy link
Collaborator

#587 does not fix importing on Nightly. I have reported this and #589 to Mozilla: https://bugzilla.mozilla.org/show_bug.cgi?id=1643649

Added a comment and example extension with the issue.
https://github.com/toasted-nutbread/firefox-web-extension-send-message-bug

@SimonPilkington I'm not sure what exactly your scenario is for using Yomichan on Nightly, but Firefox Developer Edition (currently) doesn't have this problem. You could consider using that instead, as it's usually more stable.

@SimonPilkington
Copy link
Author

Thank you. That helps.

I don't have any particular reason. I just enjoy using bleeding-edge software and don't mind occasionally encountering issues. Nightly is not especially unstable. This is the first serious issue I've encountered in the last six months or more. I am currently working around this by building from 3c48290.

@SimonPilkington
Copy link
Author

https://bugzilla.mozilla.org/show_bug.cgi?id=1642967 appears to have fixed importing dictionaries, however enumerating them still fails for me. Can you confirm?

@toasted-nutbread
Copy link
Collaborator

I am seeing some intermittent issues with the database request failing, yes. Unclear why it is happening, but the indexedDB.getAll call seems to be closing the extension message connection early.

@SimonPilkington
Copy link
Author

Seems to have been fixed by unknown other Firefox bug in the meantime.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

4 participants