Skip to content

Commit

Permalink
Revert "Add capture_graphql_errors option to Python HTTP client int…
Browse files Browse the repository at this point in the history
…egrations" (#7585)

This reverts commit 24ca1fe.
  • Loading branch information
sentrivana authored Aug 3, 2023
1 parent 6312fe0 commit 8aa9e4b
Show file tree
Hide file tree
Showing 3 changed files with 0 additions and 33 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -60,17 +60,6 @@ The stdlib integration instruments certain modules in the standard library to em
- Subprocesses spawned with the `subprocess` module will result in a <PlatformLink to="/enriching-events/breadcrumbs/">breadcrumb</PlatformLink> being logged.
### Options
You can pass the following keyword arguments to `StdlibIntegration()`:
- `capture_graphql_errors`:
If enabled, the SDK will automatically create an error event whenever the response
to a request to an endpoint called `/graphql` contains an `errors` array. The
error event will have both the request and the response content if the SDK has
been set up with `send_default_pii=True` (which is the default). If you don't want to capture client GraphQL errors, set `send_default_pii=False`.
## Modules
_Import name: `sentry_sdk.integrations.modules.ModulesIntegration`_
Expand Down
11 changes: 0 additions & 11 deletions src/platforms/python/common/configuration/integrations/httpx.mdx
Original file line number Diff line number Diff line change
Expand Up @@ -34,17 +34,6 @@ sentry_sdk.init(
)
```

## Options

You can pass the following keyword arguments to `HttpxIntegration()`:

- `capture_graphql_errors`:

If enabled, the SDK will automatically create an error event whenever the response
to a request to an endpoint called `/graphql` contains an `errors` array. The
error event will have both the request and the response content if the SDK has
been set up with `send_default_pii=True` (which is the default). If you don't want to capture client GraphQL errors, set `send_default_pii=False`.

## Supported Versions

- HTTPX: 0.16+
Expand Down
11 changes: 0 additions & 11 deletions src/platforms/python/guides/aiohttp/index.mdx
Original file line number Diff line number Diff line change
Expand Up @@ -60,8 +60,6 @@ web.run_app(app)

- All exceptions leading to an Internal Server Error are reported.

- AIOHTTP client requests will be instrumented.

- _The AIOHTTP integration currently does not attach the request body._ See
[the relevant GitHub
issue](https://github.com/getsentry/sentry-python/issues/220).
Expand All @@ -70,8 +68,6 @@ web.run_app(app)
the [Logging](/platforms/python/guides/logging/)
integration is enabled (done by default).

- GraphQL errors encountered when using the AIOHTTP client will be captured.

## Options

You can pass the following keyword arguments to `AioHttpIntegration()`:
Expand All @@ -98,10 +94,3 @@ Configure the way Sentry names transactions:
- `<module_name>.hello` if you set `transaction_style="handler_name"`

The default is `"handler_name"`.

### `capture_graphql_errors`

If enabled, the SDK will automatically create an error event whenever the response
to a request to an endpoint called `/graphql` contains an `errors` array. The
error event will have both the request and the response content if the SDK has
been set up with `send_default_pii=True` (which is the default). If you don't want to capture client GraphQL errors, set `send_default_pii=False`.

1 comment on commit 8aa9e4b

@vercel
Copy link

@vercel vercel bot commented on 8aa9e4b Aug 3, 2023

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Successfully deployed to the following URLs:

sentry-docs – ./

sentry-docs.sentry.dev
sentry-docs-git-master.sentry.dev
docs.sentry.io

Please sign in to comment.