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

Requests permissions with no clear way to grant them #49

Open
danaradg opened this issue Jul 30, 2023 · 9 comments
Open

Requests permissions with no clear way to grant them #49

danaradg opened this issue Jul 30, 2023 · 9 comments

Comments

@danaradg
Copy link

I'm a long time user of this amazing extension, thank you!
Today, the extension refused to connect to my hosts, and instead showed the following message:
image
Clicking on "Grant permissions" sends me to the extension settings page with no clear action to take.
I tried reinstalling but it didn't help.

@sdefresne
Copy link
Owner

This is likely related to the recent change to use Manifest V3.

It appears to work when running the extension in an unpackaged fashion, but fails with the attached error when packaged and delivered via the store. I've reverted the CL that added the support for Manifest V3 and redeployed the old version (even though the store does not allow to rollback, so the version number has been increased).

Screenshot 2023-07-31 at 10 37 20

@sdefresne
Copy link
Owner

@patrickkettner

Do you have an idea of what causes this error when using Manifest V3 and shipping the app through the store? There is no additional error message, nor any call stack, so this is really difficult to debug. I guess the runtime refers to chrome.runtime object, but I don't know why it works when unpackaged but not when running from the extension store.

@patrickkettner
Copy link
Contributor

That error happens when you try to send a message via chrome.runtime.sendMessage but there isn't a listener set up for it. I will grab the last version in the store and try to figure out whats going on

@patrickkettner
Copy link
Contributor

patrickkettner commented Jul 31, 2023

I am not able to replicate with the 1.3.0 (the mv3 build), at least the default flow. Would @danaradg or @sdefresne be able to share more about their config? My hunch is that setOverlayText being called with undefined, so either setGrantPermissionsButtonVisible has an error that is undefined (which seems extremely unlikely), or setLogginButtonVisible has errors who don't map to the current logic to get the error message.
Since I am not able to reproduce it, I would assume that there is an issue with the call to gerrit.fetchAllowedInstances, and that it is happening for some but not others. Hence, a config would be very helpful to debug.

@danaradg
Copy link
Author

danaradg commented Aug 1, 2023

First of all, the extension is indeed working again. Thanks for the quick response.
I'm using default settings and have 3 hosts defined and enabled.
Not sure what other information I can provide, but let me know if I can be of assistance. I'm not an expert on extensions, so please provide instructions if you want me to downgrade or collect logs.

@patrickkettner
Copy link
Contributor

Thanks @danaradg!
Are the 3 hosts all custom ones? Or are they the default ones?

@danaradg
Copy link
Author

danaradg commented Aug 1, 2023

Custom ones.

@patrickkettner
Copy link
Contributor

patrickkettner commented Aug 1, 2023

Would you be open to me contacting you directly to give further debugging steps?

@danaradg
Copy link
Author

danaradg commented Aug 1, 2023

Sure. You can contact me at [email protected]

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

No branches or pull requests

3 participants