Magic-Mirror/gecko-dev
1
0
Fork 0
mirror of https://github.com/mozilla/gecko-dev.git synced 2024-11-28 07:13:20 +00:00
Code Issues Actions Packages Projects Releases Wiki Activity

Bug 1852223 - [bidi] Handle intercepted requests in the authRequired phase r=webdriver-reviewers,Sasha

Browse Source 
Depends on D189517

Differential Revision: https://phabricator.services.mozilla.com/D191236
This commit is contained in:
Julian Descottes 2023-11-08 08:27:44 +00:00
parent 95085f677e
commit 371b0dfbb0
1 changed files with 25 additions and 6 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

31
remote/webdriver-bidi/modules/root/network.sys.mjs
View File

@ -490,11 +490,13 @@ class NetworkModule extends Module {
contextId,
isNavigationRequest,
redirectCount,
requestChannel,
requestData,
responseData,
timestamp,
} = data;
let isBlocked = false;
try {
const browsingContext = lazy.TabManager.getBrowsingContextById(contextId);
if (!browsingContext) {
@ -547,11 +549,28 @@ class NetworkModule extends Module {
authRequiredEvent,
this.#getContextInfo(browsingContext)
);
if (authRequiredEvent.isBlocked) {
isBlocked = true;
// requestChannel.suspend() is not needed here because the request is
// already blocked on the authentication prompt notification until
// one of the authCallbacks is called.
this.#blockedRequests.set(authRequiredEvent.request.request, {
request: requestChannel,
phase: InterceptPhase.AuthRequired,
});
// TODO: Once we implement network.continueWithAuth, we should create a
// promise here which will wait until the request is resumed and removes
// the request from the blockedRequests. See Bug 1826196.
}
} finally {
// Bug 1852223: Until we handle intercepted requests in the authRequired
// phase, we should always forward the auth prompt notification so that it
// can be handled by the browser.
authCallbacks.forwardAuthPrompt();
if (!isBlocked) {
// If the request was not blocked, forward the auth prompt notification
// to the next consumer.
authCallbacks.forwardAuthPrompt();
}
}
};
@ -712,9 +731,9 @@ class NetworkModule extends Module {
phase: InterceptPhase.ResponseStarted,
});
// TODO: Once we implement network.continueRequest, we should create a
// TODO: Once we implement network.continueResponse, we should create a
// promise here which will wait until the request is resumed and removes
// the request from the blockedRequests. See Bug 1850680.
// the request from the blockedRequests. See Bug 1853887.
}
};