Maven 1

be able to use sftp as a maven.repo.remote

Details

  • Type: Wish Wish
  • Status: Closed Closed
  • Priority: Major Major
  • Resolution: Fixed
  • Affects Version/s: None
  • Fix Version/s: 1.1-beta-3
  • Component/s: core
  • Labels:
    None
  • Number of attachments :
    1

Description

It would be nice to be able to use sftp remote private repositories, as they are easier to admin than https, with custom CA.

Issue Links

Activity

Hide
Juan F. Codagnone added a comment -

As a quick hack, this patch makes sftp work. Its limitation is that the username and password must be hardcoded in the url.
Depends on WAGONSSH-11 . Instead of adding another if, i have created a WagonFactory.

Example:

$ cat project.properties
maven.repo.remote=sftp://${mytest.user}:${mytest.password}@localhost/tmp/ble

$ cat build.properties
mytest.user=test
mytest.password=test

Show
Juan F. Codagnone added a comment - As a quick hack, this patch makes sftp work. Its limitation is that the username and password must be hardcoded in the url. Depends on WAGONSSH-11 . Instead of adding another if, i have created a WagonFactory. Example: $ cat project.properties maven.repo.remote=sftp://${mytest.user}:${mytest.password}@localhost/tmp/ble $ cat build.properties mytest.user=test mytest.password=test
Hide
Brett Porter added a comment -

hsa patch

Show
Brett Porter added a comment - hsa patch
Hide
Arnaud Heritier added a comment -

Brett, Why don't you apply yourself ?

Show
Arnaud Heritier added a comment - Brett, Why don't you apply yourself ?
Hide
Brett Porter added a comment -

I needed more time to look at whether it was a good implementation, just wanted to make sure it didn't get missed.

Show
Brett Porter added a comment - I needed more time to look at whether it was a good implementation, just wanted to make sure it didn't get missed.
Hide
Lukas Theussl added a comment -

Patch looks good, just update wagon dependencies and docs.

Show
Lukas Theussl added a comment - Patch looks good, just update wagon dependencies and docs.
Hide
Arnaud Heritier added a comment -

I just updated the wagon dependencies : MAVEN-1762

Show
Arnaud Heritier added a comment - I just updated the wagon dependencies : MAVEN-1762
Hide
Arnaud Heritier added a comment -

We can try to fix these issues together

Show
Arnaud Heritier added a comment - We can try to fix these issues together
Hide
Lukas Theussl added a comment -

Patch applied. Thanks!

Show
Lukas Theussl added a comment - Patch applied. Thanks!

People

Vote (0)
Watch (0)

Dates

  • Created:
    Updated:
    Resolved: