fix(SMB): use of @ and : chars in passwords (#8)

Fixes #4.

packages/xo-remote-parser/src/index.js

@@ -18,8 +18,12 @@ export const parse = (remote) => {
     remote.share = share
   } else if (type === 'smb') {
     remote.type = 'smb'
-    const [auth, smb] = rest.split('@')
+    const lastArobas = rest.lastIndexOf('@')
-    const [username, password] = auth.split(':')
+    const smb = rest.slice(lastArobas)
+    const auth = rest.slice(0, lastArobas)
+    const firstColon = auth.indexOf(':')
+    const username = auth.slice(0, firstColon)
+    const password = auth.slice(firstColon)
     const [domain, sh] = smb.split('\\\\')
     const [host, path] = sh.split('\0')
     remote.host = host

packages/xo-remote-parser/src/index.spec.js

@@ -24,6 +24,26 @@ const SMB = {
   password: 'password'
 }
 
+const SMB_AROBAS = {
+  url: 'smb://Administrator:pass@word@toto\\\\192.168.100.225\\smb\0',
+  type: 'smb',
+  host: '192.168.100.225\\smb',
+  path: undefined,
+  domain: 'toto',
+  username: 'Administrator',
+  password: 'pass@word'
+}
+
+const SMB_COLON = {
+  url: 'smb://Administrator:pass:word@toto\\\\192.168.100.225\\smb\0',
+  type: 'smb',
+  host: '192.168.100.225\\smb',
+  path: undefined,
+  domain: 'toto',
+  username: 'Administrator',
+  password: 'pass:word'
+}
+
 const parseData = {
   file: {
     url: 'file://var/lib/xoa/backup', // Remotes formatted before fixing #7 will not break when reparses
@@ -31,7 +51,9 @@ const parseData = {
     path: '/var/lib/xoa/backup'
   },
   fileFixed: FILE_FIXED,
-  smb: SMB
+  smb: SMB,
+  'smb@inPassword': SMB_AROBAS,
+  'smb:inPassword': SMB_COLON
 }
 
 const formatData = {
@@ -41,7 +63,9 @@ const formatData = {
     path: '/var/lib/xoa/backup'
   },
   fileFixed: FILE_FIXED,
-  smb: SMB
+  smb: SMB,
+  'smb@inPassword': SMB_AROBAS,
+  'smb:inPassword': SMB_COLON
 }
 
 // -------------------------------------------------------------------