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

Add logging transport for stdout #47

Closed
iromli opened this Issue Oct 6, 2018 · 1 comment

Comments

Projects
None yet
2 participants
@iromli

iromli commented Oct 6, 2018

In docker environment, it's easier to check logs via stdout. This is an example on how we patch logger.js to use console instead of rotated file: https://github.com/GluuFederation/docker-oxPassport/blob/0129ae05059e71f87063c936418f9aff903a4f35/logger.js#L38.

One of the reasons why we prefer stdout is because we want to utilize external tools such as fluentd, logstash, filebeat, etc, to retrieve logs and transport them to another backend.

We would like to see built-in support for stdout transport for our usecase.

@jgomer2001 jgomer2001 self-assigned this Nov 11, 2018

@jgomer2001 jgomer2001 added this to the 3.1.5 milestone Nov 11, 2018

jgomer2001 added a commit to GluuFederation/community-edition-setup that referenced this issue Nov 16, 2018

jgomer2001 added a commit that referenced this issue Nov 16, 2018

@jgomer2001

This comment has been minimized.

Contributor

jgomer2001 commented Nov 16, 2018

@iromli I added a property to passport-config.json so log output is redirected to console only. Use "consoleLogOnly" : true

@jgomer2001 jgomer2001 closed this Nov 16, 2018

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment