Remove redundant "identified" setting.

Author: RubenVerborgh

lib/api/authn/force-user.js

@@ -9,7 +9,6 @@ function initialize (app, argv) {
   app.use('/', (req, res, next) => {
     debug(`Identified user (override): ${forceUserId}`)
     req.session.userId = forceUserId
-    req.session.identified = true
     res.set('User', forceUserId)
     next()
   })

lib/api/authn/webid-tls.js

@@ -13,7 +13,7 @@ function initialize (app, argv) {
 
 function handler (req, res, next) {
   // User already logged in? skip
-  if (req.session.userId && req.session.identified) {
+  if (req.session.userId) {
     debug('User: ' + req.session.userId)
     res.set('User', req.session.userId)
     return next()
@@ -34,7 +34,6 @@ function handler (req, res, next) {
       return next()
     }
     req.session.userId = result
-    req.session.identified = true
     debug('Identified user: ' + req.session.userId)
     res.set('User', req.session.userId)
     return next()
@@ -88,7 +87,6 @@ function getCertificateViaHeader (req) {
 
 function setEmptySession (req) {
   req.session.userId = ''
-  req.session.identified = false
 }
 
 /**

lib/create-app.js

@@ -159,7 +159,6 @@ function initWebId (argv, app, ldp) {
         debug(`Rejecting session for ${userId} from ${origin}`)
         // Destroy session data
         delete req.session.userId
-        delete req.session.identified
         // Ensure this modified session is not saved
         req.session.save = (done) => done()
       }

lib/requests/auth-request.js

@@ -155,7 +155,6 @@ class AuthRequest {
     debug('Initializing user session with webId: ', userAccount.webId)
 
     session.userId = userAccount.webId
-    session.identified = true
     session.subject = {
       _id: userAccount.webId
     }

package.json

@@ -59,7 +59,7 @@
     "node-forge": "^0.6.38",
     "nodemailer": "^3.1.4",
     "nomnom": "^1.8.1",
-    "oidc-auth-manager": "^0.7.2",
+    "oidc-auth-manager": "^0.7.3",
     "oidc-op-express": "^0.0.3",
     "rdflib": "^0.15.0",
     "recursive-readdir": "^2.1.0",

test/unit/auth-request.js

@@ -92,7 +92,6 @@ describe('AuthRequest', () => {
       request.initUserSession(alice)
 
       expect(request.session.userId).to.equal(webId)
-      expect(request.session.identified).to.be.true()
       let subject = request.session.subject
       expect(subject['_id']).to.equal(webId)
     })