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

Add logging transport for stdout #47

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

Add logging transport for stdout #47

iromli opened this issue Oct 6, 2018 · 1 comment
Assignees
Milestone

Comments

@iromli
Copy link

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
Copy link
Contributor

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

yurem pushed a commit to GluuFederation/community-edition-setup that referenced this issue Jan 15, 2019
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

No branches or pull requests

2 participants