1899711 - After each authentication attempt the number of authRequired events received increases

Status: RESOLVED FIXED

(Remote Protocol :: WebDriver BiDi, defect, P3)

Description

[Julian Descottes [:jdescottes]]

STRs:

• subscribe to network.authRequired
• perform a request to a URL using http auth
• submit wrong credentials
• (repeat that step a few times)

ER: For each attempt, we should get a single authRequired event
AR: After each attempt the number of events received increases by 1

We never "remove" the AuthListener we setup on the original request, and when the network event for the authentication attempt request is created, we setup another AuthListener here. When we perform the next authentication attempt, both listeners will be triggered.

Comment 1

[BugBot [:suhaib / :marco/ :calixte]]

The severity field is not set for this bug.
:whimboo, could you have a look please?

For more information, please visit BugBot documentation.

Comment 2

[Alexandra Borovova [:Sasha]]

Attachment: Bug 1899711 - Set only one NetworkAuthListener per channel.

Comment 3

[Alexandra Borovova [:Sasha]]

Attachment: Bug 1899711 - [wdspec] Add a test for the correct amount of "network.authRequired" events in case of wrong credentials.

Comment 4

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/2f8b064fac6c
Set only one NetworkAuthListener per channel. r=devtools-reviewers,jdescottes
https://hg.mozilla.org/integration/autoland/rev/822828ed7732
[wdspec] Add a test for the correct amount of "network.authRequired" events in case of wrong credentials. r=webdriver-reviewers,jdescottes

Comment 5

[Web Platform Test Sync Bot [:wpt-sync] (Matrix: #interop:mozilla.org)]

Created web-platform-tests PR https://github.com/web-platform-tests/wpt/pull/47386 for changes under testing/web-platform/tests

Comment 6

[Sandor Molnar[:smolnar]]

https://hg.mozilla.org/mozilla-central/rev/2f8b064fac6c
https://hg.mozilla.org/mozilla-central/rev/822828ed7732

Comment 7

[Web Platform Test Sync Bot [:wpt-sync] (Matrix: #interop:mozilla.org)]

Upstream PR merged by moz-wptsync-bot

Comment 8

[Web Platform Test Sync Bot [:wpt-sync] (Matrix: #interop:mozilla.org)]

Upstream PR merged by moz-wptsync-bot