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

Define default ssl context parameter #1060

Merged
merged 1 commit into from
Jan 13, 2023

Conversation

ramunasd
Copy link
Member

Fix #1013

Follow up forgotten promise 🤔
#674 (comment)

@ramunasd ramunasd added the bug label Jan 10, 2023
@ramunasd ramunasd added this to the 3.4.1 milestone Jan 10, 2023
@codecov
Copy link

codecov bot commented Jan 10, 2023

Codecov Report

Merging #1060 (ac26c18) into master (2a4cd8e) will decrease coverage by 0.36%.
The diff coverage is 83.33%.

@@             Coverage Diff              @@
##             master    #1060      +/-   ##
============================================
- Coverage     73.42%   73.05%   -0.37%     
+ Complexity     1009     1007       -2     
============================================
  Files            37       37              
  Lines          3055     3069      +14     
============================================
- Hits           2243     2242       -1     
- Misses          812      827      +15     
Impacted Files Coverage Δ
PhpAmqpLib/Connection/AMQPSSLConnection.php 87.09% <66.66%> (-2.56%) ⬇️
PhpAmqpLib/Connection/AMQPConnectionConfig.php 68.75% <100.00%> (ø)
PhpAmqpLib/Wire/IO/StreamIO.php 67.66% <100.00%> (-2.73%) ⬇️
PhpAmqpLib/Wire/AMQPIOReader.php 91.89% <0.00%> (-2.56%) ⬇️
PhpAmqpLib/Connection/AbstractConnection.php 75.35% <0.00%> (-0.80%) ⬇️
PhpAmqpLib/Channel/AMQPChannel.php 60.79% <0.00%> (-0.32%) ⬇️
PhpAmqpLib/Wire/AMQPAbstractCollection.php 88.39% <0.00%> (+0.06%) ⬆️
PhpAmqpLib/Channel/AbstractChannel.php 68.92% <0.00%> (+0.17%) ⬆️

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

@lukebakken lukebakken force-pushed the fix_ssl_connection_default_params branch from ac26c18 to 26b4c34 Compare January 13, 2023 15:01
@lukebakken lukebakken merged commit c686869 into master Jan 13, 2023
@lukebakken lukebakken deleted the fix_ssl_connection_default_params branch January 13, 2023 15:01
@lukebakken
Copy link
Collaborator

Thanks @ramunasd

lukebakken added a commit that referenced this pull request Jan 18, 2023
lukebakken added a commit that referenced this pull request Jan 18, 2023
lukebakken added a commit that referenced this pull request Jan 18, 2023
lukebakken added a commit that referenced this pull request Jan 18, 2023
lukebakken added a commit that referenced this pull request Jan 18, 2023
Fixes #1065

Add deprecation error

Add non-TLS SSL test

Only trigger_error in version 8 and higher
lukebakken added a commit that referenced this pull request Jan 20, 2023
Fixes #1065

Add deprecation error

Add non-TLS SSL test

Only trigger_error in version 8 and higher

do not report user deprecations as errors in phpunit

Update PhpAmqpLib/Connection/AMQPSSLConnection.php

Co-authored-by: Andrii Dembitskyi <andrew.dembitskiy@gmail.com>
lukebakken added a commit that referenced this pull request Jan 20, 2023
Fixes #1065

Add deprecation error

Add non-TLS SSL test

Only trigger_error in version 8 and higher

do not report user deprecations as errors in phpunit

Update PhpAmqpLib/Connection/AMQPSSLConnection.php

Co-authored-by: Andrii Dembitskyi <andrew.dembitskiy@gmail.com>

Co-authored-by: Andrii Dembitskyi <andrew.dembitskiy@gmail.com>
kratkyzobak pushed a commit to kratkyzobak/php-amqplib that referenced this pull request Feb 9, 2024
…on_default_params

Define default ssl context parameter
kratkyzobak pushed a commit to kratkyzobak/php-amqplib that referenced this pull request Feb 9, 2024
Fixes php-amqplib#1065

Add deprecation error

Add non-TLS SSL test

Only trigger_error in version 8 and higher

do not report user deprecations as errors in phpunit

Update PhpAmqpLib/Connection/AMQPSSLConnection.php

Co-authored-by: Andrii Dembitskyi <andrew.dembitskiy@gmail.com>

Co-authored-by: Andrii Dembitskyi <andrew.dembitskiy@gmail.com>
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