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

Never make ports public unless explicitly configured #4159

Closed
corneliusludmann opened this issue May 6, 2021 · 0 comments · Fixed by #4548
Closed

Never make ports public unless explicitly configured #4159

corneliusludmann opened this issue May 6, 2021 · 0 comments · Fixed by #4548
Assignees
Labels
feature: ports priority: must have This issue must be fixed before others - use sparingly type: improvement Improves an existing feature or existing code

Comments

@corneliusludmann
Copy link
Contributor

Currently, ports that are listed in the .gitpod.yaml file are public by default. That means everyone that knows the port URL can access them without authentication/authorization. We do this for backward compatibility reasons.

However, public ports can be a security reason in some cases. Therefore, we should break backward compatibility in favor of more secure defaults and make ports public only when the user has configured it explicitly with the visibility property in the .gitpod.yaml file or by clicking the “Make Public” button.

We should probably mention this change in the changelog dialog to inform our users.

@corneliusludmann corneliusludmann added feature: ports type: improvement Improves an existing feature or existing code labels May 6, 2021
@csweichel csweichel added this to the [backlog] June 2021 milestone May 6, 2021
@csweichel csweichel added the priority: must have This issue must be fixed before others - use sparingly label May 6, 2021
@corneliusludmann corneliusludmann self-assigned this Jun 1, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature: ports priority: must have This issue must be fixed before others - use sparingly type: improvement Improves an existing feature or existing code
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants