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

Bug in bounding box creation for WMS 1.1.1 #878

Closed
om-henners opened this issue Jun 23, 2023 · 3 comments
Closed

Bug in bounding box creation for WMS 1.1.1 #878

om-henners opened this issue Jun 23, 2023 · 3 comments
Labels

Comments

@om-henners
Copy link

Currently this line causes an issue when building the URL parameter for the bounding box in WMS 1.1.1

request['bbox'] = ','.join([repr(x) for x in bbox])

and likewise in WMS 1.3.0, although only when the projection for the bounding box is in a non-geographic projection (because above if in a geographic projection the bbox is remapped to reverse the axis).

request['bbox'] = ','.join([repr(x) for x in bbox])

The result is that it adds quotes into the parameter string which causes downstream issues (or at least does in geoserver). For example a bounding box of

bbox = (0, 0, 1, 1, 'EPSG:28356')

is reformatted as 0,0,1,1,'EPSG:28356' when the server is expecting 0,0,1,1,EPSG:28356.

The solution is to use str instead of repr for the join.

Copy link

github-actions bot commented Oct 6, 2024

This Issue has been inactive for 90 days. In order to manage maintenance burden, it will be automatically closed in 7 days.

@github-actions github-actions bot added the stale label Oct 6, 2024
Copy link

This Issue has been closed due to there being no activity for more than 90 days.

@github-actions github-actions bot closed this as not planned Won't fix, can't repro, duplicate, stale Oct 20, 2024
@geographika geographika reopened this Nov 6, 2024
@geographika
Copy link
Contributor

Fixed with #955

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

No branches or pull requests

2 participants