Skip to content

Conversation

@dezhishen
Copy link
Member

@dezhishen dezhishen commented Aug 12, 2025

@xrgzs xrgzs requested a review from Copilot August 12, 2025 03:09

This comment was marked as outdated.

dezhishen and others added 5 commits August 12, 2025 11:18
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Signed-off-by: 我怎么就不是一只猫呢? <26274059+dezhishen@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Signed-off-by: 我怎么就不是一只猫呢? <26274059+dezhishen@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Signed-off-by: 我怎么就不是一只猫呢? <26274059+dezhishen@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Signed-off-by: 我怎么就不是一只猫呢? <26274059+dezhishen@users.noreply.github.com>
@dezhishen dezhishen requested a review from Copilot August 12, 2025 03:26

This comment was marked as outdated.

dezhishen and others added 4 commits August 12, 2025 19:27
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Signed-off-by: 我怎么就不是一只猫呢? <26274059+dezhishen@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Signed-off-by: 我怎么就不是一只猫呢? <26274059+dezhishen@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Signed-off-by: 我怎么就不是一只猫呢? <26274059+dezhishen@users.noreply.github.com>
@dezhishen dezhishen requested a review from Copilot August 12, 2025 11:55

This comment was marked as outdated.

@dezhishen dezhishen changed the title doc(docker): Add instructions on the use of --user in Docker doc(docker): Add instructions on docker about user in container Aug 12, 2025
@dezhishen dezhishen requested a review from Copilot August 12, 2025 12:08

This comment was marked as outdated.

dezhishen and others added 2 commits August 12, 2025 20:16
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
@dezhishen dezhishen requested a review from Copilot August 12, 2025 12:18

This comment was marked as outdated.

Signed-off-by: MadDogOwner <xiaoran@xrgzs.top>
@dezhishen dezhishen requested a review from Copilot August 12, 2025 12:52

This comment was marked as outdated.

dezhishen and others added 2 commits August 12, 2025 20:56
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Signed-off-by: 我怎么就不是一只猫呢? <26274059+dezhishen@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Signed-off-by: 我怎么就不是一只猫呢? <26274059+dezhishen@users.noreply.github.com>
@dezhishen dezhishen requested a review from Copilot August 12, 2025 12:56
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request Overview

This PR adds comprehensive documentation about user and permission handling in Docker for OpenList, specifically addressing changes introduced in version v4.1.0 and later where PUID/PGID environment variables were deprecated in favor of a built-in openlist user (UID 1001).

  • Documents the transition from PUID/PGID to built-in user management in v4.1.0+
  • Provides separate installation and upgrade instructions for different version ranges
  • Adds troubleshooting information for permission-related Docker errors

Reviewed Changes

Copilot reviewed 2 out of 3 changed files in this pull request and generated 3 comments.

File Description
pages/guide/installation/docker.md Adds extensive documentation about user management changes, installation commands for different versions, and updated docker-compose examples
pages/faq/error.md Adds FAQ entry for permission denied errors with solutions using --user parameter or chown commands

@dezhishen dezhishen requested a review from xrgzs August 13, 2025 01:11
@dezhishen dezhishen merged commit 6257ca7 into OpenListTeam:main Aug 13, 2025
@dezhishen dezhishen deleted the docker branch August 13, 2025 02:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants