Skip to content

Conversation

@cloutierMat
Copy link
Member

@cloutierMat cloutierMat commented Jan 8, 2026

Motivation

#13551 introduced an error in the openapi spec. The /init/stage-invalid endpoint is meant to return plain text, not xml.

Looking at the response, I also noticed that the text was not sent, essentially returning an empty message. so I took the opportunity to improve the response returned in case of a stage not found.

Changes

  • updated the content type of the openapi specs
  • fix error handling in the init script handler

@cloutierMat cloutierMat added area: documentation Issues related to LocalStack documentation semver: patch Non-breaking changes which can be included in patch releases docs: skip Pull request does not require documentation changes notes: skip Pull request does not have to be mentioned in the release notes labels Jan 8, 2026
@cloutierMat cloutierMat marked this pull request as ready for review January 8, 2026 20:22
@cloutierMat cloutierMat changed the title fix openapi spec and response fix/openapi spec Jan 8, 2026
@github-actions
Copy link

github-actions bot commented Jan 8, 2026

Test Results - Preflight, Unit

23 046 tests  ±0   21 201 ✅ ±0   6m 13s ⏱️ -5s
     1 suites ±0    1 845 💤 ±0 
     1 files   ±0        0 ❌ ±0 

Results for commit 58f561f. ± Comparison against base commit 3e92156.

@github-actions
Copy link

github-actions bot commented Jan 8, 2026

Test Results (amd64) - Acceptance

7 tests  ±0   5 ✅ ±0   2m 58s ⏱️ -7s
1 suites ±0   2 💤 ±0 
1 files   ±0   0 ❌ ±0 

Results for commit 58f561f. ± Comparison against base commit 3e92156.

Copy link
Member

@pinzon pinzon left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for the fix. 👍

@github-actions
Copy link

github-actions bot commented Jan 8, 2026

Test Results (amd64) - Integration, Bootstrap

    5 files      5 suites   2h 36m 2s ⏱️
5 569 tests 5 008 ✅ 561 💤 0 ❌
5 575 runs  5 008 ✅ 567 💤 0 ❌

Results for commit 58f561f.

@github-actions
Copy link

github-actions bot commented Jan 8, 2026

LocalStack Community integration with Pro

    2 files  ±0      2 suites  ±0   1h 56m 50s ⏱️ -19s
5 162 tests ±0  4 764 ✅ ±0  398 💤 ±0  0 ❌ ±0 
5 164 runs  ±0  4 764 ✅ ±0  400 💤 ±0  0 ❌ ±0 

Results for commit 58f561f. ± Comparison against base commit 3e92156.

@cloutierMat cloutierMat merged commit eb26e3b into main Jan 8, 2026
58 of 61 checks passed
@cloutierMat cloutierMat deleted the fix/openapi-spec branch January 8, 2026 21:37
@carole-lavillonniere
Copy link
Contributor

🙈 Thanks for the fix!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area: documentation Issues related to LocalStack documentation docs: skip Pull request does not require documentation changes notes: skip Pull request does not have to be mentioned in the release notes semver: patch Non-breaking changes which can be included in patch releases

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants