New issue
Advanced search Search tips
Note: Color blocks (like or ) mean that a user may not be available. Tooltip shows the reason.

Issue 767025 link

Starred by 2 users

Issue metadata

Status: Archived
Owner: ----
Closed: Jan 3
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Android
Pri: 3
Type: Bug



Sign in to add a comment

Allow setting Accept-Encoding header to "identity"

Project Member Reported by xunji...@chromium.org, Sep 20 2017

Issue description

[Moved from internal bug b/66132021 to crbug]

Cronet currently ignores Accept-Encoding header set by consumer ( Issue 581399 ). This was to prevent consumers from setting encoding types that are not supported by Cronet. We should allow setting Accept-Encoding to identity, so consumer has a way to disable the default encoding types (e.g. gzip, deflate).
 
Owner: xunji...@chromium.org
Status: Started (was: Available)
Cc: pauljensen@chromium.org
Related Issue 711810.
My initial idea is to make URLRequestHttpJob to ignore custom-set Accept-Encodings that are not "identity" or its variants (e.g. "identity; q=1"). We already have parsing code in http_util.cc.

That way we could address the underlying issue in //net Issue 711810 as well as this bug. Let me start a CL and we can discuss alternative approaches there. 
Cc: xunji...@chromium.org
Labels: -Pri-2 -M-63 Pri-3
Owner: ----
Status: Available (was: Started)
Unassigning... i am not actively working on this.
Project Member

Comment 4 by sheriffbot@chromium.org, Nov 12

Labels: Hotlist-Recharge-Cold
Status: Untriaged (was: Available)
This issue has been Available for over a year. If it's no longer important or seems unlikely to be fixed, please consider closing it out. If it is important, please re-triage the issue.

Sorry for the inconvenience if the bug really should have been left as Available.

For more details visit https://www.chromium.org/issue-tracking/autotriage - Your friendly Sheriffbot
Status: Archived (was: Untriaged)

Sign in to add a comment