Don't show Content-Encoding

[Goooler]

:writing_hand: Describe the bug

Don't show Content-Encoding in this case.

:bomb: Steps to reproduce

1. Add my HeaderInterceptor
2. Add ChuckerInterceptor.Builder(context).build()
3. Did not show Content-Encoding in Chucker, but showed in Charles

:camera: Screenshots

:iphone: Tech info

• Device: OnePlus 8t
• OS: 11
• Chucker version: 3.4.0

:page_facing_up: Additional context

[MiSikora]

How do you add ChuckerInterceptor to OkHttp? As a network or application interceptor? If as an application interceptor then this is expected.

[Goooler]

How do you add ChuckerInterceptor to OkHttp? As a network or application interceptor? If as an application interceptor then this is expected.

Why designed for this ?

[MiSikora]

Because this is how OkHttp works. If you want to inspect network traffic apply ChuckerInterceptor as a network interceptor.

One thing we could do, because I see a lot of confusion about this, is to display both network and application headers if it is applied as an application interceptor. But this is a topic for a different discussion. I wonder what other maintainers think of such an addition.

[Goooler]

@MiSikora Thx, I got it.

[vbuberen]

I don't think we need to show both types of headers. I would just update README to explain the topic better.

[cortinico]

One thing we could do, because I see a lot of confusion about this, is to display both network and application headers if it is applied as an application interceptor. But this is a topic for a different discussion. I wonder what other maintainers think of such an addition.

To be honest I'm a bit unsure on this point. On one hand, I think that showing all the Headers we can provide to the user will reduce frictions (i.e. probably a user just want to see the HTTP headers without having to know at which stage of the chain a header was added). At the same time, I'm afraid this could create more confusion for users that are actually aware of the network/application interceptors differences 🤔

[MiSikora]

I meant to show it in two sections. Network section and application section. Personally, I wouldn't bother, but I see that it comes up once in a while like in this issue so I just threw an idea.

[cortinico]

I meant to show it in two sections. Network section and application section.

+1 on having them separated