Fuel UI should by default redirect to https port, not 8000 http

Bug #1497397 reported by Mike Scherbakov
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Fuel for OpenStack
Fix Released
Medium
Stanislaw Bogatkin
7.0.x
Won't Fix
Medium
Stanislaw Bogatkin

Bug Description

Build RC3
http://10.20.0.2/
redirects to http://10.20.0.2:8000/#login
while I expect it to be redirected to https://10.20.0.2:8443/

Tags: area-library
Revision history for this message
Stanislaw Bogatkin (sbogatkin) wrote :

So we just need to add redirection to HTTPS port 8443 when trying to access a root directory for apache.

Changed in fuel:
status: New → Triaged
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to fuel-library (master)

Fix proposed to branch: master
Review: https://review.openstack.org/225938

Changed in fuel:
status: Triaged → In Progress
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to fuel-library (stable/7.0)

Fix proposed to branch: stable/7.0
Review: https://review.openstack.org/226653

Changed in fuel:
importance: High → Medium
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to fuel-library (master)

Reviewed: https://review.openstack.org/225938
Committed: https://git.openstack.org/cgit/stackforge/fuel-library/commit/?id=94027ba91b3cd37222cf885cc1456f70151ae512
Submitter: Jenkins
Branch: master

commit 94027ba91b3cd37222cf885cc1456f70151ae512
Author: Stanislaw Bogatkin <email address hidden>
Date: Mon Sep 21 19:48:15 2015 +0300

    Redirect all but cobbler and api from port 80

    Redirect almost all from port 80 to port 8443 with SSL. Exclusions:
    * API still have redirected to port 8000.
    * Cobbler still uses port 80.

    Change-Id: I872bb1c29df4b4775166944db7941296b352d93c
    Closes-Bug: #1497397

Changed in fuel:
status: In Progress → Fix Committed
Revision history for this message
Vitaly Sedelnik (vsedelnik) wrote :

Won't Fix for 7.0-updates because of Medium importance

Revision history for this message
Anastasia Palkina (apalkina) wrote :

I did not see redirection for API and cobbler (see screens).

"build_id": "103", "openstack_version": "2015.1.0-8.0", "build_number": "103", "release_versions": {"2015.1.0-8.0": {"VERSION": {"build_id": "103", "openstack_version": "2015.1.0-8.0", "build_number": "103", "api": "1.0", "fuel-library_sha": "0ab2c2f2c122f82085685c15fdc7867a0963cdd1", "feature_groups": ["mirantis"], "fuel-nailgun-agent_sha": "d66f188a1832a9c23b04884a14ef00fc5605ec6d", "fuel-nailgun_sha": "180a39eae65300144a8f0a41e335b0845570d88b", "fuel-agent_sha": "e881f0dabd09af4be4f3e22768b02fe76278e20e", "production": "docker", "python-fuelclient_sha": "9f0c5c86eba94d3061a1cd715f3135dc4bfba0d6", "astute_sha": "0f753467a3f16e4d46e7e9f1979905fb178e4d5b", "fuel-ostf_sha": "37c5d6113408a29cabe0f416fe99cf20e2bca318", "release": "8.0", "fuelmain_sha": "8e5e75302b2534fd38e4b41b795957111ac75543"}}}, "auth_required": true, "api": "1.0", "fuel-library_sha": "0ab2c2f2c122f82085685c15fdc7867a0963cdd1", "feature_groups": ["mirantis"], "fuel-nailgun-agent_sha": "d66f188a1832a9c23b04884a14ef00fc5605ec6d", "fuel-nailgun_sha": "180a39eae65300144a8f0a41e335b0845570d88b", "fuel-agent_sha": "e881f0dabd09af4be4f3e22768b02fe76278e20e", "production": "docker", "python-fuelclient_sha": "9f0c5c86eba94d3061a1cd715f3135dc4bfba0d6", "astute_sha": "0f753467a3f16e4d46e7e9f1979905fb178e4d5b", "fuel-ostf_sha": "37c5d6113408a29cabe0f416fe99cf20e2bca318", "release": "8.0", "fuelmain_sha": "8e5e75302b2534fd38e4b41b795957111ac75543"

Revision history for this message
Anastasia Palkina (apalkina) wrote :
Revision history for this message
Anastasia Palkina (apalkina) wrote :
Changed in fuel:
status: Fix Committed → Confirmed
Revision history for this message
Stanislaw Bogatkin (sbogatkin) wrote :

Hi Anastasia.
If you'll look into fix commit message, there will be next text:

 Redirect almost all from port 80 to port 8443 with SSL. Exclusions:
    * API still have redirected to port 8000.
    * Cobbler still uses port 80.

So what you speaking about is expected behavior. We can't move links to cobbler and API due backward compatibility reasons - not yet. Moreover, bug itself was about redirection only root path and fix doing exactly this.

So, I move it back to 'fix committed'.

Changed in fuel:
status: Confirmed → Fix Committed
Revision history for this message
Anastasia Palkina (apalkina) wrote :

Verified

"fuel-createmirror_sha": "fe81731b76071dc0a4ffdf039080732243f5a871", "api": "1.0", "fuelmain_sha": "f5b83e3f0ccfce90b9ca797bc039c9ab50b5ee55", "astute_sha": "a090546d43c770ac27ca81c6f8c78ff0ba4a93e0", "fuel-ostf_sha": "e308a7ad68838f6e012d6d80243eb32b4f7e6c18", "openstack_version": "2015.1.0-8.0", "build_id": "151", "build_number": "151", "auth_required": true, "fuel-library_sha": "d4b67bca618c48332486339f8342413ea9b4bc3b", "fuel-nailgun-agent_sha": "08e0a11cf1f29b705e4b910d9b9db5e9b708b6e3", "fuel-nailgun_sha": "56a1ca6c1889ae5a8378198db65c0168c8c06484", "fuel-agent_sha": "5d2d5e2e9ad4f8bb01e450240d85447608d84b92", "production": "docker", "python-fuelclient_sha": "655db5e96282bbafc4dae54462a01823b6b464cb", "feature_groups": ["mirantis"], "release": "8.0", "release_versions": {"2015.1.0-8.0": {"VERSION": {"fuel-createmirror_sha": "fe81731b76071dc0a4ffdf039080732243f5a871", "api": "1.0", "fuelmain_sha": "f5b83e3f0ccfce90b9ca797bc039c9ab50b5ee55", "astute_sha": "a090546d43c770ac27ca81c6f8c78ff0ba4a93e0", "fuel-ostf_sha": "e308a7ad68838f6e012d6d80243eb32b4f7e6c18", "openstack_version": "2015.1.0-8.0", "build_id": "151", "build_number": "151", "fuel-library_sha": "d4b67bca618c48332486339f8342413ea9b4bc3b", "fuel-nailgun-agent_sha": "08e0a11cf1f29b705e4b910d9b9db5e9b708b6e3", "fuel-nailgun_sha": "56a1ca6c1889ae5a8378198db65c0168c8c06484", "fuel-agent_sha": "5d2d5e2e9ad4f8bb01e450240d85447608d84b92", "production": "docker", "python-fuelclient_sha": "655db5e96282bbafc4dae54462a01823b6b464cb", "feature_groups": ["mirantis"], "release": "8.0"}}}

Changed in fuel:
status: Fix Committed → Fix Released
Dmitry Pyzhov (dpyzhov)
tags: added: area-library
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Change abandoned on fuel-library (stable/7.0)

Change abandoned by Stanislaw Bogatkin (<email address hidden>) on branch: stable/7.0
Review: https://review.openstack.org/226653
Reason: Don't apply to 7.0

To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.