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

[releases/24.x] [OAuth2] Additional SecretText usage on Certificates #841

Merged
merged 3 commits into from
Apr 3, 2024

Conversation

darjoo
Copy link
Contributor

@darjoo darjoo commented Mar 27, 2024

This pull request backports #792 to releases/24.x

Fixes AB#521447

<!-- Thank you for submitting a Pull Request. If you're new to
contributing to BCApps please read our pull request guideline below
* https://github.com/microsoft/BCApps/Contributing.md
-->
#### Summary <!-- Provide a general summary of your changes -->
SecretText usages for Certificate parameters in OAuth2 

#### Work Item(s) <!-- Add the issue number here after the #. The issue
needs to be open and approved. Submitting PRs with no linked issues or
unapproved issues is highly discouraged. -->
Fixes
[AB#520778](https://dynamicssmb2.visualstudio.com/1fcb79e7-ab07-432a-a3c6-6cf5a88ba4a5/_workitems/edit/520778)
@darjoo darjoo requested a review from a team as a code owner March 27, 2024 09:12
@github-actions github-actions bot added this to the Version 24.1 milestone Mar 27, 2024
JesperSchulz
JesperSchulz previously approved these changes Mar 27, 2024
@darjoo darjoo enabled auto-merge (squash) April 3, 2024 07:14
@darjoo darjoo merged commit d54af1d into releases/24.x Apr 3, 2024
23 checks passed
@darjoo darjoo deleted the backport/releases/24.x/792/20240327101239 branch April 3, 2024 12:35
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.

3 participants