Grammes add error context for connection failure

northwesternmutual / grammes

A Go package built to communicate with Apache TinkerPop™ Graph computing framework using Gremlin; a graph traversal language used by graph databases such as JanusGraph®, MS Cosmos DB, AWS Neptune, and DataStax® Enterprise Graph.

Apache License 2.0

125 stars 45 forks source link

Grammes add error context for connection failure #17

Open AdallomRoy opened 4 years ago

AdallomRoy commented 4 years ago

Currently, when a connection fails, only the "bad handshake" error is returned. the context (httpresponse) that is returned from the gorilla library is omitted. This helps to include that context into the error to have a clear view of what failed, when a connection fails.

rwsegura commented 3 years ago

I have made a new org and forked the repo at https://github.com/grammes-go/grammes. Feel free to redirect PRs there and reach out if you'd like to contribute