ExternalOauthAuthenticator::getUser() - Code Metrics - Inspection of "OAuth Server (#925)" - superdesk/web-publisher - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Push — master ( 65f314...a56b5d )

by Paweł

created 2019-07-30 12:00 UTC

ExternalOauthAuthenticator::getUser() B

↳ Parent: ExternalOauthAuthenticator

Complexity

Conditions	5
Paths	5

Size

Total Lines

Duplication

Lines	0
Ratio	0 %

Importance

Changes

Metric	Value
dl	0
loc	44
rs	8.9048
c	0
b	0
f	0
cc	5
nc	5
nop	2

<?php

declare(strict_types=1);

namespace SWP\Bundle\CoreBundle\Security\Authenticator;

use Symfony\Component\Security\Core\User\UserInterface;
use KnpU\OAuth2ClientBundle\Security\Authenticator\SocialAuthenticator;
use KnpU\OAuth2ClientBundle\Client\ClientRegistry;
use KnpU\OAuth2ClientBundle\Client\OAuth2Client;
use Symfony\Component\HttpFoundation\Request;
use Symfony\Component\HttpFoundation\Response;
use Symfony\Component\HttpFoundation\RedirectResponse;
use Symfony\Component\Security\Core\Authentication\Token\TokenInterface;
use Symfony\Component\Security\Core\Exception\AuthenticationException;
use Symfony\Component\Security\Core\User\UserProviderInterface;
use FOS\UserBundle\Model\UserManagerInterface;
use Symfony\Component\Security\Core\Security;
use League\OAuth2\Client\Token\AccessToken;
use function uniqid;

class ExternalOauthAuthenticator extends SocialAuthenticator
{
    protected $clientRegistry;

    protected $um;

    protected $security;

    public function __construct(
        ClientRegistry $clientRegistry,
        UserManagerInterface $um,
        Security $security
    ) {
        $this->clientRegistry = $clientRegistry;
        $this->um = $um;
        $this->security = $security;
    }

    public function supports(Request $request): bool
    {
        if (!$this->security->getUser() && ($request->query->get('code') && $request->get('state'))) {
            return true;
        }

        return false;
    }

    public function getCredentials(Request $request): AccessToken
    {
        return $this->fetchAccessToken($this->getOauthClient());
    }

    /**
     * Get the user given the access token. If the user exists as a local user,
     * fetch that one, if it does not, create a new user using the OAuth user
     * fetched from the resource server using the access token.
     *
     * @inehritdoc
     */
    public function getUser($credentials, UserProviderInterface $userProvider): UserInterface
    {
        // Fetch the user from the resource server
        $oauthUser = $this->getOauthClient()->fetchUserFromToken($credentials);
        $oauthEmail = $oauthUser->getEmail();
        $oauthId = $oauthUser->getId();

        if (!$oauthUser) {
            return null;
        }

        // Is there an existing user with the same oauth id?
        /** @var \SWP\Bundle\CoreBundle\Model\UserInterface $user */
        $user = $userProvider->findOneByExternalId($oauthId);
interface User
{
    /** @return string */
    public function getPassword();
}

class MyUser implements User
{
    public function getPassword()
    {
        // return something
    }

    public function getDisplayName()
    {
        // return some name.
    }
}

class AuthSystem
{
    public function authenticate(User $user)
    {
        $this->logger->info(sprintf('Authenticating %s.', $user->getDisplayName()));
        // do something.
    }
}
        if ($user) {
            if ($user->getEmail() !== $oauthEmail) {
                // If the email has changed for the user, update it here as well
                $user->setEmail($oauthEmail);
                $user->setUsername($oauthEmail);
                $this->um->updateUser($user);
            }

            return $user;
        }

        // Is there an existing user with the same email address?
        $user = $userProvider->findOneByEmail($oauthEmail);
interface User
{
    /** @return string */
    public function getPassword();
}

class MyUser implements User
{
    public function getPassword()
    {
        // return something
    }

    public function getDisplayName()
    {
        // return some name.
    }
}

class AuthSystem
{
    public function authenticate(User $user)
    {
        $this->logger->info(sprintf('Authenticating %s.', $user->getDisplayName()));
        // do something.
    }
}
        if ($user) {
            return $user;
        }

        // No user found, create one using the user info provided by resource server
        $user = $this->um->createUser();
        $user->setEmail($oauthEmail);
        $user->setUsername($oauthEmail);
        $user->setExternalId($oauthId);
        $user->setPassword(uniqid('', true));
        $user->setEnabled(true);
        $user->setSuperAdmin(false);

        $this->um->updateUser($user);

        return $user;
    }

    public function onAuthenticationSuccess(Request $request, TokenInterface $token, $providerKey): void
    {
    }

    public function onAuthenticationFailure(Request $request, AuthenticationException $exception): void
    {
    }

    public function start(Request $request, AuthenticationException $authException = null): RedirectResponse
    {
        return new RedirectResponse(
            '/connect/oauth/',
            Response::HTTP_TEMPORARY_REDIRECT
        );
    }

    private function getOauthClient(): OAuth2Client
    {
        return $this->clientRegistry->getClient('external_oauth');
    }
}


1			<?php
2
3			declare(strict_types=1);
4
5			namespace SWP\Bundle\CoreBundle\Security\Authenticator;
6
7			use Symfony\Component\Security\Core\User\UserInterface;
8			use KnpU\OAuth2ClientBundle\Security\Authenticator\SocialAuthenticator;
9			use KnpU\OAuth2ClientBundle\Client\ClientRegistry;
10			use KnpU\OAuth2ClientBundle\Client\OAuth2Client;
11			use Symfony\Component\HttpFoundation\Request;
12			use Symfony\Component\HttpFoundation\Response;
13			use Symfony\Component\HttpFoundation\RedirectResponse;
14			use Symfony\Component\Security\Core\Authentication\Token\TokenInterface;
15			use Symfony\Component\Security\Core\Exception\AuthenticationException;
16			use Symfony\Component\Security\Core\User\UserProviderInterface;
17			use FOS\UserBundle\Model\UserManagerInterface;
18			use Symfony\Component\Security\Core\Security;
19			use League\OAuth2\Client\Token\AccessToken;
20			use function uniqid;
21
22			class ExternalOauthAuthenticator extends SocialAuthenticator
23			{
24			protected $clientRegistry;
25
26			protected $um;
27
28			protected $security;
29
30			public function __construct(
31			ClientRegistry $clientRegistry,
32			UserManagerInterface $um,
33			Security $security
34			) {
35			$this->clientRegistry = $clientRegistry;
36			$this->um = $um;
37			$this->security = $security;
38			}
39
40			public function supports(Request $request): bool
41			{
42			if (!$this->security->getUser() && ($request->query->get('code') && $request->get('state'))) {
43			return true;
44			}
45
46			return false;
47			}
48
49			public function getCredentials(Request $request): AccessToken
50			{
51			return $this->fetchAccessToken($this->getOauthClient());
52			}
53
54			/**
55			* Get the user given the access token. If the user exists as a local user,
56			* fetch that one, if it does not, create a new user using the OAuth user
57			* fetched from the resource server using the access token.
58			*
59			* @inehritdoc
60			*/
61			public function getUser($credentials, UserProviderInterface $userProvider): UserInterface
62			{
63			// Fetch the user from the resource server
64			$oauthUser = $this->getOauthClient()->fetchUserFromToken($credentials);
65			$oauthEmail = $oauthUser->getEmail();
66			$oauthId = $oauthUser->getId();
67
68			if (!$oauthUser) {
69			return null;
70			}
71
72			// Is there an existing user with the same oauth id?
73			/** @var \SWP\Bundle\CoreBundle\Model\UserInterface $user */
74			$user = $userProvider->findOneByExternalId($oauthId);
			0 ignored issues – show Bug introduced 2019-07-11 07:08 UTC by Report Bug Copy Issue Report It seems like you code against a concrete implementation and not the interface `Symfony\Component\Securi...r\UserProviderInterface` as the method `findOneByExternalId()` does only exist in the following implementations of said interface: `SWP\Bundle\CoreBundle\Se...y\Provider\UserProvider`. Let’s take a look at an example: interface User { /** @return string / public function getPassword(); } class MyUser implements User { public function getPassword() { // return something } public function getDisplayName() { // return some name. } } class AuthSystem { public function authenticate(User $user) { $this->logger->info(sprintf('Authenticating %s.', $user->getDisplayName())); // do something. } } In the above example, the authenticate() method works fine as long as you just pass instances of MyUser. However, if you now also want to pass a different implementation of User which does not have a getDisplayName() method, the code will break. Available Fixes Change the type-hint for the parameter: class AuthSystem { public function authenticate(MyUser $user) { / ... / } } Add an additional type-check: class AuthSystem { public function authenticate(User $user) { if ($user instanceof MyUser) { $this->logger->info(/* ... /); } // or alternatively if ( ! $user instanceof MyUser) { throw new \LogicException( '$user must be an instance of MyUser, ' .'other instances are not supported.' ); } } } Note:* PHP Analyzer uses reverse abstract interpretation to narrow down the types inside the if block in such a case. Add the method to the interface: interface User { /** @return string / public function getPassword(); /* @return string */ public function getDisplayName(); } Loading history...
75			if ($user) {
76			if ($user->getEmail() !== $oauthEmail) {
77			// If the email has changed for the user, update it here as well
78			$user->setEmail($oauthEmail);
79			$user->setUsername($oauthEmail);
80			$this->um->updateUser($user);
81			}
82
83			return $user;
84			}
85
86			// Is there an existing user with the same email address?
87			$user = $userProvider->findOneByEmail($oauthEmail);
			0 ignored issues – show Bug introduced 2019-07-11 07:08 UTC by Report Bug Copy Issue Report It seems like you code against a concrete implementation and not the interface `Symfony\Component\Securi...r\UserProviderInterface` as the method `findOneByEmail()` does only exist in the following implementations of said interface: `SWP\Bundle\CoreBundle\Se...y\Provider\UserProvider`. Let’s take a look at an example: interface User { /** @return string / public function getPassword(); } class MyUser implements User { public function getPassword() { // return something } public function getDisplayName() { // return some name. } } class AuthSystem { public function authenticate(User $user) { $this->logger->info(sprintf('Authenticating %s.', $user->getDisplayName())); // do something. } } In the above example, the authenticate() method works fine as long as you just pass instances of MyUser. However, if you now also want to pass a different implementation of User which does not have a getDisplayName() method, the code will break. Available Fixes Change the type-hint for the parameter: class AuthSystem { public function authenticate(MyUser $user) { / ... / } } Add an additional type-check: class AuthSystem { public function authenticate(User $user) { if ($user instanceof MyUser) { $this->logger->info(/* ... /); } // or alternatively if ( ! $user instanceof MyUser) { throw new \LogicException( '$user must be an instance of MyUser, ' .'other instances are not supported.' ); } } } Note:* PHP Analyzer uses reverse abstract interpretation to narrow down the types inside the if block in such a case. Add the method to the interface: interface User { /** @return string / public function getPassword(); /* @return string */ public function getDisplayName(); } Loading history...
88			if ($user) {
89			return $user;
90			}
91
92			// No user found, create one using the user info provided by resource server
93			$user = $this->um->createUser();
94			$user->setEmail($oauthEmail);
95			$user->setUsername($oauthEmail);
96			$user->setExternalId($oauthId);
97			$user->setPassword(uniqid('', true));
98			$user->setEnabled(true);
99			$user->setSuperAdmin(false);
100
101			$this->um->updateUser($user);
102
103			return $user;
104			}
105
106			public function onAuthenticationSuccess(Request $request, TokenInterface $token, $providerKey): void
107			{
108			}
109
110			public function onAuthenticationFailure(Request $request, AuthenticationException $exception): void
111			{
112			}
113
114			public function start(Request $request, AuthenticationException $authException = null): RedirectResponse
115			{
116			return new RedirectResponse(
117			'/connect/oauth/',
118			Response::HTTP_TEMPORARY_REDIRECT
119			);
120			}
121
122			private function getOauthClient(): OAuth2Client
123			{
124			return $this->clientRegistry->getClient('external_oauth');
125			}
126			}
127

superdesk / web-publisher

Push — master ( 65f314...a56b5d )

ExternalOauthAuthenticator::getUser() B

Complexity

Size

Duplication

Importance

Available Fixes

Available Fixes

Duplication Side-by-Side

Filter issues like