[edge] MAAS sends to network discovery on login

Bug #1939032 reported by Stéphane Graber
8
This bug affects 1 person
Affects Status Importance Assigned to Milestone
MAAS
Fix Released
High
Caleb Ellis
maas-ui
Fix Released
Unknown

Bug Description

Recently on latest/edge, MAAS has started sending everyone to /MAAS/r/dashboard after login.
This seems reasonable in theory except that instead of being some kind of dashboard primarily showing the machine list as one would expect, it's instead showing the network discovery page.

This is made a bit worse because in my setup, network discovery is disabled, so that results in everyone logging into my MAAS getting an error/warning about network discovery being disabled and then having to figure out that they need to hit the machine tab to get back to what they want :)

Tags: ui
Revision history for this message
Stéphane Graber (stgraber) wrote :
tags: added: ui
Revision history for this message
Caleb Ellis (caleb-ellis) wrote :

We recently made some changes to the intro pages which redirects to the "dashboard" on completion. Are you able to check if the browser first redirects to "/MAAS/r/intro" before redirecting then to "/MAAS/r/dashboard"? Also are you using external authentication?

I believe the same bug is what's causing https://bugs.launchpad.net/maas/+bug/1939033

Revision history for this message
Stéphane Graber (stgraber) wrote :

I'm using RBAC authentication indeed.

The URL is being rewritten a bunch of times and the network log isn't helping a whole lot.
I may have blinked and missed one but it looks like it's doing:
 - /maas
 - /MAAS/r
 - /MAAS/r/machines
 - /MAAS/r/intro/user
 - /MAAS/r/dashboard

Revision history for this message
Bill Wear (billwear) wrote :

Without RBAC authentication, i'm not able to reproduce this problem. i'll let Caleb take this one.

Revision history for this message
Caleb Ellis (caleb-ellis) wrote :

It looks like this has been inadvertently fixed as of this commit:
https://git.launchpad.net/maas/commit/?id=4b8977764d95f286867a7539aeb1ac437da5169a

A separate bug regarding setting cookies was fixed, specifically the line referenced here: https://github.com/canonical-web-and-design/maas-ui/pull/2944/files#diff-d89226ec31d158da289f4b79e877ca3b12b70d993555b6bf4fec186446181e61R99

At least in my own testing, the issue is present before that commit and not present after that commit, so I will assume that nothing more needs to be done. Please re-open if you're still running into this issue on the edge snap. Thanks!

Changed in maas:
status: New → Fix Committed
milestone: none → 3.0.1
assignee: nobody → Caleb Ellis (caleb-ellis)
importance: Undecided → High
Changed in maas:
milestone: 3.0.1 → none
Changed in maas-ui:
importance: Undecided → Unknown
status: New → Fix Released
Changed in maas:
milestone: none → 3.1.0
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.