Add basic git repo host keys to known_hosts #751
Merged
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
#475 (comment) describes how to populate a remote server's
known_hosts
via deploy hook. The purpose is to avoid thehost key verification failed
error during the composer install task.This PR makes the
known_hosts
handling part of Trellis core:wordpress-install
role) tooI included both
rsa
anded25519
key types for GitLab. These work with current master and with theHostKeyAlgorithms
that #744 will apply to the SSH client. I avoided theecdsa
type for GitLab because it will be disallowed after #744.GitHub and Bitbucket seem to only offer the
rsa
type, before and after #744.Docs in roots/docs#78