Skip to content

Fix bug #3350 Do not display access token #3352

New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

#

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Merged
merged 3 commits into from
Sep 2, 2023
Merged

Fix bug #3350 Do not display access token #3352

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
3 commits
Select commit Hold shift + click to select a range
8ef716f
Fix bug #3350 Do not display access token
Aug 17, 2023
0cc1d0a
Adding changelog entry
KoenZomers Sep 2, 2023
aa81a42
Merge branch 'dev' into feature/3350
KoenZomers Sep 2, 2023
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
2 changes: 2 additions & 0 deletions CHANGELOG.md
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -39,13 +39,15 @@ The format is based on [Keep a Changelog](http://keepachangelog.com/en/1.0.0/).

- Improved `Set-PnPListItem` cmdlet handling of Purview labels. [#3340](https://github.com/pnp/powershell/pull/3340)
- The `Publish-PnPCompanyApp` cmdlet is now obsolete. It will be removed in the next version. [#3349](https://github.com/pnp/powershell/pull/3349)
- Verbose output will no longer show the access token [#3352](https://github.com/pnp/powershell/pull/3352)

### Removed

- Removed `-DisableListSync` and `-SyncAadB2BManagementPolicy` from `Set-PnPTenant` as the underlying properties have been removed from SharePoint CSOM as well [#3388](https://github.com/pnp/powershell/pull/3388)

### Contributors

- Rodel Pacurib [ryder-cayden]
- [CatSchneider]
- [msjennywu]
- Reshmee Auckloo [reshmee011]
Expand Down
2 changes: 1 addition & 1 deletion src/Commands/Base/TokenHandling.cs
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -81,7 +81,7 @@ internal static string GetAccessToken(Cmdlet cmdlet, string appOnlyDefaultScope,

cmdlet.WriteVerbose($"Acquiring oAuth token for {(requiredScopes.Length != 1 ? requiredScopes.Length + " " : "")}permission scope{(requiredScopes.Length != 1 ? "s" : "")} {string.Join(",", requiredScopes)}");
var accessToken = authManager.GetAccessTokenAsync(requiredScopes).GetAwaiter().GetResult();
cmdlet.WriteVerbose($"Access token acquired: {accessToken}");
cmdlet.WriteVerbose($"Access token acquired");
return accessToken;
}
return null;
Expand Down