[#4638]fixbug content type is not case insensitive #4641
Merged
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
- Modify the names of factories variables in multiple classes to factories to unify the naming style - Update the factories parameter in the method signature to factories to maintain consistency - Fixed typos in some variables and methods
…ification - Modify the content type comparison logic in the BodyProcessorCreator class and use a case-ignoring comparison method - Add test interfaces to ConsumerController and ProviderController to verify content types in different cases - Added User class to test the data transmission of the interface - Added ProviderIT class for integration testing to verify the processing of different case content types
liubao68
changed the title
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
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.
support RFC 7231, ignore case for content-type
issue：#4638