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

added, PHP_LEGACY_OPENSSL (Fix for OpenSSL 3) #3471

Merged
merged 1 commit into from Dec 12, 2023

Conversation

erikn69
Copy link
Contributor

@erikn69 erikn69 commented Oct 30, 2023

Description

This fix signing errors, or reading errors in electronic signatures (PHP 8.2, 8.3).
Legacy is only added when OpenSSL 3 is used and PHP_LEGACY_OPENSSL=true

Since OpenSSL 3, some ciphers are not available unless users configure some openssl.cnf file ... so allowing this would unbreak applications depending on this feature.
...the .p12 file uses the quite common RC2-40-CBC cipher suite and let the rest of PHP without legacy provider

php/php-src#12369 (comment)
php/php-src#12128 (comment)

Types of Changes

  • Bug fix (non-breaking change which fixes an issue).
  • [] New feature (non-breaking change which adds functionality).
  • [] Breaking change (fix or feature that would cause existing functionality to not work as expected).

Definition of Done Checklist:

  • I've read the Contribution Guide.
  • [] I've updated the documentation. (refer to this for how to do so).
  • I enjoyed my time contributing and making developer's life easier :)

@erikn69 erikn69 force-pushed the patch-14 branch 2 times, most recently from 3a98c40 to b676fd6 Compare October 31, 2023 15:52
@erikn69 erikn69 changed the title added, PHP_LEGACY_OPENSSL added, PHP_LEGACY_OPENSSL (Fix for OpenSSL 3) Dec 4, 2023
@bestlong bestlong merged commit 06cc77a into laradock:master Dec 12, 2023
28 checks passed
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.

None yet

3 participants