@semapps/auth 0.7.0 → 0.8.0

package/mixins/auth.js

@@ -19,7 +19,7 @@ const AuthMixin = {
     accountsDataset: 'settings',
     podProvider: false
   },
-  dependencies: ['api', 'webid'],
+  dependencies: ['api'],
   async created() {
     const { jwtPath, reservedUsernames, accountsDataset, podProvider } = this.settings;
 

package/mixins/auth.sso.js

@@ -33,8 +33,7 @@ const AuthSSOMixin = {
         webId = accountData.webId;
         newUser = false;
 
-        // TODO update account with recent information
-        // await ctx.call('webid.edit', profileData, { meta: { webId } });
+        // TODO update account with recent profileData information
 
         ctx.emit('auth.connected', { webId, accountData, ssoData }, { meta: { webId: null, dataset: null } });
       } else {
@@ -47,7 +46,7 @@ const AuthSSOMixin = {
           email: profileData.email,
           username: profileData.username
         });
-        webId = await ctx.call('webid.create', this.pickWebIdData({ nick: accountData.username, ...profileData }));
+        webId = await ctx.call('webid.createWebId', this.pickWebIdData({ nick: accountData.username, ...profileData }));
         newUser = true;
 
         // Link the webId with the account

package/package.json

@@ -1,14 +1,14 @@
 {
   "name": "@semapps/auth",
-  "version": "0.7.0",
+  "version": "0.8.0",
   "description": "Authentification module for SemApps",
   "license": "Apache-2.0",
   "author": "Virtual Assembly",
   "dependencies": {
-    "@semapps/ldp": "0.7.0",
-    "@semapps/middlewares": "0.7.0",
-    "@semapps/mime-types": "0.7.0",
-    "@semapps/triplestore": "0.7.0",
+    "@semapps/ldp": "0.8.0",
+    "@semapps/middlewares": "0.8.0",
+    "@semapps/mime-types": "0.8.0",
+    "@semapps/triplestore": "0.8.0",
     "bcrypt": "^5.0.1",
     "express-session": "^1.17.0",
     "jsonwebtoken": "^8.5.1",
@@ -29,5 +29,5 @@
   "engines": {
     "node": ">=14"
   },
-  "gitHead": "793f935c61e5f4e87ac17ee1dd41a968f12678b4"
+  "gitHead": "9ab99a4751d5240ec9d6df985bf00dc21c348ef4"
 }

package/services/account.js

@@ -88,6 +88,12 @@ module.exports = {
       const accounts = await this._find(ctx, { query: { email } });
       return accounts.length > 0;
     },
+    /** Overwrite find method, to filter accounts with tombstone. */
+    async find(ctx) {
+      /** @type {object[]} */
+      const accounts = await this._find(ctx, ctx.params);
+      return accounts.filter(account => !account.deletedAt);
+    },
     async findByUsername(ctx) {
       const { username } = ctx.params;
       const accounts = await this._find(ctx, { query: { username } });
@@ -184,6 +190,34 @@ module.exports = {
         '@id': account['@id'],
         ...params
       });
+    },
+    async deleteByWebId(ctx) {
+      const { webId } = ctx.params;
+      const account = await ctx.call('auth.account.findByWebId', { webId });
+
+      if (account) {
+        await this._remove(ctx, { id: account['@id'] });
+        return true;
+      }
+
+      return false;
+    },
+    // Remove email and password from an account, set deletedAt timestamp.
+    async setTombstone(ctx) {
+      const { webId } = ctx.params;
+      const account = await ctx.call('auth.account.findByWebId', { webId });
+
+      return await this._update(ctx, {
+        // Set all values to undefined...
+        ...Object.fromEntries(Object.keys(account).map(key => [key, null])),
+        '@id': account['@id'],
+        // ...except for
+        webId: account.webId,
+        username: account.username,
+        podUri: account.podUri,
+        // And add a deletedAt date.
+        deletedAt: new Date().toISOString()
+      });
     }
   },
   methods: {

package/services/auth.local.js

@@ -4,6 +4,7 @@ const sendToken = require('../middlewares/sendToken');
 const { MoleculerError } = require('moleculer').Errors;
 const AuthMailService = require('./mail');
 
+/** @type {import('moleculer').ServiceSchema} */
 const AuthLocalService = {
   name: 'auth',
   mixins: [AuthMixin],
@@ -27,6 +28,7 @@ const AuthLocalService = {
       }
     }
   },
+  dependencies: ['webid'],
   async created() {
     const { mail } = this.settings;
 
@@ -43,6 +45,8 @@ const AuthLocalService = {
   actions: {
     async signup(ctx) {
       const { username, email, password, interactionId, ...rest } = ctx.params;
+      // This is going to get in our way otherwise when waiting for completions.
+      ctx.meta.skipObjectsWatcher = true;
 
       if (username && username.length < 2) {
         throw new Error('The username must be at least 2 characters long');
@@ -56,7 +60,7 @@ const AuthLocalService = {
       });
 
       const profileData = { nick: username, email, ...rest };
-      const webId = await ctx.call('webid.create', this.pickWebIdData(profileData));
+      const webId = await ctx.call('webid.createWebId', this.pickWebIdData(profileData));
 
       // Link the webId with the account
       accountData = await ctx.call('auth.account.attachWebId', { accountUri: accountData['@id'], webId });