search

SUSE / velum

Dashboard for CaaS Platform clusters (v1, v2 and v3)
https://www.suse.com/
Apache License 2.0
54 stars 30 forks source link

prevent accessing discovery after bootstrap #633

Closed MaximilianMeister closed 6 years ago

MaximilianMeister commented 6 years ago

PR #627 introduced a weird behaviour of skip_before_action when 'only' and 'if' is used at the same time

see: https://github.com/rails/rails/issues/9703

we can just omit the 'if setup_done?' check because that's already implicitly happening through the application_controller

Signed-off-by: Maximilian Meister mmeister@suse.de

MaximilianMeister commented 6 years ago

to reproduce the bug, bootstrap a cluster and access /setup/discovery