Closed (fixed)
Project:
Memcache API and Integration
Version:
8.x-2.x-dev
Component:
Code
Priority:
Normal
Category:
Feature request
Assigned:
Reporter:
Created:
20 Mar 2017 at 18:16 UTC
Updated:
26 Apr 2017 at 19:10 UTC
Jump to comment: Most recent, Most recent file
Comments
Comment #2
Rok Žlender commentedI am working on this and should have a patch shortly.
Comment #3
Rok Žlender commentedI tested the patch with memcached server running with and without SASL and was able to connect.
Comment #4
Rok Žlender commentedComment #5
damiankloip commentedThe format of these is incorrect I think? These are all in settings now, like you have above for the sasl credentials.
I think this should be:
These are old D7 settings are duplicates of the correct sasl array above?
I think just @sasl is an ok name. Although, looking at this property, it doesn't seem to be used in the patch?
Comment #6
Rok Žlender commentedThanks for the review should be all updated in the new patch.
Comment #8
damiankloip commentedThis looks good now - thanks. We could check that username and password are explicitly set to avoid warnings but I think this is fine. Committed and pushed.