Skip to content

sssctl: add input for user-checks#254

Open
sumit-bose wants to merge 1 commit into
SSSD:masterfrom
sumit-bose:user_checks_input
Open

sssctl: add input for user-checks#254
sumit-bose wants to merge 1 commit into
SSSD:masterfrom
sumit-bose:user_checks_input

Conversation

@sumit-bose

Copy link
Copy Markdown
Contributor

To be able to provide a password or a pin during authentication an optional input option is added.

@gemini-code-assist gemini-code-assist Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Code Review

This pull request updates the user_checks method in sssd_test_framework/utils/sssctl.py to accept an optional auth_input parameter, which is passed to the underlying execution command. Feedback is provided to correct a minor typo ('pasword' instead of 'password') in the newly added docstring.

Important

The consumer version of Gemini Code Assist on GitHub is being sunset. Starting June 18, 2026, new organization installations will be blocked, and all code review activity will officially cease on July 17, 2026.
For more details on the timeline and next steps, please review the Help Documentation.

Comment thread sssd_test_framework/utils/sssctl.py Outdated
To be able to provide a password or a pin during authentication an
optional input option is added.
@alexey-tikhonov alexey-tikhonov self-requested a review July 2, 2026 12:40
@alexey-tikhonov alexey-tikhonov self-assigned this Jul 2, 2026
@alexey-tikhonov alexey-tikhonov requested a review from danlavu July 2, 2026 12:41
sumit-bose added a commit to sumit-bose/sssd that referenced this pull request Jul 9, 2026

@danlavu danlavu left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LG2M

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants