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.

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

Output logs are garbled when sending concurrent requests #47

Closed
sgrif opened this issue Dec 24, 2016 · 1 comment
Closed

Output logs are garbled when sending concurrent requests #47

sgrif opened this issue Dec 24, 2016 · 1 comment
Labels
accepted An accepted request or suggestion duplicate This issue or pull request already exists enhancement A minor feature request

Comments

@sgrif
Copy link
Contributor

sgrif commented Dec 24, 2016

Steps to reproduce:

  • Submit multiple concurrent requests to an application which is serving some content at /.

Expected behavior:

  • Log output is grouped per-request, or at least uses line-buffering and includes sufficient information per-line to link that line to the relevant request.

Actual behavior:

  • Logs are mixed in an unreadable fashion, such as:
GET /:
    => Matched: GET /
GET /:
    => Matched: GET /
    =>     => Outcome: Succcess
Outcome: Succcess
    => Response succeeded.
    => Response succeeded.

In particular, the => => Outcome: Succcess line seems especially undesirable.

@SergioBenitez
Copy link
Member

Yes, this is indeed an issue that I'm aware of. I don't have a good solution yet, but I'll merge this in with the existing logging issue #21.

@SergioBenitez SergioBenitez added accepted An accepted request or suggestion duplicate This issue or pull request already exists enhancement A minor feature request labels Dec 24, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
accepted An accepted request or suggestion duplicate This issue or pull request already exists enhancement A minor feature request
Projects
None yet
Development

No branches or pull requests

2 participants