getsentry / getsentry/sentry-javascript

HTTP Headers are not sent by default, unless `sendDefaultPii` is enabled.

Abierto
#20,706 4 comentarios 0 reacciones 0 asignados Ver en GitHub
Docs
Lenguaje dominante
TypeScript
Estrellas
8.7k
Forks
1.8k
Merge medio
1 d 17 h
PR fusionados (30 d)
515

Descripción

### SDK

JavaScript SDK

### Description

Sentry version:
```
"@sentry/browser": "10.51.0",
"@sentry/nextjs": "10.51.0",
```

The docs mention [here](https://docs.sentry.io/platforms/javascript/guides/nextjs/data-management/data-collected/#http-headers) that

> By default, the Sentry SDK sends HTTP response or request headers.

However, this is not true or needs clarifying since at least the NextJS SDK requires the SentryConfig to have `sendDefaultPii` to be true.

Response headers;
`sendDefaultPii` disabled;
Image

`sendDefaultPii` enabled;
Image

Request headers do seem to send headers but with some entries omitted, though this isn't mentioned on the docs.

### Suggested Solution

Depending what the expected behavior is here;

- If we do not expect to send headers by default, the docs should be updated here to mention that `sendDefaultPii` should be enabled for response headers and most request headers.
- If we do expect to send headers by default, there may be a bug in the Sentry JavaScript SDK.

Also, it would be useful to understand what exactly is and isn't included behind the `sendDefaultPii` field to make a decision if we should enable this or not. Perhaps also linking to the data-collected page from the interface/config API [here](https://docs.sentry.io/platforms/javascript/configuration/options/#sendDefaultPii).

Guía de contribución

Abrir la guía de contribución

Evaluación

Este issue todavía no se ha evaluado.

Recibe los nuevos issues en tu correo

Un resumen breve de issues de GitHub para principiantes.