1. Sebastian Sdorra
  2. scm-manager
Issue #282 resolved

Error reading spooled REPORT request response

Anonymous created an issue

http://domain.com/scm/svn/isb-algorithm/RT-MoDiCoVi for robustness test/controller.cpp SCM-Server 1.22

svn diff -r 84:86 controller.cpp
svn: '/scm/svn/isb-algorithm/!svn/ver/86/RT-MoDiCoVi%2520for%2520robustness%2520test/controller.cpp' path not found
svn: Error reading spooled REPORT request response

Comments (14)

  1. Sebastian Sdorra repo owner

    Script to reproduce this issue:

    URL=http://localhost:8081/scm/svn/issue
    CO=repo
    FOLDER="folder with spaces"
    FILE=a.txt
    
    svn co $URL $CO
    cd $CO
    mkdir "$FOLDER"
    svn add "$FOLDER"
    svn commit -m "added $FOLDER"
    cd "$FOLDER"
    echo a > $FILE
    svn add $FILE
    svn commit -m "added $FILE"
    echo b >> $FILE
    svn commit -m "improve $FILE"
    echo c >> $FILE
    svn commit -m "finalize $FILE"
    svn diff -r 2:3
    
  2. Sebastian Sdorra repo owner

    The script above ends with following message:

    svn: '/scm/svn/issue/!svn/ver/3/folder%2520with%2520spaces/a.txt' path not found
    svn: Error reading spooled REPORT request response
    

    And this looks like a double encoding.

  3. Sebastian Sdorra repo owner

    I found one possible issue, the location header of the MKCOL method should not url encoded:

    SCM-Manager output:

    Location: http://localhost:8081/scm/svn/issue/!svn/wrk/13ec8e1e-9dd8-49d8-8950-db20a5419ff1/folder%20with%20spaces
    

    Correct output:

    Location: http://localhost:8081/scm/svn/issue/!svn/wrk/13ec8e1e-9dd8-49d8-8950-db20a5419ff1/folder with spaces
    
  4. Sebastian Sdorra repo owner

    The subversion encoding of the Location header looks very strange. Here is the output of all location headers from a mod_svn test:

    Location: http://localhost/repos/issue/!svn/act/f2fb363b-a4eb-40d0-9611-c9d50aeb1ac8.
    Location: http://localhost/repos/issue/!svn/wbl/f2fb363b-a4eb-40d0-9611-c9d50aeb1ac8/0.
    Location: http://localhost/repos/issue/!svn/wrk/f2fb363b-a4eb-40d0-9611-c9d50aeb1ac8/.
    Location: http://localhost/repos/issue/!svn/wrk/f2fb363b-a4eb-40d0-9611-c9d50aeb1ac8/folder with spaces.
    Location: http://localhost/repos/issue/!svn/act/537ef084-6048-4fe3-9e11-7eb69a5fb346.
    Location: http://localhost/repos/issue/!svn/wbl/537ef084-6048-4fe3-9e11-7eb69a5fb346/1.
    Location: http://localhost/repos/issue/!svn/wrk/537ef084-6048-4fe3-9e11-7eb69a5fb346/folder%20with%20spaces.
    Location: http://localhost/repos/issue/!svn/wrk/537ef084-6048-4fe3-9e11-7eb69a5fb346/folder with spaces/a.txt.
    Location: http://localhost/repos/issue/!svn/act/12acc4ea-5041-47f6-bc60-ab42db784112.
    Location: http://localhost/repos/issue/!svn/wbl/12acc4ea-5041-47f6-bc60-ab42db784112/2.
    Location: http://localhost/repos/issue/!svn/wrk/12acc4ea-5041-47f6-bc60-ab42db784112/folder%20with%20spaces/a.txt.
    Location: http://localhost/repos/issue/!svn/act/a1ed271a-7773-42fa-a6ff-fe6dc7fd500f.
    Location: http://localhost/repos/issue/!svn/wbl/a1ed271a-7773-42fa-a6ff-fe6dc7fd500f/3.
    Location: http://localhost/repos/issue/!svn/wrk/a1ed271a-7773-42fa-a6ff-fe6dc7fd500f/folder%20with%20spaces/a.txt.
    
  5. Sebastian Sdorra repo owner

    There was another bug, the checked-in href of the REPORT method was duble encoded:

    <D:checked-in>
      <D:href>/scm/svn/issue/!svn/ver/3/folder%2520with%2520spaces</D:href>
    </D:checked-in>
    
  6. Log in to comment