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

[release/8.0] Move comments on generated password entropy to public API (#3729) #3781

Merged
merged 1 commit into from
Apr 18, 2024

Conversation

eerhardt
Copy link
Member

@eerhardt eerhardt commented Apr 17, 2024

Backport of #3729 to release/8.0

Customer Impact

A work item that came from our security review was to document our generated password entropy guarantees. This moves these comments to XML doc comments on public APIs so consumers can understand these guarantees.

Testing

N/A

Risk

Extremely Low. Only changing XML doc comments.

Regression?

No


  • Move comments on generated password entropy to public API

This allows for these comments to be seen by callers of the API so they can make educated decisions on what parameters should be set for the password generation.

Contributes to #237

  • Format the remarks section.
Microsoft Reviewers: Open in CodeFlow

* Move comments on generated password entropy to public API

This allows for these comments to be seen by callers of the API so they can make educated decisions on what parameters should be set for the password generation.

Contributes to dotnet#237

* Format the remarks section.
@dotnet-issue-labeler dotnet-issue-labeler bot added the area-app-model Issues pertaining to the APIs in Aspire.Hosting, e.g. DistributedApplication label Apr 17, 2024
@dotnet-policy-service dotnet-policy-service bot added the Servicing-consider Issue for next servicing release review label Apr 17, 2024
@danmoseley danmoseley added Servicing-approved Approved for servicing release and removed Servicing-consider Issue for next servicing release review labels Apr 18, 2024
@danmoseley danmoseley merged commit 40b3961 into dotnet:release/8.0 Apr 18, 2024
8 checks passed
@eerhardt eerhardt deleted the Port3729 branch April 18, 2024 17:46
@github-actions github-actions bot locked and limited conversation to collaborators May 20, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
area-app-model Issues pertaining to the APIs in Aspire.Hosting, e.g. DistributedApplication Servicing-approved Approved for servicing release
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants