change sso authentication success route to front page

[CocoPoops]

Changes the page which a sso user is redirected to when logged in to the front page so that it is the same as non sso users

[ghost]

Good catch! Maybe we should make each of them do what the KbinAuthenticator.php onAuthenticationSuccess() method does for consistency?

mbin/src/Security/KbinAuthenticator.php

Lines 49 to 56 in 635c58a

	public function onAuthenticationSuccess(Request $request, TokenInterface $token, string $firewallName): ?Response
	{
	if ($targetPath = $this->getTargetPath($request->getSession(), $firewallName)) {
	return new RedirectResponse($targetPath);
	} else {
	return new RedirectResponse($this->urlGenerator->generate('front'));
	}
	}

Honestly I'm not sure why it was different to begin with... maybe an oversight as the parameter list appears to be the same.

[CocoPoops]

Good catch! Maybe we should make each of them do what the KbinAuthenticator.php onAuthenticationSuccess() method does for consistency?

mbin/src/Security/KbinAuthenticator.php

Lines 49 to 56 in 635c58a

	public function onAuthenticationSuccess(Request $request, TokenInterface $token, string $firewallName): ?Response
	{
	if ($targetPath = $this->getTargetPath($request->getSession(), $firewallName)) {
	return new RedirectResponse($targetPath);
	} else {
	return new RedirectResponse($this->urlGenerator->generate('front'));
	}
	}

Honestly I'm not sure why it was different to begin with... maybe an oversight as the parameter list appears to be the same.

i get a 500 error when logging with the function the way you suggested
i don't really know enough to know why that's the case

[ghost]

Good catch! Maybe we should make each of them do what the KbinAuthenticator.php onAuthenticationSuccess() method does for consistency?

mbin/src/Security/KbinAuthenticator.php

Lines 49 to 56 in 635c58a

	public function onAuthenticationSuccess(Request $request, TokenInterface $token, string $firewallName): ?Response
	{
	if ($targetPath = $this->getTargetPath($request->getSession(), $firewallName)) {
	return new RedirectResponse($targetPath);
	} else {
	return new RedirectResponse($this->urlGenerator->generate('front'));
	}
	}

Honestly I'm not sure why it was different to begin with... maybe an oversight as the parameter list appears to be the same.

i get a 500 error when logging with the function the way you suggested i don't really know enough to know why that's the case

lol, that probably explains why it's different then.