kovasaurus commented on PR #13178:
URL: https://github.com/apache/apisix/pull/13178#issuecomment-4258840665

   I would like to chip into this discussion.
   
   Prior to lua-resty-session 4.x we could use the nginx variables to setup the 
various features from lua-resty-session which were not exposed directly through 
openid-connect plugin in Apisix. After 4.x version, due to dependency chain of 
openid-connect apisix plugin -> lua-resty-openidc -> lua-resty-session it is 
only possible to configure those features by configuration which is sent from 
openid-connect plugin.
   
   Effectively, the limited schema capabilities are now blocking us from using 
the features that exist in lua-resty-session.
   This PR from OP addresses the cookie configuration, but I have additional 
example.
   
   lua-resty-session supports Redis with sentinel configuration -> 
https://github.com/bungle/lua-resty-session/blob/master/README.md#redis-sentinels-configuration
   
   I have tried to play around with it for a bit, and I've managed to create 
custom docker image from apisix:3.16.0 with lua-resty-redis-connector 
installed, adjusted the schema for openid-connect to support sentinel 
properties and were able to successfully connect to Redis sentinel. The setup 
is still being tested but so far looks good. 


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to