Skip to content
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

Bump packer-plugin-sdk to address legacy SSH key algorithms in SSH communicator #218

Merged
merged 2 commits into from
May 4, 2022

Conversation

nywilken
Copy link
Contributor

@nywilken nywilken commented May 4, 2022

  • Bump github.com/hashicorp/packer-plugin-sdk with latest x/crypto/ssh fix
  • Add acceptance test case for rsa-sha2 bug

Closes hashicorp/packer#11623
Closes hashicorp/packer#11733
Closes #213

@nywilken nywilken requested a review from a team as a code owner May 4, 2022 00:03
Copy link
Contributor

@sylviamoss sylviamoss left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice! Good idea adding the test case 👍🏼

locals { timestamp = regex_replace(timestamp(), "[- TZ:]", "") }

source "amazon-ebs" "basic-example" {
ami_description = "Ubuntu 22.04 GoldenImage"
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Do you know this one tests what you need because of the ubuntu version?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yes, this particular version of ubuntu runs openssh version 8.9 which requires stronger ssh key algorithms, as of openssh 8.8. I added a comment to the HCL file. But maybe I should add one here?

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think it looks good the way it is!!

@nywilken nywilken merged commit e67f992 into main May 4, 2022
@nywilken nywilken deleted the nywilken/update-crypto-ssh branch May 4, 2022 17:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
2 participants