Skip to content
This repository was archived by the owner on Sep 17, 2026. It is now read-only.

Update artifactory.py - #136

Open
d3rnn wants to merge 7 commits into
IBM:masterfrom
d3rnn:master
Open

d3rnn wants to merge 7 commits into
IBM:masterfrom
d3rnn:master

Conversation

@d3rnn

@d3rnn d3rnn commented Sep 1, 2023

Copy link
Copy Markdown

support for Artifactory identity tokens. This one doesnt work #115

All Artifactory identity tokens are 64 characters long and always start with cmVmdGtu. I tested it against a few dozen Artifactory identity tokens with success.

support for artifactory identity tokens

Signed-off-by: d3rn <84808889+d3rnn@users.noreply.github.com>
@rossgrady

Copy link
Copy Markdown
Member

Can you also update the test coverage for that plugin? https://github.com/IBM/detect-secrets/blob/master/tests/plugins/artifactory_test.py

d3rnn added 6 commits October 4, 2023 13:23
add `ARTIFACTORY_IDENTITY_TOKEN` test case

Signed-off-by: d3rn <84808889+d3rnn@users.noreply.github.com>
Signed-off-by: d3rn <84808889+d3rnn@users.noreply.github.com>
Signed-off-by: d3rn <84808889+d3rnn@users.noreply.github.com>
Signed-off-by: d3rn <84808889+d3rnn@users.noreply.github.com>
Signed-off-by: d3rn <84808889+d3rnn@users.noreply.github.com>
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants