opendev/system-config
Code Issues Proposed changes

Merge "grafana: ensure snapshots api returns a 403"

Browse Source
This commit is contained in:
Zuul 2021-02-15 15:42:46 +00:00 committed by Gerrit Code Review
commit c3aeb3a1bb
2 changed files with 12 additions and 0 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

3
playbooks/roles/grafana/templates/grafana.vhost.j2
View File

@ -34,6 +34,9 @@
SSLCertificateKeyFile /etc/letsencrypt-certs/{{ inventory_hostname }}/{{ inventory_hostname }}.key
SSLCertificateChainFile /etc/letsencrypt-certs/{{ inventory_hostname }}/ca.cer
RewriteEngine on
RewriteRule "^/api/snapshots(.*?)$" "-" [F]
ProxyPass / http://localhost:3000/ retry=0
ProxyPassReverse / http://localhost:3000/

9
testinfra/test_grafana.py
View File

@ -25,3 +25,12 @@ def test_grafana_proxy(host):
'--resolve grafana.opendev.org:443:127.0.0.1 '
'https://grafana.opendev.org')
assert '<title>Grafana</title>' in cmd.stdout
def test_grafana_api_denial(host):
cmd = host.run('curl -s --insecure '
'--resolve grafana.opendev.org:443:127.0.0.1 '
'-XPOST -H "Accept: application/json" '
'-H "Content-Type: application/json" '
'-d \'{"dashboard": {}}\' '
'https://grafana.opendev.org/api/snapshots')
assert '403 Forbidden' in cmd.stdout