Closed Bug 1908112 Opened 4 months ago Closed 14 days ago

Ship TLS Certificate Compression in Release

Categories

(Core :: Security: PSM, enhancement, P2)

enhancement

Tracking

()

RESOLVED FIXED
133 Branch
Tracking Status
relnote-firefox --- 132+
firefox132 --- fixed
firefox133 --- fixed

People

(Reporter: anna.weine, Assigned: anna.weine)

References

(Depends on 1 open bug, )

Details

Attachments

(2 files)

As we are currently supporting certificate compression in Nightly and Beta, it's time to roll it out to Release.

See also:
certificate compression meta bug: https://bugzilla.mozilla.org/show_bug.cgi?id=1548723
enable certificate compression decoders in Beta: https://bugzilla.mozilla.org/show_bug.cgi?id=1905910

Component: Libraries → Security: PSM
Product: NSS → Core
Target Milestone: --- → 130 Branch
Version: trunk → unspecified

Can we ship it now?

Hi,

Currently we are checking that the implementation will benefit to our users (measuring the speed) without breaking any functionality. More specifically, as the certificate compression operates with the certificates that are verified before accessing the page, wrong implementation of the certificate compression on any step would lead to not being able to access certain sites.

In order to ensure that everything is going well, we are having a set of experiments. Until now we do not observe any breakages, so we hope to be able to enable the certificate compression in Release soon.

Pushed by nkulatova@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/8a1926792e04 Enable all the certificate compression mechanisms in Release r=jschanck

As discussed the last week during the NSS call, we plan

  • to enable all the cert compression algorithms;
  • uplift it to beta;
  • start (now) the roll-out to release.

Original Revision: https://phabricator.services.mozilla.com/D224936

Attachment #9429942 - Flags: approval-mozilla-beta?

beta Uplift Approval Request

  • User impact if declined: none
  • Code covered by automated testing: yes
  • Fix verified in Nightly: yes
  • Needs manual QE test: no
  • Steps to reproduce for manual QE testing: Not required
  • Risk associated with taking this patch: low
  • Explanation of risk level: the functionality is available in Nightly/Beta + we ran several experiments
  • String changes made/needed: No
  • Is Android affected?: yes
Status: NEW → RESOLVED
Closed: 27 days ago
Resolution: --- → FIXED
Attachment #9429942 - Flags: approval-mozilla-beta? → approval-mozilla-beta+

I will leave the bug opened until we successfully ship the code to Release.

Status: RESOLVED → REOPENED
Resolution: FIXED → ---
Depends on: 1924667
Target Milestone: 130 Branch → 132 Branch
Status: REOPENED → RESOLVED
Closed: 27 days ago14 days ago
Resolution: --- → FIXED

Release Note Request (optional, but appreciated)
[Why is this notable]:
This feature reduces the size and increasing the speed of the TLS handshake.
[Affects Firefox for Android]: yes
[Suggested wording]: Added support for Certificate Compression TLS Extension.
[Links (documentation, blog post, etc)]:
https://datatracker.ietf.org/doc/html/rfc8879

relnote-firefox: --- → ?
Status: RESOLVED → REOPENED
relnote-firefox: ? → ---
Resolution: FIXED → ---
Target Milestone: 132 Branch → 133 Branch
Status: REOPENED → RESOLVED
Closed: 14 days ago14 days ago
Resolution: --- → FIXED

Release Note Request (optional, but appreciated)
[Why is this notable]:
This feature reduces the size and increasing the speed of the TLS handshake.
[Affects Firefox for Android]: yes
[Suggested wording]: Added support for Certificate Compression TLS Extension.
[Links (documentation, blog post, etc)]:
https://datatracker.ietf.org/doc/html/rfc8879

relnote-firefox: --- → ?

Added to the Fx132 relnotes.

Thanks!

Depends on: 1905910
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: