Issue Details (XML | Word | Printable)

Key: MPCHANGELOG-64
Type: Bug Bug
Status: Closed Closed
Resolution: Fixed
Priority: Major Major
Assignee: David Eric Pugh
Reporter: David Eric Pugh
Votes: 0
Watchers: 0
Operations

If you were logged in you would be able to see more operations.
Maven 1.x Changelog Plugin

Changelog Plugin doesn't parse .cvspass properly

Created: 06/Jun/05 07:01 PM   Updated: 27/Oct/05 03:35 PM   Resolved: 06/Jun/05 09:33 PM
Return to search
Component/s: None
Affects Version/s: 1.7.1
Fix Version/s: 1.8.2

Time Tracking:
Not Specified


 Description  « Hide

I have been testing Maven on Linux running Java 1.5.0_02-b09, and noticed that the Changelog plugin won't run against CVS servers. Changelog uses a java library to provide CVS connectivity instead of running CVS from the command line.

I always get a message:

org.netbeans.lib.cvsclient.connection.AuthencticationException: Wrong Password.

I discovered that the format of the cvs information in the .cvspass file differs depending on if cvs inserted it, or the create-cvspass did.

CVS looks like this:
/1 :pserver:dep4b@cvs.sourceforge.net:2401/cvsroot/maven-plugins A3333 x

changelog:create-cvspass looks like this:
:pserver:dep4b@cvs.sourceforge.net:cvsroot/maven-plugins A3333 x

I think I have a small fix for CvsConnection.java's lookupPassword() method to deal with the two types of strings. It just isn't properly implemented for the cvs generated one where your password has a " " (space) in the password name.

I wanted to see if anyone else had run across this problem...



David Eric Pugh made changes - 06/Jun/05 08:26 PM
Field Original Value New Value
Status Open [ 1 ] In Progress [ 3 ]
David Eric Pugh added a comment - 06/Jun/05 08:29 PM

It seems like things aren't properly handled when the .cvspass file starts with a /1 and the password is blank, so it ends in "A ".. That line:

/1 :pserver:user@server:2401/home/cvs A

has only 3 elements, so the password is returned as null. However, it should be returned as "A "... I think....


David Eric Pugh added a comment - 06/Jun/05 09:33 PM

Looks like it is sorted out now.


David Eric Pugh made changes - 06/Jun/05 09:33 PM
Resolution Fixed [ 1 ]
Fix Version/s 1.8.2 [ 11778 ]
Status In Progress [ 3 ] Resolved [ 5 ]
Arnaud Heritier added a comment - 27/Oct/05 03:35 PM

Fixed for a long long time ago


Arnaud Heritier made changes - 27/Oct/05 03:35 PM
Status Resolved [ 5 ] Closed [ 6 ]