router

router copied to clipboard

Deprecate `experimental_when_header` once advanced telemetry features are part of a release

1

We should add a log message when this is configured saying it's deprecated and events should be used.

bnjjj

response_body is not valid under telemetry spans for custom attribute

8

https://www.apollographql.com/docs/router/configuration/telemetry/instrumentation/selectors/#supergraph It says supergraph has selector of `response_body`, but the router config schema does not contain it. Add a custom attribute under supergraph would fail. **Expected behavior** Expect `response_body` selector...

Samjin

Normalize "operation_name" value inside supergraph request

2

Currently, request.body.operation_name is a direct copy of operationName from the client request. But this behavior is very confusing to customers who expect it to have an operation name, if any,...

IvanGoncharov

Example for use case: if you want to create a counter that count the number of connection timeout error you got and add subgraph as an attribute to know which...

bnjjj

Ensure subgraph requests have access to their corresponding ExecutableDocuments

2

When we apply demand control measurements, we want to measure the cost of an operation at several points. We score the overall operation, the query plan, and each subgraph operation....

tninesling

Unable to remove the Apollo-Expose-Query-Plan with header propagation remove regex

1

**Describe the bug** I'm not entirely sure this is a bug however I'm not able to remove the `Apollo-Expose-Query-Plan` header from propagation unless I use `remove.named`. **To Reproduce** Steps to...

machados

Send response errors only to coprocessor

5

## Is your feature request related to a problem? Please describe. Today I can configure the Router to send the response, which might have `data` or `errors` or `extensions`, to...

smyrick

Clarify docs on cache warm up defaults

2

The documentation here is a bit confusing. This change may not be correct but is maybe a good starting point for a change. --- **Checklist** Complete the checklist (and note...

maschwenk

Adding a field changes the returned value of another (@requires directive with arguments involved)

1

**Describe the bug** Below a simplified description of the use case we're experiencing. The problem: Query 1 and query 2 give a different response on '`explicit`' field. When looking at...

rickbijkerk

Improve / formalize configuration of telemetry / logging / observability / metrics / tracing

32

Currently we have an `experimental_logging` section that we should bring out of experimental and in addition support json formatting options. There have also been several other users asking for much...

BrynCooke