@phpsandbox/sdk 0.1.10 → 0.1.11

package/dist/http.d.ts

@@ -0,0 +1,6 @@
+/**
+ * Cloudflare Workers does not support `redirect: "error"`, while `follow` can
+ * forward authorization headers across redirects. Handle redirects manually so
+ * authenticated requests never follow them.
+ */
+export declare function authenticatedRequest(input: URL | string, init: RequestInit): Request;

package/dist/http.js

@@ -0,0 +1,11 @@
+/**
+ * Cloudflare Workers does not support `redirect: "error"`, while `follow` can
+ * forward authorization headers across redirects. Handle redirects manually so
+ * authenticated requests never follow them.
+ */
+export function authenticatedRequest(input, init) {
+    return new Request(input, {
+        ...init,
+        redirect: 'manual',
+    });
+}

package/dist/index.js

@@ -30,6 +30,7 @@ import { RestRuntimeInvoker } from './runtime/rest.js';
 import { Feedback } from './feedback.js';
 import { IntegrationApi } from './integrations.js';
 import { SandboxIntegrationApi } from './sandbox-integrations.js';
+import { authenticatedRequest } from './http.js';
 export { LspConnection } from './lsp.js';
 export { FileChangeFilter, FileChangeType, FileType, } from './filesystem.js';
 export { notebookBuiltinServices, notebookKnownServices } from './services.js';
@@ -105,10 +106,9 @@ class ClientImplementation {
         return this.makeRequest('PATCH', path, { body: body === undefined ? undefined : JSON.stringify(body) });
     }
     async stream(path) {
-        const response = await this.fetchResponse(new Request(new URL(path.replace(/^\//, ''), this.baseUrl), {
+        const response = await this.fetchResponse(authenticatedRequest(new URL(path.replace(/^\//, ''), this.baseUrl), {
             method: 'GET',
             headers: this.headers,
-            redirect: 'error',
         }));
         if (!response.ok) {
             throw remoteError('core', errorPayload(await responsePayload(response)), response);
@@ -119,9 +119,8 @@ class ClientImplementation {
         return response.body;
     }
     async sse(url) {
-        const response = await this.fetchResponse(new Request(url, {
+        const response = await this.fetchResponse(authenticatedRequest(url, {
             method: 'GET',
-            redirect: 'error',
             headers: {
                 ...this.headers,
                 'Accept': 'text/event-stream',
@@ -136,11 +135,10 @@ class ClientImplementation {
         return response.body;
     }
     async makeRequest(method, path, init) {
-        const response = await this.fetchResponse(new Request(new URL(path.replace(/^\//, ''), this.baseUrl), {
+        const response = await this.fetchResponse(authenticatedRequest(new URL(path.replace(/^\//, ''), this.baseUrl), {
             method,
             ...init,
             headers: this.headers,
-            redirect: 'error',
         }));
         if (!response.ok) {
             throw remoteError('core', errorPayload(await responsePayload(response)), response);

package/dist/runtime/rest.js

@@ -1,4 +1,5 @@
 import { PHPSandboxError, RemoteError, TransportError, remoteError } from '../errors/index.js';
+import { authenticatedRequest } from '../http.js';
 const restActionRoutes = {
     'runtime.ready': { method: 'GET', path: '/runtime' },
     'config.get': { method: 'GET', path: '/runtime/config' },
@@ -119,12 +120,11 @@ export class RestRuntimeInvoker {
             Authorization: `Bearer ${this.ticket}`,
         };
         const body = runtimeActionRequestBody(action, data, url, headers);
-        return new Request(url, {
+        return authenticatedRequest(url, {
             method: route.method,
             headers,
             body: route.method === 'GET' ? undefined : body,
             signal,
-            redirect: 'error',
         });
     }
     async refreshAuthorization() {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@phpsandbox/sdk",
-  "version": "0.1.10",
+  "version": "0.1.11",
   "description": "A comprehensive TypeScript SDK for interacting with cloud-based PHP development environments",
   "main": "./dist/index.js",
   "module": "./dist/index.js",