Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

UDPEchoClient to close channel after response received. #1501

Merged
merged 2 commits into from
May 4, 2020

Conversation

PeterAdams-A
Copy link
Contributor

Close channel after UCPEchoClient received a response.

Motivation:

The application does not shutdown without doing this.

Modifications:

Close channel after receiving data.

Result:

Application exits after receiving data.

@swift-server-bot
Copy link

Can one of the admins verify this patch?

@Lukasa
Copy link
Contributor

Lukasa commented May 2, 2020

@swift-nio-bot add to whitelist

@Lukasa Lukasa added the 🔨 semver/patch No public API change. label May 2, 2020
@Lukasa Lukasa added this to the 2.17.0 milestone May 2, 2020
@Lukasa Lukasa merged commit c30cf9c into apple:master May 4, 2020
@PeterAdams-A PeterAdams-A deleted the upd_shutdown branch May 4, 2020 10:17
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
🔨 semver/patch No public API change.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants