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

Use reply content as an error string if it is plain text #60481

Merged
merged 1 commit into from
Feb 6, 2025

Conversation

uclaros
Copy link
Contributor

@uclaros uclaros commented Feb 6, 2025

Description

For example, a search with empty collections on the planetary computer will now show:
collection is required instead of the more cryptic Unprocessable Content, the human readable for http 422.

@uclaros uclaros added Bug Either a bug report, or a bug fix. Let's hope for the latter! STAC Related to SpatioTemporal Asset Catalog labels Feb 6, 2025
@github-actions github-actions bot added this to the 3.42.0 milestone Feb 6, 2025
Copy link

github-actions bot commented Feb 6, 2025

🪟 Windows builds

Download Windows builds of this PR for testing.
Debug symbols for this build are available here.
(Built from commit c25b136)

🪟 Windows Qt6 builds

Download Windows Qt6 builds of this PR for testing.
(Built from commit c25b136)

@nyalldawson nyalldawson merged commit c004c66 into qgis:master Feb 6, 2025
43 checks passed
@uclaros uclaros deleted the stac-better-error branch February 7, 2025 07:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Bug Either a bug report, or a bug fix. Let's hope for the latter! STAC Related to SpatioTemporal Asset Catalog
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants