Skip to content

Commit

Permalink
ASGARD-1174 - Init screen on Firefox should not do infinite redirects
Browse files Browse the repository at this point in the history
It also should not show the loading screen when the administrator still hasn't configured Asgard yet.

Instead, Firefox should immediately show the message about why Firefox is currently incompatible.
  • Loading branch information
Joe Sondow committed Jun 28, 2013
1 parent b2d0e23 commit e38e958
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 2 deletions.
2 changes: 1 addition & 1 deletion grails-app/conf/InitFilters.groovy
Original file line number Diff line number Diff line change
Expand Up @@ -20,7 +20,7 @@ class InitFilters {
def configService

def filters = {
all(controller: '(init|server)', invert: true) {
all(controller: '(init|server|firefox)', invert: true) {
before = {
if (!configService.appConfigured) {
redirect(controller: 'init')
Expand Down
2 changes: 1 addition & 1 deletion grails-app/conf/LoadingFilters.groovy
Original file line number Diff line number Diff line change
Expand Up @@ -22,7 +22,7 @@ class LoadingFilters {
def initService

def filters = {
all(controller: '(cache|init|healthcheck|server)', invert: true) {
all(controller: '(cache|init|healthcheck|server|firefox)', invert: true) {
before = {
if (!initService.cachesFilled() && !System.getProperty('skipCacheFill')) {
render(status: HttpServletResponse.SC_SERVICE_UNAVAILABLE, view: '/loading')
Expand Down

0 comments on commit e38e958

Please sign in to comment.