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

[RFE][EmbeddedAnsible] Configurable verify_ssl for git repositories #20732

Closed
NickLaMuro opened this issue Oct 26, 2020 · 1 comment
Closed

Comments

@NickLaMuro
Copy link
Member

Currently, verify_ssl is set to "false" for all GitRepository objects associated with EmbeddedAnsible:

https://github.com/ManageIQ/manageiq/blob/7ea2845fc/app/models/manageiq/providers/embedded_ansible/automation_manager/configuration_script_source.rb#L53-L79

It is assumed that it probably was just due to a first pass when implementing this originally:

#19027

Which allowed for the most permissive checks, if unfortunately less secure. Ideally this would be something that could be configured in the UI.

Adding a verify_ssl checkbox, and configuring the backend should be doable, and we most likely can default it to "unchecked" to retain the current functionality by default.

@NickLaMuro
Copy link
Member Author

Fixed via #21298 and ManageIQ/manageiq-ui-classic#7768

@Fryguy Fryguy added this to Morphy in Roadmap Aug 13, 2021
@Fryguy Fryguy added this to the Morphy milestone Aug 13, 2021
@chessbyte chessbyte moved this from Najdorf to Morphy in Roadmap Sep 16, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Roadmap
  
Morphy
Development

No branches or pull requests

3 participants