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

Remove default localhost vhost #189

Open
gbbirkisson opened this issue Aug 17, 2021 · 6 comments
Open

Remove default localhost vhost #189

gbbirkisson opened this issue Aug 17, 2021 · 6 comments
Assignees
Labels
enhancement New feature or request

Comments

@gbbirkisson
Copy link
Contributor

gbbirkisson commented Aug 17, 2021

Now we always add localhost vhost to deployments to be able to easily use kubectl port-forward. Remove that default vhost!

@gbbirkisson gbbirkisson self-assigned this Aug 17, 2021
@gbbirkisson gbbirkisson added the good first issue Good for newcomers label May 11, 2022
@gbbirkisson gbbirkisson removed their assignment May 11, 2022
@vbradnitski
Copy link
Contributor

investigate this after 1.0.0 milestone.

@sigdestad
Copy link
Member

How will I as an super admin get access to 8080/ if this localhost vhost is not pre-defined.?

@alansemenov alansemenov added this to the 1.0.0 milestone May 24, 2023
@rymsha
Copy link
Contributor

rymsha commented Jun 20, 2023

default config is located in vhost-defaults.cfg file

vbradnitski added a commit that referenced this issue Dec 6, 2023
vbradnitski added a commit that referenced this issue Dec 6, 2023
vbradnitski added a commit that referenced this issue Dec 6, 2023
@vbradnitski vbradnitski linked a pull request Dec 6, 2023 that will close this issue
@vbradnitski
Copy link
Contributor

vbradnitski commented Dec 7, 2023

NOTE: enable enableLocalhostVhost on freeplan before close the issue

@rymsha
Copy link
Contributor

rymsha commented Dec 7, 2023

The main problem with localhost vhost - is that it can be abused by developers who create a http request to localhost and think it is a "brilliant idea". As we know such creative idea leads to positive-feedback-loops and must be prevented at all costs.

@rymsha rymsha added enhancement New feature or request and removed good first issue Good for newcomers labels Dec 7, 2023
@rymsha
Copy link
Contributor

rymsha commented Dec 7, 2023

we need vhost config which would be added to all xp deployments - base vhost config. This way operator users could specify virtual hosts without need to setting up ingresses.

Use cases:

  • some xp users still depend on localhost vhost idea to work. Such users need operator being able to add localhost vhost.
  • sometimes it is necessity to access /admin endpoint on localhost. In this case it is best to open localhost:8080/admin on localhost - but not more than that. (in XP 8 we might need to allow /api as well).

Customizing base vhost config per instance seem to be out of scope right now.

We either need to find out how to configure "extra-config" from terraform or move base vhost config to values

@rymsha rymsha removed this from the 1.0.0 milestone Dec 7, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging a pull request may close this issue.

5 participants