ucphhpc / migrid-sync

MiGrid workspace where master branch is kept strictly in sync with SF upstream svn repo. Any development or experiments should use a branch. You probably want to fork your own clone or work e.g. on the edge branch if you wish to contribute.
GNU General Public License v2.0
3 stars 4 forks source link

sshfs doc. syntax error #17

Closed ARiis-63 closed 1 year ago

ARiis-63 commented 1 year ago

"MIG_GIT_REV": "6c71d37679f6750faa288365e68f4bc055e953fc" in guide for "SSHFS Drive on Windows" under setup syntax is described to seperate port using ":"

Wrong link "\sshfs\bla@bla.dk@io.erda.bla.bla:2222" Has to be like "\sshfs\bla@bla.dk@io.erda.bla.bla!2222"

according to (https://github.com/winfsp/sshfs-win#----sshfs-win--sshfs-for-windows) \sshfs[LOCUSER=]REMUSER@HOST[!PORT][\PATH] \sshfs.r[LOCUSER=]REMUSER@HOST[!PORT][\PATH] \sshfs.k[LOCUSER=]REMUSER@HOST[!PORT][\PATH] \sshfs.kr[LOCUSER=]REMUSER@HOST[!PORT][\PATH]

jonasbardino commented 1 year ago

Thanks for pointing that out - it was simple to fix so I did in the latest revision (e.g. experimental 4df9aa6b4784e54936c3c6b61c99047a7f96326d).

We always use default port 22 with local firewall port forward in production and the port is left out in that case, so we never hit the problem in practice. We recommend you make a similar port forward and use sftp_show_port=22 (i.e. SFTP_SHOW_PORT=22 in docker-migrid env) for a far more firewall- and user-friendly setup.