Skip to content

Update Get-AzAccessToken to use -AsSecureString#947

Merged
flanakin merged 1 commit into
devfrom
flanakin/dev/ps-accesstoken
Sep 1, 2024
Merged

Update Get-AzAccessToken to use -AsSecureString#947
flanakin merged 1 commit into
devfrom
flanakin/dev/ps-accesstoken

Conversation

@flanakin

@flanakin flanakin commented Sep 1, 2024

Copy link
Copy Markdown
Collaborator

🛠️ Description

Update Get-AzAccessToken to use -AsSecureString

Fixes #946

📋 Checklist

🔬 How did you test this change?

  • 🤏 Lint tests
  • 🤞 PS -WhatIf / az validate
  • 👍 Manually deployed + verified
  • 💪 Unit tests
  • 🙌 Integration tests

🙋‍♀️ Do any of the following that apply?

  • 🚨 This is a breaking change.
  • 🤏 The change is less than 20 lines of code.

📑 Did you update docs/changelog.md?

  • ✅ Updated changelog (required for dev PRs)
  • ➡️ Will add log in a future PR (feature branch PRs only)
  • ❎ Log not needed (small/internal change)

📖 Did you update documentation?

  • ✅ Public docs in docs (required for dev)
  • ✅ Internal dev docs in src (required for dev)
  • ➡️ Will add docs in a future PR (feature branch PRs only)
  • ❎ Docs not needed (small/internal change)

@flanakin flanakin added this to the 2024-08 - August milestone Sep 1, 2024
@flanakin flanakin marked this pull request as ready for review September 1, 2024 02:03
@flanakin flanakin requested a review from aromano2 as a code owner September 1, 2024 02:03
@flanakin flanakin requested a review from a team September 1, 2024 02:03
@microsoft-github-policy-service microsoft-github-policy-service Bot added Micro PR 🔬 Very small PR that should be especially easy for newcomers Needs: Review 👀 PR that is ready to be reviewed labels Sep 1, 2024
@flanakin flanakin merged commit 949d26e into dev Sep 1, 2024
@flanakin flanakin deleted the flanakin/dev/ps-accesstoken branch September 1, 2024 21:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Micro PR 🔬 Very small PR that should be especially easy for newcomers Needs: Review 👀 PR that is ready to be reviewed

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[PowerShell] Get-AzAccessToken secure string warning

2 participants