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

Added BufferedIncrementalPacketSerializer base class #174

Merged
merged 23 commits into from
Dec 2, 2023

Conversation

francis-clairicia
Copy link
Owner

What's changed

  • Added BufferedIncrementalPacketSerializer base class
    • FixedSizePacketSerializer now derives from BufferedIncrementalPacketSerializer
    • FileBasedSizePacketSerializer now derives from BufferedIncrementalPacketSerializer
    • AbstractCompressorSerializer now derives from BufferedIncrementalPacketSerializer
  • Added BufferedStreamReceiver in easynetwork.protocol
    • Added new method StreamProtocol.buffered_receiver()
  • New buffer API in easynetwork.lowlevel
    • Added BufferedStreamReadTransport base class
      • Added SelectorBufferedStreamReadTransport deriving from BufferedStreamReadTransport
      • SocketStreamTransport derives from SelectorBufferedStreamReadTransport in addition to SelectorStreamTransport
      • SSLStreamTransport derives from SelectorBufferedStreamReadTransport in addition to SelectorStreamTransport
    • Added AsyncBufferedStreamReadTransport base class
      • easynetwork.lowlevel.asyncio.RawStreamSocketAdapter derives from AsyncBufferedStreamReadTransport in addition to AsyncStreamTransport
    • Added support of buffered read in StreamEndpoint and AsyncStreamEndpoint
    • Added support of buffered read in AsyncStreamServer
  • IncrementalDeserializeError.remaining_data and StreamProtocolParseError.remaining_data can now be any byte buffer (e.g. memoryview).

@francis-clairicia francis-clairicia added enhancement New feature or request topic-serializer Relative to serializer implementations labels Dec 2, 2023
@francis-clairicia francis-clairicia self-assigned this Dec 2, 2023
Copy link

codecov bot commented Dec 2, 2023

Codecov Report

Merging #174 (390b021) into main (781739e) will not change coverage.
The diff coverage is 100.00%.

Additional details and impacted files
Flag Coverage Δ
OS-Linux 86.16% <82.30%> (-0.40%) ⬇️
OS-Windows 85.57% <82.30%> (-0.36%) ⬇️
OS-macOS 86.19% <82.30%> (-0.40%) ⬇️
Py-3.11 99.90% <100.00%> (+<0.01%) ⬆️
Py-3.12 99.90% <100.00%> (+<0.01%) ⬆️
test-functional 86.42% <82.30%> (-0.42%) ⬇️
test-unit 84.08% <94.61%> (+0.55%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

@francis-clairicia francis-clairicia merged commit 3beb7c4 into main Dec 2, 2023
17 checks passed
@francis-clairicia francis-clairicia deleted the feature/serializers/buffered-serializers branch December 2, 2023 15:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request topic-serializer Relative to serializer implementations
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant