Skip to content

CURLOPT_SSH_KNOWNHOSTS.md: strongly recommend using this#16204

Closed
bagder wants to merge 1 commit into
masterfrom
bagder/libcurl-knownhosts
Closed

CURLOPT_SSH_KNOWNHOSTS.md: strongly recommend using this#16204
bagder wants to merge 1 commit into
masterfrom
bagder/libcurl-knownhosts

Conversation

@bagder

@bagder bagder commented Feb 6, 2025

Copy link
Copy Markdown
Member

Make setopt fail for SSH backends not supporting knownhosts or pub md5

Make setopt fail for SSH backends not supporting knownhosts or pub md5
@vszakats

vszakats commented Feb 6, 2025

Copy link
Copy Markdown
Member

Would it be a good time to deprecate the MD5 public key option, in favour of SHA256?

@bagder

bagder commented Feb 6, 2025

Copy link
Copy Markdown
Member Author

Would it be a good time to deprecate the MD5 public key option, in favour of SHA256?

I think not, since only one of the SSH backends supports the sha256 one at this point.

@bagder bagder closed this in 3f7f180 Feb 6, 2025
@bagder bagder deleted the bagder/libcurl-knownhosts branch February 6, 2025 12:52
pps83 pushed a commit to pps83/curl that referenced this pull request Apr 26, 2025
Make setopt fail for SSH backends not supporting knownhosts or pub md5

Closes curl#16204
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Development

Successfully merging this pull request may close these issues.

2 participants