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

HOT FIX - Fixes Elements by reverting bug in #1441 #1474

Merged
merged 1 commit into from
Oct 25, 2024

Conversation

hayescode
Copy link
Contributor

@hayescode hayescode commented Oct 23, 2024

The project/file/{file_id} endpoint always returns 401 HTTP error. This prevent any Element from being displayed. Fixes #1472

Important

Users should not upgrade to versions 1.3.0 or 2.0.dev1

This comments out the problematic code and adds TODOs. This reopens #1101 and #1438 .

@dosubot dosubot bot added the size:S This PR changes 10-29 lines, ignoring generated files. label Oct 23, 2024
@dosubot dosubot bot added backend Pertains to the Python backend. bug Something isn't working labels Oct 23, 2024
Copy link
Collaborator

@dokterbob dokterbob left a comment

Choose a reason for hiding this comment

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

Solving the issue properly requires moving to HTTP only cookies, which is out of scope for now (we want to properly clean up auth). We're gonna ship this ASAP.

Thanks for the contrib!

@dokterbob dokterbob merged commit 49dd1a5 into Chainlit:main Oct 25, 2024
17 checks passed
dokterbob pushed a commit that referenced this pull request Oct 25, 2024
Solving the underlying issue properly requires moving to HTTP only cookies, which is out of scope for now (we want to properly clean up auth). We're gonna ship this ASAP.

This reopens #1101 and #1438 .
dokterbob added a commit that referenced this pull request Oct 25, 2024
* Update server.py (#1474)

Solving the underlying issue properly requires moving to HTTP only cookies, which is out of scope for now (we want to properly clean up auth). We're gonna ship this ASAP.

This reopens #1101 and #1438 .

* Changelog for 1.3.1 and 2.0.dev2.

* Bump version to 1.3.1.

---------

Co-authored-by: Josh Hayes <[email protected]>
@hayescode hayescode deleted the patch-3 branch October 25, 2024 15:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backend Pertains to the Python backend. bug Something isn't working size:S This PR changes 10-29 lines, ignoring generated files.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Elements don't display when using authentication in 1.3.0 & 2.0.dev1
2 participants