ref(api): Move getVerifiedAccounts to api

3box · Feb 5, 2019 · 6abc2fa · 6abc2fa
1 parent 1039ac1
commit 6abc2fa
Show file tree

Hide file tree

Showing 4 changed files with 29 additions and 25 deletions.
diff --git a/README.md b/README.md
@@ -189,7 +189,7 @@ This runs a simple server at `http://localhost:3000/` that serves the static `ex
 If you only want to fetch profile data from 3Box's profile APIs you can optimize by importing only those functions or the API specific dist file. Since this includes minimal dependencies, file size is ~ 80kb vs 4+mb for the full build.
 
 ```js
-const { profileGraphQL, getProfile, getProfiles } = require('3box/lib/api')
+const { profileGraphQL, getProfile, getProfiles, getVerifiedAccounts } = require('3box/lib/api')
 ```
 ```html
 <script src="https://unpkg.com/3box/dist/3box.api.min.js"></script>

diff --git a/readme-template.md b/readme-template.md
@@ -189,7 +189,7 @@ This runs a simple server at `http://localhost:3000/` that serves the static `ex
 If you only want to fetch profile data from 3Box's profile APIs you can optimize by importing only those functions or the API specific dist file. Since this includes minimal dependencies, file size is ~ 80kb vs 4+mb for the full build.
 
 ```js
-const { profileGraphQL, getProfile, getProfiles } = require('3box/lib/api')
+const { profileGraphQL, getProfile, getProfiles, getVerifiedAccounts } = require('3box/lib/api')
 ```
 ```html
 <script src="https://unpkg.com/3box/dist/3box.api.min.js"></script>

diff --git a/src/3box.js b/src/3box.js
@@ -11,7 +11,6 @@ const PrivateStore = require('./privateStore')
 const Verified = require('./verified')
 const Space = require('./space')
 const utils = require('./utils/index')
-const verifier = require('./utils/verifier')
 const config = require('./config.js')
 const API = require('./api')
 
@@ -281,27 +280,7 @@ class Box {
    * @return    {Object}                                    An object containing the accounts that have been verified
    */
   static async getVerifiedAccounts (profile) {
-    let verifs = {}
-    try {
-      const did = await verifier.verifyDID(profile.proof_did)
-      if (profile.proof_github) {
-        try {
-          verifs.github = await verifier.verifyGithub(did, profile.proof_github)
-        } catch (err) {
-          // Invalid github verification
-        }
-      }
-      if (profile.proof_twitter) {
-        try {
-          verifs.twitter = await verifier.verifyTwitter(did, profile.proof_twitter)
-        } catch (err) {
-          // Invalid twitter verification
-        }
-      }
-    } catch (err) {
-      // Invalid proof for DID return an empty profile
-    }
-    return verifs
+    return API.getVerifiedAccounts(profile)
   }
 
   /**

diff --git a/src/api.js b/src/api.js
@@ -1,5 +1,6 @@
 const graphQLRequest = require('graphql-request').request
 const utils = require('./utils/index')
+const verifier = require('./utils/verifier')
 const config = require('./config.js')
 
 const GRAPHQL_SERVER_URL = config.graphql_server_url
@@ -39,4 +40,28 @@ async function profileGraphQL (query, opts = {}) {
   return graphQLRequest(opts.graphqlServer, query)
 }
 
-module.exports = { profileGraphQL, getProfile, getRootStoreAddress, getProfiles }
+async function getVerifiedAccounts (profile) {
+  let verifs = {}
+  try {
+    const did = await verifier.verifyDID(profile.proof_did)
+    if (profile.proof_github) {
+      try {
+        verifs.github = await verifier.verifyGithub(did, profile.proof_github)
+      } catch (err) {
+        // Invalid github verification
+      }
+    }
+    if (profile.proof_twitter) {
+      try {
+        verifs.twitter = await verifier.verifyTwitter(did, profile.proof_twitter)
+      } catch (err) {
+        // Invalid twitter verification
+      }
+    }
+  } catch (err) {
+    // Invalid proof for DID return an empty profile
+  }
+  return verifs
+}
+
+module.exports = { profileGraphQL, getProfile, getRootStoreAddress, getProfiles, getVerifiedAccounts }