#531 closed defect (duplicate)

SFTP frontend needs tests

Reported by: warner Owned by: davidsarah
Priority: major Milestone: 1.7.0
Component: code-frontend Version: 1.2.0
Keywords: sftp test Cc:
Launchpad Bug:

Description

It would be nice to have an SFTP front end (the encrypted kind that's a part of the ssh protocol, as opposed to the unencrypted old protocol covered in #512). In particular, there is a good fuse-based 'sshfs' layer that works well on OS-X.

Attachments (3)

test_sftp.py (43.6 KB) - added by davidsarah at 2010-05-12T06:13:30Z.
Tests for new SFTP implementation
sftp-all-darcspatch.txt (115.5 KB) - added by davidsarah at 2010-05-12T06:18:21Z.
All patches needed for new SFTP implementation and tests
shouldFail-darcspatch.txt (6.3 KB) - added by davidsarah at 2010-05-12T06:21:38Z.
Change shouldFail to avoid Unicode errors when converting Failure to str. (This patch is included in sftp-all-darcspatch.txt but is easier to review as a separate file.)

Download all attachments as: .zip

Change History (14)

comment:1 Changed at 2010-02-06T04:15:00Z by zooko

Ed Pimentl and Jack Lloyd are offering bounties for SFTP: http://allmydata.org/pipermail/tahoe-dev/2010-February/003807.html

comment:2 Changed at 2010-02-06T04:20:49Z by zooko

Brian implemented an SFTP server in Tahoe-LAFS fifteen months ago: http://allmydata.org/trac/tahoe/browser/src/allmydata/frontends/sftpd.py

But it lacks tests. There is an open issue against it: #645 (connecting to sftp frontend using sshfs fails from linux client). Alberto Berti contributed a patch to fix this issue but his patch allegedly causes a different issue. Since there aren't tests I couldn't tell what to do with this situation and left Alberto's patch out of trunk pending someone writing tests for the two alleged issues.

comment:3 Changed at 2010-02-07T16:13:05Z by davidsarah

  • Keywords sftp test added
  • Milestone changed from undecided to 1.7.0
  • Summary changed from SFTP frontend to SFTP frontend needs tests
  • Type changed from enhancement to defect

comment:4 Changed at 2010-03-12T04:06:19Z by davidsarah

  • Owner set to davidsarah
  • Status changed from new to assigned

comment:5 Changed at 2010-05-12T06:11:36Z by davidsarah

  • Status changed from assigned to new

comment:6 Changed at 2010-05-12T06:12:05Z by davidsarah

  • Status changed from new to assigned

Changed at 2010-05-12T06:13:30Z by davidsarah

Tests for new SFTP implementation

Changed at 2010-05-12T06:18:21Z by davidsarah

All patches needed for new SFTP implementation and tests

Changed at 2010-05-12T06:21:38Z by davidsarah

Change shouldFail to avoid Unicode errors when converting Failure to str. (This patch is included in sftp-all-darcspatch.txt but is easier to review as a separate file.)

comment:7 Changed at 2010-05-12T06:23:08Z by davidsarah

  • Keywords review-needed added

comment:8 Changed at 2010-05-12T06:26:10Z by davidsarah

  • Owner changed from davidsarah to somebody
  • Status changed from assigned to new

comment:9 Changed at 2010-05-16T23:43:39Z by zooko

  • Owner changed from somebody to davidsarah

comment:10 Changed at 2010-05-18T21:03:23Z by zooko

  • Resolution set to duplicate
  • Status changed from new to closed

Further work on this is on #1037.

comment:11 Changed at 2010-06-03T04:20:06Z by davidsarah

  • Keywords review-needed removed
Note: See TracTickets for help on using tickets.