-
Notifications
You must be signed in to change notification settings - Fork 278
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
WFS GetFeature via POST #439
Labels
Comments
We're hitting size limits on our GET feature requests, so having support for POST would be great. |
Hello Post is not working for me. Is this issue solved in current branch? |
7 tasks
cehbrecht
pushed a commit
that referenced
this issue
Nov 26, 2020
* adds quick workaround to make filter post request Small change to allow testing the use of a xml query via post. * adds working draft of getPOSTGetFeatureRequest * Adds documentation and unimplemented args Unimplemented args have been added for consistency with other getfeature functions qui a log warning * Adds post for wfs 1.1.0 + cleanup * Refactors __init__ for PostRequest * removes print command * adds unit tests for PostRequest_1_1_0 * adds unit test for PostRequest_2_0_0 * add fixtures to postrequest tests * reformat of __init__ and typo fix * Formatting * fix set_featureid according to standard * Add bbox formating function for post * fix list args of getPOSTGetFeatureRequest * Removes repetitive condition * Removes unused imports * Add docstring to postrequest.py * Add featureversion to postrequest.py * Add storedquery to postrequest.py + refactor * Remove spaces from tag names * add test for stored query * add check to stored query for version * Modified docstring for storedQueries * Moved propertyname='*' from def to inside method For WFS 1.1.0 With Post method now being available, having propertyname='*' as a default value causes problems for the post request. Default value is now None, just like for 2.0.0. Since I don't know if there are cases where this default was necessary, added a condition when `method=get` to set propertynames to '*' is it's not assigned a value. * Removed uncessary conditional check * Change docstring for better wording
This can be closed now with #706. |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Is there support for sending WFS GetFeature requests via POST? I'd like to query the WFS service using a prepared WFS GetFeature filter in XML, but I can't get it to work with the current version.
The 'method' parameter seems to suggest it should work:
OWSLib/owslib/feature/wfs110.py
Lines 154 to 157 in d2a9722
But changing this to
method='Post'
and providing a filter as XML doesn't seem to work.I think the culprit is in the
OWSLib/owslib/feature/wfs110.py
Line 247 in d2a9722
I can prepare a PR to support a (fourth) way of using the GetFeature method, with just a filter (in XML) and method = 'Post'? Like so:
Are we in favor of supporting such a use-case?
The text was updated successfully, but these errors were encountered: