Fix Maven plugin content-type for YAML AsyncAPI files (#6712) #7083
+207
−2
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Summary
application/x-yamlfor YAML files during artifact registrationRoot Cause
When using the Maven plugin with
autoRefsenabled, the plugin determined the correct content-type from the file extension but failed to set it on the artifact object before registration. This caused the registration code to default toapplication/json, resulting in YAML content being registered with the wrong content-type.Changes Made
Maven Plugin Fix (
RegisterRegistryMojo.java:177-180):.yml,.yaml) are registered withapplication/x-yamlTest Coverage (
RegisterAsyncApiAvroAutoRefsTest.java):application/x-yamlapplication/jsonIntegration Test (
GroupsResourceTest.java):?references=REWRITEasyncapi-shopping-cart.ymlandavro/ShoppingCartCreated.avscRelated Issue
Fixes #6712
Test Plan
RegisterAsyncApiAvroAutoRefsTestto verify Maven plugin sets correct content-typesGroupsResourceTest#testGetArtifactVersionWithReferencesYamlContentTypeto verify YAML handlingapplication/x-yamlnotapplication/json