New issue
Advanced search Search tips

Issue 714185 link

Starred by 6 users

Issue metadata

Status: Fixed
Owner:
Closed: May 2017
Components:
EstimatedDays: ----
NextAction: ----
OS: Chrome
Pri: 3
Type: Feature



Sign in to add a comment

nassh: sftp: allow people to control cwd

Project Member Reported by vapier@chromium.org, Apr 21 2017

Issue description

currently sftp mounts can only look at paths under the remote user's home directory.  would be nice if people could change that on the fly, or at least control it on the initial connect.

integration with fsp makes things a bit trickier.  not sure if you can invalidate all previous lookups ... otherwise, we could expose the ".." pseudo path in the root directory for people to click through.
 

Comment 1 by cvmil...@gmail.com, May 2 2017


It would be nice to have to ability to access other remote directories. Something like:
sftp me@remotehost.com:/tmp


Project Member

Comment 2 by bugdroid1@chromium.org, May 17 2017

The following revision refers to this bug:
  https://chromium.googlesource.com/apps/libapps/+/0425eccbd4546e71d51448e72e85e412b0223701

commit 0425eccbd4546e71d51448e72e85e412b0223701
Author: Mike Frysinger <vapier@chromium.org>
Date: Wed May 17 18:00:16 2017

nassh: rework label padding

Rather than hardcode the labels that we need via id & relative position,
use a dedicated class.  This way the html file can be the single source
of truth.

BUG= chromium:714185 

Change-Id: I4bcd39ac0cb31d4af36ad380800ef0bcdf4b85e1
Reviewed-on: https://chromium-review.googlesource.com/503867
Reviewed-by: Brandon Gilmore <varz@google.com>
Tested-by: Mike Frysinger <vapier@chromium.org>

[modify] https://crrev.com/0425eccbd4546e71d51448e72e85e412b0223701/nassh/js/nassh_connect_dialog.js
[modify] https://crrev.com/0425eccbd4546e71d51448e72e85e412b0223701/nassh/html/nassh_connect_dialog.html

Comment 4 by vapier@chromium.org, May 18 2017

Owner: vapier@chromium.org
Status: Fixed (was: Available)
this will be part of the 0.8.36.4+ release

Comment 5 by dchan@chromium.org, Aug 1 2017

Labels: VerifyIn-61

Comment 6 by dchan@chromium.org, Jan 22 2018

Status: Archived (was: Fixed)

Comment 7 by vapier@chromium.org, Jun 21 2018

Status: Fixed (was: Archived)

Sign in to add a comment