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

Document reason for "Access your data for all websites" browser permission (more prominently?) #7314

Closed
cooljeanius opened this issue Mar 22, 2024 · 11 comments

Comments

@cooljeanius
Copy link

While trying to figure out why RefinedGithub was causing me some issues on a non-GitHub website (gcc.gnu.org to be specific), I noticed that RefinedGithub has "Access your data for all websites" under "Optional permissions for added functionality" enabled on my browser. I may very well have enabled that permission myself at some point, but if so, I forget why. If there was some sort of prompt or dialog that showed up explaining why the extension wanted the permission, then I've forgotten what it might have said. If documentation for why the extension might want this browser permission is already available, it should be made more prominent, and linked to from (more of) the places in which a user might look for documentation.

@cooljeanius
Copy link
Author

Screenshot of the permissions I currently have enabled: Screen Shot 2024-03-22 at 18 18 24
(this is RefinedGithub 24.3.20 for Firefox 123.0.1)

@busches
Copy link
Member

busches commented Mar 22, 2024

It's needed for GitHub Enterprise. It's an optional permission, so it doesn't do anything until you enable it on that domain. Example where I just enabled it on espn in chrome:
image

I'm curious what issue you thought it was causing on another site.

@fregante fregante closed this as not planned Won't fix, can't repro, duplicate, stale Mar 23, 2024
@fregante
Copy link
Member

RGH's own code doesn't have any permission requests, adding permissions is only possible via:

In your case, you most likely clicked Chrome's native UI toggle because we never ask for "all"

@busches The problem is most likely related to extraneous CSS injection

@cooljeanius
Copy link
Author

cooljeanius commented Mar 23, 2024

RGH's own code doesn't have any permission requests, adding permissions is only possible via:

In your case, you most likely clicked Chrome's native UI toggle because we never ask for "all"

I'm on Firefox, though, not Chrome

@fregante
Copy link
Member

s/Chrome/Firefox/

same

@jwakely
Copy link

jwakely commented Mar 25, 2024

I'm curious what issue you thought it was causing on another site.

It keeps sending POST /api/graphql HTTP/2.0 which gets 404 every time and then can trigger fail2ban to block your IP, and then you can't access the website at all.

@thesamesam
Copy link

thesamesam commented Mar 25, 2024

I observed the same and I had the permission OP reported already off. Additionally, if I right click the icon in the menu bar, "Enable for this domain" is unchecked for the sites I was inadvertently spamming...

@fregante
Copy link
Member

fregante commented Mar 25, 2024

I observed the same and I had the permission OP reported already off.

Literally impossible. Refined GitHub does not have any more permissions than what you give. Permissions are handled and limited by the browser.

"Enable for this domain" is unchecked for the sites I was inadvertently spamming...

That checkbox doesn't work well when the "all sites" permission was granted

@cooljeanius
Copy link
Author

I'm curious what issue you thought it was causing on another site.

It keeps sending POST /api/graphql HTTP/2.0 which gets 404 every time and then can trigger fail2ban to block your IP, and then you can't access the website at all.

Apparently this is to identify whether a server is GitHub Enterprise or not? Anyways, one other thing worth noting: when using the "Start process to identify feature..." button under the add-on's preferences, and the "all sites" permission is granted, it can cause the feature identification UI to appear on other sites, such as the GCC bugzilla, which can look really weird when the site in question isn't designed for it.

@fregante
Copy link
Member

Duplicate of #7319 (comment)

@fregante fregante marked this as a duplicate of #7319 Mar 29, 2024
@refined-github refined-github locked as spam and limited conversation to collaborators Mar 29, 2024
@fregante
Copy link
Member

In hands entrusted, a tool I gave,
Yet in your grasp, you find it grave.
"Why does it hurt?" you plead with me,
As if my answer could set you free.

In your grip, it's you who's harmed,
Release, find peace, be disarmed.
Lay it down, let healing start,
And mend the wounds within your heart.

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

No branches or pull requests

5 participants