Don't force port 22 for rsync backup
SoMuchForSubtlety opened this issue · comments
Currently, the backup script has port 22 hardcoded.
mailinabox/management/backup.py
Line 19 in 3a7de05
This prevents setting a non-standard port via ~/.ssh/config
and I don't see any upsides since port 22 is the default anyway.
I need to use port 23 since that is what's being used by hetzner storage box.
I have this implemented on a fork of mine - if anyone wants to make a PR you can start by cherry-picking this commit:
ddavness/power-mailinabox@8a0805d
Yeah I can't think of a reason why -p 22
should be there, and I would accept a pull request removing it.
@SoMuchForSubtlety this is merged in 61 and works now, can this be closed?
Thanks.