PowerShellForGitHub icon indicating copy to clipboard operation
PowerShellForGitHub copied to clipboard

Not forwarding AccessToken to sub-commands for Repo Team Permissions

Open HowardWolosky opened this issue 3 years ago • 0 comments

Description

The Get/Set/Remove-GitHubRepositoryTeamPermission functions will sometimes call Get-GitHubTeam to get missing information. In those instances, they weren't passing along the AccessToken parameter if it had been provided by the user. In those scenarios, the result will be a 404 since the Get-GitHubTeam call won't have the same auth permissions as the core API being called.

Issues Fixed

Fixes #353

Checklist

  • [x] You actually ran the code that you just wrote, especially if you did just "one last quick change".
  • [x] ~~Comment-based help added/updated, including examples.~~
  • [x] Static analysis is reporting back clean.
  • [x] New/changed code adheres to our coding guidelines.
  • [x] ~~Formatters were created for any new types being added.~~
  • [x] ~~New/changed code continues to support the pipeline.~~
  • [x] ~~Changes to the manifest file follow the manifest guidance.~~
  • [x] ~~Unit tests were added/updated and are all passing. See testing guidelines. This includes making sure that all pipeline input variations have been covered.~~
  • [x] ~~Relevant usage examples have been added/updated in USAGE.md.~~
  • [x] ~~If desired, ensure your name is added to our Contributors list~~

HowardWolosky avatar Jul 26 '22 22:07 HowardWolosky