Skip to content

Conversation

@github-actions
Copy link
Contributor

@github-actions github-actions bot commented Feb 11, 2022

Backport of #65216 to release/6.0

The chrome://inspect CDP logic is rejecting /json responses that don't include the content length in the response, this fixes that and allows chrome://inspect to connect to the proxy properly

/cc @lewing

Customer Impact

Without this change the official browser debugger front end is no longer able to connect properly to proxy preventing customers from using chrome or edge as the debugger frontend.

Testing

Manual testing.

Risk

Very low. Simply sets the content-length in the response header

@ghost ghost added the area-Debugger-mono label Feb 11, 2022
@ghost
Copy link

ghost commented Feb 11, 2022

Tagging subscribers to this area: @thaystg
See info in area-owners.md if you want to be subscribed.

Issue Details

Backport of #65216 to release/6.0

/cc @lewing

Customer Impact

Testing

Risk

Author: github-actions[bot]
Assignees: -
Labels:

area-Debugger-mono

Milestone: -

@lewing lewing added the Servicing-consider Issue for next servicing release review label Feb 11, 2022
@lewing lewing added this to the 6.0.4 milestone Feb 11, 2022
@lewing lewing added the arch-wasm WebAssembly architecture label Feb 12, 2022
@ghost
Copy link

ghost commented Feb 12, 2022

Tagging subscribers to 'arch-wasm': @lewing
See info in area-owners.md if you want to be subscribed.

Issue Details

Backport of #65216 to release/6.0

The chrome://inspect CDP logic is rejecting /json responses that don't include the content length in the response, this fixes that and allows chrome://inspect to connect to the proxy properly

/cc @lewing

Customer Impact

Without this change the official browser debugger front end is no longer able to connect properly to proxy preventing customers from using chrome or edge as the debugger frontend.

Testing

Manual testing.

Risk

Very low. Simply sets the content-length in the response header

Author: github-actions[bot]
Assignees: -
Labels:

Servicing-consider, arch-wasm, area-Debugger-mono

Milestone: 6.0.4

@rbhanda rbhanda added Servicing-approved Approved for servicing release and removed Servicing-consider Issue for next servicing release review labels Feb 15, 2022
@lewing
Copy link
Member

lewing commented Feb 24, 2022

can someone with the appropriate authorization please merge this?

@carlossanlop
Copy link
Contributor

PR has sign-off, the servicing-approved label applied, and the CI is passing. Ready to merge.

@carlossanlop carlossanlop merged commit 51ecbf3 into release/6.0 Mar 8, 2022
@carlossanlop carlossanlop deleted the backport/pr-65216-to-release/6.0 branch March 8, 2022 20:55
@ghost ghost locked as resolved and limited conversation to collaborators Apr 8, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

arch-wasm WebAssembly architecture area-Debugger-mono Servicing-approved Approved for servicing release

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants