FIX for false-positive href rewrites

- Closes #43
ipfs · Jan 16, 2016 · f696599 · f696599
1 parent 701443c
commit f696599
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 1 deletion.
diff --git a/lib/gateways.js b/lib/gateways.js
@@ -5,6 +5,8 @@ const {Cc, Ci} = require('chrome')
 const prefs = require('sdk/simple-prefs').prefs
 const ioservice = Cc['@mozilla.org/network/io-service;1'].getService(Ci.nsIIOService)
 
+// TODO: replace regex with 3rd party library provided by IPFS community
+// Ref. https://github.com/ipfs/notes/issues/92#issuecomment-172135885
 const IPFS_RESOURCE = /^https?:\/\/[^\/]+\/ip(f|n)s\//
 
 exports.IPFS_RESOURCE = IPFS_RESOURCE

diff --git a/lib/protocols.js b/lib/protocols.js
@@ -10,7 +10,9 @@ const IPNS_SCHEME = 'ipns'
 const FS_SCHEME = 'fs'
 const WEB_SCHEME_PREFIX = 'web+' // https://github.com/lidel/ipfs-firefox-addon/issues/36
 
-const IPFS_PATH = new RegExp('^/ip(f|n)s/')
+// TODO: replace regex with 3rd party library provided by IPFS community
+// Ref. https://github.com/ipfs/notes/issues/92#issuecomment-172135885
+const IPFS_PATH = new RegExp('^/(ipns/|ipfs/Qm[1-9A-HJ-NP-Za-km-z]{44})')
 
 function CommonProtocolHandler () {}