Mysql Nio Versions Save

What's Changed

Tidy up package a bit by @gwynne in #103

Fixes #102.

Also does some rather overdue general cleanup of the repo. No functional changes.

Reviewers

Thanks to the reviewers for their help:

• @MahdiBM

This patch was released by @gwynne

Full Changelog: https://github.com/vapor/mysql-nio/compare/1.7.0...1.7.1

This patch was authored by @samalone and released by @gwynne.

Add support for reading timestamp columns in text format.

Fixes #71

This patch was authored and released by @gwynne.

Items of note:

• Replaced all the assert()s, precondition()s, and fatalError()s with thrown errors as appropriate. Removes almost every source of crashes - especially ones caused by input the user has no control over - in the package.
• Fixes #87 (unexpected packets with no active command cause an assertion failure)
• Fixes #91 (server error before handshake throws invalidProtocolVersion() instead of the appropriate .server(ERR_Packet))
• CI and README updates
• Fixes the unit tests so they don't leave stale tables lying around.

With the rewrite finally back to being actively worked on, it became really obvious how easy these fixes were to make even in the existing code, and I decided users shouldn't have to wait any longer for them, even if the rewrite drops the next day.

This patch was authored and released by @gwynne.

This patch was authored by @elGeekalpha and released by @0xTim.

Adds the correct platform versions to match the Swift Crypto dependency to enable building on all platforms.

Solves #81

This patch was authored and released by @gwynne.

• "Connection closed" errors are now correctly reported when trying to issue queries on closed connections, instead of throwing various NIO errors.
• Query syntax errors are now explicitly reported.
• Unique constraint violation errors are now more consistently reported.

semver-minor due to adding a new case on MySQLError.

This patch was authored by @fabianfett and released by @gwynne.

Needs patch release.

This patch was authored and released by @gwynne.

• Replace the readNullTerminatedString() implementation with the much more efficient version used by PostgresNIO.
• Made writeNullTerminatedString(), writeLengthEncodedInteger(), and writeLengthEncodedSlice() return the number of bytes written, to match ByteBuffer method conventions.
• Simplified [read|write]LengthEncodedInteger(), might be slightly faster but probably not enough to notice.
• Remove a couple of unused utility methods altogether.

This patch was authored and released by @gwynne.

Fixes #63.

Also fixes a minor thread-safety issue and clears up deprecation warnings from NIOSSL.

This patch was authored by @NobodyNada and released by @tanner0101.

When TextResultSetRow encountered a null column, it did not increment the buffer index, and so it would repeatedly read NULL into every remaining column. This patch fixes the issue and adds regression tests (#57).