Skip to content

Added support for multiple certs#45

Closed
ab77 wants to merge 8 commits intochef-partners:masterfrom
ab77:master
Closed

Added support for multiple certs#45
ab77 wants to merge 8 commits intochef-partners:masterfrom
ab77:master

Conversation

@ab77
Copy link
Copy Markdown

@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
Copy link
Copy Markdown
Author

ab77 commented Jul 2, 2015

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

@siddheshwar-more
Copy link
Copy Markdown

@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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants