API key needs to work in the header

drivly / ctx.do

Context Enrichment API

https://ctx.do

MIT License

6 stars 1 forks source link

API key needs to work in the header #34

Closed nathanclevenger closed 2 years ago

nathanclevenger commented 2 years ago

I'm seeing the API key enrichment if it's a query parameter, but I'm not seeing it if it's an 'x-api-key' header.

That said, we should also honor 'authorization' headers where it's often listed as {apikey} Bearer {apikey} or APIKey {apikey} so I think we should try to honor all of those.

sam-lippert commented 2 years ago

Fixed in https://github.com/drivly/apikeys.do/commit/4a2f6749f2b7bd1ee6306a0c1756e3cd77000631