[SSL] User was redirected to http usage after login in horizon

Bug #1479318 reported by Tatyanka
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Fuel for OpenStack
Fix Released
Medium
Stanislaw Bogatkin

Bug Description

Steps:
1. Deploy cluster:
1 controller
1 compute
1 cinder with enabled ssl (public endpoints horizon)

2. As soon as cluster becomes ready, navigate to horizon by https
3. Login as admin

Expected:
Logged in user works over https

Actual:
After login user was redirected to http

[root@nailgun ~]# cat /etc/fuel/version.yaml
VERSION:
  feature_groups:
    - mirantis
  production: "docker"
  release: "7.0"
  openstack_version: "2015.1.0-7.0"
  api: "1.0"
  build_number: "103"
  build_id: "2015-07-28_12-51-16"
  nailgun_sha: "d7fe1047caeb4503970c7d39689e133b28b85b22"
  python-fuelclient_sha: "f04e6c46783ecd6000df31b61b6749da66d4d828"
  fuel-agent_sha: "2a65f11c10b0aeb5184247635a19740fc3edde21"
  fuel-nailgun-agent_sha: "1512b9af6b41cc95c4d891c593aeebe0faca5a63"
  astute_sha: "34e0493afa22999c4a07d3198ceb945116ab7932"
  fuel-library_sha: "8e64ae8e018d5b119c8e2ea49f6a83467b970a66"
  fuel-ostf_sha: "9e32b35e9a774eec41b86cd5a6c63b71a3efa4bd"
  fuelmain_sha: "0d6ff4c9ec156196cc4d9a36db314ba7346942f7"

Revision history for this message
Tatyanka (tatyana-leontovich) wrote :
summary: - [SSL] USer was redirected to http usage after login in horizon
+ [SSL] User was redirected to http usage after login in horizon
Changed in fuel:
status: New → Confirmed
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/207955

Changed in fuel:
status: Confirmed → In Progress
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to fuel-library (master)

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

commit c4a53707ca30cf7728be220bfff0a829cecab2da
Author: Stanislaw Bogatkin <email address hidden>
Date: Fri Jul 31 18:18:57 2015 +0300

    Let horizon understand that it should use HTTPS

    We should provide extra condition in apache virtual host
    to let haproxy understand cases when it should redirect
    to https address instead of plain http.

    Change-Id: I4239ea9c730fa847a1d2369a053e63a3e48df444
    Closes-Bug: #1479318

Changed in fuel:
status: In Progress → Fix Committed
Revision history for this message
Alex Schultz (alex-schultz) wrote :

Reopening as it is still redirecting incorrectly.

    "api": "1.0",
    "astute_sha": "e1d3a435e5df5b40cbfb1a3acf80b4176d15a2dc",
    "auth_required": true,
    "build_id": "2015-08-12_17-24-26",
    "build_number": "165",
    "feature_groups": [
        "mirantis"
    ],
    "fuel-agent_sha": "57145b1d8804389304cd04322ba0fb3dc9d30327",
    "fuel-library_sha": "1176b634eeafb8465a88ff357fdcf40005fba610",
    "fuel-nailgun-agent_sha": "e01693992d7a0304d926b922b43f3b747c35964c",
    "fuel-ostf_sha": "58220583f10fa47f12291488ef77854809c68310",
    "fuelmain_sha": "67e5214c0dc5d4ba6da4ae651cef9934800459a9",
    "nailgun_sha": "68642a8207d6f12543f244bab0c130e2510536ee",
    "openstack_version": "2015.1.0-7.0",
    "production": "docker",
    "python-fuelclient_sha": "26fc025e0fc5791b62e5ed8561a6016bf8a406bc",

Changed in fuel:
status: Fix Committed → Confirmed
importance: Medium → High
Revision history for this message
Alex Schultz (alex-schultz) wrote :

root@node-1:~# curl -k https://10.122.6.3
<!DOCTYPE HTML PUBLIC "-//IETF//DTD HTML 2.0//EN">
<html><head>
<title>301 Moved Permanently</title>
</head><body>
<h1>Moved Permanently</h1>
<p>The document has moved <a href="http://10.122.6.3/horizon">here</a>.</p>
</body></html>

Revision history for this message
Alex Schultz (alex-schultz) wrote :

Actually my issue is a different bug, where a user is redirected to http without logging in. I'll create a new different bug for this issue.

Changed in fuel:
importance: High → Medium
status: Confirmed → Fix Committed
Revision history for this message
Tatyanka (tatyana-leontovich) wrote :

verified 288

Changed in fuel:
status: Fix Committed → Fix Released
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.