MultivaluedStringStringHashMap crd representation

[shawkins]

From keycloak/keycloak#20937 the schema generated for

public class MultivaluedStringStringHashMap extends HashMap<String, List<String>> {
}

The same schema is generated for the original type MultivaluedHashMap<String, String>.

@andreaTP what is / was the expectation here?

[shawkins]

The generated schema either way is:

config:
  additionalProperties:
    items:
      type: "string"
    type: "array"
  type: "object"

I had missed in my first glance at the keycloakrealmimports crd that the additionalProperties are an array type as expected. There are also configs present with the schema

config:
  additionalProperties:
    type: string
  type: object

but those are not the ones targeted by the swaps.

If the MultivaluedHashMap<String, String> schema is now correct, then I'll just remove those schemaswaps in keycloak.

[andreaTP]

@shawkins do you mind pasting the generated type?

I think that some improvements have been done in the meanwhile in this area, i.e. #4501
But I can't tell on top of my head if this is completely handled.

I confirm that this encoding is the expected one:

config:
  additionalProperties:
    items:
      type: "string"
    type: "array"
  type: "object"

[shawkins]

Great, so I'll close this and resolve the keycloak issue by removing the MultivaluedStringStringHashMap and swaps.