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

should "val at released request" be a debug level message rather than warn #252

Open
kajtzu opened this issue Jul 9, 2021 · 1 comment

Comments

@kajtzu
Copy link

kajtzu commented Jul 9, 2021

We recently upgraded from 0.4.4 to 0.5.2 and noticed that our logs fill up with messages like the following:

2021-07-09 11:21:41[warn][23159][main]val at released request 22908064, idx 1
2021-07-09 11:21:41[warn][23159][main]val at released request 22908064, idx 3

Is there from a developer perspective any objection to changing the following log message from level warn to level debug? Or is it an indication that we are using nginx-clojure wrong as we seem to get a lot of these ;)

NginxClojureRT.getLog().warn("val at released request %s, idx %d", r, i);

@xfeep
Copy link
Member

xfeep commented Jul 28, 2021

It seems there are some invalid access to those closed requests.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants