Skip to content
This repository has been archived by the owner on Nov 28, 2021. It is now read-only.

More verbose logging #107

Closed
3 tasks done
BradMclain opened this issue Apr 10, 2016 · 2 comments
Closed
3 tasks done

More verbose logging #107

BradMclain opened this issue Apr 10, 2016 · 2 comments
Labels
enhancement
Milestone
2.0.15

Comments

@BradMclain
Copy link
Contributor

BradMclain commented Apr 10, 2016 •
edited
Loading

Currently we log like this:

ERROR:prestans.sample-api:field attribute is required and does not provide a default value

It would be nice to include extra information about:

  • Method
  • URL
  • User agent
@BradMclain BradMclain modified the milestones: 2.0.14, 2.0.15 Apr 10, 2016
@BradMclain
Copy link
Contributor Author

We really probably need a common base class for both ParserException and HandlerException which accepts a method and path to add to the logging message.

@BradMclain
Copy link
Contributor Author

BradMclain commented Apr 15, 2016 •
edited
Loading

Exceptions to update:

Auth Provider

  • AuthenticationError
  • AuthorizationError

Request

Response

  • ServiceUnavailable
  • BadRequest
  • Conflict
  • NotFound
  • Unauthorized
  • MovedPermanently
  • PaymentRequired
  • Forbidden

BradMclain added a commit that referenced this issue Apr 15, 2016
@BradMclain
first cut of #107, needs more testing before we can consider it stable
9501461
BradMclain added a commit that referenced this issue Jun 3, 2016
@BradMclain
added verbose logging for exceptions in auth provider #107
c640941
@BradMclain BradMclain mentioned this issue Jun 3, 2016
Open
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees
No one assigned
Labels
enhancement
Projects
None yet
Milestone
2.0.15
Development

No branches or pull requests
1 participant
@BradMclain