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

Adapt all scripts to yescrypt #216

Merged
merged 2 commits into from
Nov 21, 2023
Merged

Adapt all scripts to yescrypt #216

merged 2 commits into from
Nov 21, 2023

Conversation

speed47
Copy link
Collaborator

@speed47 speed47 commented Nov 21, 2023

No description provided.

@speed47
Revert "fix: clean obsolete check 99.5.4.5.1, now handled by 5.3.4 (#215
dbf4358 
)"

This reverts commit 670c8c6.

We still want to verify the preexisting hashes in /etc/shadow,
even if the PAM configuration is correct for new passwords (5.3.4).
@github-actions GitHub Actions
Copy link

sh-checker report

To get the full details, please check in the job output.

shellcheck errors 
shellcheck checking is disabled.
shfmt errors 

'shfmt -l -i 4 -w' returned error 1 finding the following formatting issues:

----------
bin/hardening/99.5.4.5.2_acc_shadow_algorithm.sh:55:5: "done" can only be used to end a loop
----------

You can reformat the above files to meet shfmt's requirements by typing:

  shfmt -l -i 4 -w -w filename

@speed47 speed47 force-pushed the yescrypt branch 5 times, most recently from a0fc1b7 to 6bc427a Compare November 21, 2023 16:18
@ThibaultDewailly ThibaultDewailly merged commit de295b3 into master Nov 21, 2023
11 checks passed
@ThibaultDewailly ThibaultDewailly deleted the yescrypt branch December 27, 2023 07:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@ThibaultDewailly ThibaultDewailly ThibaultDewailly approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
2 participants
@speed47 @ThibaultDewailly