This appears to fail because the AttributeEncoder is defined at the end, after the ValueMap, which I don't think is schema-valid. https://wiki.shibboleth.net/confluence/display/IDP30/MappedAttributeDefinition I just want to make sure we're right about that before I change it.