Skip to content

Commit 53267d2

Browse files
authored
Use bcrypt with password_hash (#35)
Changes: - Reinstate the `bcrypt` option for `password_hash` This has been done since `node_exporter` expects hashed password for basic authentication to be created with `brypt`. I initially removed because the following appears when running the `monitoring_client` role: ``` AttributeError: module 'bcrypt' has no attribute '__about__' ``` The play proceeds and `node_exporter` is correctly installed. I tried pinning `bcrypt` to `4.0.1` as suggested in pyca/bcrypt#684 but the traceback still appears. Without `bcrypt` the `node_exporter` service fails to start.
1 parent 075d789 commit 53267d2

File tree

1 file changed

+1
-1
lines changed

1 file changed

+1
-1
lines changed

roles/monitoring_client/templates/node_exporter_web.yml.j2

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -2,4 +2,4 @@ tls_server_config:
22
cert_file: node_exporter.crt
33
key_file: node_exporter.key
44
basic_auth_users:
5-
{{ monitoring_client_exporter_username }}: {{ monitoring_client_exporter_password | password_hash }}
5+
{{ monitoring_client_exporter_username }}: {{ monitoring_client_exporter_password | password_hash('bcrypt') }}

0 commit comments

Comments
 (0)