diff options
author | Richard van der Hoff <richard@matrix.org> | 2020-04-07 17:21:13 +0100 |
---|---|---|
committer | Richard van der Hoff <richard@matrix.org> | 2020-04-07 17:43:46 +0100 |
commit | e13c6c7a96dc71200eef9f966bee21c27ae54426 (patch) | |
tree | 8fd324ea975e9bdb3977abbff95674275e7e5b8a /synapse/replication/tcp/protocol.py | |
parent | Fix warnings about not calling superclass constructor (diff) | |
download | synapse-e13c6c7a96dc71200eef9f966bee21c27ae54426.tar.xz |
Handle one-word replication commands correctly
`REPLICATE` is now a valid command, and it's nice if you can issue it from the console without remembering to call it `REPLICATE ` with a trailing space.
Diffstat (limited to 'synapse/replication/tcp/protocol.py')
-rw-r--r-- | synapse/replication/tcp/protocol.py | 14 |
1 files changed, 11 insertions, 3 deletions
diff --git a/synapse/replication/tcp/protocol.py b/synapse/replication/tcp/protocol.py index 9aabb9c586..9276ed2965 100644 --- a/synapse/replication/tcp/protocol.py +++ b/synapse/replication/tcp/protocol.py @@ -201,15 +201,23 @@ class BaseReplicationStreamProtocol(LineOnlyReceiver): ) self.send_error("ping timeout") - def lineReceived(self, line): + def lineReceived(self, line: bytes): """Called when we've received a line """ if line.strip() == "": # Ignore blank lines return - line = line.decode("utf-8") - cmd_name, rest_of_line = line.split(" ", 1) + linestr = line.decode("utf-8") + + # split at the first " ", handling one-word commands + idx = linestr.index(" ") + if idx >= 0: + cmd_name = linestr[:idx] + rest_of_line = linestr[idx + 1 :] + else: + cmd_name = linestr + rest_of_line = "" if cmd_name not in self.VALID_INBOUND_COMMANDS: logger.error("[%s] invalid command %s", self.id(), cmd_name) |