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

Updated checks to new PAT format #20436

Merged
merged 2 commits into from
Sep 20, 2024

Conversation

max-zaytsev
Copy link
Contributor

@max-zaytsev max-zaytsev commented Sep 18, 2024

Copied from #20433

Task name: AppCenterTestV1, AzureTestPlanV0

Description: Azure DevOps has implemented a new PAT format that is no longer 52 characters long but 84 characters long with a fixed signature 'AZDO'. To prepare for the transition between the new and old PAT format, I would like to change the PAT checks. We have already changed our documentation on the new PAT format: https://learn.microsoft.com/en-us/azure/devops/organizations/accounts/use-personal-access-tokens-to-authenticate?view=azure-devops&tabs=Windows#changes-to-format

Documentation changes required: (Y/N) N

Added unit tests: (Y/N) N

Attached related issue: (Y/N) N

Checklist:

  • Task version was bumped - please check instruction how to do it
  • Checked that applied changes work as expected

@martin-toman martin-toman enabled auto-merge (squash) September 20, 2024 08:02
@martin-toman
Copy link
Contributor

/azp run

Copy link

Azure Pipelines successfully started running 3 pipeline(s).

Co-authored-by: Jan Kolarik jankolarik@microsoft.com
Co-authored-by: Jan Kolarik jankolarik@microsoft.com
@max-zaytsev max-zaytsev force-pushed the users/max-zaytsev/update-pat-check branch from ce55efc to 247765b Compare September 20, 2024 12:14
@martin-toman martin-toman merged commit e37b9a7 into master Sep 20, 2024
11 checks passed
@max-zaytsev max-zaytsev deleted the users/max-zaytsev/update-pat-check branch September 20, 2024 13:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants