fix: use no-cors in page fetch

fix: use no-cors in page fetch

build(pkg): v0.1.2

fix(event): clean unused record

Author: WittBulter

examples/main.ts

@@ -5,7 +5,7 @@ const handler = async() => {
    const backLoader = new BackLoader({
      // scripts: ['https://code.jquery.com/jquery-3.2.1.slim.min.js'],
      // pages: ['https://google.com'],
-     images: ['https://cn.bing.com/az/hprichbg/rb/BarHarborCave_ZH-CN8055769470_1920x1080.jpg'],
+     images: ['https://www.bing.com/az/hprichbg/rb/TadamiTrain_ROW14602114613_1920x1080.jpg'],
    })
   backLoader.start().on(event => {
     console.log(event)

package-lock.json

@@ -1,6 +1,6 @@
 {
   "name": "back-loader",
-  "version": "0.1.1",
+  "version": "0.1.2",
   "description": "",
   "main": "release/index",
   "unpkg": "release/index.js",

package.json

@@ -39,22 +39,28 @@ export class EventHub {
     // just remove one
     if (done) return this.removeOne(eventType, done)
     // remove this type
-    this.eventRecords
-      .filter(re => re.type === eventType)
-      .forEach(re => this.source.removeEventListener(re.type, re.handle))
+    this.eventRecords = this.eventRecords
+    .map(re => {
+      if (re.type !== eventType) return re
+      this.source.removeEventListener(re.type, re.handle)
+      return null
+    })
+    .filter(r => r)
   }
 
   removeAll(): void {
     this.eventRecords
-      .forEach(re => this.source.removeEventListener(re.type, re.handle))
+    .forEach(re => this.source.removeEventListener(re.type, re.handle))
+    this.eventRecords = []
   }
 
   private removeOne(eventType: string, done?: EventHubListener): void {
-    const record: EventRecord = this.eventRecords.find(record => {
+    const index: number = this.eventRecords.findIndex(record => {
       return record.type === eventType && record.done === done
     })
-    if (!record) return
-    this.source.removeEventListener(eventType, record.handle)
+    if (index < 0) return
+    this.source.removeEventListener(eventType, this.eventRecords[index].handle)
+    this.eventRecords.splice(index, 1)
   }
 
 }

src/core/event.ts

@@ -22,12 +22,12 @@ export class Loader {
 
   pages(urls: string[]): void {
     urls.forEach(url => {
-      $fetch(url, { mode: 'cors' })
+      $fetch(url, { mode: 'no-cors' })
       .then(html => {
         this.scripts(filterResources(html, 'script'))
         this.styles(filterResources(html, 'style'))
       })
-      .catch(() => {
+      .catch((e) => {
         this.emit(Object.assign({}, this.baseEvent, {
           type: 'page', source: url, success: false,
         }))

src/core/loader.ts

@@ -1,7 +1,7 @@
 // resource load use no-cors mode default
 // html load must use cors
 export const $fetch = (url: string, init: RequestInit = {}) => {
-  return fetch(url, Object.assign({ mode: 'no-cors' }, init))
+  return fetch(url, Object.assign({ mode: 'cors', cache: 'reload' }, init))
   .then(r => r.text())
 }