Skip Menu |

This queue is for tickets about the Sudo CPAN distribution.

Report information
The Basics
Id: 105380
Status: new
Priority: 0/
Queue: Sudo

People
Owner: Nobody in particular
Requestors: smonff [...] gmail.com
Cc: LANDMAN [...] cpan.org
smonff [...] gmail.com
wagthepitbull [...] gmail.com
AdminCc:

Bug Information
Severity: Normal
Broken in: 0.33
Fixed in: (no value)



CC: smonff [...] gmail.com, LANDMAN [...] cpan.org, wagthepitbull [...] gmail.com
Subject: Missing things in documentation
Hi, Thank you for your Sudo module, it's very useful. As I didn't found a Github repository for this one and you don't mention a way to contribute, I post a report here. This is about documentation, and maybe not directly your fault. There is two different things : 1) When using the -S (stdin) sudo option, the "password must be followed by a newline character" - "\n\r" - (from sudo man page). Then, why not including it directly in the command building of your module? If it's not possible for some reasons, I would definitively propose to mention it in the Sudo.pm documentation. Personally, it took me some time to understand why the password wasn't correctly recognized only because of the lack of this newline character. 2) No documentation available for the Sudo.pm "timeout" attribute: you're required to read the code to discover it's possible to set it, or have a minimal knowledge of IPC::Run. Personally, I was just about to try to add the timeout feature to your module when I discovered it was already possible to set the timeout. If you don't have time for this, I would be glad to propose a pull request for these changes, if a git repository is available for this module. Thanks for your work and I hope this comments would be useful. I send it because I hope it could help people in later use of Sudo.pm. Sorry for any English mistakes. -- smonff