Skip to content

chore: docker compose runs without auth#1250

Open
inimaz wants to merge 2 commits into
masterfrom
chore/docker-setup-noauth
Open

chore: docker compose runs without auth#1250
inimaz wants to merge 2 commits into
masterfrom
chore/docker-setup-noauth

Conversation

@inimaz

@inimaz inimaz commented Jun 21, 2026

Copy link
Copy Markdown
Collaborator

Description

So far it was hard to test in local the webapp because of the authentication mechanism. The goal of this is to provide a simple way to deploy in local the server without auth. This makes it simple for devs to test the frontend + backend + db in one shot.

Try it

docker compose up -d --build

Then go to localhost:3000 and you should be able to test everything with just the auth mocked.

AI Usage Disclosure

Please refer to docs/how-to/ai-policy.md for detailed guidelines on how to disclose AI usage in your PR. Accurately completing this section is mandatory.

  • 🟥 AI-vibecoded: You cannot explain the logic. Car analogy : the car drive by itself, you are outside it and just tell it where to go.
  • 🟠 AI-generated: Car analogy : the car drive by itself, you are inside and give instructions.
  • ⭐ AI-assisted. Car analogy : you drive the car, AI help you find your way.
  • ♻️ No AI used. Car analogy : you drive the car.

@inimaz inimaz requested a review from a team as a code owner June 21, 2026 10:44

@SaboniAmine SaboniAmine left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks Inigo! This will be very useful!!

"""
if auth_provider is None:
raise HTTPException(status_code=501, detail="Authentication not configured")
return RedirectResponse(get_redirect_url(request))

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

CAN it be computed at startup of server?
Redirect url shouldn't change during app lifetime

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Good catch! We don't even need to read the redirect from the request, changing it here da71fbd

"""
if auth_provider is None:
raise HTTPException(status_code=501, detail="Authentication not configured")
return RedirectResponse(get_redirect_url(request))

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Same

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.

2 participants