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
type/bug The PR fixed a bug or issue reported a bug

Comments

@ivankelly
Copy link
Contributor

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
Copy link
Contributor

massakam commented Jun 7, 2018

This issue has been fixed by #1862

@massakam massakam closed this as completed Jun 7, 2018
@massakam massakam self-assigned this Jun 7, 2018
@massakam massakam added the type/bug The PR fixed a bug or issue reported a 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
Labels
type/bug The PR fixed a bug or issue reported a bug
Projects
None yet
Development

No branches or pull requests

2 participants