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

Credential types #373

Closed
cs278 opened this issue Sep 11, 2015 · 2 comments · Fixed by #531
Closed

Credential types #373

cs278 opened this issue Sep 11, 2015 · 2 comments · Fixed by #531
Labels
enhancement New feature or request

Comments

@cs278
Copy link
Contributor

cs278 commented Sep 11, 2015

I'm trying to configure the GitHub plugin using Puppet as part of that I need to provide a credential for the API token which requires a 'secret text' credential, currently this is not possible using this module.

Would it not be better to add a defined type for each kind of credential rather than one cluttered type?

Also jenkins::credentials really should be jenkins::credential, seems there is a PR to address this #311.

@georgemihalcea
Copy link

I am also trying to configure a plain text credential to store a GitHub token in it and there is no such option. It would be useful.

@jhoblitt
Copy link
Member

jhoblitt commented Oct 2, 2015

@cs278 @georgemihalcea I am actively using/contributing to the github-oauth plugin. I have posted a PR that provides two new types jenkins_security_realm/ jenkins_authorization_strategy that do support configuring this plugin. #382 I wouldn't suggest trying it "in production" but it may be worth testing.

@jhoblitt jhoblitt added the needs-feedback Further information is requested label Oct 8, 2015
@jhoblitt jhoblitt added enhancement New feature or request and removed needs-feedback Further information is requested labels Jan 20, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants