fix: reverts read_state polling expiry changes (#971)

Author: krpeacock

docs/CHANGELOG.md

@@ -8,6 +8,7 @@
 
 ### Changed
 
+- fix: reverts read_state polling expiry changes due to mismatched signature introduced in 2.1.3. Polling will re-use the original request as before, up to the point where the request expires
 - feat: HttpAgent uses anonymous identity to make syncTime call, which can allow readState calls to work beyond 5 minutes
 - chore: bumps .nvmrc and nodejs version in CI to 22
 - HttpAgent now awaits fetching rootkey before making network calls if `shouldFetchRootKey` is set

packages/agent/src/agent/http/index.ts

@@ -1062,13 +1062,7 @@ export class HttpAgent implements Agent {
 
     const transformedRequest = request ?? (await this.createReadStateRequest(fields, identity));
 
-    // With read_state, we should always use a fresh expiry, even beyond the point where the initial request would have expired
-    const bodyWithAdjustedExpiry = {
-      ...transformedRequest.body,
-      ingress_expiry: new Expiry(DEFAULT_INGRESS_EXPIRY_DELTA_IN_MSECS),
-    };
-
-    const body = cbor.encode(bodyWithAdjustedExpiry);
+    const body = cbor.encode(transformedRequest.body);
 
     this.log.print(
       `fetching "/api/v2/canister/${canister}/read_state" with request:`,

packages/agent/src/polling/index.ts

@@ -86,11 +86,6 @@ export async function pollForResponse(
   const path = [new TextEncoder().encode('request_status'), requestId];
   const currentRequest = request ?? (await agent.createReadStateRequest?.({ paths: [path] }));
 
-  // Use a fresh expiry for the repeated readState call
-  if (request && isSignedReadStateRequestWithExpiry(currentRequest)) {
-    currentRequest.body.content.ingress_expiry = new Expiry(DEFAULT_INGRESS_EXPIRY_DELTA_IN_MSECS);
-  }
-
   const state = await agent.readState(canisterId, { paths: [path] }, undefined, currentRequest);
   if (agent.rootKey == null) throw new Error('Agent root key not initialized before polling');
   const cert = await Certificate.create({