Skip to content

optionally use POST insetad of GET for WMSGetFeatureInfo - trac ticket 3357#1507

Open
bkiselka wants to merge 1 commit intoopenlayers:masterfrom
bkiselka:3357_WMSGetFeatureInfo_POST
Open

optionally use POST insetad of GET for WMSGetFeatureInfo - trac ticket 3357#1507
bkiselka wants to merge 1 commit intoopenlayers:masterfrom
bkiselka:3357_WMSGetFeatureInfo_POST

Conversation

@bkiselka
Copy link
Copy Markdown
Contributor

@bkiselka bkiselka commented Nov 22, 2016

add option usePOST for optionally using POST instead of GET for WMSGetFeatureInfo requests (idea from Protocol/HTTP.js)

fixes https://trac.osgeo.org/openlayers/ticket/3357

@ahocevar
Copy link
Copy Markdown
Member

You decided to reuse layers, but not wmsOptions? A bonus would be to make this consistent and reuse both.

Also, what happens when using drillDown with POST? I think the change would be clearer if you added unit tests.

@bkiselka
Copy link
Copy Markdown
Contributor Author

I must admit, I did only tested the this.drillDown === false section.
Though I think it should work, I'll add some unit tests.

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