Created
September 9, 2015 22:10
-
-
Save pypt/94d747fe5180851196eb to your computer and use it in GitHub Desktop.
PyYAML: raise exception on duplicate keys on the same document hierarchy level
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
import yaml | |
from yaml.constructor import ConstructorError | |
try: | |
from yaml import CLoader as Loader | |
except ImportError: | |
from yaml import Loader | |
def no_duplicates_constructor(loader, node, deep=False): | |
"""Check for duplicate keys.""" | |
mapping = {} | |
for key_node, value_node in node.value: | |
key = loader.construct_object(key_node, deep=deep) | |
value = loader.construct_object(value_node, deep=deep) | |
if key in mapping: | |
raise ConstructorError("while constructing a mapping", node.start_mark, | |
"found duplicate key (%s)" % key, key_node.start_mark) | |
mapping[key] = value | |
return loader.construct_mapping(node, deep) | |
yaml.add_constructor(yaml.resolver.BaseResolver.DEFAULT_MAPPING_TAG, no_duplicates_constructor) | |
# Works fine (no duplicate keys) | |
yaml_data = yaml.load(''' | |
--- | |
foo: bar | |
baz: qux | |
''' | |
) | |
# Works fine (no duplicate keys on the same level) | |
yaml_data = yaml.load(''' | |
--- | |
foo: | |
bar: baz | |
baz: qux | |
bar: | |
bar: baz | |
baz: qux | |
''' | |
) | |
# Raises exception (has duplicate keys) | |
yaml_data = yaml.load(''' | |
--- | |
foo: bar | |
foo: qux | |
''' | |
) |
Nice! Thanks guys!
It does not seem to work when the duplicate key comes from an anchor:
build_template: &BUILD_TEMPLATE eks_container: cpu: 4 build_task: eks_container: cpu: 10 env: DEPLOY_PULL_REQUEST: true BUILD_ARGUMENTS: "-DtrafficInspection=false --parallel --profile -x test -x sonar" <<: *BUILD_TEMPLATE # Duplicate key eks_container and CPU not detectedAny idea on how to support this case? Thanks.
YAML allows to override keys from an anchor so I don't think it should be considered an error in the first place.
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Nice! Thanks guys!
It does not seem to work when the duplicate key comes from an anchor:
Any idea on how to support this case?
Thanks.