github-plugin icon indicating copy to clipboard operation
github-plugin copied to clipboard

WIP [JENKINS-33228] Consolidation of the GitHub Server list

Open stephenc opened this issue 8 years ago • 5 comments

Todo

  • [x] Refactor credentials usage to use authentication tokens
  • [ ] Migrate existing credentials to GitHub OAuth credential type
  • [ ] Change server configuration to enable extension by other plugins that need to record information

This change is Reviewable

stephenc avatar Nov 01 '17 17:11 stephenc

@lanwen could you add a Work In Progress label to this PR please?

stephenc avatar Nov 01 '17 17:11 stephenc

Migrate existing credentials to GitHub OAuth credential type

Reason?

KostyaSha avatar Nov 01 '17 22:11 KostyaSha

Because they should not be SecretText. They are specific to connecting to GitHub. The use of SecretText is against the design principles of the credentials API. Additionally it does not allow for filtering based on permissions granted to the credentials.

Should be transparent when I get it implemented

stephenc avatar Nov 01 '17 22:11 stephenc

I. Additionally it does not allow for filtering based on permissions granted to the credentials.

https://github.com/KostyaSha/github-integration-plugin/blob/master/github-pullrequest-plugin/src/main/java/com/github/kostyasha/github/integration/generic/repoprovider/GitHubPluginRepoProvider.java#L32

Exists for one year. You still reinventing bicycle with github features. Plus we have fully working and integrated with oauth instance based on github-integration-plugin .

The use of SecretText is against the design principles of the credentials API.

DomainRequirements?

KostyaSha avatar Nov 01 '17 22:11 KostyaSha

  1. Did you even look at the linked PR, which is mapping EndPoint to the global config in this plugin.

  2. Domain requirements are for a different set of use cases

Seriously, just wait and see what I have planned, I discussed all this with @lanwen previously and told me he agreed with the plan I have.

stephenc avatar Nov 01 '17 23:11 stephenc

tumbleweed

stephenc avatar Feb 13 '23 15:02 stephenc