802649 - Change Network Priorities for CRL (low) and OCSP (high)

Status: RESOLVED FIXED

(Core :: Security: PSM, enhancement)

Description

[Patrick McManus [:mcmanus]]

I hope to make channel priorities a more useful attribute in necko so I'm cleaning up the priority some resource requests are classified with.

OCSP and CRL are both currently executed with default priority.

This patch changes OCSP to high priority as it can block SSL handshake completion, and CRL fetching to low as it is a timer driven process that could compete with latency sensitive browsing operations.

Neither of these changes will have an immediate impact because current prioritization only happens among resources with the same hostname; this is about better meta data to allow experimentation/measurement with other approaches.

Comment 1

[Patrick McManus [:mcmanus]]

Attachment: patch 0

Comment 2

[Patrick McManus [:mcmanus]]

Comment on attachment 672342 [details] [diff] [review]
patch 0

I need a second PSM peer r?

Comment 3

[Honza Bambas (:mayhemer)]

Comment on attachment 672342 [details] [diff] [review]
patch 0

Review of attachment 672342 [details] [diff] [review]:
-----------------------------------------------------------------

r=honzab

Comment 4

[Brian Smith (:briansmith, :bsmith, use NEEDINFO?)]

The CRL Manager part of this patch needs to be removed, since the CRL manager was removed.

This has two r+s. Should we check it in after rebasing it.

Comment 5

[Patrick McManus [:mcmanus]]

thanks - I lost track of this one somehow. I'll rebase it without the crl and check it in.

Comment 6

[Patrick McManus [:mcmanus]]

Attachment: network priority for ocsp

Comment 7

[Patrick McManus [:mcmanus]]

https://hg.mozilla.org/integration/mozilla-inbound/rev/51a006875f24

Comment 8

[Carsten Book [:Tomcat]]

https://hg.mozilla.org/mozilla-central/rev/51a006875f24