Navigation Menu

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

Jump to bottom

HttpURLConnection does not support PATCH #4199

Closed
Maijh97 opened this issue Nov 11, 2020 · 0 comments · Fixed by #4211
Closed

HttpURLConnection does not support PATCH #4199

Maijh97 opened this issue Nov 11, 2020 · 0 comments · Fixed by #4211
Assignees
@Maijh97
Labels
area/Naming kind/bug Category issues or prs related to bug.
Milestone
1.4.1

Comments

@Maijh97
Copy link
Collaborator

Maijh97 commented Nov 11, 2020 •
edited

Describe the bug

When DistroFilter forwards to the responsible node, if the http method is PATCH, an exception will occur: java.net.ProtocolException: Invalid HTTP method: PATCH.

Expected behavior
support forwarding requests with Http Method being PATCH.

Acutally behavior
A clear and concise description of what you actually to happen.

How to Reproduce
Steps to reproduce the behavior:

  1. Cluster starts, three nodes.
  2. Execute curl -X PATCH'http://127.0.0.1:8848/nacos/v1/ns/instance?port=8848&healthy=true&ip=11.11.11.11&weight=1.0&serviceName=nacos.test.3&encoding=UTF-8'

Desktop (please complete the following information):

  • OS: MAC OS
  • Version nacos-server 1.2.0 - 1.4.0
  • Module naming
  • SDK

Additional context
Add any other context about the problem here.
@Maijh97 Maijh97 self-assigned this Nov 11, 2020
@Maijh97 Maijh97 added the kind/bug Category issues or prs related to bug. label Nov 11, 2020
@Maijh97 Maijh97 added this to the 1.4.1 milestone Nov 11, 2020
@Maijh97 Maijh97 mentioned this issue Nov 12, 2020
Merged
5 tasks
KomachiSion pushed a commit that referenced this issue Nov 12, 2020
@Maijh97
for #4199, Replace the http client implementation used by DistroFilte…
a47e726 
…r. (#4211)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@Maijh97 Maijh97

Labels
area/Naming kind/bug Category issues or prs related to bug.
Projects
None yet
Milestone
1.4.1
Development

Successfully merging a pull request may close this issue.

[ISSUE #4199] Replace the http client implementation used by DistroFilter
1 participant
@Maijh97