Skip to content

Conversation

FranzBusch
Copy link
Contributor

## Motivation

To aid debugging and troubleshooting, it is helpful to understand which Service led to the error.

Modifications

Add the service name to the serviceFinishedUnexpectedly error.

Result

Easier to debug failing services.

Copy link
Collaborator

@czechboy0 czechboy0 left a comment

Choose a reason for hiding this comment

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

One suggestion, otherwise lgtm

## Motivation

To aid debugging and troubleshooting, it is helpful to understand which Service led to the error.

## Modifications

Add the service name to the `serviceFinishedUnexpectedly` error.

## Result

Easier to debug failing services.
@areese
Copy link

areese commented Oct 7, 2025

Thank you for filing and fixing this.

@FranzBusch FranzBusch added the 🆕 semver/minor Adds new public API. label Oct 7, 2025
@FranzBusch
Copy link
Contributor Author

Merging over the API break since this is actually an ABI break

@FranzBusch FranzBusch merged commit 0fcc4c9 into swift-server:main Oct 8, 2025
33 of 35 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

🆕 semver/minor Adds new public API.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants