ports/182449: [databases/pecl-memcached] Memcached::setSaslAuthData unavailable, by lack of SASL authentication option,

[Sébastien Santoro]

>Number:         182449
>Category:       ports
>Synopsis:       [databases/pecl-memcached] Memcached::setSaslAuthData unavailable, by lack of SASL authentication option,
>Confidential:   no
>Severity:       non-critical
>Priority:       low
>Responsible:    freebsd-ports-bugs
>State:          open
>Quarter:        
>Keywords:       
>Date-Required:
>Class:          sw-bug
>Submitter-Id:   current-users
>Arrival-Date:   Sat Sep 28 00:00:01 UTC 2013
>Closed-Date:
>Last-Modified:
>Originator:     Sébastien Santoro
>Release:        
>Organization:
>Environment:
(current bug isn't environment dependant)
>Description:
PECL extension by default doesn't include SASL authentication, it should be explicitly asked at configure time.

Excerpt from http://www.php.net/manual/en/memcached.installation.php:
“SASL authentication support is disabled by default. To enable it, use --enable-memcached-sasl switch. This requires that libsasl2 has been installed and that libmemcached has been built with SASL support enabled.”

Excerpt from http://www.php.net/manual/en/memcached.setsaslauthdata.php:
“This method is only available when the memcached extension is build with SASL support. Please refer to Memcached setup for how to do this.”
>How-To-Repeat:
<?php
    $memcached = new Memcached();
    $memcached->addServer('localhost', 11211);
    $memcached->setSaslAuthData('username', 'password');
    //Fails, as setSaslAuthData doesn't exist

>Fix:
Add a SASL option:
- adding --enable-memcached-sasl to ./configure
- adding security/cyrus-sasl2 as dependency (should have been done by libmemcached)

If you wish to do it without the option:
- Argument pro: databases/libmemcached now always add SASL support
- Argument against: databases/memcached still provides the option (but we're in the client side, not the server side, so shouldn't we be coherent with libmemcached choice?)


>Release-Note:
>Audit-Trail:
>Unformatted: