Clojurians Log v2

Hi again @souenzzo If you are interested the following code solved the use case: https://github.com/pedestal/pedestal/issues/665#issuecomment-651652532 It's not perfect, but it works.

Hi because of a health check I see every second logging like this:

{"message":"{:msg \"GET /\", :line 80}","severity":"INFO","thread":"qtp1569164069-15","logger":"io.pedestal.http"}

Is there a way to modify the logger in such a way that requests to “\/” are ignored?

I think that we are a bit confused about async and lazy things If you are sending a huge payload, your client probably will not handle it "as a json array", it will prefer "a steam of jsons"

@erwinrooijakkers change your logback.xml file to change the log level that is reported to be above "INFO"

The thing is I do want other INFO logging

Other requests are fine

Only not GET requests to /

I think it can be done by modifying the request-logger

Default is this:

(def log-request
  "Log the request's method and uri."
  (interceptor/on-request
    ::log-request
    (fn [request]
      (log/info :msg (format "%s %s"
                             (string/upper-case (name (:request-method request)))
                             (:uri request)))
      (log/meter ::request)
      request)))

So I can change that one to check the :uri and then not log. 🙂

I really think you should make that change in the logback.xml, but I don't know the details off the top of my head. This sounds like it should be a configuration change though, not a code change.

you could decorate the default request-logger and then pass yours in when you create the service https://github.com/pedestal/pedestal/blob/master/service/src/io/pedestal/http.clj#L205

Uhm well logback only has access to the string content. It might be able to create a filter based on a regex.