Skip to content

add support for TLS configuration for Redis Store#1828

Merged
tvdijen merged 1 commit intosimplesamlphp:masterfrom
UiP9AV6Y:feature_redis_store_tls
Jul 26, 2023
Merged

add support for TLS configuration for Redis Store#1828
tvdijen merged 1 commit intosimplesamlphp:masterfrom
UiP9AV6Y:feature_redis_store_tls

Conversation

@UiP9AV6Y
Copy link
Copy Markdown
Contributor

generate selected SSL context options for the Predis client.

see https://www.php.net/manual/en/context.ssl.php

closes #1804

generate selected SSL context options for the Predis client.

see https://www.php.net/manual/en/context.ssl.php

closes #1804
@codecov
Copy link
Copy Markdown

codecov bot commented Jul 22, 2023

Codecov Report

Merging #1828 (73db1b6) into master (d81adb9) will decrease coverage by 0.07%.
The diff coverage is 0.00%.

Additional details and impacted files
@@             Coverage Diff              @@
##             master    #1828      +/-   ##
============================================
- Coverage     45.13%   45.07%   -0.07%     
- Complexity     3641     3649       +8     
============================================
  Files           162      162              
  Lines         12335    12353      +18     
============================================
  Hits           5568     5568              
- Misses         6767     6785      +18     

@tvdijen tvdijen merged commit 968e6e6 into simplesamlphp:master Jul 26, 2023
tvdijen pushed a commit that referenced this pull request Jul 26, 2023
generate selected SSL context options for the Predis client.

see https://www.php.net/manual/en/context.ssl.php

closes #1804
tvdijen pushed a commit that referenced this pull request Jul 26, 2023
generate selected SSL context options for the Predis client.

see https://www.php.net/manual/en/context.ssl.php

closes #1804
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Oct 25, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Redis Store doesn't Support TLS/SSL encryption

2 participants