Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Fallback to root when user's home directory is not accessible #47524

Open
wants to merge 5 commits into
base: master
Choose a base branch
from
Open

Fallback to root when user's home directory is not accessible #47524

wants to merge 5 commits into from
+216 −30

Commits on Oct 11, 2024

  1. check user home dir perms

    @rosstimothy
    rosstimothy committed Oct 11, 2024
    Configuration menu
    Copy the full SHA
    ed24597 View commit details
    Browse the repository at this point in the history

  2. refactoring out some shared code and adding test coverage

    @eriktate
    eriktate committed Oct 11, 2024
    Configuration menu
    Copy the full SHA
    f23e7c3 View commit details
    Browse the repository at this point in the history

Commits on Oct 16, 2024

  1. ensuring access check is handled correctly based on the current user … 

    …and using a call to os.Chdir since os.Stat doesn't necessarily tell us what we need to know
    @eriktate
    eriktate committed Oct 16, 2024
    Configuration menu
    Copy the full SHA
    a3e2b84 View commit details
    Browse the repository at this point in the history

  2. fixing broken unit test

    @eriktate
    eriktate committed Oct 16, 2024
    Configuration menu
    Copy the full SHA
    66351ed View commit details
    Browse the repository at this point in the history

  3. fixing TestOSCommandPrep test by using a test user other than root 

    because of mismatched HomeDir values when calling user.Current()
    @eriktate
    eriktate committed Oct 16, 2024
    Configuration menu
    Copy the full SHA
    865d79f View commit details
    Browse the repository at this point in the history