Maven Wagon

WebDAV Wagon putDirectory fails when multiple directories in path don't exist

Details

  • Type: Bug Bug
  • Status: Closed Closed
  • Priority: Blocker Blocker
  • Resolution: Fixed
  • Affects Version/s: 1.0-beta-1, 1.0-beta-2
  • Fix Version/s: 1.0-beta-2
  • Component/s: wagon-webdav
  • Labels:
    None
  • Number of attachments :
    2

Description

The implementation of the 'putDirectory' method isn't properly checking that all directories (WebDAV collections) have been created. I've attached a patch that reworks the 'putDirectory' such that it relies on the 'put' method for each upload. This will guarantee that all file uploads are treated the same way and resolves this issue, as the 'put' method implements the appropriate directory checks.

  1. File
    wagontestcase_deepdest.diff
    12/Jul/06 12:32 PM
    3 kB
    Nathan Beyer (Cerner)
  2. File
    webdavwagon_putdirectory.diff
    12/Jul/06 11:35 AM
    2 kB
    Nathan Beyer (Cerner)

Activity

Ascending order - Click to sort in descending order
Hide
Permalink
Nathan Beyer (Cerner) added a comment -

The attachement 'wagontestcase_deepdest.diff' is a patch for the base WagonTestCase class that adds a regression test method that demonstrates the issue. This should cause the WebDAV provider to fail.

Show
Nathan Beyer (Cerner) added a comment - The attachement 'wagontestcase_deepdest.diff' is a patch for the base WagonTestCase class that adds a regression test method that demonstrates the issue. This should cause the WebDAV provider to fail.
Nathan Beyer (Cerner) made changes -
Field Original Value New Value
Attachment wagontestcase_deepdest.diff [ 21481 ]
Hide
Permalink
Carlos Sanchez added a comment -

Thanks, I wanted to do that refactoring some time ago

Show
Carlos Sanchez added a comment - Thanks, I wanted to do that refactoring some time ago
Carlos Sanchez made changes -
Status Open [ 1 ] Closed [ 6 ]
Assignee Carlos Sanchez [ carlos ]
Resolution Fixed [ 1 ]
Fix Version/s 1.0-beta-2 [ 12683 ]
Brett Porter made changes -
Workflow jira [ 59019 ] Maven New [ 88447 ]

People

  • Assignee:
    Carlos Sanchez
    Reporter:
    Nathan Beyer (Cerner)
Vote (0)
Watch (1)

Dates

  • Created:
    Updated:
    Resolved: