Domain Mapping: Q re. debug log entries after recent bug fix...

Hi WPMUDEV,

Hope your day is going really awesomely!

I have a Q re. an entry we are seeing very frequently in our debug log after @Jose recently completed some work to patch our DM for issue when defining DM_FORCE_PROTOCOL_ON_MAPPED_DOMAIN to be true...

There is a bug in the parsing logic for the original url. Therefore, it returns this broken url:
https:disappointed:htts:disappointed:tivism-secure-1.wpms.network/wp-login.php
When the browser is not able to find a valid protocol/schema it will treat the url as a relative one, prepending the base url https://tivism.com/.

https://premium.wpmudev.org/forums/topic/domain-mapping-bug-4033-breaks-https-permalinks-causes-mixed-content#post-852806

In debug log we are now seeing many (many!) entries
PHP Notice: Trying to get property of non-object in /home/xxxx/public_html/ext/plugins/domain-mapping/classes/Domainmap/Module/Mapping.php on line 675

In our patched DM, that file location has the following code

/**
		 * Force single page
		 */
		if( !is_admin() && ( $this->is_ssl_forced_by_id( $post->ID ) || $this->is_ssl_forced_by_request() ) && !is_ssl() ){
			wp_redirect( $current_url_secure  );
			exit();
		}elseif(  $this->is_mapped_domain() && self::force_ssl_on_mapped_domain() !== 2 && !( $this->is_ssl_forced_by_id( $post->ID ) || $this->is_ssl_forced_by_request() ) ){
			/**
			 * Force mapped domains
			 */
			if( self::force_ssl_on_mapped_domain() === 1 && !is_ssl()  ){ // force https
				wp_redirect( $current_url_secure  );
				exit();
			}elseif( self::force_ssl_on_mapped_domain() === 0 && is_ssl() ){ //force http
				wp_redirect( $current_url);
				exit();
			}
		}

wherein line 675 is
wp_redirect( $current_url_secure );

Basically, we're wondering if this is related to the recent work? Please advise =)

Kind Regards, Max