Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fixes #5194: Properly loads Bastion UI after session timeout and subsequ... #10

Closed
wants to merge 1 commit into from

Conversation

ehelms
Copy link
Owner

@ehelms ehelms commented Jul 9, 2014

...ent redirect.

Bastion pages are driven off routes attached to the HTML anchor. The
anchor is not passed down to the web server, thus whenever a session
timeout occurs the user is redirected to the proper page but lacks
the anchor which tells our angular routing what to load. Since all of our
pages include the base route, this adds a check if the hash-path is empty
to inspect the full url, extract the current entity page and add that
to the hash-path. This assumes that the structure of our pages is
'/katello/entity#/entity'.

…nd subsequent redirect.

Bastion pages are driven off routes attached to the HTML anchor. The
anchor is not passed down to the web server, thus whenever a session
timeout occurs the user is redirected to the proper page but lacks
the anchor which tells our angular routing what to load. Since all of our
pages include the base route, this adds a check if the hash-path is empty
to inspect the full url, extract the current entity page and add that
to the hash-path. This assumes that the structure of our pages is
'/katello/entity#/entity'.
@ehelms ehelms closed this Jul 9, 2014
@ehelms ehelms reopened this Jul 9, 2014
@ehelms ehelms closed this Jul 9, 2014
@ehelms ehelms reopened this Jul 9, 2014
@ehelms ehelms closed this Jul 9, 2014
@ehelms ehelms reopened this Jul 9, 2014
@ehelms ehelms closed this Jul 9, 2014
ehelms pushed a commit that referenced this pull request Dec 18, 2015
Refs Katello#10970 - Remove deprecation warnings from test runs
ehelms added a commit that referenced this pull request Aug 25, 2017
ehelms added a commit that referenced this pull request Aug 25, 2017
Fixes #10: Generate issue and release cache
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant