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

rec: Backport 12961 to rec-4.7.x: Work around Red Hat 8 pooping the bed in OpenSSL's headers #13075

Merged

Conversation

omoerbeek
Copy link
Member

The openssl/kdf.h header on EL8 is invalid because someone backported a work-in-progress feature to an older OpenSSL branch and did not bother to backport the fixes that were added later.

Red Hat declined to fix their mess and helpfully suggested we do the work instead in https://bugzilla.redhat.com/show_bug.cgi?id=2215856

(cherry picked from commit 3dabf2d)

Backport of #12961

Short description

Checklist

I have:

  • read the CONTRIBUTING.md document
  • compiled this code
  • tested this code
  • included documentation (including possible behaviour changes)
  • documented the code
  • added or modified regression test(s)
  • added or modified unit test(s)
  • checked that this code was merged to master

The openssl/kdf.h header on EL8 is invalid because someone backported
a work-in-progress feature to an older OpenSSL branch and did not
bother to backport the fixes that were added later.

Red Hat declined to fix their mess and helpfully suggested we do the
work instead in https://bugzilla.redhat.com/show_bug.cgi?id=2215856

(cherry picked from commit 3dabf2d)
@omoerbeek omoerbeek added the rec label Jul 26, 2023
@omoerbeek omoerbeek added this to the rec-4.7.x milestone Jul 26, 2023
@omoerbeek
Copy link
Member Author

Failing test should be fixed by #13058, which is merged.

@omoerbeek omoerbeek merged commit 256aa62 into PowerDNS:rel/rec-4.7.x Jul 31, 2023
44 of 47 checks passed
@omoerbeek omoerbeek deleted the backport-12961-to-rec-4.7.x branch July 31, 2023 17:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants