Skip to content

Commit

Permalink
Fix Azure authentication (#1117)
Browse files Browse the repository at this point in the history
The callback path for authentication in Azure AD, and possibly others, was
unintentionally broken in 8ecd415 by modifying the user-facing path.

This commit reverts one line of that change to restore it to its previous
state.
  • Loading branch information
maciejwie authored Jul 5, 2024
1 parent 7dbdbc5 commit add7238
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion backend/chainlit/server.py
Original file line number Diff line number Diff line change
Expand Up @@ -332,7 +332,7 @@ def get_user_facing_url(url: URL):
if config_url.path.endswith("/"):
config_url = config_url.replace(path=config_url.path[:-1])

return config_url.__str__()
return config_url.__str__() + url.path


@router.get("/auth/config")
Expand Down

1 comment on commit add7238

@OmidS
Copy link

@OmidS OmidS commented on add7238 Jul 15, 2024

Choose a reason for hiding this comment

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

Thanks! This fix not being in the latest stable release took me hours to debug...

Please sign in to comment.