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

Added support for multiple certs #45

Closed
wants to merge 8 commits into
base: master
from

Conversation

Projects
None yet
2 participants
@ab77

ab77 commented Jul 1, 2015

The current enable.rb expects a single certificate in the Certificates.pem bundle. This fails in a scenario where a VM role is deployed with SSH key authentication. In this instance, both the SSH cert/key and the Azure Management certificate/key are present in the bundle file.

This update solves this case by iterating over the cert/key pair and attempting to decrypt using each of these, until a successful decryption is achieved or end of bundle is reached,

-- ab1

@ab77

This comment has been minimized.

Show comment
Hide comment
@ab77

ab77 Jul 2, 2015

If anyone has experience writing unit tests, please update azure_enable_spec.rb as I am struggling..

ab77 commented Jul 2, 2015

If anyone has experience writing unit tests, please update azure_enable_spec.rb as I am struggling..

@siddheshwar-more

This comment has been minimized.

Show comment
Hide comment
@siddheshwar-more

siddheshwar-more Jul 2, 2015

Contributor

@ab77 - We are verifying this changes.

Contributor

siddheshwar-more commented Jul 2, 2015

@ab77 - We are verifying this changes.

@ab77 ab77 closed this Apr 25, 2016

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