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

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
Labels
area/Naming kind/bug Category issues or prs related to bug.
Milestone

Comments

@Maijh97
Copy link
Collaborator

Maijh97 commented Nov 11, 2020

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
penghao03 pushed a commit to penghao03/nacos that referenced this issue Nov 14, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/Naming kind/bug Category issues or prs related to bug.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant