Update protobuf

.changeset/violet-olives-doubt.md

@@ -0,0 +1,5 @@
+---
+'@apollo/usage-reporting-protobuf': minor
+---
+
+Update protobuf which includes updates for supporting (notably) ConditionNode in the gateway

cspell.yaml

@@ -19,6 +19,7 @@ dictionaries:
 useGitignore: true
 ignorePaths:
   - '**/generated/**'
+  - 'packages/usage-reporting-protobuf/src/reports.proto'
   - '**/*.sketch'
   - '**/*.svg'
   - cspell.yaml

packages/server/src/__tests__/plugin/usageReporting/plugin.test.ts

@@ -447,8 +447,6 @@ describe('sendHeaders makeHTTPRequestHeaders helper', () => {
   function makeTestHTTP(): Trace.HTTP {
     return new Trace.HTTP({
       method: Trace.HTTP.Method.UNKNOWN,
-      host: null,
-      path: null,
     });
   }
 

packages/server/src/plugin/usageReporting/plugin.ts

@@ -425,15 +425,6 @@ export function ApolloServerPluginUsageReporting<TContext extends BaseContext>(
               Trace.HTTP.Method[
                 http.method as keyof typeof Trace.HTTP.Method
               ] || Trace.HTTP.Method.UNKNOWN,
-            // Host and path are not used anywhere on the backend, so let's not bother
-            // trying to parse request.url to get them, which is a potential
-            // source of bugs because integrations have different behavior here.
-            // On Node's HTTP module, request.url only includes the path
-            // (see https://nodejs.org/api/http.html#http_message_url)
-            // The same is true on Lambda (where we pass event.path)
-            // But on environments like Cloudflare we do get a complete URL.
-            host: null,
-            path: null,
           });
 
           if (options.sendHeaders) {

packages/server/src/plugin/usageReporting/stats.ts

@@ -35,6 +35,8 @@ export class SizeEstimator {
   bytes = 0;
 }
 export class OurReport implements Required<IReport> {
+  tracesPreAggregated = false;
+
   constructor(readonly header: ReportHeader) {}
   readonly tracesPerQuery: Record<string, OurTracesAndStats> =
     Object.create(null);