php-buildpack icon indicating copy to clipboard operation
php-buildpack copied to clipboard

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

Open atgitwk opened this issue 5 months ago • 2 comments

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

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

  • [x] I have made this pull request to the master branch

  • [ ] I have added an integration test

atgitwk avatar Jul 13 '25 22:07 atgitwk

Hi team My PR is still in pending review but cloudfoundry/wg-app-runtime-interfaces-buildpacks-php-approvers returns HTTP 404 "Page not found" :disappointed: Is this normal ? May anyone help ?

atgitwk avatar Aug 23 '25 20:08 atgitwk