Skip to content

[java] attempt to pass all the headers from the paused request #15535

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Closed
wants to merge 4 commits into from
Closed

[java] attempt to pass all the headers from the paused request #15535

Changes from all commits
Commits
Show all changes
4 commits
Select commit Hold shift + click to select a range
b1f4ca5
attempt to pass all the headers from the paused reuqest
iampopovich Mar 30, 2025
a898f9d
applied format.sh
iampopovich Mar 30, 2025
10932a5
Merge branch 'trunk' into fix/15530-passing-headers-for-paused-request
iampopovich Mar 30, 2025
c8e7060
Merge branch 'trunk' into fix/15530-passing-headers-for-paused-request
iampopovich Apr 23, 2025
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
16 changes: 15 additions & 1 deletion java/src/org/openqa/selenium/devtools/v134/v134Network.java
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -191,7 +191,21 @@ protected Command<Void> continueRequest(RequestPaused pausedReq, HttpRequest req
}

List<HeaderEntry> headers = new ArrayList<>();
req.forEachHeader((name, value) -> headers.add(new HeaderEntry(name, value)));

// First add the original headers from the paused request to preserve them
pausedReq
.getRequest()
.getHeaders()
.forEach((name, value) -> headers.add(new HeaderEntry(name, String.valueOf(value))));

// Then add any new headers from the modified request, which will override existing ones if
// present
req.forEachHeader(
(name, value) -> {
// Remove any existing header with the same name
headers.removeIf(h -> h.getName().equalsIgnoreCase(name));
headers.add(new HeaderEntry(name, value));
});

return Fetch.continueRequest(
pausedReq.getRequestId(),
Expand Down
Loading