Redact Authorization header from signJWT debug logs - #536
diegoDrp-Dev wants to merge 1 commit into
Conversation
|
Thanks for your pull request! It looks like this may be your first contribution to a Google open source project. Before we can look at your pull request, you'll need to sign a Contributor License Agreement (CLA). View this failed invocation of the CLA check for more information. For the most up to date status, view the checks section at the bottom of the pull request. |
0879f09 to
197b22b
Compare
|
Notei que os workflows obrigatórios (action_scanning, lint, test) estão parados há 30 dias aguardando aprovação de execução nenhum chegou a rodar, todos expiraram com a mensagem "required approval but was not approved before expiring". O CLA já passou. Poderiam aprovar a execução para que o CI processe o PR? Fico à disposição para qualquer ajuste necessário. Obrigado pela atenção! |
Prevents the access token used by signJWT() from being included in debug logging.
The request continues to use the original Authorization header, while a sanitized copy of the headers is passed to the logger.
This also adds a regression test verifying that:
Related Google Issue Tracker report: 550959018
Validation:
npm test: 55/55 passingnpm run lint: passingnpm run build: passinggit diff --check: passing