search

Smile-SA / odoo_addons

Odoo addons developed by Smile
GNU Affero General Public License v3.0
287 stars 308 forks source link

Add UNIX socket support for redis session store #58

Open telac opened 5 years ago

telac commented 5 years ago

This commit adds unix socket option for redis session store. Using the option requires that both redis_host and redis_port should be None. If no unix socket is supplied, the behavior doesn't change at all. This commit can be applies to both branches 11.0 and 12.0 as is.

This is one possible solution to issue #56

telac commented 5 years ago

Closing the PR for now. This commit is missing a check that the port value is not None before trying to convert port value to integer.

telac commented 5 years ago

The temporary fix works for our testing environment, you'll probably want to come up with something more sophisticated though.