Skip to content

[FormRecognizer] Whitelist redacted headers for logging#12285

Merged
kinelski merged 2 commits intoAzure:masterfrom
kinelski:fr-headers
May 27, 2020
Merged

[FormRecognizer] Whitelist redacted headers for logging#12285
kinelski merged 2 commits intoAzure:masterfrom
kinelski:fr-headers

Conversation

@kinelski
Copy link
Copy Markdown
Contributor

@kinelski kinelski commented May 26, 2020

Fixes #11660.

Headers descriptions, grabbed from the internet and from internal sources:

  • x-envoy-upstream-service-time: contains the time in milliseconds spent by the upstream host processing the request.
  • apim-request-id: GUID request Id generated from APIM side.
  • Strict-Transport-Security: lets a web site tell user agents that it should only be accessed using HTTPS, instead of using HTTP.
  • X-Content-Type-Options: a marker used by the server to indicate that the MIME types advertised in the Content-Type headers should not be changed and be followed.

Location and Operation-Location headers expose the endpoint, which means resource names are exposed as well. Resource names are not considered PII and are safe to log.

EDIT: PR for adding query parameters here.

@kinelski kinelski added Cognitive Services Client This issue is related to a non-management package FormRecognizer labels May 26, 2020
@kinelski kinelski self-assigned this May 26, 2020
@kinelski kinelski added the Docs label May 26, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Client This issue is related to a non-management package Cognitive - Form Recognizer Cognitive Services Docs

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Design] [FormRecognizer] Whitelist redacted header for logging

2 participants