Skip to content

Conversation

@dpopp07
Copy link
Contributor

@dpopp07 dpopp07 commented Feb 28, 2023

The RFC for content-type header format allows optional whitespace before the semicolon. Our regular expressions for recognizing various mime types do not account for this, so valid mime types may not be recognized. This commit adjusts the expression to close that gap.

The RFC for content-type header format allows optional whitespace before the semicolon. Our regular expressions for recognizing various mime types do not account for this, so valid mime types may not be recognized. This commit adjusts the expression to close that gap. Signed-off-by: Dustin Popp <dpopp07@gmail.com>
@dpopp07 dpopp07 requested a review from padamstx February 28, 2023 15:15
Copy link
Contributor

@padamstx padamstx left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@CLAassistant
Copy link

CLAassistant commented Feb 28, 2023

CLA assistant check
All committers have signed the CLA.

@dpopp07 dpopp07 merged commit 58a7c7f into main Feb 28, 2023
@dpopp07 dpopp07 deleted the dp/json-mimetype-whitespace branch February 28, 2023 16:14
ibm-devx-sdk pushed a commit that referenced this pull request Feb 28, 2023
## [9.17.6](9.17.5...9.17.6) (2023-02-28) ### Bug Fixes * recognize content types with whitespace ([#200](#200)) ([58a7c7f](58a7c7f))
@ibm-devx-sdk
Copy link
Contributor

🎉 This PR is included in version 9.17.6 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

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

Labels

4 participants