Skip to content

Add the ability to use TLS to the redis-session extension #1182

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

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

atgitwk
Copy link

@atgitwk atgitwk commented Jul 13, 2025

Hello,
PHP buildpack does not seem to support the automatic setting of php ini session_save_path variable for redis-sessions services with TLS.
extensions/sessions/extension.py only handles the 'tcp:' scheme.

    def session_save_path(self):
        return "tcp://%s:%s?auth=%s" % (
            self.creds.get('hostname',
                           self.creds.get('host', 'not-found')),
            self.creds.get('port', 'not-found'),
            self.creds.get('password', ''))

To use TLS, the Redis 'uri' specification indicates that the ‘uri’ parameter could be set with scheme ‘rediss://’ instead of ‘redis://’.
This 'uri' parameter could be initialized in the json binding credentials configuration provided by the service broker.
Here's a proposal PR for managing redis-session with TLS

  • I have viewed signed and have submitted the Contributor License Agreement
    Go an error at link Individual CLA : Error 404 Not Found 😟

  • I have made this pull request to the master branch

  • I have added an integration test

@atgitwk atgitwk requested a review from a team as a code owner July 13, 2025 22:08
Copy link

linux-foundation-easycla bot commented Jul 13, 2025

CLA Signed


The committers listed above are authorized under a signed CLA.

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.

1 participant