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

Proxy doesn't strip the request URL for admin requests correctly #1797

Closed
ivankelly opened this issue May 17, 2018 · 1 comment
Closed

Proxy doesn't strip the request URL for admin requests correctly #1797

ivankelly opened this issue May 17, 2018 · 1 comment
Assignees
Labels

Comments

@ivankelly
Copy link
Contributor

@ivankelly ivankelly commented May 17, 2018

Expected behavior

Using proxy to do admin tasks should work.

Actual behavior

On client

org.apache.pulsar.client.admin.PulsarAdminException$NotAllowedException: HTTP 405 HTTP method PUT is not supported by this URL

From proxy log.

09:33:50.627 [pulsar-external-web-4-8] INFO  org.eclipse.jetty.server.RequestLog - 172.29.0.1 - - [17/May/2018:09:33:50 +0000] "PUT //172.29.0.4:8080/admin/v2/persistent/s3-offload-test/ns1/topic1/offload HTTP/1.1" 405 406 "-" "Jersey/2.25 (HttpUrlConnection 1.8.0_151)" 10

Steps to reproduce

Set up a broker with mostly default config (just set up zk stuff)
Set up a proxy with mostly default config (just set zk and brokerServiceURL)

I had

brokerServiceURL=http://pulsar-broker1:8080

where pulsar-broker1 resolves to 172.29.0.5
172.29.0.4 is the ip of the proxy itself.

Admin client is created with:

PulsarAdmin admin = new PulsarAdmin(new URL("http://172.29.0.4:8080"), "", "")
@massakam

This comment has been minimized.

Copy link
Contributor

@massakam massakam commented Jun 7, 2018

This issue has been fixed by #1862

@massakam massakam closed this Jun 7, 2018
@massakam massakam self-assigned this Jun 7, 2018
@massakam massakam added the type/bug label Jun 7, 2018
@massakam massakam added this to the 2.0.1-incubating milestone Jun 7, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
2 participants
You can’t perform that action at this time.