Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Request, Depends dependency fix #1449

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

sahilk0108
Copy link

fixed #1446

Description

Removed Request type hint from get_current_user in auth.py

Related Issues

#1446
fastapi/fastapi#9897

Solution and Design

#1426
fastapi/fastapi#9897

Test Plan

Run docker compose -f docker-compose.yaml up backend
Confirm Application startup complete. message showing up.

Type of change

  • [✔ ] Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)
  • Docs update

Checklist

  • [✔ ] My pull request is atomic and focuses on a single change.
  • [✔ ] I have read the contributing guide and my code conforms to the guidelines.
  • [ ✔] I have documented my changes clearly and comprehensively.
  • [✔ ] I have added the required tests.

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

superAGI-backend throws error in logs
2 participants