Skip to content

Created a decoder handler to deserialize a MemcachedResponse #11

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.

Sign up for GitHub

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

Merged
merged 14 commits into from
Jun 1, 2023
Merged

Created a decoder handler to deserialize a MemcachedResponse #11

Changes from 1 commit
Commits
Show all changes
14 commits
Select commit Hold shift + click to select a range
2838923
created a MemcachedResponse decoder
dkz2 May 26, 2023
4ce410d
flags are parsed but not handled
dkz2 May 26, 2023
c2e44ec
soundness fix
dkz2 May 26, 2023
5624259
response status name cleanup
dkz2 May 26, 2023
633e79b
eof not needed
dkz2 May 26, 2023
067a943
accessor not needed
dkz2 May 26, 2023
6ebbea2
Response and decoder refactor
dkz2 May 30, 2023
72b883e
reimpl decoder as state machine
dkz2 May 31, 2023
abb43c8
decoder test clean up
dkz2 May 31, 2023
077d046
no need to move reader in decodeEndOfLine
dkz2 May 31, 2023
9a4f3be
ignore flags
dkz2 May 31, 2023
7e385c3
consume flag but do nothing with it
dkz2 May 31, 2023
b7772ce
revamp of decoder, removed un needed state
dkz2 Jun 1, 2023
250f680
remove unused state, added fatal error
dkz2 Jun 1, 2023
File filter

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
response status name cleanup
  • Loading branch information
@dkz2
dkz2 committed May 26, 2023
commit 56242598c1ab0a1f09a2fe2a90ab2fcf726c950d
8 changes: 4 additions & 4 deletions Sources/SwiftMemcache/MemcachedResponseDecoder.swift
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -26,9 +26,9 @@ struct MemcachedResponseDecoder: ByteToMessageDecoder {
return .needMoreData
}

guard let asciiValue1 = buffer.readInteger(as: UInt8.self),
let asciiValue2 = buffer.readInteger(as: UInt8.self),
let responseCode = ResponseStatus(asciiValues: (asciiValue1, asciiValue2)) else {
guard let firstResponseDigit = buffer.readInteger(as: UInt8.self),
let secondResponseDigit = buffer.readInteger(as: UInt8.self),
let responseStatus = ResponseStatus(asciiValues: (firstResponseDigit, secondResponseDigit)) else {
preconditionFailure("Response code could not be read.")
}

Expand All @@ -45,7 +45,7 @@ struct MemcachedResponseDecoder: ByteToMessageDecoder {
preconditionFailure("Line ending '\r\n' not found after the flags.")
}

let setResponse = MemcachedResponse.SetResponse(status: responseCode)
let setResponse = MemcachedResponse.SetResponse(status: responseStatus)
context.fireChannelRead(self.wrapInboundOut(.set(setResponse)))
return .continue
}
Expand Down