374282 - Need to allow anonymous read-only pserver users

Status: VERIFIED INVALID

(Webtools Graveyard :: Despot, defect)

Description

[Aravind Gottipati [:aravind]]

Attachment: Add an anonymous user to the generated CVSROOT/passwd file.

Bugs 369013 and 373373 need to access the cvs.m.o repository using the anonymous cvs pserver for read-only checkouts.

The current CVSROOT/passwd file is generated automatically by despot and does not contain an entry for the anonymous user.

I am attaching a one-liner patch that will add this entry.

Comment 1

[Dave Miller [:justdave]]

does a user named anonymous automatically get read-only access?

Comment 2

[Aravind Gottipati [:aravind]]

no, I need to create the corresponding reader/writer files.  But that is not a part of despot.

The patch above will need to change to

print PASSWD "anonymous::cvsuser\n";

Comment 3

[Reed Loden [:reed]]

Attachment: Move the system user to the config file

Instead of hardcoding "nobody", move the system user to use to the config file. I really don't like the name "anonymous_sysuser" as a variable name, but it does describe what it is. If anybody has other suggestions, I'm open!

Comment 4

[Reed Loden [:reed]]

Checking in config.pl.dist;
/cvsroot/mozilla/webtools/despot/config.pl.dist,v  <--  config.pl.dist
new revision: 1.3; previous revision: 1.2
done
Checking in syncit.pl;
/cvsroot/mozilla/webtools/despot/syncit.pl,v  <--  syncit.pl
new revision: 1.39; previous revision: 1.38
done

I will backout aravind's changes to commitcheck.tmpl now.

Comment 5

[Reed Loden [:reed]]

Backed out aravind's commitcheck.templ changes:

Checking in commitcheck.templ;
/cvsroot/mozilla/webtools/despot/commitcheck.templ,v  <--  commitcheck.templ
new revision: 1.11; previous revision: 1.10
done

Comment 6

[Aravind Gottipati [:aravind]]

Thank you!

Comment 7

[Reed Loden [:reed]]

Backed out due to the use of a different fix for bug 369013.

Checking in config.pl.dist;
/cvsroot/mozilla/webtools/despot/config.pl.dist,v  <--  config.pl.dist
new revision: 1.4; previous revision: 1.3
done
Checking in syncit.pl;
/cvsroot/mozilla/webtools/despot/syncit.pl,v  <--  syncit.pl
new revision: 1.40; previous revision: 1.39
done