Skip Menu |

Preferred bug tracker

Please visit the preferred bug tracker to report your issue.

This queue is for tickets about the Redis CPAN distribution.

Report information
The Basics
Id: 75562
Status: resolved
Priority: 0/
Queue: Redis
People
Owner: Nobody in particular
Requestors: ether [...] cpan.org
Cc:
AdminCc:
Bug Information
Severity: Normal
Broken in: 1.926
Fixed in: (no value)

History Show all quoted text
  Mon Mar 05 19:34:12 2012 ether [...] cpan.org - Ticket created
Subject: Explicit username/password support
It doesn't look like the constructor accepts username/password options, which is the usual place where a client would specify its credentials. I see that authentication can be done by sending the AUTH command (http://redis.io/commands/auth), which should be available via $redis->auth and the magic of AUTOLOAD, but this is not instantly obvious to all users of the client, so it would be nice if the constructor handled this. I'd be happy to patch in this support, if I could only figure out where you keep the source... it doesn't seem to be on github, and there is no relevant meta information in your META.yml. (I also see some documentation errors, which I'd also be happy to supply a patch for.)
  Tue Mar 06 04:29:25 2012 melo [...] cpan.org - Correspondence added
Hi, On Mon Mar 05 19:34:12 2012, ETHER wrote: Show quoted text
> It doesn't look like the constructor accepts username/password options, > which is the usual place where a client would specify its credentials.
True, it should. Show quoted text
> I see that authentication can be done by sending the AUTH command > (http://redis.io/commands/auth), which should be available via > $redis->auth and the magic of AUTOLOAD, but this is not instantly > obvious to all users of the client, so it would be nice if the > constructor handled this.
Agreed. Show quoted text
> I'd be happy to patch in this support, if I could only figure out where > you keep the source... it doesn't seem to be on github, and there is no > relevant meta information in your META.yml.
It is on Github. And thanks for pointing out that the information is not in the documentation. Time to convert that dist to Dist::Zilla to avoid this oversights. http://github.com/melo/perl-redis/ Show quoted text
> (I also see some documentation errors, which I'd also be happy to supply > a patch for.)
Feel free to send pull requests, I'm planning on working on Redis the next weekend, to make a release next week. At least the auth information in the constructor would be nice to have. Also, if you don't mind, I'll be moving this ticket to github issues. I find it easier to use than RT. https://github.com/melo/perl-redis/issues/11 Bye, -- Hi, how are you?
  Tue Mar 06 04:29:27 2012 The RT System itself - Status changed from 'new' to 'open'
  Tue Mar 06 04:29:27 2012 melo [...] cpan.org - Status changed from 'open' to 'resolved'