Skip to content
This repository was archived by the owner on Mar 17, 2026. It is now read-only.

Add normalization to suppress false positives for booleans in diff mode#3

Merged
clefrancq merged 1 commit intoflow_configfrom
CLOUDTRUST-5987-diff-dict-false-positives
Apr 15, 2025
Merged

Add normalization to suppress false positives for booleans in diff mode#3
clefrancq merged 1 commit intoflow_configfrom
CLOUDTRUST-5987-diff-dict-false-positives

Conversation

@clefrancq
Copy link
Copy Markdown

Summary

Addresses: https://projectportal.elca.ch/jira/browse/CLOUDTRUST-5987
Boolean values in dictionaries show up as false positive diffs when quotes are not included in script.

This PR adds a util normalize_diffmode_boolean(dictionary) that converts the boolean values in the dictionary to lower-case strings, which will match the API response.

Modules corrected

keycloak_client: attributes, protocol mapper config
keycloak_clientscope: protocol mapper config

@clefrancq clefrancq self-assigned this Apr 14, 2025
@clefrancq clefrancq requested review from basbeu, damu46 and vloup April 14, 2025 12:52
@clefrancq clefrancq merged commit fdcf660 into flow_config Apr 15, 2025
1 of 2 checks passed
@clefrancq clefrancq deleted the CLOUDTRUST-5987-diff-dict-false-positives branch April 15, 2025 12:11
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants