Skip to content

Update sql-server-linux-encrypted-connections.md#6609

Merged
ktoliver merged 2 commits intoMicrosoftDocs:livefrom
MitchellSternke:MitchellSternke-patch-2
Jul 20, 2021
Merged

Update sql-server-linux-encrypted-connections.md#6609
ktoliver merged 2 commits intoMicrosoftDocs:livefrom
MitchellSternke:MitchellSternke-patch-2

Conversation

@MitchellSternke
Copy link
Copy Markdown
Contributor

Add documentation related to recent Linux distribution behavior with regards to OpenSSL certificates.

Users may encounter this issue out-of-the box if they are using their own custom certificates. This makes this issue searchable and documents the solution.

Mentioning @tejasaks / @amvin87 for review.

@PRMerger17
Copy link
Copy Markdown
Contributor

@MitchellSternke : Thanks for your contribution! The author(s) have been notified to review your proposed change.

@ktoliver ktoliver added the aq-pr-triaged tracking label for the PR review team label Jul 19, 2021
@amvin87-zz
Copy link
Copy Markdown
Contributor

#sign-off
@VanMSFT could you please help merge this change to the document.

@ktoliver
Copy link
Copy Markdown
Contributor

ktoliver commented Jul 20, 2021

Hi @amvin87 Before we merge, "note" should be removed from the new H2 heading. We can either change the current H2 heading to "Ubuntu 20.04 and other recent Linux distribution releases," or use that same text string, but make the heading an H3, to demote it in accordance with the meaning of "note" (which is like an aside or subordinate idea). Which do you prefer, or is there another option you'd like to use?

Thanks.

@amvin87-zz
Copy link
Copy Markdown
Contributor

Thanks @ktoliver for the suggestion, let us please go with the latter option, which is : "use that same text string, but make the heading an H3, to demote it in accordance with the meaning of "note""

@PRMerger9 PRMerger9 requested a review from amvin87-zz July 20, 2021 15:59
@PRMerger10
Copy link
Copy Markdown
Contributor

@ktoliver : Thanks for your contribution! The author(s) have been notified to review your proposed change.

@ktoliver
Copy link
Copy Markdown
Contributor

Thanks @amvin87; we'll merge this now.

@ktoliver ktoliver merged commit ee9dfc0 into MicrosoftDocs:live Jul 20, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants