John Children

Nested message names can collide silently in swagger generation and result in randomized swagger message definitions

6

Generated swagger files can silently have name collisions between message names when messages are nested. For example: ```protobuf syntax = "proto3"; message FooBar { string value = 1; } message...

Diesel-Async support

2

I tend to use [diesel-async](https://github.com/weiznich/diesel_async) for connections now, so it might be nice to support that. Though perhaps it ought to be a separate crate.

net.peer.ip and net.peer.host are incorrect

1

Both fields are just what the server is listening on, not what the client connects to. Need to find a better way of discovering this information.

Add an examples directory with some code examples

1

I've found it's quite hard to explain how this library works with just words, so some code examples would be useful.

Hook into diesel's upcoming instrumentation module

1

https://github.com/diesel-rs/diesel/commits/master/diesel/src/connection/instrumentation.rs I think using this module rather than wrapping connections could be a lot nicer, it will be a big breaking API change for this crate however.

In order to get more connection information for opentelemetry fields there are two possible approaches: 1. Use the libpg C FFI to parse the connection string 2. Implement a custom...

Rather than re-implement parsing for connection options it would be nice if we could just use their existing structure instead. Also means that we can be sure that we won't...

The instrumented connection for postgresql assumes that you are connecting over a network and cannot handle unix sockets

Fixes some type errors that have been seen by users and in other internal code. Also makes some methods public that were being used by QuantinuumBackend and fixes some mismatched...