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.

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

Fix unexpected keyword argument 'crypt_id' on FreeBSD 14.1 #1955

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

Commits on Aug 15, 2024

  1. fix(distroutils): add **kwargs to FreeBSDDistro.chpasswd()

    When operating FreeBSD, the argument crypt_id is passed to an
    overridden method chpasswd() in the FreeBSDDistro class. The definition
    of this method does not contain an argument for 'crypt_id' or
    'salt_len'. To address this issue, add **kwargs to the definition of
    chpasswd() in the FreeBSDDistro class.
    kro-cat committed Aug 15, 2024
    Configuration menu
    Copy the full SHA
    f6b9f35 View commit details
    Browse the repository at this point in the history
  2. fix(distroutils): encode cmd_input argument for ext_utils.run_send_st…

    …din()
    
    The method ext_utils.run_send_stdin() (in extensionutils.py) requires
    the second argument, cmd_input, to be a bytes object, not a string
    object. This is due to the usage of: subprocess.Popen().communicate().
    Encode the password as utf-8 when passing to ext_utils.run_send_stdin().
    kro-cat committed Aug 15, 2024
    Configuration menu
    Copy the full SHA
    b22c688 View commit details
    Browse the repository at this point in the history

Commits on Aug 23, 2024

  1. fix(distroutils): use /etc/pw.conf to discern accounting information.

    /etc/user.defs is the shadow configuration file. FreeBSD user management
    uses the pw utility.
    kro-cat committed Aug 23, 2024
    Configuration menu
    Copy the full SHA
    245cc50 View commit details
    Browse the repository at this point in the history