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

Incompatible with AWS SDK due to paragonie/sodium_compat dependency #392

Closed
BenSeDev opened this issue Sep 11, 2024 · 8 comments · Fixed by #394
Closed

Incompatible with AWS SDK due to paragonie/sodium_compat dependency #392

BenSeDev opened this issue Sep 11, 2024 · 8 comments · Fixed by #394

Comments

@BenSeDev
Copy link

Hello,

I'm unable to install the package due to incompatibility with paragonie/sodium_compat.
We're using the AWS php SDK, which has a dev dependency of paragonie/sodium_compat v2.xx.
So I can't downgrade paragonie/sodium_compat.
I've checked the open PRs and I believe #391 & #384 combined would solve my issues.

- pusher/pusher-php-server[7.2.0, ..., 7.2.4] 
require paragonie/sodium_compat ^1.6 -> found paragonie/sodium_compat[v1.6.0, ..., v1.21.1] 
but the package is fixed to v2.0.1 (lock file version) by a partial update 
and that version does not match. Make sure you list it as an argument for the update command.
@xinix
Copy link

xinix commented Sep 12, 2024

+1

1 similar comment
@JensLeirens
Copy link

+1

@sbaeckelant
Copy link

I need this as well. Any news about this?

@IanSimpson
Copy link
Contributor

+1 here too... please?

@BenSeDev
Copy link
Author

For those following this ticket:
#391 (review)

PR was approved and will be merged.

❤️

@aonemd
Copy link
Contributor

aonemd commented Oct 18, 2024

Hey, thank you for reporting the issue. Both PRs are now merged. I'm preparing a release now.

@aonemd
Copy link
Contributor

aonemd commented Oct 18, 2024

Just made a new release https://github.com/pusher/pusher-http-php/releases/tag/7.2.6

@BenSeDev
Copy link
Author

Thank you @aonemd
This help us a lot and is much appreciated!

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 a pull request may close this issue.

6 participants