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

Update dependencies #2470

Merged
merged 3 commits into from Aug 12, 2020
Merged

Update dependencies #2470

merged 3 commits into from Aug 12, 2020

Conversation

tanner0101
Copy link
Member

@tanner0101 tanner0101 commented Aug 12, 2020

Updates to swift-nio 2.18, async-http-client 1.2, and swift-nio-http2 1.13 (#2470, #2467, fixes #2466).

  • Fixes any usages of deprecated methods.
  • Fixes a reentrancy bug causing "body stream deinit before closing" assertion that popped up in tests with latest dependencies.
  • Makes trace logs from the HTTP decoder more concise.

@tanner0101 tanner0101 closed this Aug 12, 2020
tanner0101 and others added 2 commits August 12, 2020 13:38
As noted in #2466 - the docs refer to a ByteBuffer convenience initialiser, which is not implemented in SwiftNIO until version 2.18
@tanner0101 tanner0101 reopened this Aug 12, 2020
@tanner0101 tanner0101 added the enhancement New feature or request label Aug 12, 2020
@tanner0101 tanner0101 added this to Awaiting Review in Vapor 4 via automation Aug 12, 2020
@tanner0101 tanner0101 added the semver-patch Internal changes only label Aug 12, 2020
@tanner0101 tanner0101 changed the title bump nio Update to latest dependencies Aug 12, 2020
@tanner0101 tanner0101 marked this pull request as ready for review August 12, 2020 17:39
@tanner0101 tanner0101 changed the title Update to latest dependencies Update dependencies Aug 12, 2020
@tanner0101 tanner0101 merged commit a8a641c into master Aug 12, 2020
Vapor 4 automation moved this from Awaiting Review to Done Aug 12, 2020
@tanner0101 tanner0101 deleted the tn-bump-nio branch August 12, 2020 17:48
@tanner0101
Copy link
Member Author

These changes are now available in 4.27.3

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request semver-patch Internal changes only
Projects
Vapor 4
  
Done
Development

Successfully merging this pull request may close these issues.

File write documentation example relies on VaporTest Utility helper
2 participants