Merge pull request #513 from joao-p-marques/fix/mx-puppet-bridge

Fix/mx puppet bridge
This commit is contained in:
Slavi Pantaleev 2020-05-22 10:53:37 +03:00 committed by GitHub
commit 4111e42070
No known key found for this signature in database
GPG key ID: 4AEE18F83AFDEB23
2 changed files with 3 additions and 3 deletions

View file

@ -68,7 +68,7 @@ matrix_mx_puppet_slack_configuration_yaml: |
# Slack OAuth settings. Create a slack app at https://api.slack.com/apps # Slack OAuth settings. Create a slack app at https://api.slack.com/apps
oauth: oauth:
enabled: false enabled: true
# Slack app credentials. # Slack app credentials.
# N.B. This must be quoted so YAML wouldn't parse it as a float. # N.B. This must be quoted so YAML wouldn't parse it as a float.
clientId: "{{ matrix_mx_puppet_slack_client_id }}" clientId: "{{ matrix_mx_puppet_slack_client_id }}"

View file

@ -50,7 +50,7 @@
}} }}
tags: tags:
- always - always
when: matrix_appservice_slack_enabled|bool when: matrix_mx_puppet_slack_enabled|bool
- name: Warn about reverse-proxying if matrix-nginx-proxy not used - name: Warn about reverse-proxying if matrix-nginx-proxy not used
debug: debug:
@ -60,7 +60,7 @@
Please make sure that you're proxying the `{{ something }}` Please make sure that you're proxying the `{{ something }}`
URL endpoint to the matrix-appservice-slack container. URL endpoint to the matrix-appservice-slack container.
You can expose the container's port using the `matrix_appservice_slack_container_http_host_bind_port` variable. You can expose the container's port using the `matrix_appservice_slack_container_http_host_bind_port` variable.
when: "matrix_appservice_slack_enabled|bool and matrix_nginx_proxy_enabled is not defined" when: "matrix_mx_puppet_slack_enabled|bool and matrix_nginx_proxy_enabled is not defined"
# ansible lower than 2.8, does not support docker_image build parameters # ansible lower than 2.8, does not support docker_image build parameters
# for self buildig it is explicitly needed, so we rather fail here # for self buildig it is explicitly needed, so we rather fail here