-
Notifications
You must be signed in to change notification settings - Fork 27
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
[WIP] Better auth #60
Conversation
…' into feat/apps-sql
@vporton apps here is chatflow app. We can not run more than one at the time |
@apssouza22 A chatflow app is any API that user documented, right? Or what is it? Why can't we run more than one at a time? |
Yes, but os docs will be linked to one app. It is just one way of organizing things. One user can have multiple apps and one app can have multiple docs |
@apssouza22 What are "os docs"? You didn't answer why we need to store the app in session data, not instead decide the app by the AI at each step (not the entire session) separately. |
@vporton because u need to search only docs for the the given app. Don't make a big deal of it. It is just app id in the session to tell the backend where to retrieve docs from |
A good behavior of the app is to expire authentication after a certain time interval after the last operation that the user did. Because expiration time is stored on the client side, to implement this delayed expiration, we need to re-download JWT token on each request (not only on login/signup requests). I propose either:
@apssouza22 I need your principal agreement for either 1 or 2. I prefer 2. |
Let's stick with the standard. If the token is expired the user need to loging again. |
This is a work in progress to solve issue #10. This PR is dependent on PR #56.