Skip to content

Commit

Permalink
fix(debian): Add pwquality dictionary check dependencies (#425)
Browse files Browse the repository at this point in the history
In order to do password dictionary checks we need to use cracklib, but
the runtime isn't installed by default in debian so add a test
dependency for it and, just in case, to debian too.

Also use the simplest password ever that would fail any dictionary check
(and that's part of cracklib's cracklib-small dictionary)

It built fine at
https://launchpad.net/~ubuntu-enterprise-desktop/+archive/ubuntu/authd/+sourcepub/16261392/+listing-archive-extra

UDENG-3575
  • Loading branch information
3v1n0 authored Jul 11, 2024
2 parents 5672b51 + ddff1f3 commit 7883243
Show file tree
Hide file tree
Showing 4 changed files with 5 additions and 2 deletions.
2 changes: 2 additions & 0 deletions .github/workflows/qa.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -15,7 +15,9 @@ env:
libpwquality-dev
test_apt_deps: >-
cracklib-runtime
ffmpeg
# In Rust the grpc stubs are generated at build time
# so we always need to install the protobuf compilers
# when building the NSS crate.
Expand Down
1 change: 1 addition & 0 deletions debian/control
Original file line number Diff line number Diff line change
Expand Up @@ -3,6 +3,7 @@ Section: admin
Priority: optional
Maintainer: Ubuntu Developers <[email protected]>
Build-Depends: debhelper-compat (= 13),
cracklib-runtime <!nocheck>,
dbus <!nocheck>,
dh-apport,
dh-cargo,
Expand Down
2 changes: 1 addition & 1 deletion pam/integration-tests/gdm_test.go
Original file line number Diff line number Diff line change
Expand Up @@ -222,7 +222,7 @@ func TestGdmModule(t *testing.T) {
Challenge: "goodpass",
}),
gdm_test.IsAuthenticatedEvent(&authd.IARequest_AuthenticationData_Challenge{
Challenge: "foolinux",
Challenge: "password",
}),
gdm_test.IsAuthenticatedEvent(&authd.IARequest_AuthenticationData_Challenge{
Challenge: "newpass",
Expand Down
2 changes: 1 addition & 1 deletion pam/internal/adapter/gdmmodel_test.go
Original file line number Diff line number Diff line change
Expand Up @@ -485,7 +485,7 @@ func TestGdmModel(t *testing.T) {
Challenge: "newpass",
}}),
sendEvent(gdmTestSendAuthDataWhenReady{&authd.IARequest_AuthenticationData_Challenge{
Challenge: "foolinux",
Challenge: "password",
}}),
sendEvent(gdmTestSendAuthDataWhenReady{&authd.IARequest_AuthenticationData_Challenge{
Challenge: "gdm-good-password",
Expand Down

0 comments on commit 7883243

Please sign in to comment.