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

Fixed proxyUrl in Cesium Layers, add demo to cors #1815

Merged
merged 1 commit into from
May 12, 2017

Conversation

offtherailz
Copy link
Member

Two problems:

  1. Cors was used if you ask from https to http server (as the map in test)
  2. the proxyUrl was not correctly parsed in the case of object

Now WMTSLayer, WMSLayer and TileProvider layers in cesium get the proxy from global ConfigUtils object so I used ConfigUtils also in the proxyUrl retrival parts of the code, For the future we could implement it in a more functional style.

@coveralls
Copy link

Coverage Status

Coverage remained the same at 79.314% when pulling b55dc76 on offtherailz:fix_wmts into a99bac6 on geosolutions-it:master.

@offtherailz offtherailz merged commit 4ce1460 into geosolutions-it:master May 12, 2017
@offtherailz offtherailz deleted the fix_wmts branch May 3, 2019 13:15
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.

2 participants