[PR #2320] [MERGED] Import thumbnail photos when LDAP users are created. #5981

New Issue

OVERLORD · 2026-02-05T10:21:47+03:00

OVERLORD commented

2026-02-05 10:21:47 +03:00

📋 Pull Request Information

Original PR: https://github.com/BookStackApp/BookStack/pull/2320
Author: @jasonhoule
Created: 10/12/2020
Status: ✅ Merged
Merged: 5/24/2021
Merged by: @ssddanbrown

Base: master ← Head: master

📝 Commits (2)

b714652 Import thumbnail photos when LDAP users are created.
a192b60 Missed a variable when updating LdapService.

📊 Changes

3 files changed (+15 additions, -1 deletions)

View changed files

📝 app/Auth/Access/Guards/LdapSessionGuard.php (+11 -1)
📝 app/Auth/Access/LdapService.php (+2 -0)
📝 app/Config/services.php (+2 -0)

📄 Description

My BookStack instance is using Active Directory for authentication. We wanted to be able to automatically import the user photos that we already have stored in Active Directory.

This change, controlled by config values in the config/services.php file, allows for user avatars to be imported from the LDAP thumbnailPhoto attribute. Users can still update their images from the profile page.

_{🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.}

## 📋 Pull Request Information **Original PR:** https://github.com/BookStackApp/BookStack/pull/2320 **Author:** [@jasonhoule](https://github.com/jasonhoule) **Created:** 10/12/2020 **Status:** ✅ Merged **Merged:** 5/24/2021 **Merged by:** [@ssddanbrown](https://github.com/ssddanbrown) **Base:** `master` ← **Head:** `master` --- ### 📝 Commits (2) - [`b714652`](https://github.com/BookStackApp/BookStack/commit/b714652e1020246d8365d9f9c307370168e5b792) Import thumbnail photos when LDAP users are created. - [`a192b60`](https://github.com/BookStackApp/BookStack/commit/a192b600fc818313ef26b139a9a2d11d4d17f0a4) Missed a variable when updating LdapService. ### 📊 Changes **3 files changed** (+15 additions, -1 deletions) <details> <summary>View changed files</summary> 📝 `app/Auth/Access/Guards/LdapSessionGuard.php` (+11 -1) 📝 `app/Auth/Access/LdapService.php` (+2 -0) 📝 `app/Config/services.php` (+2 -0) </details> ### 📄 Description My BookStack instance is using Active Directory for authentication. We wanted to be able to automatically import the user photos that we already have stored in Active Directory. This change, controlled by config values in the config/services.php file, allows for user avatars to be imported from the LDAP thumbnailPhoto attribute. Users can still update their images from the profile page. --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>

OVERLORD added the pull-request label 2026-02-05 10:21:47 +03:00

OVERLORD closed this issue

2026-02-05 10:21:48 +03:00

Sign in to join this conversation.

Branches Tags

1 Participants

Notifications

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: starred/BookStack#5981