Chris

Btw for some context on where this is coming from: The next [Mattermost](https://github.com/mattermost/mattermost-server) release will include support for proxying hot-linked, user-posted images via this or Camo. It's primarily because we...

I don't think it's passing the headers to the origin server, but I don't think that's really much of an issue either. It's not fetching the image bytes on every...

Yep, that's correct.

Thank you for the kind words! 1. In case you (or anyone that stumbles upon this issue) haven't already seen it, there is a demo application in the [examples/chat](https://github.com/ccbrown/api-fu/tree/master/examples/chat) directory....

Thanks, I'll try to investigate soon.

Fixed for forums. I can't seem to find Yolandi's Reddit username though.

There's now a `LogFields` implementation, but... it seems to do something totally wrong? https://github.com/DataDog/dd-trace-go/blob/v1.27.1/ddtrace/opentracer/span.go#L43-L64 It just sets some tags for a few of the common fields and assumes that logged...

> Names in GraphQL are limited to this ASCII subset of possible characters to support interoperation with as many other systems as possible. – https://spec.graphql.org/June2018/#sec-Names A really nice property of...

I think we could kill two birds with one stone if instead of adding a `parse` method to `GraphQLRequest`, we just added a method to get the query. This would...

> Are you suggesting that: > > 1. We move the parse method elsewhere (as a free fn) AND > 2. Add a simple accessor method to get the query...