Skip to content

API Request with Optional 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.

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

Open
wants to merge 26 commits into
base: develop
Choose a base branch
from

Conversation

ericevans-nv
Copy link
Contributor

@ericevans-nv ericevans-nv commented May 13, 2025

Description

This feature implements the ability to send API requests with optional authentication, handled automatically on the user’s behalf. Users can upload their authentication credentials via the main AIQ configuration file, then call the make_api_request function to send authenticated or non-authenticated requests and handle the appropriate responses. The library builds and sends the HTTP request, managing any necessary authentication using OAuth 2.0 or API keys based on the provided configuration. The corresponding front-end UI consent prompt (aiq-auth) page, which enables users to log into their API provider via delegated access, has already been merged. However, the corresponding pull request can be found below.

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 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 ericevans-nv added feature request New feature or request non-breaking Non-breaking change and removed DO NOT MERGE PR should not be merged; see PR for details labels Jun 3, 2025
@ericevans-nv ericevans-nv changed the title Feature/workflow tool authentication API Request with Optional Authentication Jun 3, 2025
@ericevans-nv ericevans-nv marked this pull request as ready for review June 3, 2025 19:04
@ericevans-nv ericevans-nv force-pushed the feature/workflow-tool-authentication branch 3 times, most recently from 39f0ac9 to 5248f36 Compare June 3, 2025 21:39
Signed-off-by: Eric Evans <[email protected]>
…ebsocket, and console execution modes.

Signed-off-by: Eric Evans <[email protected]>
@ericevans-nv ericevans-nv force-pushed the feature/workflow-tool-authentication branch 2 times, most recently from 4cdc2bb to 3574697 Compare June 4, 2025 01:51
@ericevans-nv ericevans-nv added the DO NOT MERGE PR should not be merged; see PR for details label Jun 4, 2025
@ericevans-nv ericevans-nv force-pushed the feature/workflow-tool-authentication branch 2 times, most recently from d87fe30 to ed23085 Compare June 5, 2025 17:09
…orkflow-tool-authentication

Signed-off-by: Eric Evans <[email protected]>
@ericevans-nv ericevans-nv force-pushed the feature/workflow-tool-authentication branch from ed23085 to 4e9a590 Compare June 5, 2025 18:32
@ericevans-nv ericevans-nv removed the DO NOT MERGE PR should not be merged; see PR for details label Jun 25, 2025
@ericevans-nv ericevans-nv force-pushed the feature/workflow-tool-authentication branch from 816f3e9 to 5cdfc8d Compare June 25, 2025 22:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature request New feature or request non-breaking Non-breaking change
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant