Merge branch 'remake' of OpenServer/OSSecureFileWrapper into master

.npmignore

@@ -1,3 +1,3 @@
 tsconfig.json
+src/
 node_modules/
-.vscode/

.vscode/settings.json

@@ -0,0 +1,3 @@
+{
+   "cSpell.enabled": false
+}

package.json

@@ -1,6 +1,6 @@
 {
-   "name": "@hibas123/secure-file-wrapper",
-   "version": "2.3.3",
+   "name": "secure-file-wrapper",
+   "version": "2.0.0",
    "main": "lib/index.js",
    "author": "Fabian Stamm <dev@fabianstamm.de>",
    "license": "MIT",
@@ -8,22 +8,20 @@
    "scripts": {
       "build": "tsc",
       "watch": "tsc --watch",
-      "prepublishOnly": "tsc",
+      "prepublish": "tsc",
       "test": "mocha lib/test.js"
    },
    "dependencies": {
-      "@hibas123/utils": "^1.0.1",
-      "cross-fetch": "^3.0.1",
-      "uuid": "^3.3.2"
+      "isomorphic-fetch": "^2.2.1"
    },
    "devDependencies": {
       "@types/chai": "^4.1.4",
-      "@types/mocha": "^5.2.6",
-      "@types/node": "^11.10.5",
-      "@types/node-fetch": "^2.1.6",
-      "@types/uuid": "^3.4.4",
+      "@types/isomorphic-fetch": "^0.0.34",
+      "@types/mocha": "^5.2.2",
+      "@types/node": "^10.12.18",
+      "@types/node-fetch": "^2.1.4",
       "chai": "^4.1.2",
-      "mocha": "^6.0.2",
-      "typescript": "^3.3.3333"
+      "mocha": "^5.2.0",
+      "typescript": "^3.2.2"
    }
 }

src/index.ts

@@ -1,6 +1,5 @@
-import { Observable, Lock } from "@hibas123/utils";
-//this references global on node and window in browser
-const fetch = this.fetch ? this.fetch : require("cross-fetch").default;
+import Observable from "./observable";
+import Lock from "./lock";
 
 export interface IFileVersion {
    version: string;
@@ -67,6 +66,8 @@ export class BadRequest extends Error {
    }
 }
 
+import * as fetch from "isomorphic-fetch";
+
 function statusParser(res: Response) {
    if (res.status !== 200) {
       switch (res.status) {
@@ -85,12 +86,12 @@ function statusParser(res: Response) {
 }
 
 export default class SecureFileWrapper {
-   private _jwtObservableServer: Observable<(jwt: string) => void> = new Observable(false);
+   private _jwtObservableServer: Observable<(jwt: string) => void> = new Observable();
    jwtObservable = this._jwtObservableServer.getPublicApi();
 
-   private jwt: string;
+   jwt: string;
 
-   private auth_lock = new Lock();
+   auth_lock = new Lock();
 
    constructor(private server: string) {
       if (this.server.endsWith("/")) {
@@ -130,11 +131,13 @@ export default class SecureFileWrapper {
          "cache-control": "no-cache"
       };
 
+      let body_n;
       if (body) {
          headers["Content-Type"] = "application/octet-stream"
+         body_n = Buffer ? Buffer.from(body instanceof ArrayBuffer ? body : body.buffer) : body;
       }
       try {
-         let res = await fetch(this.server + endpoint + query_str, { method, body, headers });
+         let res = await fetch(this.server + endpoint + query_str, { method, body: body_n, headers });
          if (res.status === 401 && !second) {
             await this.getJWT();
             return this.makeRequest(endpoint, method, query, body, true);
@@ -149,19 +152,6 @@ export default class SecureFileWrapper {
       }
    }
 
-   private fixIFileVersion(version: IFileVersion): IFileVersion {
-      version.time = new Date(version.time)
-      return version;
-   }
-
-   private fixIFile(file: IFile): IFile {
-      file.active.time = new Date(file.active.time)
-      if (file.versions) {
-         file.versions = file.versions.map(e => this.fixIFileVersion(e))
-      }
-      return file;
-   }
-
    // async test(jwt): Promise<{ user: string, test: true }> {
    //    let res = await this.makeRequest("/test", "GET", {}, undefined, this.jwt_enabled);
    //    statusParser(res);
@@ -172,27 +162,21 @@ export default class SecureFileWrapper {
       let query: any = {}
       if (folder) query.folder = folder;
       let res = await this.makeRequest("/files", "GET", query);
-      let d: { files: IFile[] } = await res.json();
-      return d.files.map(e => this.fixIFile(e));
+      let d = await res.json();
+      return d.files;
    }
 
-   async create(name: string, data: ArrayBuffer | ArrayBufferView, type: "text" | "binary", folder?: string, preview?: string, id?: string, date?: Date): Promise<IFile> {
+   async create(name: string, data: ArrayBuffer | ArrayBufferView, type: "text" | "binary", folder?: string, preview?: string): Promise<IFile> {
       let params: any = { type: type, name: name };
-      if (preview)
+      if (preview) {
          params.preview = preview;
-
-      if (folder)
+      }
+      if (folder) {
          params.folder = folder;
-
-      if (id)
-         params.id = id
-
-      if (date)
-         params.date = date.toJSON()
-
+      }
 
       let res = await this.makeRequest("/files", "POST", params, data);
-      return this.fixIFile((await res.json()).file);
+      return (await res.json()).file;
    }
 
    async get(id: string, version?: string): Promise<ArrayBuffer> {
@@ -203,30 +187,34 @@ export default class SecureFileWrapper {
          res = await this.makeRequest("/files/" + id, "GET", {});
       }
 
-      return res.arrayBuffer()
+      if (res.arrayBuffer) {
+         return res.arrayBuffer()
+      } else {
+         let blob: Buffer = await (<any>res).buffer()
+         // console.log(blob.length);
+         return Uint8Array.from(blob).buffer;
+      }
    }
 
-   async update(id: string, data: ArrayBuffer | ArrayBufferView, preview?: string, date?: Date, old = false): Promise<IFile> {
-      let params: any = { old };
-      if (preview) params.preview = preview;
-      if (date)
-         params.date = date.toJSON()
-      let res = await this.makeRequest("/files/" + id, "PUT", params, data);
+   async update(id: string, data: ArrayBuffer | ArrayBufferView, preview?: string): Promise<IFile> {
+      let put: any = {};
+      if (preview) put.preview = preview;
+      let res = await this.makeRequest("/files/" + id, "PUT", put, data);
 
       let json = await res.json()
-      return this.fixIFile(json.file);
+      return json.file;
    }
 
-   async delete(id: string): Promise<void> {
+   async delete(id: string): Promise<boolean> {
       let res = await this.makeRequest("/files/" + id, "DELETE", {});
+
+      return res.json();
    }
 
    async history(id: string): Promise<IHistory> {
       let res = await this.makeRequest(`/files/${id}/history`, "GET", {});
-      let data: IHistory = await res.json();
-      data.file = this.fixIFile(data.file)
-      data.history = data.history.map(v => this.fixIFileVersion(v));
-      return data;
+      statusParser(res);
+      return res.json();
    }
 
    async restore(id: string, version: string) {

src/lock.ts

@@ -0,0 +1,36 @@
+export type Release = { release: () => void };
+export default class Lock {
+   private _locked: boolean = false;
+   get locked() {
+      return this._locked;
+   }
+   private toCome: (() => void)[] = [];
+
+   constructor() {
+      this.release = this.release.bind(this);
+   }
+
+   async getLock(): Promise<Release> {
+      if (!this._locked) return { release: this.lock() };
+      else {
+         return new Promise<Release>((resolve) => {
+            this.toCome.push(() => {
+               resolve({ release: this.lock() });
+            })
+         })
+      }
+   }
+
+   private lock() {
+      this._locked = true;
+      return this.release;
+   }
+
+   private async release() {
+      if (this.toCome.length > 0) {
+         this.toCome.shift()();
+      } else {
+         this._locked = false;
+      }
+   }
+}

src/observable.ts

@@ -0,0 +1,38 @@
+export type ObserverCallback<T> = (data: T) => void;
+
+export default class Observable<T = any> {
+   private subscriber: ObserverCallback<T[]>[] = [];
+   private events: T[] = [];
+   private timeout = undefined;
+
+   constructor(private collect: boolean = true, private collect_intervall: number = 100) { }
+
+   getPublicApi() {
+      return {
+         subscribe: (callback: ObserverCallback<T[]>) => {
+            if (this.subscriber.indexOf(callback) < 0)
+               this.subscriber.push(callback)
+         },
+         unsubscribe: (callback: ObserverCallback<T[]>) => {
+            let idx = this.subscriber.indexOf(callback);
+            if (idx >= 0) {
+               this.subscriber.splice(idx, 1);
+            }
+         }
+      }
+   }
+
+   send(data: T) {
+      if (!this.collect)
+         this.subscriber.forEach(e => e([data]));
+      else {
+         this.events.push(data);
+         if (!this.timeout) {
+            this.timeout = setTimeout(() => {
+               this.subscriber.forEach(e => e(this.events));
+               this.timeout = 0;
+            }, this.collect_intervall);
+         }
+      }
+   }
+}

src/test.ts

@@ -1,5 +1,5 @@
 import SecureFile, { NotFound } from "./index";
-import * as v4 from "uuid/v4"
+
 import { TextEncoder, TextDecoder } from "util";
 
 const testname = "ouiavgbsop687463743"
@@ -7,7 +7,6 @@ const encoder = new TextEncoder();
 const decoder = new TextDecoder();
 const testdata = encoder.encode("Ich bin ein Test");
 const newTestData = encoder.encode("neue test daten");
-const newTestDataOld = encoder.encode("neue test daten asd");
 const testprev = "Ich bin...";
 
 const testfolder = "iabos";
@@ -54,15 +53,6 @@ function test(sf: SecureFile) {
       expect(found, "Element not in List").to.be.true;
    })
 
-   it("update to history", async () => {
-      let res = await sf.update(testid, newTestDataOld, undefined, undefined, true);
-      expect(res, "No data returned").to.exist;
-      expect(res._id, "_id missing").to.exist;
-      expect(res.active.version, "New version was created").to.equal(testver);
-      let res2 = await sf.get(testid);
-      expect(decoder.decode(res2), "Fetched data not updated").to.be.equal(decoder.decode(testdata));
-   })
-
    it("update", async () => {
       let res = await sf.update(testid, newTestData, undefined);
       expect(res, "No data returned").to.exist;
@@ -78,9 +68,9 @@ function test(sf: SecureFile) {
       expect(his, "no data returned").to.exist;
       expect(his.file, "file not set").to.exist;
       expect(his.history, "history not set").to.exist;
-      expect(his.history.length, `Not expected history length. Expected 1 got ${his.history.length}`).to.be.equal(2);
+      expect(his.history.length, `Not expected history length. Expected 1 got ${his.history.length}`).to.be.equal(1);
 
-      expect(his.history[1].version, "Wrong version on history").to.be.equal(testver);
+      expect(his.history[0].version, "Wrong version on history").to.be.equal(testver);
       expect(his.file.active.version, "Wrong version on file").to.be.equal(testver2);
    });
 
@@ -98,42 +88,8 @@ function test(sf: SecureFile) {
    })
 
    it("delete", async () => {
-      await sf.delete(testid);
-   })
-
-
-   describe("fixed id", () => {
-      let id = v4();
-      it("create", async () => {
-         let res = await sf.create(testname, testdata, "text", undefined, testprev, id)
-         expect(res, "Res isnnot set").to.exist;
-         expect(res._id, "Res has no _id").to.exist;
-         expect(res._id, "Res has invalid _id").to.be.equal(id)
-      })
-
-      it("get", async () => {
-         let res = await sf.get(id);
-         expect(res, "No data returned").to.exist;
-         expect(decoder.decode(res), "Returned data not equal to stored").to.be.equal(decoder.decode(testdata));
-      })
-   })
-
-   describe("predefined date", () => {
-      let id = v4();
-      it("create", async () => {
-         const date = new Date("2017-01-01T00:00:00.000Z")
-         let res = await sf.create(testname, testdata, "text", undefined, testprev, id, date)
-         expect(res, "Res isnnot set").to.exist;
-         expect(res._id, "Res has no _id").to.exist;
-         expect(res._id, "Res has invalid _id").to.be.equal(id)
-         expect(res.active.time.toJSON()).to.be.equal(date.toJSON())
-      })
-
-      it("list", async () => {
-         let res = await sf.get(id);
-         expect(res, "No data returned").to.exist;
-         expect(decoder.decode(res), "Returned data not equal to stored").to.be.equal(decoder.decode(testdata));
-      })
+      let res = await sf.delete(testid);
+      expect(res, "Res not set").to.exist;
    })
 
    describe("folder", () => {
@@ -159,7 +115,8 @@ function test(sf: SecureFile) {
       })
 
       it("delete", async () => {
-         await sf.delete(ftestid);
+         let res = await sf.delete(ftestid);
+         expect(res, "Res not set").to.exist;
       });
    })
 }