avatar not display correct in latest image 0.7.5-rc2

[elvis-cai]

What happened?

the image is not displayed correctly after login with okta. Let me know if I am missing anything, thx.

Steps to Reproduce

deploy with image: ghcr.io/danny-avila/librechat:v0.7.5-rc2, login with sso and check user avatar.

What browsers are you seeing the problem on?

No response

Relevant log output

No response

Screenshots

Code of Conduct

• I agree to follow this project's Code of Conduct

[danny-avila]

likely your OIDC provider doesn't return an image or rather it doesn't return an avatar in the expected userinfo field (currently only userinfo.picture, not unique to Entra but not universally implemented across all OIDC providers)

[elvis-cai]

Thanks @danny-avila for reproducing the issue, interesting, I found my profile image is not rendered correctly either, it works perfectly in 0.7.4

[elvis-cai]

0.7.4

[danny-avila]

do you mind sharing the user object of your account? you can get this from the network tab in dev tools, filter by "user" and refresh:

[elvis-cai]

here you go @danny-avila :)

[elvis-cai]

thanks @sn-smaier 👍 , as you can see from the screenshot avator is "" in my case , cc @danny-avila is there anything we could do to fix this issue? 🙏

[drewdresser]

I'm hitting this with Okta too. Did something related to avatar display or is it a change on the Okta side?

[danny-avila]

File history: https://github.com/danny-avila/LibreChat/commits/3e0f95458f1e1085b1268b7dea2c602d8f902c56/api/strategies/openidStrategy.js

when was your last update?

[danny-avila]

Would be helpful to see this particular log from debug logs, with any sensitive info redacted: [openidStrategy] very login tokenset and userinfo

[sn-smaier]

It's caused by the change from user?.avatar || avatarSrc to user?.avatar ?? avatarSrc
the operator causes this issue if user.avatar in the database is saved as empty string instead of null

[danny-avila]

Thanks for the tip, I'll fix this today

[danny-avila]

Please let me know if this PR fully addresses the issue when merged: #4321

[elvis-cai]

thanks @danny-avila , can confirm it works perfectly for me now 👍