Closed Bug 1592927 Opened 8 months ago Closed 8 months ago

Test test_anonymous-coalescing is racy

Categories

(Core :: Networking: HTTP, defect)

defect
Not set
normal

Tracking

()

RESOLVED FIXED
mozilla72
Tracking Status
firefox72 --- fixed

People

(Reporter: dragana, Assigned: dragana)

References

Details

Attachments

(1 file)

test_anonymous-coalescing test that a non-anonymous h2 connection can be reused for a anonymous request and vice versa. The connection will be reuse only if it experienced. The connection is experience if a non-null transaction is added to the connection when tls handshake was been finished. The test use only one transaction to make a connection "experienced" but this is racy and may not mark the connection "experienced":

  • if a null transaction is used to drive tls handshake, the non-null transaction will be added after the handshake is finished and it will mark the connection "experienced".
  • if a non-null transaction is used to drive tls handshake, the connection will not be marked "experienced" and it will be only marked "experience when another transaction is added"

To make sure the connection is marked "experienced" we will always performe 2 transaction (in one case anonymous and in the other non-anonymous) before we try a transaction with reverted anonymous flag.

Blocks: 1581637
Pushed by ncsoregi@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/597370891189
Test test_anonymous-coalescing is racy. r=kershaw
Status: ASSIGNED → RESOLVED
Closed: 8 months ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla72
You need to log in before you can comment on or make changes to this bug.