[PR #284] [MERGED] server: Add support for users' avatars in GrahpQL #661

New issue

Closed

opened 2026-02-27 08:18:22 +03:00 by kerem · 0 comments

kerem commented

2026-02-27 08:18:22 +03:00

Owner

📋 Pull Request Information

Original PR: https://github.com/lldap/lldap/pull/284
Author: @nitnelave
Created: 8/3/2022
Status: ✅ Merged
Merged: 8/9/2022
Merged by: @nitnelave

Base: main ← Head: jpeg

📝 Commits (8)

f29c888 app: Switch yew_form dependency back to main repo
61eb71e server: Add support for users' avatars in GrahpQL
f3f27be server: Change attribute values to bytes
d7f1907 ldap: return user's avatar
b148fea ldap: Stop returning empty attributes
be6ba02 app: Add support for modifying an avatar
4bbabe3 migration-tool: Import users' avatars
9e14af5 migration-tool: misc cleanup

📊 Changes

19 files changed (+558 additions, -180 deletions)

View changed files

📝 Cargo.lock (+66 -9)
📝 Cargo.toml (+4 -8)
📝 app/Cargo.toml (+15 -2)
📝 app/queries/get_user_details.graphql (+1 -0)
📝 app/src/components/create_user.rs (+1 -0)
📝 app/src/components/user_details.rs (+1 -2)
📝 app/src/components/user_details_form.rs (+159 -20)
📝 app/src/infra/common_component.rs (+43 -6)
📝 migration-tool/Cargo.toml (+1 -0)
📝 migration-tool/src/ldap.rs (+22 -19)
📝 migration-tool/src/lldap.rs (+4 -12)
📝 schema.graphql (+3 -0)
📝 server/Cargo.toml (+9 -3)
📝 server/src/domain/handler.rs (+79 -1)
📝 server/src/domain/sql_backend_handler.rs (+5 -0)
📝 server/src/infra/graphql/mutation.rs (+25 -1)
📝 server/src/infra/graphql/query.rs (+4 -0)
📝 server/src/infra/ldap_handler.rs (+115 -96)
📝 server/src/infra/ldap_server.rs (+1 -1)

📄 Description

Does most of the heavy lifting required for #171 . With this, users can update their avatar in LLDAP and it is served in LDAP as jpegPhoto attribute.

Note that this only supports JPEG, not PNG.

This does not include support for the migration tool.

_{🔄 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/lldap/lldap/pull/284 **Author:** [@nitnelave](https://github.com/nitnelave) **Created:** 8/3/2022 **Status:** ✅ Merged **Merged:** 8/9/2022 **Merged by:** [@nitnelave](https://github.com/nitnelave) **Base:** `main` ← **Head:** `jpeg` --- ### 📝 Commits (8) - [`f29c888`](https://github.com/lldap/lldap/commit/f29c88842ac877c304851e06b2ccb5535d99e2a9) app: Switch yew_form dependency back to main repo - [`61eb71e`](https://github.com/lldap/lldap/commit/61eb71ebd3377bd33fb7249a92ebf0743e1cd2c9) server: Add support for users' avatars in GrahpQL - [`f3f27be`](https://github.com/lldap/lldap/commit/f3f27bedbf61a2f49a4d41c04ff96232048d3e60) server: Change attribute values to bytes - [`d7f1907`](https://github.com/lldap/lldap/commit/d7f19071620dfb9adb3413d57518b76600e87b3d) ldap: return user's avatar - [`b148fea`](https://github.com/lldap/lldap/commit/b148fea008f6302467eb01791881c8473e894a8d) ldap: Stop returning empty attributes - [`be6ba02`](https://github.com/lldap/lldap/commit/be6ba020c8dfa1d78e20852efcfe0233ac5e034a) app: Add support for modifying an avatar - [`4bbabe3`](https://github.com/lldap/lldap/commit/4bbabe3c0b90cd5f286fe6ab94eb1d535fea5aa4) migration-tool: Import users' avatars - [`9e14af5`](https://github.com/lldap/lldap/commit/9e14af56d285139eb422964c80c84ca696b427b2) migration-tool: misc cleanup ### 📊 Changes **19 files changed** (+558 additions, -180 deletions) <details> <summary>View changed files</summary> 📝 `Cargo.lock` (+66 -9) 📝 `Cargo.toml` (+4 -8) 📝 `app/Cargo.toml` (+15 -2) 📝 `app/queries/get_user_details.graphql` (+1 -0) 📝 `app/src/components/create_user.rs` (+1 -0) 📝 `app/src/components/user_details.rs` (+1 -2) 📝 `app/src/components/user_details_form.rs` (+159 -20) 📝 `app/src/infra/common_component.rs` (+43 -6) 📝 `migration-tool/Cargo.toml` (+1 -0) 📝 `migration-tool/src/ldap.rs` (+22 -19) 📝 `migration-tool/src/lldap.rs` (+4 -12) 📝 `schema.graphql` (+3 -0) 📝 `server/Cargo.toml` (+9 -3) 📝 `server/src/domain/handler.rs` (+79 -1) 📝 `server/src/domain/sql_backend_handler.rs` (+5 -0) 📝 `server/src/infra/graphql/mutation.rs` (+25 -1) 📝 `server/src/infra/graphql/query.rs` (+4 -0) 📝 `server/src/infra/ldap_handler.rs` (+115 -96) 📝 `server/src/infra/ldap_server.rs` (+1 -1) </details> ### 📄 Description Does most of the heavy lifting required for #171 . With this, users can update their avatar in LLDAP and it is served in LDAP as `jpegPhoto` attribute. Note that this only supports JPEG, not PNG. This does not include support for the migration tool. --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>