Fuel redirect to wrong HTTPS port: 8443

Bug #1545014 reported by Thomas Goirand
8
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Fuel for OpenStack
Confirmed
Medium
Vitaly Kramskikh
Mitaka
Won't Fix
Medium
Vitaly Kramskikh
Newton
Confirmed
Medium
Fuel UI Team

Bug Description

After installing Fuel master node on Debian (yes, you're reading well ... :) no, it's not uploaded to Debian ... YET !), I have found that the web interface for Fuel, at the login prompt, has a link to HTTPS, though this link redirects to port 443.

not serioius:
What have you guys drink to redirect to this port number? Do you know it's forbidden to drink and type code?
/joke

More seriously, while it may make sense to use such a funny port, probably the default should be 443. I would advise a specific configuration option to use any other port.

Ilya Kutukov (ikutukov)
Changed in fuel:
assignee: nobody → Fuel Python Team (fuel-python)
milestone: none → 9.0
importance: Undecided → High
status: New → Confirmed
tags: added: area-python
tags: added: debian
tags: added: area-debian
tags: added: area-ui
Revision history for this message
Maciej Kwiek (maciej-iai) wrote :

If this bug is indeed a bug, it needs changes in both Library and UI, probably in files
In library port number is configured in deployment/puppet/nailgun/templates/nginx_nailgun.conf.erb
In UI Port is set in nailgun/static/views/login_page.js#L121

Changed in fuel:
status: Confirmed → Triaged
assignee: Fuel Python Team (fuel-python) → Fuel Library Team (fuel-library)
importance: High → Low
Revision history for this message
Maciej Kwiek (maciej-iai) wrote :

Also, changing to Low, it seems that this doesn't really affect anything. It's just non-standard port.

Revision history for this message
Thomas Goirand (thomas-goirand) wrote :

Maciej Kwiek (maciej-iai), please don't turn this into "low", as Debian really *IS* affected. Indeed, fuel gets installed in the normal Apache configuration, where the port 443 is in use, not this non-standard port. The packaging will *NOT* be changed to switch to such a non-standard port. As a consequence, this is just a broken link, which we would like to avoid.

Revision history for this message
Thomas Goirand (thomas-goirand) wrote :

Oh, btw, I'm just interested in the JS file fix, as the Debian package installs the Apache config itself. Thanks for pointing at the correct place to patch, which I have added as a debian/patches file in my package.

Revision history for this message
Dmitry Pyzhov (dpyzhov) wrote :

It doesn't look like an issue from the user point of view. So I'm marking it as a tech-debt. However I think this is a quite important tech-debt.

Changed in fuel:
importance: Low → Medium
tags: added: tech-debt
Changed in fuel:
assignee: Fuel Library Team (fuel-library) → Maksim Malchuk (mmalchuk)
Revision history for this message
Thomas Goirand (thomas-goirand) wrote :

Is there a patch available? Has it been merged yet?

Revision history for this message
Maksim Malchuk (mmalchuk) wrote :

Not yet. As You can see the status is Triaged, not In Progress.

Revision history for this message
Vitaly Kramskikh (vkramskikh) wrote :

I'm still unsure how this should be fixed. Currently UI doesn't have its own config, it only relies on nailgun and the only "configurable" value which is used by UI - is "auth enabled" flag which is obtained via /api/version output. The approach should be discussed, so moving the status back to Confirmed

Changed in fuel:
status: Triaged → Confirmed
Changed in fuel:
assignee: Maksim Malchuk (mmalchuk) → Vitaly Kramskikh (vkramskikh)
tags: added: tricky
no longer affects: fuel/newton
Changed in fuel:
milestone: 10.0 → 10.1
Changed in fuel:
milestone: 10.1 → 11.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.