Skip to content
This repository was archived by the owner on Dec 27, 2022. It is now read-only.

556 protocol version #650

Merged
LayneHaber merged 4 commits into from
Jun 7, 2021
Merged

556 protocol version #650

LayneHaber merged 4 commits into from
Jun 7, 2021

Conversation

@LayneHaber
Copy link
Contributor

@LayneHaber LayneHaber commented Jun 7, 2021 •
edited
Loading

The Problem

  • No clear error on protocol issues

The Solution

  • Adds error on protocol version incompatibility issues
  • Adds error if a lock message is received

@LayneHaber LayneHaber changed the base branch from main to 556-no-lock June 7, 2021 21:36
@LayneHaber LayneHaber requested a review from rhlsthrm June 7, 2021 22:01
@LayneHaber LayneHaber merged commit 0da1b16 into 556-no-lock Jun 7, 2021
@LayneHaber LayneHaber deleted the 556-protocol-version branch June 7, 2021 22:39
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Reviewers

@rhlsthrm rhlsthrm Awaiting requested review from rhlsthrm

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@LayneHaber