Fix vhosts extra_parameters indentation

[paulgration]

Uses the indent Jinja filter to ensure that all lines of the extra_parameters variable are indented to the same level (2 to match the rest of the vhosts template)

Example playbook snippet:

- name: Example play
  become: true
  vars:
    apache_vhosts:
      - servername: example.com
        extra_parameters: |
          Header always set Strict-Transport-Security "max-age=63072000; includeSubDomains; preload"
          Header always set X-Content-Type-Options: "nosniff"
          Header always set X-Frame-Options: "SAMEORIGIN"

Would previously produce this output:

<VirtualHost *:80>
  ServerName example.com

  Header always set Strict-Transport-Security "max-age=63072000; includeSubDomains; preload"
Header always set X-Content-Type-Options: "nosniff"
Header always set X-Frame-Options: "SAMEORIGIN"

</VirtualHost>

With this change the output is:

<VirtualHost *:80>
  ServerName example.com

  Header always set Strict-Transport-Security "max-age=63072000; includeSubDomains; preload"
  Header always set X-Content-Type-Options: "nosniff"
  Header always set X-Frame-Options: "SAMEORIGIN"

</VirtualHost>

I added first and indentfirst keyword arguments to cover all bases (indentfirst was renamed to first in Jinja 2.10).

[stale]

This pull request has been marked 'stale' due to lack of recent activity. If there is no further activity, the PR will be closed in another 30 days. Thank you for your contribution!

Please read this blog post to see the reasons why I mark pull requests as stale.

[stale]

This pull request has been closed due to inactivity. If you feel this is in error, please reopen the pull request or file a new PR with the relevant details.

[jpiron]

The indentation issue is still here.
Is there anyway to reopen this PR ?

[paulgration]

The indentation issue is still here.
Is there anyway to reopen this PR ?

Not that I can see - shame the stale bot closed it before review

[jpiron]

Indeed, I can understand the usage of a stale bot for the issues but for MRs it discourages people from submitting !

[paulgration]

@geerlingguy are you able to review?

[stale]

This pull request is no longer marked for closure.

[geerlingguy]

Letting tests run on it again. Sorry about that—issues like this are issues I do want to merge (they fix actual bugs/broken-ish functionality in the role), but sometimes they seemingly get lost in the barrage.