WebSVN – oidplus – Diff – /trunk/includes/classes/OIDplusAuthContentStoreSession.class.php

 \defined('INSIDE_OIDPLUS') or die;
 // phpcs:enable PSR1.Files.SideEffects
 class OIDplusAuthContentStoreSession extends OIDplusAuthContentStore {
-        /**
-         * @return OIDplusSessionHandler
-         */
-        protected static function getSessionHandler(): OIDplusSessionHandler {
-                static $sesHandler = null;
-                if (is_null($sesHandler)) {
-                        $sesHandler = new OIDplusSessionHandler();
-                }
-                return $sesHandler;
-        }
         // Override abstract functions
-        # TODO: shouldn't we just include OIDplusSessionHandler in this class?
         /**
          * @param string $name
          * @param mixed|null $default
          * @return mixed|null
          * @throws OIDplusException
          */
         public function getValue(string $name, $default = NULL) {
                 try {
+                        if (isset($this->cacheSetValues[$name])) return self::decrypt($this->cacheSetValues[$name], $this->secret);
+                        if (!$this->isActive()) return $default; // GDPR: Only start a session when we really need one
+                        $this->sessionSafeStart();
+                        OIDplus::cookieUtils()->setcookie(session_name(),session_id(),time()+$this->sessionLifetime);
+                        if (!isset($_SESSION[$name])) return $default;
-                        return self::getSessionHandler()->getValue($name, $default);
+                        return self::decrypt($_SESSION[$name], $this->secret);
                 } catch (\Exception $e) {
-                        self::getSessionHandler()->destroySession();
+                        $this->destroySession();
                         // TODO: For some reason If destroySession() is called, we won't get this Exception?!
                         throw new OIDplusException(_L('Internal error with session. Please reload the page and log-in again. %1', $e->getMessage()));
+                }
+        }
          * @param mixed $value
          * @return void
          * @throws OIDplusException
          */
         public function setValue(string $name, $value) {
+                $enc_data = self::encrypt($value, $this->secret);
-                self::getSessionHandler()->setValue($name, $value);
+                $this->cacheSetValues[$name] = $enc_data;
+                $this->sessionSafeStart();
+                OIDplus::cookieUtils()->setcookie(session_name(),session_id(),time()+$this->sessionLifetime);
+                $_SESSION[$name] = $enc_data;
+        }
         /**
          * @param string $name
          * @return bool
          * @throws OIDplusException
          */
         public function exists(string $name): bool {
+                if (isset($this->cacheSetValues[$name])) return true;
+                if (!$this->isActive()) return false; // GDPR: Only start a session when we really need one
+                $this->sessionSafeStart();
+                OIDplus::cookieUtils()->setcookie(session_name(),session_id(),time()+$this->sessionLifetime);
-                return self::getSessionHandler()->exists($name);
+                return isset($_SESSION[$name]);
+        }
         /**
          * @param string $name
          * @return void
          * @throws OIDplusException
          */
         public function delete(string $name) {
+                if (isset($this->cacheSetValues[$name])) unset($this->cacheSetValues[$name]);
+                if (!$this->isActive()) return; // GDPR: Only start a session when we really need one
-                self::getSessionHandler()->delete($name);
+                $this->sessionSafeStart();
+                OIDplus::cookieUtils()->setcookie(session_name(),session_id(),time()+$this->sessionLifetime);
+                unset($_SESSION[$name]);
+        }
         /**
          * @return void
          * @throws OIDplusException
          */
         public function destroySession() {
+                if (!$this->isActive()) return;
+                $this->sessionSafeStart();
+                OIDplus::cookieUtils()->setcookie(session_name(),session_id(),time()+$this->sessionLifetime);
+                $_SESSION = array();
-                self::getSessionHandler()->destroySession();
+                session_destroy();
+                session_write_close();
+                OIDplus::cookieUtils()->unsetcookie(session_name()); // remove cookie, so GDPR people are happy
+        }
         /**
          * @return OIDplusAuthContentStoreSession|null
+         * @throws OIDplusException
          */
         public static function getActiveProvider()/*: ?OIDplusAuthContentStore*/ {
                 static $contentProvider = null;
                 $rel_url = substr($_SERVER['REQUEST_URI'], strlen(OIDplus::webpath(null, OIDplus::PATH_RELATIVE_TO_ROOT)));
                         // For REST, we must only allow JWT from Bearer and nothing else! So disable cookies if we are accessing the REST plugin
                         return null;
+                }
                 if (!$contentProvider) {
-                        if (self::getSessionHandler()->isActive()) {
+                        if (self::isActive()) {
                                 $contentProvider = new OIDplusAuthContentStoreSession();
+                        }
+                }
                 return $contentProvider;
         /**
          * @param string $email
          * @param string $loginfo
          * @return void
+         * @throws OIDplusException
          */
         public function raLoginEx(string $email, string &$loginfo) {
                 $this->raLogin($email);
                 if (is_null(self::getActiveProvider())) {
                         $loginfo = 'into new PHP session';
+        }
         /**
          * @param string $loginfo
          * @return void
+         * @throws OIDplusException
          */
         public function adminLoginEx(string &$loginfo) {
                 $this->adminLogin();
                 if (is_null(self::getActiveProvider())) {
                         $loginfo = 'into new PHP session';
          */
         public function activate() {
                 # Sessions automatically activate during setValue()
+        }
+        # ------------------------------------------------------------------------------------------------------------------
+        /**
+         * @var string|null
+         */
+        private $secret = '';
+        /**
+         * @var int|null
+         */
+        protected $sessionLifetime = 0;
+        /**
+         * @throws OIDplusException
+         */
+        public function __construct() {
+                $this->sessionLifetime = OIDplus::baseConfig()->getValue('SESSION_LIFETIME', 30*60);
+                $this->secret = OIDplus::authUtils()->makeSecret(['b118abc8-f4ec-11ed-86ca-3c4a92df8582']);
+                // **PREVENTING SESSION HIJACKING**
+                // Prevents javascript XSS attacks aimed to steal the session ID
+                @ini_set('session.cookie_httponly', '1');
+                // **PREVENTING SESSION FIXATION**
+                // Session ID cannot be passed through URLs
+                @ini_set('session.use_only_cookies', '1');
+                @ini_set('session.use_trans_sid', '0');
+                // Uses a secure connection (HTTPS) if possible
+                @ini_set('session.cookie_secure', OIDplus::isSslAvailable() ? '1' : '0');
+                $path = OIDplus::webpath(null,OIDplus::PATH_RELATIVE);
+                if (empty($path)) $path = '/';
+                @ini_set('session.cookie_path', $path);
+                @ini_set('session.cookie_samesite', OIDplus::baseConfig()->getValue('COOKIE_SAMESITE_POLICY', 'Strict'));
+                @ini_set('session.use_strict_mode', '1');
+                @ini_set('session.gc_maxlifetime', $this->sessionLifetime);
+        }
+        /**
+         * @return void
+         * @throws OIDplusException
+         */
+        protected function sessionSafeStart() {
+                if (!isset($_SESSION)) {
+                        // TODO: session_name() makes some problems. Leave it away for now.
+                        //session_name('OIDplus_SESHDLR');
+                        if (!session_start()) {
+                                throw new OIDplusException(_L('Session could not be started'));
+                        }
+                }
+                if (!isset($_SESSION['ip'])) {
+                        if (!isset($_SERVER['REMOTE_ADDR'])) return;
+                        // Remember the IP address of the user
+                        $_SESSION['ip'] = $_SERVER['REMOTE_ADDR'];
+                } else {
+                        if ($_SERVER['REMOTE_ADDR'] != $_SESSION['ip']) {
+                                // Was the session hijacked?! Get out of here!
+                                // We don't use $this->destroySession(), because this calls sessionSafeStart() again
+                                $_SESSION = array();
+                                session_destroy();
+                                session_write_close();
+                                OIDplus::cookieUtils()->unsetcookie(session_name()); // remove cookie, so GDPR people are happy
+                        }
+                }
+        }
+        /**
+         * @return void
+         */
+        function __destruct() {
+                session_write_close();
+        }
+        private $cacheSetValues = array(); // Important if you do a setValue() followed by an getValue()
+        /**
+         * @return bool
+         */
+        public static function isActive(): bool {
+                return isset($_COOKIE[session_name()]);
+        }
+        /**
+         * @param string $data
+         * @param string $key
+         * @return string
+         * @throws \Exception
+         */
+        protected static function encrypt(string $data, string $key): string {
+                if (function_exists('openssl_encrypt')) {
+                        $iv = random_bytes(16); // AES block size in CBC mode
+                        // Encryption
+                        $ciphertext = openssl_encrypt(
+                                $data,
+                                'AES-256-CBC',
+                                hash_pbkdf2('sha512', $key, '', 10000, 32/*256bit*/, true),
+                                OPENSSL_RAW_DATA,
+                                $iv
+                        );
+                        // Authentication
+                        $hmac = sha3_512_hmac($iv . $ciphertext, $key, true);
+                        return $hmac . $iv . $ciphertext;
+                } else {
+                        // When OpenSSL is not available, then we just do a HMAC
+                        $hmac = sha3_512_hmac($data, $key, true);
+                        return $hmac . $data;
+                }
+        }
+        /**
+         * @param string $data
+         * @param string $key
+         * @return string
+         * @throws OIDplusException
+         */
+        protected static function decrypt(string $data, string $key): string {
+                if (function_exists('openssl_decrypt')) {
+                        $hmac       = mb_substr($data, 0, 64, '8bit');
+                        $iv         = mb_substr($data, 64, 16, '8bit');
+                        $ciphertext = mb_substr($data, 80, null, '8bit');
+                        // Authentication
+                        $hmacNew = sha3_512_hmac($iv . $ciphertext, $key, true);
+                        if (!hash_equals($hmac, $hmacNew)) {
+                                throw new OIDplusException(_L('Authentication failed'));
+                        }
+                        // Decryption
+                        $cleartext = openssl_decrypt(
+                                $ciphertext,
+                                'AES-256-CBC',
+                                hash_pbkdf2('sha512', $key, '', 10000, 32/*256bit*/, true),
+                                OPENSSL_RAW_DATA,
+                                $iv
+                        );
+                        if ($cleartext === false) {
+                                throw new OIDplusException(_L('Decryption failed'));
+                        }
+                        return $cleartext;
+                } else {
+                        // When OpenSSL is not available, then we just do a HMAC
+                        $hmac       = mb_substr($data, 0, 64, '8bit');
+                        $cleartext  = mb_substr($data, 64, null, '8bit');
+                        $hmacNew    = sha3_512_hmac($cleartext, $key, true);
+                        if (!hash_equals($hmac, $hmacNew)) {
+                                throw new OIDplusException(_L('Authentication failed'));
+                        }
+                        return $cleartext;
+                }
+        }
+}

Subversion Repositories oidplus

oidplus/trunk/includes/classes/OIDplusAuthContentStoreSession.class.php – Rev 1300 → 1301