Skip to content

fix: remove stray )} characters from redirect chain output#1808

Open
firat-bcfm wants to merge 1 commit intoChromeDevTools:mainfrom
firat-bcfm:fix/remove-stray-chars-in-redirect-chain
Open

fix: remove stray )} characters from redirect chain output#1808
firat-bcfm wants to merge 1 commit intoChromeDevTools:mainfrom
firat-bcfm:fix/remove-stray-chars-in-redirect-chain

Conversation

@firat-bcfm
Copy link
Copy Markdown

The redirect chain formatter in converNetworkRequestDetailedToStringDetailed had a stray )} at the end of the template literal, causing every redirect chain entry to be suffixed with those characters in tool output.

@google-cla
Copy link
Copy Markdown

google-cla bot commented Apr 3, 2026

Thanks for your pull request! It looks like this may be your first contribution to a Google open source project. Before we can look at your pull request, you'll need to sign a Contributor License Agreement (CLA).

View this failed invocation of the CLA check for more information.

For the most up to date status, view the checks section at the bottom of the pull request.

@nroscino
Copy link
Copy Markdown
Contributor

nroscino commented Apr 8, 2026

Hi @firat-bcfm!
Before proceeding we need you to sign the CLA.
Also, can you please cover this with a test so that next time something similar happens we will catch it with a failing test?

@firat-bcfm
Copy link
Copy Markdown
Author

Okey

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants