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

Interceptor wms reflector option for interceptor #280

Merged
merged 2 commits into from
Oct 17, 2017

Conversation

ahennr
Copy link
Member

@ahennr ahennr commented Oct 12, 2017

This PR introduces an option for GeoServer interceptor (parameter useReflect) to call WMS reflector endpoint of GeoServer.

An example call to interceptor (layer dumpy in namespace humpty) would be:

http://localhost:9090/geoserver.action?layers=humpty:dumpy&useReflect=true

@ahennr ahennr force-pushed the interceptor-wms-reflector-option branch from 5f50ba1 to a021299 Compare October 13, 2017 06:58
@buehner
Copy link
Member

buehner commented Oct 16, 2017

LGTM 👍
very nice addition

@ahennr
Copy link
Member Author

ahennr commented Oct 17, 2017

THX for review @buehner

@ahennr ahennr merged commit bd55e96 into terrestris:master Oct 17, 2017
@ahennr ahennr deleted the interceptor-wms-reflector-option branch October 17, 2017 13:06
@marcjansen
Copy link
Member

marcjansen commented Oct 25, 2017

I think this breaks backwards compatibility by effectively removing the previously public method URI getGeoServerBaseURIFromNameSpace(String geoServerNamespace).

I think this should have at least deserved a release.

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

Successfully merging this pull request may close these issues.

3 participants