Merge pull request #61 from humanmade/backport-59-to-v6-branch

[Backport v6-branch] Fix port for SAML login
humanmade · Jul 30, 2021 · 5b6dd57 · 5b6dd57
2 parents e58c0a3 + 9b1647f
commit 5b6dd57
Showing 1 changed file with 6 additions and 0 deletions.
diff --git a/inc/saml/namespace.php b/inc/saml/namespace.php
@@ -15,6 +15,12 @@
  * @return void
  */
 function bootstrap() {
+	// Set server port to 443 or 80 for SAML, port 8080 breaks validation.
+	if ( class_exists( '\\OneLogin\\Saml2\\Utils' ) ) {
+		$is_https = ! empty( $_SERVER['HTTPS'] ) && $_SERVER['HTTPS'] === 'on';
+		\OneLogin\Saml2\Utils::setSelfPort( $is_https ? '443' : '80' );
+	}
+
 	add_filter( 'wpsimplesaml_network_activated', '__return_true' );
 	add_filter( 'wpsimplesaml_idp_metadata_xml_path', __NAMESPACE__ . '\\get_idp_metadata_file_path' );
 	add_filter( 'pre_site_option_sso_sp_base', __NAMESPACE__ . '\\get_sp_client_id' );