I need the hashbang to work pushState = false #30

chdev77 · 2017-08-19T05:12:51Z

About 6 months ago I created a proto of Identity Server 3 with AspNet Identity and an Aurelia app. Also with the suggested pushState = true for the Aurelia router. At the time was told to shelve the proto but, was working great. Fast-forward to 6+ months of Aurelia development and now dusting off the proto for OIDC. We now have many items using the hashbang, google tag manager and a WordPress homepage to name a few. So, everything works until I set pushState = true. I've tried many different configs with the redirect urls without any luck. I do not know how to create a hack to make this work or even where to start.

Any ideas would be very helpful.

chdev77 · 2017-08-22T22:59:12Z

I have the open-id-connect-configuration

export const oidcConfig = {
  loginRedirectModuleId: 'home',
  logoutRedirectModuleId: 'home',
userManagerSettings: {
    post_logout_redirect_uri: `${environment.openidConfig.urls.host}/#/signout-oidc`,
    redirect_uri: `${environment.openidConfig.urls.host}/#/signin-oidc`,
}

app.js
config.options.pushState = false;

I can get the IdSvr login and authenticate but redirecting to the aurelia app I get the following...

nmocruz · 2018-01-09T11:44:33Z

I think that we can't use # simbols on the redirect urls because the identity server is using fragements to send tokens states etc. something ${callbackUrl}#token=(...) and so token will mess with routing match.

I solved this be using a callback.html page that redirects back to '/' after. Not sure if is a good soluction but seems like the aurelia-router is not able to detect redicted to routes because fragments are used to send values in the redirect url.

Maybe instead of register redirect_uri as a aurelia router, it should be handled by a different mechanism just for these urls

pimvd · 2018-01-11T09:14:10Z

Please can you share your callback.html? What do I have to do in callback.html besides the redirect?

nmocruz · 2018-01-11T10:34:33Z

is a regular html. with a body like this <body> <script src="oidc-client.js"></script> <script> new Oidc.UserManager().signinRedirectCallback().then(function () { window.location = "/"; }).catch(function (e) { console.error(e); }); </script> </body> was doing the job a few version ago, not sure about now.

…

pimvd · 2018-01-11T10:46:14Z

Thanks! It works!

koenbeuk · 2018-09-13T06:04:54Z

You can also handle authentication yourself in aurelia's configure method (by default located in main.ts). in one of my projects:

async function configure() {
    .......
    await aurelia.start();

    // this code runs before the aurelia-router kicks in
    if (window.location.hash.indexOf("#id_token") > -1) {
            let openIdConnect = aurelia.container.get(OpenIdConnect) as OpenIdConnect;
            let isSilentLogin = window.top !== window.self;
            if (isSilentLogin) {
                await openIdConnect.userManager.signinSilentCallback();
            } else {
                await openIdConnect.userManager.signinRedirectCallback();
            }
    }

    aurelia.setRoot(PLATFORM.moduleName("app"));
}

arnederuwe · 2019-09-25T11:23:14Z

I'll see if we can get this baked in, or at least documented

chdev77 changed the title ~~I need the hashbang to work pushState = false~~ I need the hashbang to work pushState = true Aug 22, 2017

chdev77 changed the title ~~I need the hashbang to work pushState = true~~ I need the hashbang to work pushState = false Aug 22, 2017

chdev77 mentioned this issue Aug 22, 2017

Aurelia pushState = false IdentityModel/oidc-client-js#407

Closed

shaunluttin added the help wanted label Oct 13, 2017

nmocruz mentioned this issue Jan 9, 2018

Links containing hash anchor tags in URL not supported aurelia/router#225

Open

arnederuwe added the enhancement label Sep 25, 2019

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

I need the hashbang to work pushState = false #30

I need the hashbang to work pushState = false #30

chdev77 commented Aug 19, 2017 •

edited

Loading

chdev77 commented Aug 22, 2017

nmocruz commented Jan 9, 2018 •

edited

Loading

pimvd commented Jan 11, 2018

nmocruz commented Jan 11, 2018 via email •

edited

Loading

pimvd commented Jan 11, 2018

koenbeuk commented Sep 13, 2018

arnederuwe commented Sep 25, 2019

I need the hashbang to work pushState = false #30

I need the hashbang to work pushState = false #30

Comments

chdev77 commented Aug 19, 2017 • edited Loading

chdev77 commented Aug 22, 2017

nmocruz commented Jan 9, 2018 • edited Loading

pimvd commented Jan 11, 2018

nmocruz commented Jan 11, 2018 via email • edited Loading

pimvd commented Jan 11, 2018

koenbeuk commented Sep 13, 2018

arnederuwe commented Sep 25, 2019

chdev77 commented Aug 19, 2017 •

edited

Loading

nmocruz commented Jan 9, 2018 •

edited

Loading

nmocruz commented Jan 11, 2018 via email •

edited

Loading