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

GitHub Enterprise Template - set storage account's minimum TLS version to TLSv1.2 #14284

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

kyanny
Copy link

@kyanny kyanny commented Jan 6, 2025

PR Checklist

Check these items before submitting a PR...

Contribution Guide

Best Practice Guide

  • - Please check this box once you've submitted the PR if you've read through the Contribution Guide and best practices checklist.

Changelog

  • Add minimumTlsVersion to specify TLS 1.2 as the minimum TLS version of the created storage account

Considering IETF officially deprecates TLS 1.0 and TLS 1.1 (RFC 8996 Deprecating TLS 1.0 and TLS 1.1) and Microsoft Azure will stop supporting TLS 1.0 and 1.1 on 2025-08-31, there is no valid reason to deploy a GitHub Enterprise Server to Azure with storage account that uses TLS 1.0 (default value) as the minimum TLS version.

This pull request adds a property to specify TLS 1.2 as the minimum TLS version of the created storage account.

@azure-quickstarts azure-quickstarts added duplicate foldername folder name is a duplicate metadata violations metadata violations during PR best practices violations labels Jan 6, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
best practices violations duplicate foldername folder name is a duplicate metadata violations metadata violations during PR
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants