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

Remove AUTH_PROVIDER references #1299

Merged
merged 5 commits into from
Dec 15, 2024
Merged

Conversation

Josh-XT
Copy link
Owner

@Josh-XT Josh-XT commented Dec 15, 2024

This pull request removes the dependency on the magicalauth authentication provider across various parts of the codebase. The most important changes include updates to the verify_api_key function, removal of the AUTH_PROVIDER environment variable, and adjustments in the MagicalAuth class.

Removal of magicalauth dependency:

  • agixt/ApiClient.py: Removed the conditional check for AUTH_PROVIDER in the verify_api_key function.
  • agixt/Globals.py: Removed the AUTH_PROVIDER entry from the default environment variables.
  • start.py: Removed the AUTH_PROVIDER entry from the default environment variables in the get_default_env_vars function.

Changes in MagicalAuth class:

  • agixt/MagicalAuth.py: Removed the conditional check for AUTH_PROVIDER in the verify_api_key function and adjusted the initialization of the encryption_key in the MagicalAuth class. [1] [2] [3] [4]

Docker configuration updates:

  • docker-compose-dev.yml, docker-compose-nostreamlit-dev.yml, docker-compose-nostreamlit.yml, docker-compose.yml: Removed the AUTH_PROVIDER environment variable from the services configuration. [1] [2] [3] [4]

@Josh-XT Josh-XT marked this pull request as ready for review December 15, 2024 12:44
@Josh-XT Josh-XT merged commit 66d45e6 into main Dec 15, 2024
7 checks passed
@Josh-XT Josh-XT deleted the Remove-AUTH_PROVIDER-references branch December 15, 2024 12:44
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