search

panicbit / rust-rcon

An RCON implementation in Rust
Apache License 2.0
32 stars 14 forks source link

Improve error handling #10

Closed klemens closed 7 years ago

klemens commented 7 years ago

The first tree commits improve the error handling in general and fix #6 and #7.

The last commit introduces the new CommandTooLong error variant that is used to reject commands that exceed 1413 byte (see #5).

All commits are breaking API changes, so we should probably bump the version to at least 0.1.0.

This change is Reviewable

panicbit commented 7 years ago

Reviewed 2 of 2 files at r1. Review status: all files reviewed at latest revision, all discussions resolved.

Comments from Reviewable

panicbit commented 7 years ago

Nice work! Thanks a lot! 👍

panicbit commented 7 years ago

I'm going to release a new crate version now.