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

Confirmation popup for workspace sharing #4726

Closed
fntlnz opened this issue Jul 7, 2021 · 3 comments · Fixed by #4743
Closed

Confirmation popup for workspace sharing #4726

fntlnz opened this issue Jul 7, 2021 · 3 comments · Fixed by #4743

Comments

@fntlnz
Copy link
Contributor

fntlnz commented Jul 7, 2021

This feature is particularly sensitive to leaking workspace data and the user needs to be extremely aware that they are enabling it when they do.

A possible solution is to add an "Are you sure?" popup that explains the implications for workspace sharing so that the user can have a chance to reflect and decide before they do the action.

This was already present in the Theia editor, we need it in Code too now.

@fntlnz fntlnz added aspect: security Anything related to preventing vulnerabilities editor: code (browser) labels Jul 7, 2021
@shaal
Copy link
Contributor

shaal commented Jul 7, 2021

When I'm sharing a workspace, users who enter that workspace, get access and same git permissions to all my repos, outside this workspace.

@AlexTugarev
Copy link
Member

AlexTugarev commented Jul 7, 2021

... get access and same git permissions to all my repos, outside this workspace.

That should be limited to the current workspace session.

OTOH, thinking more about this, I realize you probably mean that the guest may use a terminal to access your other repositories.

@shaal
Copy link
Contributor

shaal commented Jul 7, 2021

@AlexTugarev Yes, that's exactly what I meant.
It has similar abilities to sharing my local terminal with someone else (ie. https://tmate.io)

On Gitpod, I already authorized Github and Gitlab access, so someone accessing my Gitpod terminal - has read/write access to all the repositories hosted on these services.

I think the option of sharing workspace should be removed from the main menu, because of the security risk.
The alternative, sharing instead just the snapshot of a workspace, provides most of the benefits and none of the security concerns.

Another option would be sharing workspace with specific user/email, but I don't know how much effort it would take to achieve that.

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

Successfully merging a pull request may close this issue.

5 participants