Skip to content

Commit

Permalink
fix(core): Do not set static document base URL (#9890)
Browse files Browse the repository at this point in the history
The previous change to enable optional path-based routing (#9802) added a `<base>` tag, which regressed serving hash-based routing at non-root public paths.

This change returns to AngularJS's default behavior when using hash-based routing.  Note that path-based routing will continue to only function when served from a domain root.

Resolves spinnaker/spinnaker#6723

(cherry picked from commit 5ac7516)
  • Loading branch information
jcavanagh authored and mergify[bot] committed Oct 6, 2022
1 parent 8cc8293 commit beb2bf7
Show file tree
Hide file tree
Showing 2 changed files with 1 addition and 1 deletion.
1 change: 0 additions & 1 deletion packages/app/index.deck
Original file line number Diff line number Diff line change
Expand Up @@ -5,7 +5,6 @@
<meta charset="utf-8">
<meta name="description" content="">
<meta name="viewport" content="width=device-width">
<base href="/">

<!-- Sample loading styles -->
<style>
Expand Down
1 change: 1 addition & 0 deletions packages/core/src/bootstrap/angular.config.ts
Original file line number Diff line number Diff line change
Expand Up @@ -38,6 +38,7 @@ bootstrapModule.config([
$locationProvider.html5Mode({
enabled: SETTINGS.feature.html5Routing,
rewriteLinks: false,
requireBase: false,
});
},
]);

0 comments on commit beb2bf7

Please sign in to comment.