Better Loggin - Githubissues

Notifiarr / notifiarr

Client for Notifiarr.com

https://notifiarr.com

MIT License

531 stars 31 forks source link

Better Loggin #601

Open bakerboy448 opened 1 year ago

bakerboy448 commented 1 year ago

[x] cleanse plex token from logs
[x] enhance request response logging to include response time
[ ] enhance request logging to include request id
[ ] enhance request logging to include request and response together or easily identifiable and connectable
[x] improve plex session timeout logging

davidnewhall commented 7 months ago

@bakerboy448 how's it lookin' on unstable?

bakerboy448 commented 7 months ago

So far so good from what I've glanced at

davidnewhall commented 3 months ago

Do you remember which requests we needed to tie together? enhance request logging to include request and response together or easily identifiable and connectable

bakerboy448 commented 3 months ago

I believe all the requests to apps and the site? Basically any HTTP request outbound from the client

davidnewhall commented 3 months ago

That's already all tied. What I was thinking we meant was telling which outbound requests were created due to an inbound request. eg. snapshot request. But that's a lot of code to write and not easily accomplished. Not sure how useful it'd be either since the app doesn't do that much at once typically.