-
Notifications
You must be signed in to change notification settings - Fork 2k
Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
- Loading branch information
1 parent
2c8106c
commit a1c725e
Showing
1 changed file
with
17 additions
and
0 deletions.
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,17 @@ | ||
--- | ||
'@apollo/server': patch | ||
--- | ||
|
||
Ensure API keys are valid header values on startup | ||
|
||
Apollo Server previously performed no sanitization or validation of API keys on startup. In the case that an API key was provided which contained characters that are invalid as header values, Apollo Server could inadvertently log the API key in cleartext. | ||
|
||
This only affected users who: | ||
- Provide an API key with characters that are invalid as header values | ||
- Use either schema or usage reporting | ||
- Use the default fetcher provided by Apollo Server or configure their own `node-fetch` fetcher | ||
|
||
Apollo Server now trims whitespace from API keys and validates that they are valid header values. If an invalid API key is provided, Apollo Server will throw an error on startup. | ||
|
||
For more details, see the security advisory: | ||
https://github.com/apollographql/apollo-server/security/advisories/GHSA-j5g3-5c8r-7qfx |