Maven 1.x Changelog Plugin

Plugin does not find the password for the CVSROOT

Details

  • Type: Bug Bug
  • Status: Closed Closed
  • Priority: Major Major
  • Resolution: Fixed
  • Affects Version/s: 1.4
  • Fix Version/s: None
  • Labels:
    None
  • Environment:
    Windows XP and 2000, cvs client version 1.11.2
  • Number of attachments :
    1

Description

Plugin does not find the password for the CVSROOT in .cvspass if the CVSROOT in .cvspass is preceded by a "/1". I do not know what the "/1" means, but is was placed their by CVS.

***

  • From Maven's DEBUG logging. Notice the "Didn't find password"
  • message
    ***
    maven-changelog-plugin:report:
    [echo] Generating the changelog report
    Didn't find password for CVSROOT ':pserver:anoncvs@castor.exolab.org:/cvs/castor'.
    org.netbeans.lib.cvsclient.connection.AuthenticationException: Authentication failed. Response from server was: "E PAM authenticate error: Authentication failure".
    ChangeLog found: 0 entries
    attaining goal build:end

***

  • From .cvspass
    ***
    /1 :pserver:anoncvs@castor.exolab.org:2401/cvs/castor Ay=0=h<Z

Activity

Ascending order - Click to sort in descending order
Hide
Permalink
Paul Spencer added a comment -

This problem was introduced in rc2. I did not have this problem in rc1 or beta-10

Show
Paul Spencer added a comment - This problem was introduced in rc2. I did not have this problem in rc1 or beta-10
Hide
Permalink
analogue added a comment -

Patch while loop in CVSConnection.lookupPassword() as follows:

while ((line = reader.readLine()) != null)
{
if (line.startsWith("/1 "))
line = line.substring("/1 ".length());

if (line.startsWith(cvsRoot))

{ password = line.substring(cvsRoot.length() + 1); break; }

}

Show
analogue added a comment - Patch while loop in CVSConnection.lookupPassword() as follows: while ((line = reader.readLine()) != null) { if (line.startsWith("/1 ")) line = line.substring("/1 ".length()); if (line.startsWith(cvsRoot)) { password = line.substring(cvsRoot.length() + 1); break; } }
Hide
Permalink
Gary Gregory added a comment -

I am seing the same "Didn't find password for CVSROOT" in RC-3. Is the "/1" an index since .cvspass can have multiple entries?

Show
Gary Gregory added a comment - I am seing the same "Didn't find password for CVSROOT" in RC-3. Is the "/1" an index since .cvspass can have multiple entries?
Hide
Permalink
Rafael Santiago, Jr. added a comment -

Currently, it seems that the plugin only onderstands cvspass lines in the format created by running "maven -Dpassword=XXXXXX changelog:create-cvspass", namely:

1. The line in CVS cannot start with /1 .
2. The line has to match what you specify in project.xml. In other words, if you have a port number in your <repository><connection>, it HAS to be in the .cvspass. If you don't specify a port, it can't be in the .cvspass line either.

Show
Rafael Santiago, Jr. added a comment - Currently, it seems that the plugin only onderstands cvspass lines in the format created by running "maven -Dpassword=XXXXXX changelog:create-cvspass", namely: 1. The line in CVS cannot start with /1 . 2. The line has to match what you specify in project.xml. In other words, if you have a port number in your <repository><connection>, it HAS to be in the .cvspass. If you don't specify a port, it can't be in the .cvspass line either.
Hide
Permalink
ksaito added a comment -

I made the patch.

Show
ksaito added a comment - I made the patch.
Hide
Permalink
dion gillard added a comment -

This looks good.

Show
dion gillard added a comment - This looks good.

People

  • Assignee:
    Unassigned
    Reporter:
    Paul Spencer
Vote (5)
Watch (5)

Dates

  • Created:
    Updated:
    Resolved: