Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

TLS Enabled by Default (CRDs) #535

Open
1 of 22 tasks
NickLarsenNZ opened this issue Feb 28, 2024 · 0 comments
Open
1 of 22 tasks

TLS Enabled by Default (CRDs) #535

NickLarsenNZ opened this issue Feb 28, 2024 · 0 comments
Labels

Comments

@NickLarsenNZ
Copy link
Member

NickLarsenNZ commented Feb 28, 2024

There is a general consensus within the team that TLS should be enabled by default. This was discussed at the February 2024 on-site.

Issues with the current TLS structure:

  • TLS is disabled by default
    We want it enabled by default, and it will soon become a requirement under the CRA.
    See: Tls struct
  • Confusion
    There is too much confusion around what specifying null means (TLS disabled, or TLS default, which happens to currently be disabled. There shouldn't be so much guesswork, and the suggestion of adding an explicit enabled flag came up (which defaults to true).
    See the comment Cloudflare left on their similar piece of code

Note

I have written the following as if it's all agreed on, but maybe there is still need for discussion.

There will be breaking changes for two reasons:

  • The default will change from TLS off to TLS on (trusting WbPKI by default)
  • The structure will change, at least by adding an enabled flag, leading to other structural changes for it to make sense.

Epic Checklist

General tasks

Product specific tasks

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

1 participant