HttpLoggingInterceptor with exceptions

[mgohin]

• Feature Request

I'm using your HttpLoggingInterceptor which do a pretty good job to log http requests into a file in my android app.
I'd like to add some urls that must not be logged (all logging calls for example).

Did I miss that kind of feature or it doesn't exists ?

[mgohin]

I can make a PR if you like the idea, I had to dev my own custom logging interceptor on top of your to do it

[JakeWharton]

This sounds like a good task for composition and not something we want to bake in. .addInterceptor(new PredicateInterceptor(loggingerInterceptor, request -> !blacklisted.contains(request.url()))

…

On Mon, Dec 18, 2017 at 9:59 AM Maelig ***@***.***> wrote: I can make a PR if you like the idea, I had to dev my own custom logging interceptor on top of your to do it — You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub <#3727 (comment)>, or mute the thread <https://github.com/notifications/unsubscribe-auth/AAEEEV6KVX-JqdaJ2d0D3SSk6oiyDhgrks5tBn22gaJpZM4RFiUY> .

[mgohin]

Do you add it to your todo list or do I make a PR ? (you'll probably be better than me for the dev of it)

[JakeWharton]

I don't think we need to ship a PredicateInterceptor since it's only like 6 lines of code. Perhaps ask a StackOverflow question for this and we can add it as an answer.

…

On Mon, Dec 18, 2017 at 10:07 AM Maelig ***@***.***> wrote: Do you add it to your todo list or do I make a PR ? — You are receiving this because you commented. Reply to this email directly, view it on GitHub <#3727 (comment)>, or mute the thread <https://github.com/notifications/unsubscribe-auth/AAEEEVGxYvmj43XZNC_0q5AUaLnLcoJiks5tBn-8gaJpZM4RFiUY> .

[mgohin]

I was not aware of interceptor's power, came to this :

.addInterceptor(new Interceptor() {
                @Override
                public Response intercept(Chain chain) throws IOException {
                    Request request = chain.request();
                    boolean logBody = needToLogBody();
                    logging.setLevel(logBody ? HttpLoggingInterceptor.Level.BODY : HttpLoggingInterceptor.Level.HEADERS);
                    return chain.proceed(request);
                }
            })
.addInterceptor(logging)