[DE-605] Always use POST for cursors

Author: pluma

CHANGELOG.md

@@ -18,6 +18,11 @@ This driver uses semantic versioning:
 
 ### Changed
 
+- Fetching additional cursor results now uses `POST` instead of `PUT` (DE-605)
+
+  The `POST` route was deprecated and the `PUT` route is supported in all
+  actively maintained versions of ArangoDB.
+
 - User management methods now use database-relative URLs (DE-606)
 
   Previously these methods would make requests without a database prefix,

src/cursor.ts

@@ -214,17 +214,10 @@ export class BatchedArrayCursor<T = any> {
   protected async _more(): Promise<void> {
     if (!this._id || !this.hasMore) return;
     const body = await this._db.request({
-      ...(this._nextBatchId
-        ? {
-            method: "POST",
-            path: `/_api/cursor/${encodeURIComponent(this._id)}/${
-              this._nextBatchId
-            }`,
-          }
-        : {
-            method: "PUT",
-            path: `/_api/cursor/${encodeURIComponent(this._id)}`,
-          }),
+      method: "POST",
+      path: this._nextBatchId
+        ? `/_api/cursor/${encodeURIComponent(this._id)}/${this._nextBatchId}`
+        : `/_api/cursor/${encodeURIComponent(this._id)}`,
       hostUrl: this._hostUrl,
       allowDirtyRead: this._allowDirtyRead,
     });