@leondape thank you for updating the code.

What do you think about instead of using the default url for redirecting have a parameter in the url to trigger this? So we don't need to make too many changes in the code?

This will also solve the infinite redirect loop?

@rubentalstra could you elaborate your idea? I don’t quite get what you mean. :)

@leondape instead of having a default direct to the OIDC? have the parameter so this? so when you use /login it will not auto direct only while doing /login?automatic-redirect=true or something. so you don't need to have a check for the infinite loop check and other things that makes it more complex?

I understand.
I would propose it with a small change: if .env_redirect=true it should do so on /login.
If auth fails at the IDP it should navigate to /login?automatic-redirect=false to disable the infinite loop.
That way we can keep the default url. Otherwise I don't see how I can implement users browsing to /login?automatic-redirect=true.
What do you think?

@rubentalstra I just saw you merged into main?
I did not work on it at all. It probably has the whole "ugly" cookies solution in it. Was that a mistake?

@leondape I did not merged it into main no worries. I'm working on your branch now. almost finished cleaning up some code here.

aaaah I was just to stupid to read! Thank you. Looking forward to your solution.

@leondape 17b0f35

please have a look and update your PR because I could not update it for some reason.

https://github.com/user-attachments/assets/143c252f-46ca-410a-b23d-e52cba04af0c
😏

Vey cool, I see it works? :)
Any action required from my side? I suppose you can just merge from your branch?

Also.. different question: how can you setup the startup overlay message which can be seen in the video?

@leondape if I merge from my branch then I need to close your PR. but I don;t want to take away your time and effort to start it :).

so to keep this PR it would be great if you could just update your branch with the changes from this branch:
https://github.com/danny-avila/LibreChat/compare/feat/oidc-auto-redirect

but if you don't mind then I will close this PR and create a new PR.

regarding the redirect message. this was just something I wanted to show the user with a little slower internet connection. will add a override text to it later as well was just an example.

How forsightful, thank you!
Regarding the message. I meant the overlay stating something about admin dashboard or so?
I was looking for a way to display messages to users once they come to the platform for the first time and eventuelly when there are updates to report or so..

@leondape have a look at this

config/update-banner.js

and

config/delete-banner.js

package.json

    "update-banner": "node config/update-banner.js",
    "delete-banner": "node config/delete-banner.js",

here you can set a banner. but I could not find it documented somewhere.

interesting. Looking forward to the banner release!

interesting. Looking forward to the banner release!

You can already use the banner as described here. I already used it and it works!

#3952

Amazing, but where does this overlay splash screen come from?

@leondape I have updated it a little more: https://github.com/danny-avila/LibreChat/compare/feat/oidc-auto-redirect could you have a look please

I updated it but must say for protocol that I did not have the time to review any of your changes.
But I like the approach much more than mine. If you continue to make changes, please feel free to close my PR and make your own. I don't mind not being credited on this one. But very noble of you acting that way! :)

Amazing, but where does this overlay splash screen come from?

From here. But it's only a Term and Conditions acceptance dialog, not for general info I guess.

@danny-avila
✅ ready for review