Merge ~utkarsh/ubuntu/+source/openldap:lp1921562-openldap-focal into ubuntu/+source/openldap:ubuntu/focal-devel
Status: | Merged | ||||
---|---|---|---|---|---|
Approved by: | Christian Ehrhardt | ||||
Approved revision: | bad0f80f7759e2f2b371c899e0a2975f9dfa7c49 | ||||
Merged at revision: | bad0f80f7759e2f2b371c899e0a2975f9dfa7c49 | ||||
Proposed branch: | ~utkarsh/ubuntu/+source/openldap:lp1921562-openldap-focal | ||||
Merge into: | ubuntu/+source/openldap:ubuntu/focal-devel | ||||
Diff against target: |
182 lines (+160/-0) 3 files modified
debian/changelog (+8/-0) debian/patches/ITS-8650-loop-on-incomplete-TLS-handshake.patch (+151/-0) debian/patches/series (+1/-0) |
||||
Related bugs: |
|
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
Christian Ehrhardt (community) | Approve | ||
Canonical Server | Pending | ||
Sergio Durigan Junior | Pending | ||
git-ubuntu developers | Pending | ||
Review via email: mp+400754@code.launchpad.net |
Description of the change
Hello,
This MP intends to fix LP: #1921562 for Focal. And as discussed in the standup a few days ago, I am requesting review from Sergio explicitly :)
A note for the reviewer: given the very nature of the bug, it might get a bit hard to reproduce so there aren't really very "direct" or "clear" steps to reproduce and thus the "Test Plan" section mentions only what the reporter wrote. Hope this makes sense and is OK for this case?
That said, I've asked the reporter to verify the fix as well; but since Vincent already mentioned, he has been running a similar patched version in production for a few weeks w/o any problems now, so it should be good at that front.
And finally, PPA could be found at https:/
Requesting you to review and sponsor the upload if it makes sense. TIA! \o/
I verified that change matches what we have applied since 2.4.49+dfsg-4
Also I cleaned up the bug in regard to Groovy
Changelog LTGM
I've checked upstream git if there are important pre-requisites 2.4.49+ dfsg-2ubuntu1. 7 to when it was applied (none) or follow up fixes that we'd need (none)
7cf7aa3 already is a backport by upstream to 2.4.50 from the original 735e1ab1 on 2.5.0 - that looks safe as well.
Build log and PPA did not reveal any concerns in a quick (superficial) check (and arm builds were not yet complete).
Yeah - other than the "hard to test" I think this is fine.
+1
P.S. As a general hint, building everything in one PPA seems to be appealing for being easy, but from being burnt by things that behave differently by being built against other packages in there I'd recommend you one PPA per case.