Request and response logging of the unserialized Java types

[lsuski]

Hi,
I'd like to log only request and response body along with http status code and execution time (no headers, urls and methods names). Is there possibility to make log level mechanism more flexible? For example use some interface or simple class with methods like isLogStatus(), isLogBody() etc. and ovveride it for own needs.
Thanks

[thimes]

Log is already an interface. You can create a class of your own that prints as little or as much as you want, and then set it on the RestAdapter, like so:

   RestAdapter restAdapter = new RestAdapter.Builder().setLog(new MyLogImpl()).build();

What you print is up to you. I'd also suggest looking into the Emitter class, for help with printing.

[lsuski]

Log is an interface but LogLevel is not. I would need to check message content passed to Log.log() method to reject e.g. headers, which is not very handy and require a knowledge of RestAdapter message format.
Thanks.

BTW. Where is the class Emitter. I couldn't find it in Retrofit.

[thimes]

Huh, I apologize. I thought that was part of retrofit, looking closer, it's just an interface I made to abstract away that part of output.

I could envision a LogLevel builder that contains methods for setting the logging of headers, body, url, and status, among others. What else would you want to see in the builder?

[lsuski]

Things that you mentioned + exceptions are sufficient for me.
Thanks a lot for interest.

[JakeWharton]

Writing this down so it's not in my head:

Loggers will get callbacks for request and response much like they do internally. Log level will become part of the default logging impl. You can replace the logger with one of your own.

[Turbo87]

@JakeWharton is that planned for 2.0 or also for the 1.x branch?

[JakeWharton]

This is all 2.0 stuff. I'm trying to wrap up a 1.6 this week so I can start the work on 2.0 ASAP.

[JakeWharton]

Another thing: how do we handle binary vs text data by default?

[JakeWharton]

We need to be sure to hand the Class<?> reference to the logger for which the events are being generated so that loggers can present the logs as if the interfaces themselves were generating them.

[jacobtabak]

Came here looking for a solution to logging binary data - for now, I will use a different rest adapter. Do we need to consider the case where a TypedFile contains ASCII data and should potentially be logged?