Feature #4607

control panel port redirection

Added by Jaime Melis about 5 years ago. Updated almost 4 years ago.

Status:ClosedStart date:06/28/2016
Priority:HighDue date:
Assignee:Jaime Melis% Done:

0%

Category:vCenter
Target version:Release 5.4.1
Resolution:fixed Pull request:

Description

it should redirect port but always use http, not https.

History

#1 Updated by Miguel Ángel Álvarez Cabrerizo about 4 years ago

  • Status changed from Pending to Closed

This issue has been moved to a vOneCloud ticket.

#2 Updated by Jaime Melis almost 4 years ago

  • Tracker changed from Bug to Feature
  • Status changed from Closed to Assigned
  • Priority changed from Normal to High
  • Target version set to Release 5.4.1

#3 Updated by Vlastimil Holer almost 4 years ago

  • Status changed from Assigned to Closed
  • Resolution set to fixed

Fixed by Jaime's older commit.

commit ed22ffa810f3677781b34bea012e63c2059d6dec
Author: Jaime Melis <jmelis@opennebula.org>
Date:   Fri Jul 29 12:02:49 2016 +0200

    Fix redirect port (always to http)

    (cherry picked from commit eef1ee6d4f34c3ebdf6c7cb6e764169d36e8e69d)

diff --git a/src/sunstone/public/app/tabs/upgrade-top-tab.js b/src/sunstone/public/app/tabs/upgrade-top-tab.js
index 815a217c7..7df99b754 100644
--- a/src/sunstone/public/app/tabs/upgrade-top-tab.js
+++ b/src/sunstone/public/app/tabs/upgrade-top-tab.js
@@ -33,7 +33,7 @@ define(function(require) {
       var upgrade_url = config['upgrade']['url'];

       if (redirect_port) {
-        window.location = document.URL.replace(/(https?:\/\/)([^:\/]+).*$/,"$1$2:"+redirect_port)
+        window.location = document.URL.replace(/(https?:\/\/)([^:\/]+).*$/,"http://$2:"+redirect_port)
       } else {
         window.location = upgrade_url
       }

Closing.

Also available in: Atom PDF