[TECH DEBT] support for pysss can be removed #65029
Assignees
Labels
Comments
3 tasks
nicholasmhughes
added a commit
to nicholasmhughes/salt
that referenced
this issue
Aug 29, 2023
3 tasks
felippeb
pushed a commit
to felippeb/salt
that referenced
this issue
Oct 27, 2023
felippeb
pushed a commit
to felippeb/salt
that referenced
this issue
Oct 27, 2023
m-czernek
pushed a commit
to openSUSE/salt
that referenced
this issue
Jul 2, 2024
* fixes saltstack/salt#64953 user.list_groups omits remote groups * fixes saltstack/salt#65029 support for pysss can be removed * add changlog entries * add tests for _getgrall and local vs remote group handling * add negative tests for _getgrall * root can still read the file and tests run as root * remove permission check as its probably an unreachable edge case --------- Co-authored-by: nicholasmhughes <nicholasmhughes@gmail.com>
meaksh
pushed a commit
to openSUSE/salt
that referenced
this issue
Oct 29, 2024
* fixes saltstack/salt#64953 user.list_groups omits remote groups * fixes saltstack/salt#65029 support for pysss can be removed * add changlog entries * add tests for _getgrall and local vs remote group handling * add negative tests for _getgrall * root can still read the file and tests run as root * remove permission check as its probably an unreachable edge case --------- Co-authored-by: nicholasmhughes <nicholasmhughes@gmail.com>
agraul
pushed a commit
to agraul/salt
that referenced
this issue
Jan 27, 2025
* fixes saltstack#64953 user.list_groups omits remote groups * fixes saltstack#65029 support for pysss can be removed * add changlog entries * add tests for _getgrall and local vs remote group handling * add negative tests for _getgrall * root can still read the file and tests run as root * remove permission check as its probably an unreachable edge case --------- Co-authored-by: nicholasmhughes <nicholasmhughes@gmail.com> BACKPORT-UPSTREAM=saltstack#65077
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Description of the tech debt to be addressed, include links and screenshots
...at least in the context of user management. It seems that there may have been a package called
pyssswhich presumably offered user/group management capabilities.However, that package is no longer related to that capability:
PyPi: https://pypi.org/project/pysss/
GitHub: https://github.com/mohamed/pysss
The sections below are referencing that package, and further digging should probably be done to see if it's used anywhere else.
https://github.com/saltstack/salt/blob/c9512a18537935ed44e798602920c18629125994/salt/utils/user.py#L34,L39
https://github.com/saltstack/salt/blob/c9512a18537935ed44e798602920c18629125994/salt/utils/user.py#L304,L310
Versions Report
3006.2/master (dated 2023-08-22)
The text was updated successfully, but these errors were encountered: