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

Too wide GH API permissions required just to sign a CLA #97

Closed
dolmen opened this issue Feb 2, 2016 · 4 comments
Closed

Too wide GH API permissions required just to sign a CLA #97

dolmen opened this issue Feb 2, 2016 · 4 comments

Comments

@dolmen
Copy link

dolmen commented Feb 2, 2016

CLA Assitant requires too wide permissions when I'm using it just to sign a CLA.
For example, why does it want to be able to write to my repos?

@KharitonOff
Copy link
Contributor

this question is already answered in #78
To sign a CLA you access CLA assistant via a specific link https://cla-assistant.io/<org_or_user>/<repository_name>

@dolmen
Copy link
Author

dolmen commented Feb 2, 2016

It appears that despistes I arrived on CLA assistant via a such link, the authentication process failed in some way (probably because I have cookies disabled by default for all sites), so I have probably been redirected to the home page where I clicked on the "sign up with GitHub", and that's probably the reason why I have been asked so many perms.

So I think that you should track more the users coming using a "sign link", to avoid such failures. Signing a CLA should either work without cookies (not even session cookies) enabled, or you should detect the issue (and report to the user) if they are required.

@jn64
Copy link

jn64 commented May 28, 2020

This issue still exists. Accessing CLAassistant via a specific link (not homepage), with cookies disabled, appears to work with no indication to the user. The only clue is that the user is confusingly redirected to the homepage and asked for more permissions.

@Olf0
Copy link

Olf0 commented Jun 6, 2022

@KharitonOff and @thojansen, as denoted closing this was premature. It enforced users to create duplicate issue reports, see #78, #566, #810, #863 etc.

I suggest to deal with this at issue #566, until it has been fully resolved: I.e., until the CLA assistant works properly with cookies disabled in the web-browser.

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

4 participants