[JENKINS-70897] Add support for personal access token authentication#1104
[JENKINS-70897] Add support for personal access token authentication#1104Gagarmel wants to merge 6 commits intojenkinsci:masterfrom
Conversation
MarkEWaite
left a comment
There was a problem hiding this comment.
Thanks very much for the pull request.
Can you describe the testing that you performed to confirm that the change works as expected?
src/main/java/org/jenkinsci/plugins/gitclient/CliGitAPIImpl.java
Outdated
Show resolved
Hide resolved
|
I've tested the changes by creating an organizational folder for project auto discovery. The token added in there as credentials discovers all projects/branches as expected, but the builds weren't able to checkout the repository. Tested it with a Windows build agent and Kubernetes cloud agents. I'm also working on a PR in the git-plugin to pass the credentials in here. Otherwise this change will not work at all. |
jtnord
left a comment
There was a problem hiding this comment.
does this need a corrsponding change for jgit, or is this unsupported there?
if unsupported should this not be called somewhere?
I think that there is an alternate implementation technique that would add support for a standard credential and would work with both JGit and command line git. |
|
Please kindly add these changes. |
|
Hi, what is a status of this PR? Looks like it stuck. Is anything missing? |
Did some investigation and JGit, also does not support the token authentication at all |
JENKINS-70897 - Add support for personal access token authentication
On Bitbucket Datacenter/Cloud and Azure it is required to use personal access tokens instead of username/password authentication.
Resolves jenkinsci/bitbucket-branch-source-plugin#716
Checklist
Put an
xin the boxes that apply. You can also fill these out after creating the PR. If you're unsure about any of them, don't hesitate to ask. This is simply a reminder of what we are going to look for before merging your code.Types of changes
What types of changes does your code introduce? Put an
xin the boxes that apply