-
Notifications
You must be signed in to change notification settings - Fork 16
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
Authentication options #47
Comments
correction to the first proposal if (authOption.authType == AuthorizationType[AuthorizationType.patAuth) { |
sounds good, i will integrate it |
just would like to know whether this has been implemented in the latest release? If not, any plan on this? thank you. |
I've had success with PAT authentication for the local Azure DevOps Server (could work for the service as well). I have used basicAuth with Make sure that the PAT has the "Packaging (Read)" assigned. |
On my company we had some private azure feeds where credentials are deactivated and only PAT's are accepted. Event when its possible add the PAT as password on code try to add the user
if (authOption.authType == AuthorizationType[AuthorizationType.basicAuth]) {
var bytes = utf8.encode(authOption.username + ":" + authOption.password);
var encoded = base64.encode(bytes);
requestOption.headers['Authorization'] = 'Basic ' + encoded;
}
maybe add on more authenthication type like "PAT" can help
if (authOption.authType == AuthorizationType[AuthorizationType.patAuth) {
var bytes = utf8.encode(authOption.password);
var encoded = base64.encode(bytes);
requestOption.headers['Authorization'] = 'Basic ' + encoded;
}
or expected directly the pat in base64
if (authOption.authType == AuthorizationType[AuthorizationType.patAuth) {
requestOption.headers['Authorization'] = 'Basic ' + authOption.password;
}
The text was updated successfully, but these errors were encountered: