Changeset 2520259

Timestamp:

04/23/2021 08:37:42 AM (5 years ago)

Author:

antwerpes

Message:

Fix active session detecting by Wordpress Site Health

Location:

doccheck-login/tags/1.0.9

Files:

• . (copied) (copied from doccheck-login/trunk)
• README.txt (modified) (2 diffs)
• client/class-dcl-client.php (modified) (9 diffs)
• dc-login.php (modified) (1 diff)
• includes/class-dcl-base.php (modified) (1 diff)
• includes/class-dcl.php (modified) (2 diffs)

doccheck-login/tags/1.0.9/README.txt

@@ -3 +3 @@
 Tags: DocCheck, login, medical, authentication, hcp, hwg, healthcare, OAuth2
 Requires at least: 4.5
-Tested up to: 5.4.2
-Stable tag: 1.0.8
+Tested up to: 5.7.1
+Stable tag: 1.0.9
 License: GPLv2 or later
 License URI: http://www.gnu.org/licenses/gpl-2.0.html
@@ -50 +50 @@
 
 == Changelog ==
+= 1.0.9 =
+* Fix active session detecting by Wordpress Site Health
+* Tested up to 5.7.1
+
 = 1.0.8 =
 * Bug fix profession routing and profession access management

doccheck-login/tags/1.0.9/client/class-dcl-client.php

@@ -36 +36 @@
      * Initialize the class and set its properties.
      *
-     * @param   string $plugin_name The name of the plugin.
-     * @param   string $version The version of this plugin.
+     * @param string $plugin_name The name of the plugin.
+     * @param string $version The version of this plugin.
      *
      * @since   1.0.0
@@ -47 +47 @@
         $this->plugin_name = $plugin_name;
         $this->version     = $version;
+
     }
 
@@ -235 +236 @@
     }
 
-    public function register_my_session() {
-        if ( ! session_id() ) {
-            session_start();
-        }
-    }
-
     /**
      * "Login" DocCheck user.
@@ -301 +296 @@
             );
 
-
+            session_start();
             // Success
             // Received access token
@@ -308 +303 @@
                 $this->dcl_set_session_token( $response->access_token );
 
-                if ( ! session_id() ) {
-                    session_start();
-                }
-
                 if ( $occupation_activated ) {
                     if ( $occupation_profession_parent_id != null ) {
@@ -327 +318 @@
                     }
 
-
+                    session_write_close();
                     if ( count( $occupations_routing ) > 0 ) {
                         foreach ( $occupations_routing as $item ) {
@@ -351 +342 @@
                 // Remove parameters from URL
                 wp_redirect( $redirect_permalink );
+
                 exit;
             }
@@ -378 +370 @@
         // If the nonce is okay, remove the logged in cookie
         if ( wp_verify_nonce( $_REQUEST['dcl_logout_nonce'], 'dcl_logout_nonce' ) ) {
+            session_start();
             session_destroy();
         }
@@ -478 +471 @@
         $redirect_url = get_permalink();
         $login_url    = $this->dcl_get_login_permalink( $redirect_url );
-
+        session_start();
         // Set redirect cookie
         $this->dcl_set_session_redirect_url( $redirect_url );
-
+        session_write_close();
         wp_redirect( $login_url );
     }

doccheck-login/tags/1.0.9/dc-login.php

@@ -19 +19 @@
  * Plugin Name:       DocCheck Login
  * Description:       The official DocCheck plug-in enables the authentication of certified healthcare professionals and facilitates the integration of the DocCheck login.
- * Version:           1.0.8
+ * Version:           1.0.9
  * Author:            antwerpes ag <[email protected]>
  * Author URI:        https://antwerpes.com

doccheck-login/tags/1.0.9/includes/class-dcl-base.php

@@ -302 +302 @@
     public function dcl_start_session() {
         if ( ! isset( $_SESSION ) ) {
-            session_set_cookie_params( 86400 );
             session_start();
+            session_write_close();
         }
     }

doccheck-login/tags/1.0.9/includes/class-dcl.php

@@ -66 +66 @@
 
         $this->plugin_name = 'dc-login';
-        $this->version     = '1.0.8';
+        $this->version     = '1.0.9';
 
         $this->set_loader();
@@ -160 +160 @@
 
         $this->loader->add_action( 'init', $plugin_public, 'dcl_start_session' );
-        $this->loader->add_action( 'init', $plugin_public, 'register_my_session' );
         $this->loader->add_action( 'pre_get_posts', $plugin_public, 'dcl_filter_search_results' );
         $this->loader->add_filter( 'wp_get_nav_menu_items', $plugin_public, 'dcl_filter_menus' );