Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix: content type #63

Merged
merged 1 commit into from
Mar 2, 2024
Merged

fix: content type #63

merged 1 commit into from
Mar 2, 2024

Conversation

devhaozi
Copy link
Member

@devhaozi devhaozi commented Mar 2, 2024

Closes #

📑 Description

有些情况下,Content-Type 会携带编码之类的东西,这样会导致无法匹配:
例如:multipart/form-data; boundary=--------------------------686926246495000608771553application/json;charset=UTF-8

✅ Checks

  • My pull request adheres to the code style of this project
  • My code requires changes to the documentation
  • I have updated the documentation as required
  • All the tests have passed

ℹ Additional Information

@devhaozi devhaozi requested a review from hwbrzzl March 2, 2024 04:01
@devhaozi devhaozi self-assigned this Mar 2, 2024
@devhaozi devhaozi enabled auto-merge (squash) March 2, 2024 04:39
Copy link
Contributor

@hwbrzzl hwbrzzl left a comment

Choose a reason for hiding this comment

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

Good catch, does the Gin driver need to be optimized, too?

@devhaozi devhaozi merged commit 5a7d285 into master Mar 2, 2024
7 checks passed
@devhaozi devhaozi deleted the haozi/content branch March 2, 2024 04:59
@devhaozi
Copy link
Member Author

devhaozi commented Mar 2, 2024

Good catch, does the Gin driver need to be optimized, too?

Not sure, pending further testing.

@devhaozi
Copy link
Member Author

devhaozi commented Mar 2, 2024

Good catch, does the Gin driver need to be optimized, too?

Not sure, pending further testing.

Not need, it has a filter.
image

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

Successfully merging this pull request may close these issues.

2 participants