Closed Bug 1638023 Opened 4 years ago Closed 3 years ago

TLS configuration is weak

Categories

(Cloud Services :: Operations: Firefox Profiler, task)

task

Tracking

(Not tracked)

VERIFIED FIXED

People

(Reporter: julienw, Assigned: jbuck)

Details

Our TLS configuration supports TLS 1.0 and TLS 1.1 as well as weak ciphers for TLS 1.2. Here is Mozilla own recommendations about TLS configuration that we need to follow.

This happens on both prod and non prod.

nonprod: ssltest
prod: ssltest

Assignee: edunham → oremj

We aren't able to tweak these in GCP until https://github.com/kubernetes/ingress-gce/issues/246 is fixed. I don't think we need to worry about this, all of our GCP properties have this configuration.

Status: NEW → RESOLVED
Closed: 4 years ago
Resolution: --- → WONTFIX

It looks like it has been fixed upstream: https://github.com/kubernetes/ingress-gce/issues/246#issuecomment-617991794
Jeremy, for the sake of curiosity, do you any idea what the timeline might be for having it enabled on our side? And if I understand your comment correctly, our configuration as it is should already support it when it is available, is that right?

Flags: needinfo?(oremj)

I'm not sure what the timeline will be. Judging from their previous releases, I would guess a few months away. Yeah, our configuration will support it once it is available.

Flags: needinfo?(oremj)

This seems to be available in both regular and stable channels now: https://github.com/kubernetes/ingress-gce/issues/246#issuecomment-711438978

Do you think we can fix this on our server now? As far as I know TLS 1.0/1.1 and the weak ciphers for TLS 1.2 are still present.

Thanks!

Status: RESOLVED → REOPENED
Resolution: WONTFIX → ---

Yes, this is possible to do now. I'll update this to use our standard ssl profile.

Assignee: oremj → jbuckley

This has been deployed to production

Status: REOPENED → RESOLVED
Closed: 4 years ago3 years ago
Resolution: --- → FIXED

I can confirm that all 3 servers (prod, stage, dev) are now A+ according to ssllabs!
Thanks a lot!

Status: RESOLVED → VERIFIED
You need to log in before you can comment on or make changes to this bug.