Skip to content

Stop allowing "all urls" as a permission#7407

Merged
fregante merged 2 commits intomainfrom
no-all-urls
Jun 6, 2024
Merged

Stop allowing "all urls" as a permission#7407
fregante merged 2 commits intomainfrom
no-all-urls

Conversation

@fregante
Copy link
Copy Markdown
Member

@fregante fregante commented Jun 5, 2024

I also tried to follow the removal with .request(GitHub.com) but that didn't work.

@fregante fregante added the meta Related to Refined GitHub itself label Jun 5, 2024
@fregante fregante marked this pull request as ready for review June 6, 2024 05:48
@fregante fregante merged commit f8f7a8f into main Jun 6, 2024
@fregante fregante deleted the no-all-urls branch June 6, 2024 05:56
@fregante
Copy link
Copy Markdown
Member Author

fregante commented Jun 6, 2024

@cooljeanius @jwakely @thesamesam @RokeJulianLockhart @Pandustry-Dev

The next version of Refined GitHub will automatically remove broad host permissions, will block re-enabling them, and will warn you if you try. The checkbox will still be there because it's added by the browser.

Screen.Recording.mov

@Pandustry-Dev
Copy link
Copy Markdown

Nice work, that will help save many hours of troubleshooting for people that stumble into this confluence of configurations. Thanks for the work you all do to develop this addon/mod.

@thesamesam
Copy link
Copy Markdown

Thank you so much! It's really appreciated.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

meta Related to Refined GitHub itself

Development

Successfully merging this pull request may close these issues.

Warn when the extension is mistakenly enabled everywhere

3 participants