-
Notifications
You must be signed in to change notification settings - Fork 150
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
feat: add SAS_EXPRESS_KEEP_ALIVE_TIMEOUT #1233
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.
The functionality looks good to me. I just have the one question about why the SAS_EXPRESS_LOG_MODE
line is being removed from the Readme
@@ -157,8 +157,7 @@ For more information on our configuration manager visit its readme [here](https: | |||
|
|||
- `SAS_EXPRESS_BIND_HOST`: address on which the server will be listening, defaults to `127.0.0.1`. | |||
- `SAS_EXPRESS_PORT`: port on which the server will be listening, defaults to `8080`. | |||
- `SAS_EXPRESS_LOG_MODE`: enable console logging of "all" HTTP requests, only "errors", or nothing by |
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.
Is SAS_EXPRESS_LOG_MODE
being removed as a valid configuration value?
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.
Funny, but also weirdly enough it's a very old legacy mode which has been removed for a while now.
closes: #1231
Allow sidecar to expose
SAS_EXPRESS_KEEP_ALIVE_TIMEOUT
as a enviornment variable. It expects a number value, and it defaults to 5 is if nothing inputted (That is the default express value).