starred/pocket-id
1
0
Fork 0
mirror of https://github.com/pocket-id/pocket-id.git synced 2026-03-01 03:17:28 +03:00
Code Issues 61 Packages Projects Releases 11 Wiki Activity

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

New Issue
Closed
opened 2026-02-04 17:59:55 +03:00 by OVERLORD · 2 comments
OVERLORD commented 2026-02-04 17:59:55 +03:00
Owner
Copy Link

Originally created by @johnnycube on GitHub (Mar 15, 2025).

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 (Mar 15, 2025). 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 2026-02-04 17:59:55 +03:00
OVERLORD commented 2026-02-04 17:59:59 +03:00
Author
Owner
Copy Link

@kmendell commented on GitHub (Mar 15, 2025):

This should be fixed now :) 25cfb0ba20

@kmendell commented on GitHub (Mar 15, 2025): This should be fixed now :) https://github.com/pocket-id/website/commit/25cfb0ba206aa968abd218b3166dd593fad49ccf
OVERLORD commented 2026-02-04 18:00:07 +03:00
Author
Owner
Copy Link

@johnnycube commented on GitHub (Mar 15, 2025):

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 (Mar 15, 2025): 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
Sign in to join this conversation.
No Branch/Tag Specified
Branches Tags
Labels
Clear labels
breaking
bug
bug
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#205
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?