[Fixes #14111] Fix Harvester lookup failure caused by trailing ? during the remote resource upload through the API#14112
Conversation
There was a problem hiding this comment.
Code Review
This pull request updates the WMS remote handler to correctly format the OWS URL by conditionally adding the query string, preventing trailing question marks when no query parameters are present. The feedback suggests using urllib.parse methods for a more robust and idiomatic approach to URL manipulation.
|
@Gpetrak I suspect this won't cover the cases where the URL includes parameters. |
@giohappy , I saw that get_cleaned_url_params clears the standard parameters like |
This PR was created according to this issue: #14111
Checklist
For all pull requests:
The following are required only for core and extension modules (they are welcomed, but not required, for contrib modules):
Submitting the PR does not require you to check all items, but by the time it gets merged, they should be either satisfied or inapplicable.