Browse files

Disable SASL for Memcached, references #31

It was a good idea in theory but does not work out
well in practice. Many Memcached clients do not
support SASL or it is not assumed to be enabled
by default. In addition, most developers do not
even realize Memcached has authentication support
at all. In the end, lots of test suites fail for
no obvious reason.

In addition, authentication is not optional, so
we have to either double down or go back to
running Memcached w/o authentication. The latter
is the only realistic option.
  • Loading branch information...
1 parent 1afb5f3 commit aa2b14c2da60efedc916a816920b27b72fbc8531 @michaelklishin michaelklishin committed Oct 16, 2012
Showing with 2 additions and 2 deletions.
  1. +1 −1 ci_environment/memcached/attributes/default.rb
  2. +1 −1 ci_environment/memcached/attributes/ppa.rb
@@ -2,4 +2,4 @@
default[:memcached][:port] = 11211
default[:memcached][:user] = "nobody"
default[:memcached][:listen] = ""
-default[:memcached][:sasl] = true
+default[:memcached][:sasl] = false
@@ -1,3 +1,3 @@
include_attribute "memcached::default"
-default[:memcached][:sasl] = true
+default[:memcached][:sasl] = false

0 comments on commit aa2b14c

Please sign in to comment.