Parse error when default value for string array property is a string array

I have an OpenAPI 3 apispec with the following schema defined:

            "DocFilterSettings": {
                "type": "object",
                "description": "This model describes the attributes used to filter a document.",
                "properties": {
                    "tag_list": {
                        "description": "This is the set of tags to include in the output.",
                        "type": "array",
                        "items": {
                            "type": "string"
                        },
                        "default": [
                            "h1",
                            "h2"
                        ]
                    }
                }
            }

So… a property which is an array of strings with a default value which is also an array of strings. Unfortunately, when I use swagger-parser v2.0.8 to read in this apispec, I get this error returned in the SwaggerParseResult object:

attribute components.schemas.DocFilterSettings.default is not of type `string`

I initially found this in swagger-parser v2.0.4 but it also occurs with the latest (2.0.8).

It seems like the default value for an array of strings should itself be an array of strings, no? Also, a secondary issue is that the error message doesn’t reference the actual name of the schema property which was found to be in error.

Here’s a small code snippet (part of a unit test) which shows the code I’m using to read in the apispec:

        ParseOptions options = new ParseOptions();
        options.setResolve(true);
        options.setResolveCombinators(true);
        SwaggerParseResult parseResult = new OpenAPIParser().readLocation(path, null, options);
        if (parseResult.getMessages() != null && !parseResult.getMessages().isEmpty()) {
            parseResult.getMessages().forEach(s -> System.out.println(s));
            fail("Error while loading apispec!");
        }

        OpenAPI apispec = parseResult.getOpenAPI();
        assertNotNull(apispec);

[gracekarina]

@padamstx thanks for the heads up, I’m looking into it (in parser) right now.

[padamstx]

@gracekarina Thanks for fixing this so quickly! Can’t wait to try it out 😃