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

Add GetTokens, GetTokenById, and RevokeToken functions #1

Open
wants to merge 2 commits into
base: create-token-params
Choose a base branch
from

Conversation

Smirl
Copy link
Owner

@Smirl Smirl commented May 12, 2023

Summary

NOTE - draft until create-token-params PR is merged jfrog#759

  • Following the API reference here https://jfrog.com/help/r/jfrog-rest-apis/access-tokens
  • added GetTokens (with optional filters)
  • added GetTokenById
  • added RevokeToken
  • moved to use a url.URL instead of a string for the URL so it is easier to work with. Helper method accessTokenURL
    • perhaps this should become a standard across the entire codebase (but that would be a bunch of work)

@Smirl Smirl force-pushed the more-token-commands branch from efc6015 to 0d7e947 Compare June 1, 2023 10:11
@Smirl Smirl force-pushed the more-token-commands branch from 0d7e947 to bd72936 Compare June 1, 2023 10:13
@Smirl Smirl force-pushed the create-token-params branch from 74e57a9 to deead89 Compare June 1, 2023 10:13
@Smirl Smirl force-pushed the create-token-params branch from deead89 to 9071933 Compare June 13, 2023 11:06
@Smirl Smirl force-pushed the create-token-params branch from 9071933 to 0893790 Compare September 4, 2023 21:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant