Skip to content

Commit

Permalink
Fix woke and test workload
Browse files Browse the repository at this point in the history
  • Loading branch information
amandahla committed Aug 28, 2023
1 parent 08152d6 commit 7340d93
Show file tree
Hide file tree
Showing 3 changed files with 10 additions and 1 deletion.
8 changes: 8 additions & 0 deletions .woke.yaml
Original file line number Diff line number Diff line change
@@ -0,0 +1,8 @@
ignore_files:
- lib/charms/nginx_ingress_integrator/v0/nginx_route.py
- pyproject.toml
- src/grafana_dashboards/synapse.json
- tests/unit/test_synapse_workload.py
rules:
# Ignore "grandfathered" used by SAML configuration.
- name: grandfathered
2 changes: 1 addition & 1 deletion src/synapse/workload.py
Original file line number Diff line number Diff line change
Expand Up @@ -205,7 +205,7 @@ def _create_pysaml2_config(charm_state: CharmState) -> typing.Dict:
# login.staging.canonical.com and login.canonical.com
# dont send uid in SAMLResponse so this will map
# fullname to uid
if "canonical.com" in saml_config["metadata_url"]:
if "ubuntu.com" in saml_config["metadata_url"]:
sp_config["attribute_map_dir"] = "/data/attributemaps"

return sp_config
Expand Down
1 change: 1 addition & 0 deletions tests/unit/test_synapse_workload.py
Original file line number Diff line number Diff line change
Expand Up @@ -127,6 +127,7 @@ def test_enable_saml_success():
}
},
"allow_unknown_attributes": True,
"attribute_map_dir": "/data/attributemaps",
},
"user_mapping_provider": {
"config": {
Expand Down

0 comments on commit 7340d93

Please sign in to comment.