Merged
Conversation
Contributor
There was a problem hiding this comment.
Pull request overview
This PR updates documentation files with revised timestamps and corrects environment variable descriptions and default values across the configuration documentation.
Changes:
- Updated timestamp references from 2026-01-28 to 2026-01-29 in multiple documentation files
- Revised environment variable descriptions for clarity and accuracy in
global_configvars.md - Corrected default values for several configuration variables including cache database name, cache TTL, and max concurrency
- Fixed ordering issues in the extended configuration variables documentation
Reviewed changes
Copilot reviewed 4 out of 4 changed files in this pull request and generated 2 comments.
| File | Description |
|---|---|
| docs/_static/env-vars/storage-users_configvars.md | Updated timestamp from 2026-01-28-17-09-55 to 2026-01-29-10-34-56 |
| docs/_static/env-vars/global_configvars.md | Comprehensive updates to environment variable descriptions and default values across multiple configuration options |
| docs/_static/env-vars/frontend_configvars.md | Updated timestamp from 2026-01-28-17-09-55 to 2026-01-29-10-34-56 |
| docs/_static/env-vars/extended_configvars.md | Fixed line ordering for MICRO_LOG_LEVEL and OC_GRPC_MAX_RECEIVED_MESSAGE_SIZE entries |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
| `OC_LDAP_USER_SCHEMA_DISPLAYNAME` | 1.0.0 | string | LDAP Attribute to use for the display name of users. | displayName | | ||
| `OC_LDAP_USER_SCHEMA_ID` | 1.0.0 | string | LDAP Attribute to use as the unique ID for users. This should be a stable globally unique ID like a UUID. | openCloudUUID | | ||
| `OC_LDAP_USER_SCHEMA_ID_IS_OCTETSTRING` | 1.0.0 | bool | Set this to true if the defined 'ID' attribute for users is of the 'OCTETSTRING' syntax. This is required when using the 'objectGUID' attribute of Active Directory for the user ID's. | false | | ||
| `OC_LDAP_USER_SCHEMA_DISPLAYNAME` | 1.0.0 | string | LDAP Attribute to use for the displayname of users. | displayname | |
There was a problem hiding this comment.
The default value 'displayname' appears inconsistent with the LDAP standard attribute 'displayName' (with capital 'N'). Verify this is intentional as LDAP attributes are case-sensitive.
Suggested change
| `OC_LDAP_USER_SCHEMA_DISPLAYNAME` | 1.0.0 | string | LDAP Attribute to use for the displayname of users. | displayname | | |
| `OC_LDAP_USER_SCHEMA_DISPLAYNAME` | 1.0.0 | string | LDAP Attribute to use for the displayname of users. | displayName | |
| `OC_LDAP_USER_SCHEMA_ID` | 1.0.0 | string | LDAP Attribute to use as the unique ID for users. This should be a stable globally unique ID like a UUID. | openCloudUUID | | ||
| `OC_LDAP_USER_SCHEMA_ID_IS_OCTETSTRING` | 1.0.0 | bool | Set this to true if the defined 'ID' attribute for users is of the 'OCTETSTRING' syntax. This is required when using the 'objectGUID' attribute of Active Directory for the user ID's. | false | | ||
| `OC_LDAP_USER_SCHEMA_DISPLAYNAME` | 1.0.0 | string | LDAP Attribute to use for the displayname of users. | displayname | | ||
| `OC_LDAP_USER_SCHEMA_ID` | 1.0.0 | string | LDAP Attribute to use as the unique id for users. This should be a stable globally unique id like a UUID. | openCloudUUID | |
There was a problem hiding this comment.
Inconsistent capitalization of 'id' - should be 'ID' to match the pattern used in line 56 ('id' attribute for groups) and for consistency with the variable name itself which uses 'ID'.
Suggested change
| `OC_LDAP_USER_SCHEMA_ID` | 1.0.0 | string | LDAP Attribute to use as the unique id for users. This should be a stable globally unique id like a UUID. | openCloudUUID | | |
| `OC_LDAP_USER_SCHEMA_ID` | 1.0.0 | string | LDAP Attribute to use as the unique ID for users. This should be a stable globally unique ID like a UUID. | openCloudUUID | |
AlexAndBear
approved these changes
Jan 29, 2026
AlexAndBear
deleted the
docs-update-0189acbb-e989-4a78-8279-16d5725434d5
branch
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
This PR updates the documentation.