-
Notifications
You must be signed in to change notification settings - Fork 3
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
Listen to custom port #27
base: master
Are you sure you want to change the base?
Listen to custom port #27
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think the port should be added also to the config.toml, README.md, and implement the default in the run.sh.
de44203
to
5a5c2bf
Compare
Hi @jesusfcr, thanks for having a look into this! 😄 I have added the port to the config.toml, README.md and local.env files. There was no need to add to run.toml, since it's auto generate based on the contents of config.toml plus local.env, the port will be added to file after we run envsubst on it. Let me know if it does not work like that. |
Hi, is in config.toml where you have to use the variable. |
@jesusfcr done! |
This PR introduces a new configuration field
api.port
, so users can run the Vulnerability DB API in a port different than:8080
The default value will be
8080
, so existingtoml
files are kept valid.