Merge pull request #59842 from nextcloud/fix/swallow-redirected-navigation

fix(files,files_sharing): swallow redirected navigation errors

Author: susnux

apps/files/src/router/router.ts

@@ -38,13 +38,14 @@ Router.prototype.replace = (function(this: Router, ...args: Parameters<typeof or
 }) as typeof originalReplace
 
 /**
- * Ignore duplicated-navigation error but forward real exceptions
+ * Ignore duplicated- and redirected-navigation errors but forward real exceptions
  *
  * @param error The thrown error
  */
 function ignoreDuplicateNavigation(error: unknown): void {
-	if (isNavigationFailure(error, NavigationFailureType.duplicated)) {
-		logger.debug('Ignoring duplicated navigation from vue-router', { error })
+	if (isNavigationFailure(error, NavigationFailureType.duplicated)
+		|| isNavigationFailure(error, NavigationFailureType.redirected)) {
+		logger.debug('Ignoring duplicated/redirected navigation from vue-router', { error })
 	} else {
 		throw error
 	}

apps/files_sharing/src/router/index.ts

@@ -34,13 +34,14 @@ Router.prototype.replace = (function(this: Router, ...args: Parameters<typeof or
 }) as typeof originalReplace
 
 /**
- * Ignore duplicated-navigation error but forward real exceptions
+ * Ignore duplicated- and redirected-navigation errors but forward real exceptions
  *
  * @param error The thrown error
  */
 function ignoreDuplicateNavigation(error: unknown): void {
-	if (isNavigationFailure(error, NavigationFailureType.duplicated)) {
-		logger.debug('Ignoring duplicated navigation from vue-router', { error })
+	if (isNavigationFailure(error, NavigationFailureType.duplicated)
+		|| isNavigationFailure(error, NavigationFailureType.redirected)) {
+		logger.debug('Ignoring duplicated/redirected navigation from vue-router', { error })
 	} else {
 		throw error
 	}