Merge ~ahasenack/ubuntu/+source/autofs:mantic-autofs-fix-ntlm-crammd5 into ubuntu/+source/autofs:ubuntu/devel

Proposed by Andreas Hasenack
Status: Merged
Approved by: git-ubuntu bot
Approved revision: not available
Merged at revision: a93fc88e5322f18edd557f6d5c4d90b6b627f74b
Proposed branch: ~ahasenack/ubuntu/+source/autofs:mantic-autofs-fix-ntlm-crammd5
Merge into: ubuntu/+source/autofs:ubuntu/devel
Diff against target: 96 lines (+33/-4)
6 files modified
debian/changelog (+12/-0)
debian/patches/autofs-5.1.8-support-SCRAM-for-SASL-binding.patch (+1/-1)
debian/patches/ntlm-crammd5-require-credentials.patch (+16/-0)
debian/patches/series (+1/-0)
debian/tests/control (+1/-1)
debian/tests/ldap-map-sasl-auth (+2/-2)
Reviewer Review Type Date Requested Status
git-ubuntu bot Approve
Athos Ribeiro (community) Approve
Canonical Server Core Reviewers Pending
Canonical Server Reporter Pending
Review via email: mp+446104@code.qastaging.launchpad.net

Description of the change

This MP fixes:
- support for NTLM and CRAM-MD5 (bug #2023595): these were initially thought to be working, but closer inspection showed them to also be broken in the release pocket. I still need to fill out the SRU template for this one.

In the previous autofs merge I had opted to not fix this in the devel release, but since I'm SRUing this fix, I changed my mind. Initially I thought we shouldn't fix deprecated mechanisms (which NTLM and CRAM-MD5 are), but that's not the correct way to deprecate something (not fixing a bug). If we really want to deprecate these alrogorithms, then it should be a wider effort and they should be removed from cyrus-sasl2, for example.

Testing these is a bit involved, and I wrote a DEP8 test for that. This test will go over the shared secret mechanisms, as well as the kerberos ones (gssapi/gss-spnego).

PPA: https://launchpad.net/~ahasenack/+archive/ubuntu/autofs-sasl-fixes/
DEP8: green

Note: I disabled arm* builds because they were not starting, and I waited over 1h.

To post a comment you must log in.
Revision history for this message
Andreas Hasenack (ahasenack) wrote :

I still have to fill out the sru template for bug #2023595

Revision history for this message
Athos Ribeiro (athos-ribeiro) wrote :

Thanks Andreas!

+1 on fixing this even if NTLM and CRAM-MD5 are deprecated.

LGTM. It seems we also got an upstream ack/+1 for the forwarded change :)

review: Approve
Revision history for this message
git-ubuntu bot (git-ubuntu-bot) wrote :

Approvers: ahasenack, athos-ribeiro
Uploaders: ahasenack, athos-ribeiro
MP auto-approved

review: Approve
Revision history for this message
Andreas Hasenack (ahasenack) wrote :

I addressed the same points raised in the lunar and jammy MPs

Revision history for this message
Andreas Hasenack (ahasenack) wrote :

Uploaded with rich history:

Uploading autofs_5.1.8-2ubuntu2.dsc
Uploading autofs_5.1.8-2ubuntu2.debian.tar.xz
Uploading autofs_5.1.8-2ubuntu2_source.buildinfo
Uploading autofs_5.1.8-2ubuntu2_source.changes

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
The diff is not available at this time. You can reload the page or download it.

Subscribers

People subscribed via source and target branches