Ignore accessTime updates caused by read operations (#762)

paulmillr · Feb 5, 2019 · fb8ef5e · fb8ef5e
1 parent 0a46e6f
commit fb8ef5e
Showing 1 changed file with 14 additions and 2 deletions.
diff --git a/lib/nodefs-handler.js b/lib/nodefs-handler.js
@@ -264,12 +264,24 @@ function(file, stats, initialAdd, callback) {
         if (error) {
           this._remove(dirname, basename);
         } else {
-          this._emit('change', file, newStats);
+          // check that change event was not fired coz of changed accessTime
+          if (
+            !newStats.atime.getTime() ||
+            newStats.atime.getTime() <= newStats.mtime.getTime()
+          ) {
+            this._emit('change', file, newStats);
+          }
         }
       }.bind(this));
     // add is about to be emitted if file not already tracked in parent
     } else if (parent.has(basename)) {
-      this._emit('change', file, newStats);
+      // check that change event was not fired coz of changed accessTime
+      if (
+        !newStats.atime.getTime() ||
+        newStats.atime.getTime() <= newStats.mtime.getTime()
+      ) {
+        this._emit('change', file, newStats);
+      }
     }
   }.bind(this));