Skip to content

fix: invalid array string types#209

Closed
perher wants to merge 1 commit intoeclipse-dataspace-protocol-base:mainfrom
perher:fix/array-string-type
Closed

fix: invalid array string types#209
perher wants to merge 1 commit intoeclipse-dataspace-protocol-base:mainfrom
perher:fix/array-string-type

Conversation

@perher
Copy link
Copy Markdown

@perher perher commented Jul 3, 2025

What this PR changes/adds

Fixes array schema definitions where string is used instead of a proper JSON schema type

Why it does that

To be valid according to the 2019-09 JSON schema specification

Further notes

Identified two files where this pattern was used and updated those.

Linked Issue(s)

Closes #208

Please be sure to take a look at the contributing guidelines and our etiquette for pull requests.

@jimmarino jimmarino self-requested a review July 4, 2025 10:09
@perher
Copy link
Copy Markdown
Author

perher commented Jul 4, 2025

this was included in #211

@perher perher closed this Jul 4, 2025
@perher perher deleted the fix/array-string-type branch July 4, 2025 12:50
@jimmarino
Copy link
Copy Markdown
Contributor

Thanks @perher for spotting this and also proposing a solution.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

'"items": "string"' is not a valid json schema

2 participants