search

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 3 forks source link

SFTP Setup in Webinterface shows wrong port #39

Closed benibr closed 2 months ago

benibr commented 2 months ago

When the following config is applied via docker-migrid, it should show port 2222 in the Webinterface

SFTP_PORT=2222
SFTP_SUBSYS_PORT=22222
SFTP_SHOW_PORT=2222

but instead Port 22 is shown. This would result in users not being able to access SFTP because the would use the wrong config. Screenshot

jonasbardino commented 2 months ago

Thanks for reporting it. AFAICT this is specific to setups setting daemon_show_address empty in which case the sftp_show_port does get inserted in MiGserver.conf, but the option remains commented out. That is, more generateconfs logic to iron out for the fully distributed setup as outlined in the milestone 5 on docker-migrid (https://github.com/ucphhpc/docker-migrid/milestone/5). I'll take a closer look.

jonasbardino commented 2 months ago

The migrid code should now address it correctly with svn r6005 / git experimental c3164d7f747c02b09a16e2bb508e74bf01fa0863. Please test and reopen or create an issue in docker-migrid if anything is still not working correctly there.