You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
I notice the logger.trace() calls we had previously in HTTPX are not present in this package anymore.
I suppose this was done out of simplicity, but I think we should bring them back. They were super useful to debug networking issues, connection management issues, etc.
I think it was while I was trying to do a bit of refactoring of the HTTP/2 and pulling everything across to httpcore. Is it clear what environment variable naming we ought to be using given that we've got a proper httpx/httpcore split now?
Activity
lovelydinosaur commentedon Apr 13, 2020
Yeah that's my fault,
I think it was while I was trying to do a bit of refactoring of the HTTP/2 and pulling everything across to httpcore. Is it clear what environment variable naming we ought to be using given that we've got a proper httpx/httpcore split now?
florimondmanca commentedon Apr 13, 2020
For env vars we could have an HTTPCORE_LOG_LEVEL, and make HTTPX set that when HTTPX_LOG_LEVEL is set too.
lovelydinosaur commentedon Apr 13, 2020
Sounds reasonable yeah