aws-secrets-manager-credentials-provider-plugin icon indicating copy to clipboard operation
aws-secrets-manager-credentials-provider-plugin copied to clipboard

Adds support for AWS Credentials

Open chris-h-phillips opened this issue 1 year ago • 4 comments

As suggested here: https://github.com/jenkinsci/aws-secrets-manager-credentials-provider-plugin/issues/237

Adds AwsAccessKeysCredentials which will allow other plugins that need to interact with AmazonWebServicesCredentials instances from the aws-credentials plugin e.g. the amazon-ecr plugin.

Testing done

Added AWSAccessKeysCredentialsIT which follows the testing patterns of the other credential types.

### Submitter checklist
- [x] Make sure you are opening from a **topic/feature/bugfix branch** (right side) and not your main branch!
- [x] Ensure that the pull request title represents the desired changelog entry
- [x] Please describe what you did
- [x] Link to relevant issues in GitHub or Jira
- [x] Link to relevant pull requests, esp. upstream and downstream changes
- [x] Ensure you have provided tests - that demonstrates feature works or fixes the issue

chris-h-phillips avatar Oct 14 '23 00:10 chris-h-phillips

Guys, any update on this?

asaf-the-ancient avatar Feb 01 '24 10:02 asaf-the-ancient

Would be nice to have that

pavriet-boxtal avatar Mar 25 '24 14:03 pavriet-boxtal

Hi @chris-h-phillips, sorry for not getting round to this sooner.

I have (slowly) been trying to solve the problem of how to support vendor-proprietary credential types in the plugin, but without bringing them into the plugin core (where, effectively, they would have to be kept around forever - even if a vendor were to go out of business, or deprecate their proprietary credential type in favour of another - since there is no way to remove things without breaking changes).

To that end I have created #298, which allows the factory for each credential type to be queried and loaded dynamically with the Java ServiceLoader.

If you (or someone else interested in this PR) would be able to rework this PR using #298 as a base, that would be great!

chriskilding avatar Mar 28 '24 23:03 chriskilding

(And once that's done, the next step would be to merge https://github.com/jenkinsci/aws-secrets-manager-credentials-provider-plugin/pull/298, and then create separate add-on plugins for each vendor-proprietary credential type - starting with this one.)

With this architecture people can add the respective vendor-proprietary add-on plugins that they need to their installations.

chriskilding avatar Mar 28 '24 23:03 chriskilding