Why request access to all URLs?

[Nantris]

Is there a limit to the number of URLs you can include in the manifest?

[Konnor88]

Hi, the reason I decided to use all URLs is that whenever I added new URLs before and pushed out an update, it would disable the extension for the users until they acknowledge the permission changes. This change just makes upgrading it a bit easier.

[Nantris]

That makes sense, and thanks for your reply and your great work on this plugin.

But this change has made me more wary of the plugin and I've left it disabled since it updated, so a bit counterproductive maybe. I previously was happy to enable it without needing to look over the source code since it explicitly limited the sites it allowed itself to.

My two cents, I think it would be better to go back to the old way of explicitly listing sites.

[Konnor88]

Yeah, I can see where you're coming from on that. I'll keep that in consideration and will do that if there are more requests to go back to the old way. Thanks for your feedback.

[muchqs]

just noticed it today and honestly I prefer the old way also, just my 2c.
But i can see why this was done. I didn't notice it was disabled for days since the icon was hidden on the overflow bar.

[Nantris]

Just pinging to mention I still have the extension disabled since this change. I'd love to see it reverted, but I understand it's more work from the developer side so I understand if it never is.

[boobthecat]

Bump :B

[Nantris]

I'd still be interested in a version that didn't request access to all URLs.

[Realswitzer]

I'd still be interested in a version that didn't request access to all URLs.

I made a fork to change that, though a few problems include it being unsigned and unavailable on add-on stores (AMO and CWS primarily), being disabled on every update, and not auto-updating.
Any updates upstream, if I continue to maintain the fork, would likely end up being delayed too.
https://github.com/Realswitzer/anti-grabify/releases/latest

[boobthecat]

You just forgot to add an ID to the Firefox one. I can't install it, only as temporary in the about:debugging

[Realswitzer]

Oh, didn't know that. Will look into and should be able to push a release today.
Found the problem, was looking at source code and no ID was provided there which is what threw me off.

[boobthecat]

Thanks!