starred/pocket-id-pocket-id-1
1
0
Fork 0
mirror of https://github.com/pocket-id/pocket-id.git synced 2025-12-09 23:02:59 +03:00
Code Issues 122 Packages Projects Releases 10 Wiki Activity

🐛 Bug Report: Docs wrong on LDAP env variables #297

New Issue
Closed
opened 2025-10-08 00:02:00 +03:00 by OVERLORD · 2 comments
OVERLORD commented 2025-10-08 00:02:00 +03:00
Owner
Copy Link

Originally created by @johnnycube on GitHub.

Originally assigned to: @kmendell on GitHub.

Reproduction steps

Hey,

just found a problem in the released documentation: https://pocket-id.org/docs/configuration/environment-variables

  • LDAP_ATTRIBUTE_USER_UNIQUE_ID
  • LDAP_ATTRIBUTE_GROUP_UNIQUE_ID

should instead be

  • LDAP_ATTRIBUTE_USER_UNIQUE_IDENTIFIER
  • LDAP_ATTRIBUTE_GROUP_UNIQUE_IDENTIFIER

as per the backend source code

Thanks for the nice app you are writing!

Expected behavior

It should be in the docs:

  • LDAP_ATTRIBUTE_USER_UNIQUE_IDENTIFIER
  • LDAP_ATTRIBUTE_GROUP_UNIQUE_IDENTIFIER

Actual Behavior

Wrong env flags result in wrong LDAP import

Version and Environment

v0.39.0

Log Output

No response

Originally created by @johnnycube on GitHub. Originally assigned to: @kmendell on GitHub. ### Reproduction steps Hey, just found a problem in the released documentation: https://pocket-id.org/docs/configuration/environment-variables * LDAP_ATTRIBUTE_USER_UNIQUE_ID * LDAP_ATTRIBUTE_GROUP_UNIQUE_ID should instead be * LDAP_ATTRIBUTE_USER_UNIQUE_IDENTIFIER * LDAP_ATTRIBUTE_GROUP_UNIQUE_IDENTIFIER as per the backend source code Thanks for the nice app you are writing! ### Expected behavior It should be in the docs: * LDAP_ATTRIBUTE_USER_UNIQUE_IDENTIFIER * LDAP_ATTRIBUTE_GROUP_UNIQUE_IDENTIFIER ### Actual Behavior Wrong env flags result in wrong LDAP import ### Version and Environment v0.39.0 ### Log Output _No response_
OVERLORD added the bug label 2025-10-08 00:02:00 +03:00
OVERLORD commented 2025-10-08 00:02:03 +03:00
Author
Owner
Copy Link

@johnnycube commented on GitHub:

Love the reaction time ;)

Took a bit of trial and error and source code reading to find it. Thanks for the change. Did not see the website repo - would have sent a pull request

Thanks again for the lovely tool

@johnnycube commented on GitHub: Love the reaction time ;) Took a bit of trial and error and source code reading to find it. Thanks for the change. Did not see the website repo - would have sent a pull request Thanks again for the lovely tool
OVERLORD commented 2025-10-08 00:02:04 +03:00
Author
Owner
Copy Link

@kmendell commented on GitHub:

This should be fixed now :) 25cfb0ba20

@kmendell commented on GitHub: This should be fixed now :) https://github.com/pocket-id/website/commit/25cfb0ba206aa968abd218b3166dd593fad49ccf
Sign in to join this conversation.
No Branch/Tag Specified
Branches Tags
Labels
Clear labels
breaking
bug
documentation
feature
needs more upvotes
open to pull requests
pull-request
Mirrored from GitHub Pull Request
No Label bug
Milestone
No items
No Milestone
Projects
Clear projects
No project
Assignees
Clear assignees
OVERLORD
No Assignees
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: starred/pocket-id-pocket-id-1#297
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?