Maven 1.x Changelog Plugin
  1. Maven 1.x Changelog Plugin
  2. MPCHANGELOG-22

SvnChangeLogParser breaks on Windows style usernames

    Details

    • Type: Bug Bug
    • Status: Closed Closed
    • Priority: Minor Minor
    • Resolution: Fixed
    • Affects Version/s: 1.4
    • Fix Version/s: 1.5
    • Labels:
      None
    • Environment:
      Windows and mod_svn running behind mod_auth_sspi
    • Number of attachments :
      2

      Description

      I currently use Subversion as my source repository. I also use mod_sspi to provide authentication for users connecting to the repository.

      What does this have to do with maven-changelog-plugin? Well it seems that the username Subversion stores while being protected by SSPI is in the Windows form of "DOMAIN\user". When svn outputs a revision history, this is the username it shows. Which breaks the SvnChangeLogParser since it's not expecting a '\' in the middle of a username.

        Issue Links

          Activity

          Hide
          Dan Everton added a comment -

          Simple patch to fix this issue. Basically changes the username regexp to expect any series of non-whitespace characters.

          Show
          Dan Everton added a comment - Simple patch to fix this issue. Basically changes the username regexp to expect any series of non-whitespace characters.
          Hide
          Tim Shadel added a comment -

          This patch includes the fix from Dan's patch, as well as additions to the unit tests to check it.

          Show
          Tim Shadel added a comment - This patch includes the fix from Dan's patch, as well as additions to the unit tests to check it.
          Hide
          dion gillard added a comment -

          The patch looks good.

          Show
          dion gillard added a comment - The patch looks good.
          Hide
          dion gillard added a comment -

          Applied.

          Show
          dion gillard added a comment - Applied.

            People

            • Assignee:
              Unassigned
              Reporter:
              Dan Everton
            • Votes:
              0 Vote for this issue
              Watchers:
              0 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved:

                Time Tracking

                Estimated:
                Original Estimate - 1 hour
                1h
                Remaining:
                Remaining Estimate - 1 hour
                1h
                Logged:
                Time Spent - Not Specified
                Not Specified