mirror of
https://github.com/lldap/lldap.git
synced 2026-04-25 08:15:52 +03:00
[PR #177] [MERGED] Introduce a read-only user #592
Labels
No labels
backend
blocked
bug
cleanup
dependencies
docker
documentation
duplicate
enhancement
enhancement
frontend
github_actions
good first issue
help wanted
help wanted
integration
invalid
ldap
pull-request
question
rust
rust
tests
wontfix
No milestone
No project
No assignees
1 participant
Notifications
Due date
No due date set.
Dependencies
No dependencies set.
Reference
starred/lldap-lldap#592
Loading…
Add table
Add a link
Reference in a new issue
No description provided.
Delete branch "%!s()"
Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?
📋 Pull Request Information
Original PR: https://github.com/lldap/lldap/pull/177
Author: @nitnelave
Created: 6/6/2022
Status: ✅ Merged
Merged: 6/6/2022
Merged by: @nitnelave
Base:
main← Head:readonly📝 Commits (1)
667a9e1server: Introduce a read-only user📊 Changes
5 files changed (+161 additions, -82 deletions)
View changed files
📝
server/src/infra/auth_service.rs(+42 -7)📝
server/src/infra/graphql/mutation.rs(+8 -8)📝
server/src/infra/graphql/query.rs(+5 -5)📝
server/src/infra/ldap_handler.rs(+92 -61)📝
server/src/main.rs(+14 -1)📄 Description
Fixes #176 .
This PR will create a group "lldap_readonly" on first run. Then the admin can create a user and add it to this group. It'll then be available both for web access (read-only except to update the details of the read-only user) and through LDAP (same, readonly except to change its own password).
🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.