Merge branch 'release/1.0.22'

CHANGELOG.md

@@ -1,3 +1,13 @@
+<a name="1.0.22"></a>
+## [1.0.22](https://github.com/adonisjs/adonis-session/compare/v1.0.21...v1.0.22) (2018-03-18)
+
+
+### Bug Fixes
+
+* **session:** allow readonly access to store when freezed ([49282d7](https://github.com/adonisjs/adonis-session/commit/49282d7))
+
+
+
 <a name="1.0.21"></a>
 ## [1.0.21](https://github.com/adonisjs/adonis-session/compare/v1.0.20...v1.0.21) (2018-03-16)
 

package.json

@@ -1,6 +1,6 @@
 {
   "name": "@adonisjs/session",
-  "version": "1.0.21",
+  "version": "1.0.22",
   "description": "This repo is the official session provider for Adonisjs apps. It supports multiple drivers to store session data.",
   "main": "providers/SessionProvider",
   "directories": {

src/Session/index.js

@@ -208,8 +208,6 @@ class Session {
    */
   get (...args) {
     this._ensureInitiated()
-    this._ensureNotFreezed()
-
     return this._store.get(...args)
   }
 
@@ -219,8 +217,6 @@ class Session {
    */
   all (...args) {
     this._ensureInitiated()
-    this._ensureNotFreezed()
-
     return this._store.all(...args)
   }
 

test/session.spec.js

@@ -188,6 +188,26 @@ test.group('Session', () => {
     assert.notProperty(headers, 'set-cookie')
     assert.match(text, /E_RUNTIME_ERROR: Session store is freezed and you cannot write values to session/)
   })
+
+  test('should be able to access values in freezed state', async (assert) => {
+    const server = http.createServer((req, res) => {
+      const config = new Config()
+      const cookie = new Cookie(config)
+      cookie.setRequest(helpers.getRequest(req), helpers.getResponse(res))
+      const session = new Session(helpers.getRequest(req), helpers.getResponse(res), cookie, config)
+      session
+        .instantiate(true)
+        .then(() => {
+          res.end(session.get('username'))
+        }).catch(({ message }) => {
+          res.writeHead(500)
+          res.end(message)
+        })
+    })
+
+    const { text } = await supertest(server).get('/').expect(200)
+    assert.equal(text, '')
+  })
 })
 
 test.group('Session Store', () => {