Skip to content

correction, Fixes AB#3525780#2909

Open
rpdome wants to merge 1 commit intodevfrom
rapong/correction
Open

correction, Fixes AB#3525780#2909
rpdome wants to merge 1 commit intodevfrom
rapong/correction

Conversation

@rpdome
Copy link
Member

@rpdome rpdome commented Feb 24, 2026

@rpdome rpdome requested a review from a team as a code owner February 24, 2026 17:31
Copilot AI review requested due to automatic review settings February 24, 2026 17:31
@rpdome rpdome added the No-Changelog This Pull-Request has no associated changelog entry. label Feb 24, 2026
@github-actions
Copy link

✅ Work item link check complete. Description contains link AB#3525780 to an Azure Boards work item.

@github-actions
Copy link

❌ Work item link check failed. Description contains AB#3525780 but the Bot could not link it to an Azure Boards work item.

Click here to learn more.

@github-actions
Copy link

✅ Work item link check complete. Description contains link AB#3525780 to an Azure Boards work item.

1 similar comment
@github-actions
Copy link

✅ Work item link check complete. Description contains link AB#3525780 to an Azure Boards work item.

@github-actions github-actions bot changed the title correction correction, Fixes AB#3525780 Feb 24, 2026
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Updates documentation in AuthenticationSettings#setSecretKey to more accurately describe the impact of supplying a custom key on previously encrypted data.

Changes:

  • Refines the Javadoc warning about unreadable previously encrypted content when a user-provided key is involved.

* this method must be invoked before any other MSAL classes,
* otherwise the key might not be loaded
* (The existing encrypted key will not be readable,
* (The existing data (that was encrypted by the user provided key) will not be readable,
Copy link

Copilot AI Feb 24, 2026

Choose a reason for hiding this comment

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

Doc grammar/clarity: “user provided key” should be “user-provided key”, and the nested parentheses make the sentence harder to read. Consider rewriting as a single parenthetical (e.g., ‘Existing data encrypted with a user‑provided key…’).

Copilot uses AI. Check for mistakes.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

No-Changelog This Pull-Request has no associated changelog entry.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants