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

sshd: Use '.*' for regex. Add accept pw test. #99

Merged
merged 1 commit into from
Jun 28, 2023
Merged

Conversation

stephen-fox
Copy link
Contributor

The processAcceptedPasswordEntry function was untested. I also added some additional characters to the "exp" test constants.

The processAcceptedPasswordEntry function was untested. I also
added some additional characters to the "exp" test constants.
@stephen-fox stephen-fox requested review from a team as code owners June 28, 2023 19:34
@codecov
Copy link

codecov bot commented Jun 28, 2023

Codecov Report

Merging #99 (a18d686) into main (fd163c4) will decrease coverage by 0.41%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##             main      #99      +/-   ##
==========================================
- Coverage   80.12%   79.71%   -0.41%     
==========================================
  Files          30       30              
  Lines        1972     1972              
==========================================
- Hits         1580     1572       -8     
- Misses        319      325       +6     
- Partials       73       75       +2     
Flag Coverage Δ
unittests 79.71% <ø> (-0.41%) ⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

see 1 file with indirect coverage changes

@stephen-fox stephen-fox merged commit a10db9f into main Jun 28, 2023
@stephen-fox stephen-fox deleted the regex-cleanup branch June 28, 2023 20:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants