Skip to content
This repository has been archived by the owner on Apr 26, 2023. It is now read-only.

Java: token-based security requires that the serverUrl includes /rest #5

Closed
kellyhutchins opened this issue Feb 15, 2014 · 5 comments
Closed
Assignees
Milestone

Comments

@kellyhutchins
Copy link
Member

Update the proxy so that /rest is not required.

@kellyhutchins kellyhutchins added this to the 1.0 milestone Feb 15, 2014
@bsvensson bsvensson added the Java label Feb 15, 2014
@esoekianto
Copy link
Contributor

@kellyhutchins is this where "/rest" must be included in the white list url right?

I think this has been taken care of in java, i just tested and debug the java proxy. the way it gets the infoUrl is actually from the request url, not from the url in the config.

It gets the index of "/rest" in the request url, then the infoUrl is the substring of that url from 0 to that index, plus "/rest/info?f=json"

int infoIndex = url.toLowerCase().indexOf("/rest");
String infoUrl = url.substring(0, infoIndex);
infoUrl += "/rest/info?f=json";

I think we are good here for java.

@bsvensson
Copy link
Member

@esoekianto What if your domain starts with "rest" as in http://reston.example.com ...

@esoekianto
Copy link
Contributor

@bsvensson It would not work, good catch. I guess it should check for "/rest/". I can make the changes #18

@jgravois how about in .Net for this?

@bsvensson
Copy link
Member

@MehulChoksey - Could you verify this JSP issue?

@esoekianto
Copy link
Contributor

@kellyhutchins @bsvensson

I think this can be closed and the "better" way should be address in #23

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

4 participants