Skip to content

Add a note about certificate file watching#31588

Merged
amcasey merged 3 commits intodotnet:mainfrom
amcasey:CertWatch
Jan 29, 2024
Merged

Add a note about certificate file watching#31588
amcasey merged 3 commits intodotnet:mainfrom
amcasey:CertWatch

Conversation

@amcasey
Copy link
Copy Markdown
Member

@amcasey amcasey commented Jan 27, 2024

Add a note about certificate file watching in Kestrel.

Fixes #30404


Internal previews

📄 File 🔗 Preview link
aspnetcore/release-notes/aspnetcore-8.0.md What's new in ASP.NET Core 8.0

@amcasey
Copy link
Copy Markdown
Member Author

amcasey commented Jan 27, 2024

I don't think we usually document app context switches, since they are inherently transient. Usually, we offer them as workarounds to affected customers (of which there are hopefully few).

@adityamandaleeka
Copy link
Copy Markdown
Member

Yea, I'd suggest dropping the last paragraph. Looks good to me otherwise :)

@amcasey amcasey requested a review from tdykstra January 29, 2024 17:35
@amcasey
Copy link
Copy Markdown
Member Author

amcasey commented Jan 29, 2024

@tdykstra Can you help me with the correct format for the links?

Copy link
Copy Markdown
Contributor

@tdykstra tdykstra left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Links fixed.

Comment thread aspnetcore/release-notes/aspnetcore-8.0.md Outdated
Co-authored-by: Tom Dykstra <tdykstra@microsoft.com>
@amcasey
Copy link
Copy Markdown
Member Author

amcasey commented Jan 29, 2024

Links fixed.

Thanks! Good to go?

@amcasey amcasey merged commit 602c111 into dotnet:main Jan 29, 2024
@amcasey amcasey deleted the CertWatch branch January 29, 2024 21:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Add Kestrel certificate auto rotation support

3 participants