Skip to content

Commit

Permalink
Merge pull request #147 from C2FO/CHAN-6645
Browse files Browse the repository at this point in the history
Fix SFTP Backend to consider configured ports other than 22
  • Loading branch information
dustin-schnee authored Nov 21, 2023
2 parents 6294f6d + f81cc14 commit 70f09c4
Show file tree
Hide file tree
Showing 3 changed files with 5 additions and 43 deletions.
42 changes: 0 additions & 42 deletions .github/workflows/whitesource.yml

This file was deleted.

4 changes: 4 additions & 0 deletions CHANGELOG.md
Original file line number Diff line number Diff line change
Expand Up @@ -5,6 +5,10 @@ The format is based on [Keep a Changelog](https://keepachangelog.com/en/1.0.0/),
and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0.html).
## [Unreleased]

## [6.9.1] - 2023-11-21
### Fixed
- fixed sftp backend to consider configured ports other than 22

## [6.9.0] - 2023-09-27

### Security
Expand Down
2 changes: 1 addition & 1 deletion backend/sftp/options.go
Original file line number Diff line number Diff line change
Expand Up @@ -61,7 +61,7 @@ func getClient(authority utils.Authority, opts Options) (Client, io.Closer, erro
Config: sshConfig,
}
// default to port 22
host := authority.Host()
host := fmt.Sprintf("%s:%d", authority.Host(), authority.Port())
if authority.Port() == 0 {
host = fmt.Sprintf("%s:%d", host, 22)
}
Expand Down

0 comments on commit 70f09c4

Please sign in to comment.