Gitea: New Field Request: Ability to set Profile Pic with LDAP

[benyanke]

It would be neat to be able to set profile picture with LDAP. I suppose this would be best implemented as a URL, but I know there are multiple ways of implementing images in ldap.

[tcs-ulli]

gogs can this maybe its easy to copy that feature ?

[lunny]

@tcs-ulli could you point where is the code base?

[mq2035]

In openldap attribute name is jpegPhoto (from inetOrgPerson), it is binary and it is stored in LDAP.

working code example (in java)
https://github.com/xwiki-contrib/ldap/tree/master/ldap-authenticator

[dl5rcw]

Using fusiondirectory, the jpegPhoto as mentioned by @mq2035 really makes sense. It should be updated like the other entries to pull changes from ldap.

[helmut72]

Yes, would be good, as I have stored jpegPhoto in my Samba4/LDAP server. Many Applications use the photo from LDAP server, which makes sense.

[lafriks]

Only problem could be with sync, how to check and download images only if they have been changed to not download all user images on every sync

[benyanke]

That would be an additional piece needed. Most apps I've seen do it on individual login, and then refetch on an administrator's cache clear, or when a scheduled job runs. Ideally it's something you could cache a day or two, since pictures don't change often, especially if you're updating on the login event.

[helmut72]

Only problem could be with sync, how to check and download images only if they have been changed to not download all user images on every sync

Let the user decide in his settings.

[lafriks]

I was just thinking if there is more optimal way to check if image has changed

[helmut72]

It would be a good start if there is just a sync every x day/hour setting with 0 to disable. Then it's the administrators choice. Better than no ldap photo sync.