-
-
Notifications
You must be signed in to change notification settings - Fork 4.7k
fix(files_external): Respect explicit port configuration #57793
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
b8c7080 to
79dfb62
Compare
|
The port field will now take precedence over the port parsed from the host, but the old behavior is still there to avoid breaking changes. |
|
/backport to stable33 |
|
/backport to stable32 |
|
/backport to stable31 |
It allows the use of the alternative SFTP port defined in the GUI (external storage), instead of the default port (22). Signed-off-by: pac0san <7056343+pac0san@users.noreply.github.com> Signed-off-by: provokateurin <kate@provokateurin.de>
79dfb62 to
e48f871
Compare
|
Thanks for your first pull request and welcome to the community! Feel free to keep them coming! If you are looking for issues to tackle then have a look at this selection: https://github.com/nextcloud/server/issues?q=is%3Aopen+is%3Aissue+label%3A%22good+first+issue%22 |
It allows the use of the alternative SFTP port defined in the GUI (external storage), instead of the default port (22).
Summary
TODO
Checklist
3. to review, feature component)stable32)