#777 test pam_interactive (including multistep capability) automatically - #824
#777 test pam_interactive (including multistep capability) automatically#824d-w-moore wants to merge 11 commits into
Conversation
|
Test of multistep doesn't yet pass, even when iRODS4j is run, so something is probably wrong with the test setup. Awaiting a branch of irods/irods4j in which ports are exposed to test PRC on a valid setup |
|
Is the PAM Interactive test run expected to fail in GitHub Actions? See the following: |
No, it should pass ... looking |
I can only get it to pass by reverting our change (702f723) to the native handoff from pam_interactive to the native auth phase. If I do that, all tests related to pam_interactive (including my new tests in test012...) pass once again. Not sure how this should look, for the moment. Will need to study. |
|
That's fine. Those changes do not need to be part of this PR. Keep them in a separate branch so that we can investigate them later. |
57fe12d to
238268a
Compare
One test for correct 2nd password, another for deliberately munged 2nd password. (nb, the 2nd password test does an extra check to a pam_userdb.db file to prove multistep authentication work flows are functioning for the pam_interactive auth scheme.)
This reverts commit 702f723.
|
Waiting for tests to pass. As soon as they do, let's go with a final review. |
2 participants
No description provided.