Routes for Instance Security Dashboard sub-pages inconsistent & contain typo
Summary
Currently the URL to access the instance-level security dashboard settings page is /-/security/dasboard/settings
.
There are two issues with this URL:
- The typo in the word
dasboard
- The structure of this path is inconsistent with how we've structured the group-level dashboard
/security/dashboard
/security/vulnerabilities
As part of the scope of this issue we should:
-
correct the typo dasboard
->dashboard
✅ !41440 (merged) -
change the route to be consistent with the group-level structure -
add 'dashboard' to route which is currently https://gitlab.com/-/security
Steps to reproduce
- Go to https://gitlab.com/-/security/dasboard/settings
- Observe the URL
Example Project
https://gitlab.com/-/security/dasboard/settings
What is the current bug behavior?
URL to settings page displays as https://gitlab.com/-/security/dasboard/settings
What is the expected correct behavior?
URL to settings page displays with no typeo as https://gitlab.com/-/security/dashboard/settings
[TBD] URL structure aligns with the group-level dashboard URLs
Possible fixes
Implementation plan
-
backend Modify https://gitlab.com/gitlab-org/gitlab/blob/master/ee/config/routes/security.rb to change the behavior: -
root
should redirect tosecurity/dashboard
, -
security/dashboard
should point todashboard#show
-
Edited by Alan (Maciej) Paruszewski