Skip to content

Feature/workflow tool authentication #251

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.

Sign up for GitHub

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

Jump to bottom
Draft
wants to merge 13 commits into
base: develop
Choose a base branch
from
Draft

Feature/workflow tool authentication #251

wants to merge 13 commits into from

Conversation

ericevans-nv
Copy link
Contributor

Description

Closes

By Submitting this PR I confirm:

  • I am familiar with the Contributing Guidelines.
  • We require that all contributors "sign-off" on their commits. This certifies that the contribution is your original work, or you have rights to submit it under the same license, or a compatible license.
    • Any contribution which contains commits that are not Signed-Off will not be accepted.
  • When the PR is ready for review, new or existing tests cover these changes.
  • When the PR is ready for review, the documentation is up to date with these changes.

@ericevans-nv
Convert Auth config to model and add stubs for client.
df92264
@ericevans-nv
Adding API request utility stubs.
e15761b
@ericevans-nv
Updating credential persistence, adding plumbing to build request.
285667b 
Signed-off-by: Eric Evans <194135482+ericevans-nv@users.noreply.github.com>
@ericevans-nv
Updating OAuth2.0 stubs as well as auth server.
dc17075 
Signed-off-by: Eric Evans <194135482+ericevans-nv@users.noreply.github.com>
@ericevans-nv
Completed Happy Path OAuth2.0 Code Flow
320b3f5 
Signed-off-by: Eric Evans <194135482+ericevans-nv@users.noreply.github.com>
@ericevans-nv
Happy path OAuth2.0 authorization clean up.
bc7be28 
Signed-off-by: Eric Evans <194135482+ericevans-nv@users.noreply.github.com>
@ericevans-nv
Merge branch 'develop' of github.com:NVIDIA/AIQToolkit into feature/w…
ddda33b 
…orkflow-tool-authentication
@ericevans-nv
Happy path OAuth2.0 authorization clean up after merge conflicts.
7d63b1e 
Signed-off-by: Eric Evans <194135482+ericevans-nv@users.noreply.github.com>
@ericevans-nv
Completed Happy Path authenticated api call.
b1c9a12 
Signed-off-by: Eric Evans <194135482+ericevans-nv@users.noreply.github.com>
@ericevans-nv
Refresh Token testing.
5dc1a26 
Signed-off-by: Eric Evans <194135482+ericevans-nv@users.noreply.github.com>
@ericevans-nv ericevans-nv requested a review from mdemoret-nv May 13, 2025 17:49
@ericevans-nv ericevans-nv self-assigned this May 13, 2025
@ericevans-nv ericevans-nv added the DO NOT MERGE PR should not be merged; see PR for details label May 13, 2025
@ericevans-nv ericevans-nv marked this pull request as draft May 13, 2025 17:50
@ericevans-nv
Completed handling refresh token and OAuth2.0 credential validation l…
406c93c 
…ogic.

Signed-off-by: Eric Evans <194135482+ericevans-nv@users.noreply.github.com>
@ericevans-nv
Merge branch 'develop' of github.com:NVIDIA/AIQToolkit into feature/w…
531854a 
…orkflow-tool-authentication
@ericevans-nv
Completed Happy Path consent prompt handling from frontend.
9f429fc 
Signed-off-by: Eric Evans <194135482+ericevans-nv@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@mdemoret-nv mdemoret-nv Awaiting requested review from mdemoret-nv

At least 1 approving review is required to merge this pull request.

Assignees

@ericevans-nv ericevans-nv

Labels
DO NOT MERGE PR should not be merged; see PR for details
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@ericevans-nv