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

[Security] - GitHub App Permissions #8

Closed
probably-not opened this issue Sep 6, 2022 · 3 comments
Closed

[Security] - GitHub App Permissions #8

probably-not opened this issue Sep 6, 2022 · 3 comments
Labels
enhancement New feature or request

Comments

@probably-not
Copy link

Currently, Sidekick is requesting Full Permissions for read and write to all private and public repositories, and on top of that, is requesting Full Permissions for read and write to all personal information.

image

This is a hard no in terms of security, and under no circumstances should Sidekick have full read and write permissions on Personal information data.

In addition to the issue of requesting full permissions to personal information, it's a major security issue to request write permissions on all public and private repositories, especially when this can't be scoped to a specific repository under a specific organization.

@probably-not
Copy link
Author

GitHub has GitHub Apps and OAuth Apps (see here for more information on the differences: https://docs.github.com/en/developers/apps/getting-started-with-apps/differences-between-github-apps-and-oauth-apps).

From what I can tell, Sidekick is currently using an OAuth App, which means it can't be scoped to a specific organization or repository. I would recommend moving to a GitHub App, which can be scoped and audited on an organization.

@boroskoyo
Copy link
Contributor

@probably-not thank you for this issue. we will be fixing it in the upcoming sprint, which starts next week

@boroskoyo
Copy link
Contributor

As promised we have replaced our OAuth App with a GitHub App. Now our Sidekick SaaS asks for fewer permissions and can be scoped and audited on an organization as you have mentioned.

@boroskoyo boroskoyo added the enhancement New feature or request label Sep 29, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Development

No branches or pull requests

2 participants