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

[Features/108] Support for ssl module #109

Merged
merged 7 commits into from
Jun 22, 2022
Merged

[Features/108] Support for ssl module #109

merged 7 commits into from
Jun 22, 2022

Conversation

xtianae7
Copy link
Member

Requirements

  • Filling out the template is required. Any pull request that does not include enough information to be reviewed in a timely manner may be closed at the maintainers' discretion.
  • All new code requires tests to ensure against regressions
  • Remember to set idealista:develop as base branch;

Description of the Change

  • To enable you have to set nginx_ssl_enabled to true
  • Install http_ssl_module
  • Import certificates

For test purposes, in molecule, a dummy self signed certificate has been added.

Benefits

Making the installation transparent will make it easier for the users to configure and to use.

Possible Drawbacks

None AFAIK

Applicable Issues

Closes #108

@xtianae7 xtianae7 merged commit fcfcf2a into develop Jun 22, 2022
@xtianae7 xtianae7 deleted the features/108 branch June 22, 2022 10:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants