Merge lp:~chipaca/ubuntu-push/elements-of-client-session-toponomy into lp:ubuntu-push
Proposed by
John Lenton
Status: | Merged |
---|---|
Approved by: | John Lenton |
Approved revision: | 52 |
Merged at revision: | 40 |
Proposed branch: | lp:~chipaca/ubuntu-push/elements-of-client-session-toponomy |
Merge into: | lp:ubuntu-push |
Prerequisite: | lp:~chipaca/ubuntu-push/introduction-to-quantum-client-sessions |
Diff against target: |
286 lines (+209/-10) 2 files modified
client/session/session.go (+43/-1) client/session/session_test.go (+166/-9) |
To merge this branch: | bzr merge lp:~chipaca/ubuntu-push/elements-of-client-session-toponomy |
Related bugs: |
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
Samuele Pedroni | Approve | ||
Review via email: mp+204129@code.launchpad.net |
Commit message
Starting to take shape.
Description of the change
Client session, volume 5: elements of client session toponomy.
To post a comment you must log in.
42 + err = proto.ReadMessa ge(&connAck)
around here there should be a sanity check that connAck.Type is "connack"
114 + c.Check( err.Error( ), Matches, ".*deadline.*")
gocheck has c.Check(err, ErrorMatches, ...)
I just noticed that upCh downCh are still from the point of view of the server, I suppose we need to live with that if we want to share later unless we can invent neutral names