Merge pull request #3 from is-a-dev/wdhdev/filter-noreply

filter users.noreply.github.com emails

src/routes/auth.js

@@ -6,7 +6,6 @@ const fetch = require("node-fetch");
 
 module.exports = async (req, res) => {
     const code = req.query.code;
-    const GITHUB_ID = process.env.GITHUB_ID;
 
     if (!code) {
         return res.status(400).json({ error: "No code provided" });

src/routes/profile.js

@@ -5,5 +5,9 @@ module.exports = async (req, res) => {
     const emails = user.emails;
     const message = 'test message';
 
-    return res.render("profile", { username, user_id, emails, message, error: false, user: user });
-}
+    let filteredEmails = emails.filter(function(e) {
+        return e.verified && !e.email.endsWith("@users.noreply.github.com");
+    })
+
+    return res.render("profile", { username, user_id, emails: filteredEmails, message, error: false, user: user });
+}

src/util/gitAuth.js

@@ -38,7 +38,11 @@ async function authenticateGithub(code) {
         },
     );
 
-    const emails = await emailResponse.json();
+    let emails = await emailResponse.json();
+
+    emails = emails.filter(function(e) {
+        return e.verified && !e.email.endsWith("@users.noreply.github.com");
+    });
 
     return { username, user_id, emails };
 }