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

Add opensssl extension as a required dependency #110

Merged
merged 1 commit into from
Nov 18, 2023

Conversation

martin-rueegg
Copy link
Contributor

@martin-rueegg martin-rueegg commented Sep 18, 2023

Added a dependency in composer.json ...

"ext-openssl": "*",

... due to the usage of openssl_pkey_export and openssl_pkey_get_details in

openssl_pkey_export($this->key, $privateKey);
$phar->setSignatureAlgorithm(\Phar::OPENSSL, $privateKey);
$keyDetails = openssl_pkey_get_details($this->key);

@martin-rueegg
Copy link
Contributor Author

Hi @theseer

Any chance to look at this for feedback or comments?

Thanks,
Martin.

@theseer theseer merged commit dd14846 into theseer:master Nov 18, 2023
7 checks passed
@martin-rueegg
Copy link
Contributor Author

Awesome, thanks!

@martin-rueegg martin-rueegg deleted the fix/missing-composer-dependency branch November 18, 2023 19:51
@theseer
Copy link
Owner

theseer commented Nov 18, 2023

Sorry for taking long.

Actually, the extension is only required when you make use of the openssl signing feature. So, in that sense it's optional. But I didn't properly implement that as optional. I was even considering to remove the openssl feature as it's pretty useless.

@martin-rueegg
Copy link
Contributor Author

Sorry for taking long.

No worries. Appreciating your time!

Actually, the extension is only required when you make use of the openssl signing feature. So, in that sense it's optional. But I didn't properly implement that as optional. I was even considering to remove the openssl feature as it's pretty useless.

I see. I only noticed it since my IDE complained ... :-)

I can't recall if I was actually using the related functionality!

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.

2 participants