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.

Sign up for GitHub

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

Jump to bottom

Skip the interceptor for MediaType.MULTIPART_FORM_DATA #9217

Merged
merged 2 commits into from
Jan 18, 2021
Merged

Skip the interceptor for MediaType.MULTIPART_FORM_DATA #9217

merged 2 commits into from
Jan 18, 2021

Conversation

codelipenghui
Copy link
Contributor

Motivation

Skip the interceptor for MediaType.MULTIPART_FORM_DATA

Does this pull request potentially affect one of the following parts:

If yes was chosen, please highlight the changes

  • Dependencies (does it add or upgrade a dependency): (no)
  • The public API: (no)
  • The schema: (no)
  • The default values of configurations: (no)
  • The wire protocol: (no)
  • The rest endpoints: (no)
  • The admin cli options: (no)
  • Anything that affects deployment: (no)

Documentation

  • Does this pull request introduce a new feature? (no)

@codelipenghui codelipenghui self-assigned this Jan 15, 2021
@codelipenghui codelipenghui added this to the 2.8.0 milestone Jan 15, 2021
@315157973
Copy link
Contributor

/pulsarbot run-failure-checks

1 similar comment
@codelipenghui
Copy link
Contributor Author

/pulsarbot run-failure-checks

@codelipenghui codelipenghui merged commit 93b2239 into apache:master Jan 18, 2021
@codelipenghui codelipenghui deleted the penghui/skip-multi_form_data branch January 18, 2021 09:03
@codelipenghui codelipenghui added the cherry-picked/branch-2.7 Archived: 2.7 is end of life label Jan 19, 2021
codelipenghui added a commit that referenced this pull request Jan 19, 2021
@codelipenghui
Skip the interceptor for MediaType.MULTIPART_FORM_DATA (#9217)
4dcc738 
Skip the interceptor for `MediaType.MULTIPART_FORM_DATA`

(cherry picked from commit 93b2239)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@sijie sijie sijie approved these changes

@315157973 315157973 Awaiting requested review from 315157973

@hangc0276 hangc0276 Awaiting requested review from hangc0276

@jiazhai jiazhai Awaiting requested review from jiazhai

@merlimat merlimat Awaiting requested review from merlimat

Assignees

@codelipenghui codelipenghui

Labels
cherry-picked/branch-2.7 Archived: 2.7 is end of life release/2.7.1
Projects
None yet
Milestone
2.8.0
Development

Successfully merging this pull request may close these issues.

None yet
3 participants
@codelipenghui @315157973 @sijie