Skip to content

Add option to allow logging to standard error #63

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

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

Georift
Copy link

@Georift Georift commented Feb 5, 2024

Sometimes you'd like to be able to influence the log lines and send some logs to standard error. This option add the property LOG_TO_STANDARD_ERROR which will send any failing status codes to standard error.

To confirm it's working:

docker run --rm -e LOG_WITHOUT_NEWLINE=true -e LOG_TO_STANDARD_ERROR=true -p 8080:8080 mendhak/http-https-echo:latest 2> errors

Will output only standard error to the errors file.

Test it logs to standard error:

curl -v -H "x-set-response-status-code: 401" http://localhost:8080/

Expect to see a log line appear in errors but not on the docker run output

Test it logs to standard out:

curl -v -H "x-set-response-status-code: 350" http://localhost:8080/

Expect to see a log line appear in the docker run output, but not in errors

Sometimes you'd like to be able to influence the log lines and send some
logs to standard error. This option add the property
`LOG_TO_STANDARD_ERROR` which will send any failing status codes to
standard error.
@mendhak
Copy link
Owner

mendhak commented Feb 29, 2024

Sorry for late reply, are you able to add a test here: https://github.com/mendhak/docker-http-https-echo/blob/master/tests.sh

for what you've added.

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

Successfully merging this pull request may close these issues.

2 participants