Issues (72)

src/Roles/ClientInterface.php (2 issues)

1
<?php
2
/**
3
 * Created by PhpStorm.
4
 * User: Alexandre
5
 * Date: 06/03/2018
6
 * Time: 21:30
7
 */
8
9
namespace OAuth2\Roles;
10
11
12
use OAuth2\Roles\ClientTypes\ClientMetadataInterface;
13
14
/**
15
 * Interface ClientInterface
16
 * @package OAuth2\Roles
17
 *
18
 * @see https://tools.ietf.org/html/rfc6749#section-1.1
19
 * An application making protected resource requests on behalf of the
20
 * resource owner and with its authorization.  The term "client" does
21
 * not imply any particular implementation characteristics (e.g.,
22
 * whether the application executes on a server, a desktop, or other
23
 * devices).
24
 */
25
interface ClientInterface
26
{
27
    /**
28
     * @return bool
29
     *
30
     * @see https://tools.ietf.org/html/rfc6749#section-3.2.1
31
     * Confidential clients or other clients issued client credentials MUST
32
     * authenticate with the authorization server as described in
33
     * Section 2.3 when making requests to the token endpoint.
34
     */
35
    public function hasCredentials(): bool;
36
37
    //public function isHttpBasicAuthenticationSchemeSupported(): bool;
0 ignored issues
show
Unused Code Comprehensibility introduced by
64% of this comment could be valid code. Did you maybe forget this after debugging?

Sometimes obsolete code just ends up commented out instead of removed. In this case it is better to remove the code once you have checked you do not need it.

The code might also have been commented out for debugging purposes. In this case it is vital that someone uncomments it again or your project may behave in very unexpected ways in production.

This check looks for comments that seem to be mostly valid code and reports them.

Loading history...
38
39
    //public function isTLSSupported(): bool;
0 ignored issues
show
Unused Code Comprehensibility introduced by
64% of this comment could be valid code. Did you maybe forget this after debugging?

Sometimes obsolete code just ends up commented out instead of removed. In this case it is better to remove the code once you have checked you do not need it.

The code might also have been commented out for debugging purposes. In this case it is vital that someone uncomments it again or your project may behave in very unexpected ways in production.

This check looks for comments that seem to be mostly valid code and reports them.

Loading history...
40
41
    public function getMetadata(): ClientMetadataInterface;
42
}