Skip to content

Handle project access for non-repository members #7159

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

Open
gtsiolis opened this issue Dec 9, 2021 · 6 comments
Open

Handle project access for non-repository members #7159

gtsiolis opened this issue Dec 9, 2021 · 6 comments
Labels
aspect: authentication This is a broad, abstract, and almost impractical category that we have yet to sort out. component: dashboard meta: never-stale This issue can never become stale team: webapp Issue belongs to the WebApp team type: feature request New feature or request type: improvement Improves an existing feature or existing code user experience

Comments

@gtsiolis
Copy link
Contributor

gtsiolis commented Dec 9, 2021

Problem to solve

Users that don't have access to a repository they run into errors and empty pages when access a project in Gitpod.

Steps to reproduce

  1. Create a private repository in any of the available providers
  2. Add a new project in a new team in Gitpod
  3. Invite a team member in the new team who does have access to the repository in the corresponding provider
  4. Log in with the invited team member and try accessing the project branches or opening a workspace
Empty branches page Error when opening a workspace
Screenshot 2021-12-09 at 10 04 55 PM Screenshot 2021-12-09 at 10 04 02 PM

Proposal

TBD (To be discussed)

This can become more relevant as users adopt Teams & Projects and create teams with multiple projects across different providers with team members that don't necessarily have access to corresponding repositories. 💡

@gtsiolis gtsiolis added type: feature request New feature or request user experience aspect: authentication This is a broad, abstract, and almost impractical category that we have yet to sort out. component: dashboard team: webapp Issue belongs to the WebApp team type: improvement Improves an existing feature or existing code labels Dec 9, 2021
@jldec
Copy link
Contributor

jldec commented Dec 13, 2021

Hi @gtsiolis would you mind please adding the steps required to reproduce this issue.

@gtsiolis
Copy link
Contributor Author

@jldec I've added steps to reproduce in the issue description to clarify. Let me know if these don't look clear enough.

@jldec
Copy link
Contributor

jldec commented Dec 13, 2021

Thanks - I suppose we could always improve the error message, but the behavior seems correct - users should not be able to see repositories through Gitpod which they cannot acces directly.

@gtsiolis
Copy link
Contributor Author

I suppose we could always improve the error message, but the behavior seems correct

Yes, handling this with better error messages is a good way forward. since there's no much we can do about this and we most probably don't want to limit project visibility for users that are part of a team but don't have access to the actual repository. 🧀

@jldec
Copy link
Contributor

jldec commented Dec 21, 2021

Looking at the error message which is there, that seems pretty close to what we want for workspace startup.
It is consistent with GitHub's behavior of showing 404 for private repos accessed by unauthed users.
We don't want to leak more information to unauthorized users.

What might be more promising as a way to improve the UX would be to show projects which won't work for you because of auth-failures in the project list, and team members with such project auth issues in the team members list.

@stale
Copy link

stale bot commented Mar 25, 2022

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

@stale stale bot added the meta: stale This issue/PR is stale and will be closed soon label Mar 25, 2022
@gtsiolis gtsiolis added the meta: never-stale This issue can never become stale label Mar 25, 2022
@stale stale bot removed the meta: stale This issue/PR is stale and will be closed soon label Mar 25, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
aspect: authentication This is a broad, abstract, and almost impractical category that we have yet to sort out. component: dashboard meta: never-stale This issue can never become stale team: webapp Issue belongs to the WebApp team type: feature request New feature or request type: improvement Improves an existing feature or existing code user experience
Projects
Status: No status
Development

No branches or pull requests

2 participants