storeapi: handle 500 error response when releasing snap
Metadata
Current evaluation
Merged PR #2059 via commit 3a861f74c fixed snap release handling of HTML 500 responses, added debug output for store errors, and resolved #2028.
Resolution: closed resolved
Reason: PR #2059 was merged as commit 3a861f74c. Its tests and implementation properly handle HTML-formatted Store 500 responses and provide debug logging for easier diagnosis.
Evaluated by: mr-cal-bot
No scores available.
Issue body
- [x] Have you followed the [guidelines for contributing](https://github.com/snapcore/snapcraft/blob/master/CONTRIBUTING.md)?
- [x] Have you signed the [CLA](http://www.ubuntu.com/legal/contributors/)?
- [x] If this is a bugfix. Have you checked that there is a bug report open for the issue you are trying to fix on [bug reports](https://bugs.launchpad.net/snapcraft)?
- [x] If this is a new feature. Have you discussed the design on the [forum](https://forum.snapcraft.io)?
- [x] Have you successfully run `./runtests.sh static`?
- [x] Have you successfully run `./runtests.sh unit`?
-----
This PR fixes LP: [#1750177](https://bugs.launchpad.net/snapstore/+bug/1750177) and #2028 by properly handling the store returning a 500 error formatted as HTML instead of a typical error response. It also starts printing store error responses when in debug mode in order to streamline such fixes in the future.
Evaluation history
| Date | Model | Scores | Action | Summary |
|---|---|---|---|---|
| stealth/space-bunny-alpha | — | closed resolved | Merged PR #2059 via commit 3a861f74c fixed snap release handling of HTML 500 responses, added debug output for store errors, and resolved #2028. | |
| stealth/space-bunny-alpha | — | closed resolved | Merged PR #2059 (commit 3a861f74c) fixed storeapi handling of HTML 500 responses during snap release and added debug logging for store errors. | |
| stealth/space-bunny-alpha | — | closed resolved | Merged in #2059 (commit 3a861f74): Snapcraft now handles HTML 500 responses during snap release and logs store errors in debug mode, fixing #2028 and LP #1750177. | |
| qwen/qwen3.6-35b-a3b | — | — | Merged fix that properly handles 500 HTML error responses from the store during snap release. Adds debug logging for store errors to streamline future troubleshooting. Approved by reviewer and passed CI checks. | |
| qwen3.6-35b-a3b-mtp-q6 | — | — | Merged fix for handling 500 HTML error responses from the store during snap release. Adds debug logging for store errors. Review feedback on parameter passing was addressed prior to merge. |
Update history
No update history recorded yet.
Related work
-
Likely Fixed By:
snapcraft#2028
(confidence 100%)
PR #2059 explicitly identifies #2028 as fixed by the merged Store error handling change.
Related issues
| Issue | Project | State | Summary | Similarity |
|---|---|---|---|---|
| #1936 storeapi: handle errors even for >400 responses | snapcraft | merged | Merged PR #1936 (688588a0d) fixed Store API error handling for HTTP responses above 400, preventing failed responses from being mistaken for missing responses. | |
| #3403 storeapi: remove unused MissingSnapdError | snapcraft | merged | Merged cleanup by maintainer sergiusens removing the unused storeapi.MissingSnapdError exception class (11 lines deleted in 1 file) as commit 5a7cef39d, merged via PR #3403 with 1 approving review and no CI failures. |