add profile scope

package-lock.json

@@ -1,12 +1,12 @@
 {
   "name": "archery",
-  "version": "0.2.0",
+  "version": "0.2.2",
   "lockfileVersion": 3,
   "requires": true,
   "packages": {
     "": {
       "name": "archery",
-      "version": "0.2.0",
+      "version": "0.2.2",
       "license": "MIT",
       "dependencies": {
         "body-parser": "^2.2.0",

package.json

@@ -1,6 +1,6 @@
 {
   "name": "archery",
-  "version": "0.2.0",
+  "version": "0.2.2",
   "description": "Build Arch packages through a web interface",
   "keywords": [
     "docker",

src/DB.ts

@@ -216,6 +216,15 @@ class DB extends Store {
         return user.id;
     }
 
+    public async upsertUser(user: User): Promise<string> {
+        await this.user.upsert({
+            id: user.id,
+            username: user.username,
+            displayName: user.displayName || null
+        });
+        return user.id;
+    }
+
     public async createBuild(repo: string, commit: string, patch: string, distro: string, dependencies: string, author: string, uuid: string): Promise<number> {
         const buildRec = await this.build.create({
             repo,

src/Web.ts

@@ -324,17 +324,15 @@ class Web {
             userInfoURL: openidconf.userinfo_endpoint,
             clientID: options.oidc.clientId,
             clientSecret: options.oidc.clientSecret,
-            callbackURL: `${baseUrl}cb`
+            callbackURL: `${baseUrl}cb`,
+            scope: 'profile'
         }, async (_: string, profile: passport.Profile, cb: OpenIDConnectStrategy.VerifyCallback) => {
             const userObj: User = {
                 id: profile.id,
                 username: profile.username,
                 displayName: profile.displayName
             };
-            const user = await this.db.getUser(profile.id);
+            await this.db.upsertUser(userObj);
-            if (!user) {
-                await this.db.createUser(userObj);
-            }
             return cb(null, userObj);
         });
     }

views/build.ejs

@@ -31,7 +31,7 @@
             <a href="/build?id=<%= build.sqid %>" class="button">Clone</a>
         </div>
         <% } %>
-        <% if (!ended) { %>
+        <% if (!ended && !public) { %>
         <div id="cancelRow">
             <a href="cancel" class="button">Cancel build</a>
         </div>