[p4] [SPAM] Re: Python Review Daemon Script and Passwords

[Stephen Vance]

Don't expect this to work if you have the security level set to 3.

The canonical review daemon only requires 'review' but some review 
daemons need more depending on what they do.

Steve

Simon Timms wrote:
> If you open up the script there are two lines
>
> os.environ['P4PORT'] = 'perforce:1999'
> os.environ['P4USER'] = 'review_daemon'
>
> Try adding to that a third line 
>
> os.environ['P4PASSWD'] = 'your password' 
>
> Obviously warning should abound about how you should not allow users to
> read the script containing the password.  Perhaps somebody will correct
> me if I'm wrong, but it was my impression that the only permission
> needed for a user to run a review daemon is 'review'.  You might want to
> create a user just for running the review daemon and only give that user
> the review permission.  
>
> -----Original Message-----
> From: perforce-user-bounces at perforce.com
> [mailto:perforce-user-bounces at perforce.com] On Behalf Of Mullis,
> Keri-Lyn
> Sent: Wednesday, May 31, 2006 12:04 PM
> To: perforce-user at perforce.com
> Subject: [p4] Python Review Daemon Script and Passwords
>
> Hello
>
>  
>
> I get an error from the Review Daemon when I try to make the superuser
> responsible for running the daemon to have a password.  I take away the
> password and it works fine.  
>
>  
>
> We are trying to lock down all the superusers so developers can't find a
> "workaround" to get somewhere they aren't supposed to.  
>
>  
>
> Is there a way to put password in the python script?
>
>  
>
> Any help would be greatly appreciated.
>
>  
>
> Thanks
>
> Keri-Lyn
>
>  
>
> This is the email I get when this user has a password set:
>
>  
>
> Review daemon problem:
>
>  
>
> Traceback (most recent call last):
>
>   File "c:\temp\p4review.py", line 445, in loop_body
>
>     if notify_jobs: review_jobs(mailport,limit_emails)
>
>   File "c:\temp\p4review.py", line 353, in review_jobs
>
>     start_time = int(os.popen(p4 + ' counter jobreview').read())
>
> ValueError: invalid literal for int(): 
>
>  
>
>  
>
> Keri-Lyn Mullis
>
> Release Engineer/IT Production Control
>
> First Marblehead
>
> kmullis at firstmarblehead.com
>
>  
>
>  
>
> *This e-mail and any attachments may contain content protected under
> federal law and is also confidential and proprietary in nature. 
> If you received this message in error, please notify the sender
> immediately and delete the original and destroy all copies of the
> message and any attachments. Any other use of this e-mail by you
> including retaining, using, copying, distributing, or otherwise
> disclosing this information in any manner is prohibited.
>
> _______________________________________________
> perforce-user mailing list  -  perforce-user at perforce.com
> http://maillist.perforce.com/mailman/listinfo/perforce-user
>
> _______________________________________________
> perforce-user mailing list  -  perforce-user at perforce.com
> http://maillist.perforce.com/mailman/listinfo/perforce-user
>
>