1 files changed, 6 insertions, 84 deletions

diff --git a/dht/src/Network/Tox/NodeId.hs b/dht/src/Network/Tox/NodeId.hs
index 311095ec..68888fdd 100644
--- a/dht/src/Network/Tox/NodeId.hs
+++ b/dht/src/Network/Tox/NodeId.hs
@@ -38,7 +38,7 @@ module Network.Tox.NodeId
     , ToxContact(..)
     , ToxProgress(..)
     , parseToken32
-    , show64Token32
+    , showToken32
     , show32Token32
     , nodeInfoFromJSON
     ) where
@@ -91,6 +91,7 @@ import System.Endian
 import qualified Data.Text as Text
          ;import Data.Text (Text)
 import Util (splitJID)
+import Codec.AsciiKey256
 
 -- | perform io for hashes that do allocation and ffi.
 -- unsafeDupablePerformIO is used when possible as the
@@ -163,42 +164,8 @@ zeroKey = throwCryptoError $ publicKey $ B.replicate 32 0
 zeroID :: NodeId
 zeroID = NodeId (replicate 4 0) (Just zeroKey)
 
--- | Convert to and from a Base64 variant that uses .- instead of +/.
-nmtoken64 :: Bool -> Char -> Char
-nmtoken64 False '.' = '+'
-nmtoken64 False '-' = '/'
-nmtoken64 True  '+' = '.'
-nmtoken64 True  '/' = '-'
-nmtoken64 _     c   = c
-
--- | Parse 43-digit base64 token into 32-byte bytestring.
-parseToken32 :: String -> Either String ByteString
-parseToken32 str = fmap (BA.drop 1) $ Base64.decode $ C8.pack $ 'A':map (nmtoken64 False) (take 43 str)
-
--- | Parse 52-digit z-base32 token into 32-byte bytestring.
-parse32Token32 :: String -> Either String ByteString
-parse32Token32 str = fmap (BA.drop 1) $ Base32.decode $ C8.pack $ 'y':map (fixupDigit32 . toLower) (take 52 str)
-
--- | Encode 32-byte bytestring as 43-digit base64 token.
-show64Token32 :: ByteArrayAccess bin => bin -> String
-show64Token32 bs = map (nmtoken64 True) $ C8.unpack $ BA.drop 1 $ Base64.encode $ BA.cons 0 $ BA.convert bs
-
--- | Encode 32-byte bytestring as 52-digit z-base32 token.
-show32Token32 :: ByteArrayAccess bin => bin -> String
-show32Token32 bs = map (nmtoken64 True) $ C8.unpack $ BA.drop 1 $ Base32.encode $ BA.cons 0 $ BA.convert bs
-
 instance Read NodeId where
-    readsPrec _ str
-        | (bs,_) <- Base16.decode (C8.pack $ take 64 str)
-        , CryptoPassed pub <- publicKey bs -- B.length bs == 32
-                    = [ (key2id pub, drop (2 * B.length bs) str) ]
-        | Right bs <- parse32Token32 str
-        , CryptoPassed pub <- publicKey bs -- B.length bs == 32
-                    = [ (key2id pub, drop 52 str) ]
-        | Right bs <- parseToken32 str
-        , CryptoPassed pub <- publicKey bs -- B.length bs == 32
-                    = [ (key2id pub, drop 43 str) ]
-        | otherwise = []
+    readsPrec _ str = readsPrecKey256 (fmap key2id . maybeCryptoError . publicKey) str
 
 instance Show NodeId where
     show nid = show32Token32 $ id2key nid
@@ -279,14 +246,7 @@ nodeInfoFromJSON prefer4 (JSON.Object v) = do
                       <|> maybe empty (return . IPv6) (ip6str >>= readMaybe)
                 else maybe empty (return . IPv6) (ip6str >>= readMaybe)
                       <|> maybe empty (return . IPv4) (ip4str >>= readMaybe)
-        let nidbs  = C8.pack nidstr
-            (bs,_) = Base16.decode nidbs
-            enid   = case C8.length nidbs of
-                        43 -> Base64.decode (C8.pack $ 'A' : map (nmtoken64 False) nidstr)
-                        52 -> Base32.decode (C8.pack $ 'y' : map (fixupDigit32 . toLower) nidstr)
-                        _  -> Left "Wrong size of node-id."
-        idbs <- (guard (B.length bs == 32) >> return bs)
-                <|> either fail (return . B.drop 1) enid
+        idbs <- parseKey256 nidstr
         return $ NodeInfo (bs2id idbs) ip (fromIntegral (portnum :: Word16))
 
 getIP :: Word8 -> S.Get IP
@@ -320,31 +280,6 @@ instance S.Serialize NodeInfo where
         S.put port
         S.put nid
 
-hexdigit :: Char -> Bool
-hexdigit c = ('0' <= c && c <= '9') || ( 'a' <= c && c <= 'f') || ( 'A' <= c && c <= 'F')
-
-b64digit :: Char -> Bool
-b64digit '.' = True
-b64digit '+' = True
-b64digit '-' = True
-b64digit '/' = True
-b64digit c = ('0' <= c && c <= '9') || ( 'a' <= c && c <= 'z') || ( 'A' <= c && c <= 'Z')
-
-zb32digit :: Char -> Bool
-zb32digit '1' = True
-zb32digit c = or [ '3' <= c && c <= '9'
-                 , 'a' <= c && c <= 'k'
-                 , 'm' <= c && c <= 'u'
-                 , 'w' <= c && c <= 'z'
-                 ]
-
--- Apply substitutions for mistaken z-base32 digits.
-fixupDigit32 :: Char -> Char
-fixupDigit32 'l' = '1'
-fixupDigit32 '2' = 'z'
-fixupDigit32 'v' = 'u'
-fixupDigit32  c  =  c
-
 ip_w_port :: Int -> RP.ReadP (IP, PortNumber)
 ip_w_port i = do
             ip <- RP.between (RP.char '[') (RP.char ']')
@@ -360,23 +295,10 @@ instance Read NodeInfo where
     RP.skipSpaces
     let parseAddr = RP.between (RP.char '(') (RP.char ')') (RP.munch (/=')'))
                       RP.+++ RP.munch (\c -> not (isSpace c) && not (c `elem` ("{}()"::[Char])))
-        nodeidAt = do (is64,hexhash) <- foldr1 (RP.+++)
-                        [ fmap (32,) (sequence $ replicate 52 (RP.satisfy zb32digit))
-                        , fmap (64,) (sequence $ replicate 43 (RP.satisfy b64digit))
-                        , fmap (16,) (sequence $ replicate 64 (RP.satisfy isHexDigit))
-                        ]
+        nodeidAt = do
+                      nid <- bs2id <$> readP_key256
                       RP.char '@' RP.+++ RP.satisfy isSpace
                       addrstr <- parseAddr
-                      nid <- case is64 of
-                        32 -> case Base32.decode $ C8.pack $ 'y' : map (fixupDigit32 . toLower) hexhash of
-                                  Right bs | B.length bs - 1==32 -> return (bs2id $ BA.drop 1 bs)
-                                  _                              -> fail "Bad node id."
-                        64 -> case Base64.decode $ C8.pack $ 'A' : map (nmtoken64 False) hexhash of
-                                  Right bs | B.length bs - 1==32 -> return (bs2id $ BA.drop 1 bs)
-                                  _                              -> fail "Bad node id."
-                        _  -> case Base16.decode $ C8.pack hexhash of
-                                  (bs,rem) | B.length bs == 32 &&  B.null rem -> return (bs2id bs)
-                                  _                                           -> fail "Bad node id."
                       return (nid,addrstr)
     (nid,addrstr) <- ( nodeidAt RP.+++ ( (zeroID,) <$> parseAddr) )
     (ip,port) <- case RP.readP_to_S (ip_w_port i) addrstr of