-
Notifications
You must be signed in to change notification settings - Fork 392
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
CB-4039 logout with context #2296
Conversation
# Same as #authLogout, but returns additional information | ||
@since(version: "23.3.3") | ||
authLogoutExtended(provider: ID, configuration: ID): LogoutInfo! |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
it's not clear what Extended
suffix means especially if we will remove authLogout
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Alexander decided authLogout
function to be for a while for back backward compatibility. so this is why we have new function with Extended
suffix
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
problem is that Extended
means nothing this is not good suffix for api
authLogoutInfo
sound better
// TODO handle all redirect links once we know what to do with multiple popups issue | ||
const { | ||
authLogoutExtended: { redirectLinks }, | ||
} = await this.userInfoResource.logout(providerId, configurationId); | ||
|
||
if (redirectLinks.length) { | ||
const oktaLink = redirectLinks[0]; | ||
const id = `okta-logout-id-${uuid()}`; | ||
|
||
this.windowsService.open(id, { | ||
url: oktaLink, | ||
width: 400, | ||
height: 400, | ||
}); | ||
} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
this logic must replace
if (userAuthConfiguration?.configuration.signOutLink) {
this.logoutConfiguration(userAuthConfiguration);
}
we need to rewrite or remove it or change and maybe to change logoutConfiguration
No description provided.