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

Remove the check for 52 chars in the PAT to facilitate yamlizr working in a pipeline. #181

Open
jshield opened this issue Feb 2, 2023 · 0 comments
Labels
enhancement New feature or request help wanted Extra attention is needed

Comments

@jshield
Copy link

jshield commented Feb 2, 2023

Azure DevOps agents do not use PATs to communicate with Azure DevOps, instead they use an access token issued by an oath2 token endpoint, as such this check leads to yamlizr running in a pipeline to fail.

our use case is to monitor the countless legacy pipelines running in our organization and determine porting efforts when they get updated.

if (string.IsNullOrWhiteSpace(PAT) || PAT.Trim().Length != 52)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request help wanted Extra attention is needed
Projects
None yet
Development

No branches or pull requests

2 participants