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.

Sign up for GitHub

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

Jump to bottom

add logging #8

Closed
missinglink opened this issue Sep 18, 2014 · 5 comments · Fixed by #123 or pelias/config#9
Closed

add logging #8

missinglink opened this issue Sep 18, 2014 · 5 comments · Fixed by #123 or pelias/config#9
Labels
help wanted in review v1
Milestone
Pelias v1.0.0

Comments

@missinglink
Copy link
Member

use winston or another similar logging library
@sevko
Copy link
Contributor

sevko commented Apr 16, 2015

pelias/logger is the way to go, @hkrishna.

@sevko sevko self-assigned this Apr 30, 2015
@sevko
Copy link
Contributor

sevko commented Apr 30, 2015

Is there anything we want to start logging in particular? The API currently doesn't output access logs or anything, unless I'm mistaken.

@sevko sevko added the on-deck label May 1, 2015
@sevko
Copy link
Contributor

sevko commented May 4, 2015

@hkrishna , @heffergm ?

@sevko sevko changed the title winston logger add logging May 4, 2015
@sevko
Copy link
Contributor

sevko commented May 4, 2015

Access logs would be nice, for a start.

@heffergm
Copy link
Contributor

heffergm commented May 4, 2015

Additionally, it might be useful to add the ability to log response times from the backend for each request.

@dianashk dianashk added this to the Pelias v1.0.0 milestone May 4, 2015
@sevko sevko added in progress and removed on-deck labels May 4, 2015
@sevko sevko mentioned this issue May 5, 2015
Merged
@sevko sevko removed the on-deck label May 5, 2015
@sevko sevko mentioned this issue May 5, 2015
Merged
@sevko sevko closed this as completed in #123 May 7, 2015
@sevko sevko removed the in review label May 7, 2015
@riordan riordan added the v1 label Aug 13, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
help wanted in review v1
Projects
None yet
Milestone
Pelias v1.0.0
Development

Successfully merging a pull request may close this issue.

add logging pelias/api
Define a default api.accessLog value. pelias/config
5 participants
@riordan @heffergm @missinglink @sevko @dianashk