Skip to content

AER/Default to headers for client identity#1960

Merged
zionts merged 1 commit into
masterfrom
adam/18/11/client-info-default-to-headers
Nov 14, 2018
Merged

AER/Default to headers for client identity#1960
zionts merged 1 commit into
masterfrom
adam/18/11/client-info-default-to-headers

Conversation

@zionts

@zionts zionts commented Nov 14, 2018

Copy link
Copy Markdown
Contributor

TODO:

  • Update CHANGELOG.md with your change (include reference to issue & this PR)
  • Make sure all of the significant new logic is covered by tests
  • Rebase your changes on master so that they can be merged easily
  • Make sure all tests and linter rules pass

@zionts zionts requested review from evans and jbaxleyiii November 14, 2018 01:33
lolwut

Revert "lolwut"

This reverts commit 03a2e10.
@zionts zionts force-pushed the adam/18/11/client-info-default-to-headers branch from 03a2e10 to 81d9f47 Compare November 14, 2018 01:36
@zionts zionts merged commit e40a2e9 into master Nov 14, 2018
@zionts zionts deleted the adam/18/11/client-info-default-to-headers branch November 14, 2018 01:51
@abernix

abernix commented Nov 14, 2018

Copy link
Copy Markdown
Member

@zionts Does this sound like an appropriate CHANGELOG entry?

A once suggested, slightly incorrect, now tweaked suggestion
When `generateClientInfo` is not used to define the client name, client version and
client reference ID, Apollo Server will default to the values present in the HTTP headers
of the request (`apollographql-client-name`, `apollographql-client-reference-id` and
`apollographql-client-version` respectively) and then, when those headers are not set,
the query extensions' `clientInfo` values. [PR #1960](https://github.com/apollographql/apollo-server/pull/1960)

Update: Done with a slightly different in 8f50045

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants