-
-
Notifications
You must be signed in to change notification settings - Fork 2k
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
Document about Access log cause confusion #2085
Labels
documentation
Improvements or additions to documentation
good first issue
Good for newcomers
outdated
Milestone
Comments
mapler
changed the title
Access log disable not working
Document about Access log cause confusion
Jul 13, 2017
Yes. Doc about logger disabling was created before introducing Could you create a PR for doc update? |
asvetlov
added
documentation
Improvements or additions to documentation
good first issue
Good for newcomers
labels
Jul 13, 2017
mapler
added a commit
to mapler/aiohttp
that referenced
this issue
Jul 13, 2017
mistake about disabling access logs.
asvetlov
pushed a commit
that referenced
this issue
Jul 13, 2017
Fixed by #2085 |
Sign up for free
to subscribe to this conversation on GitHub.
Already have an account?
Sign in.
Labels
documentation
Improvements or additions to documentation
good first issue
Good for newcomers
outdated
Long story short
When I followed the document here to disable access log. It seems not working.
Expected behaviour
Just use
app.make_handler(access_log=None)
for disabling access log.Steps to reproduce
After I check the source code here. I know I make mistake about the usage of
make_handler
.But I think
run_app(app, access_log=None)
is better and more clear thanapp.make_handler(access_log=None)
in document.The text was updated successfully, but these errors were encountered: