Skip to content

Commit e5ee559

Browse files
authored
Merge pull request #3 from pdsinterop/cleanup/rename-response-methods
Rename response methods
2 parents 6fa3462 + d97255a commit e5ee559

2 files changed

Lines changed: 35 additions & 18 deletions

File tree

src/Server.php

Lines changed: 30 additions & 17 deletions
Original file line numberDiff line numberDiff line change
@@ -46,23 +46,15 @@ final public function respondToAccessTokenRequest(Request $request) : Response
4646
}
4747
}
4848

49-
final public function respondToWellKnownRequest() : Response
50-
{
51-
$response = $this->response;
52-
53-
$serverConfig = $this->config->getServer();
54-
55-
return $this->createJsonResponse($response, $serverConfig);
56-
}
57-
58-
final public function respondToJwksRequest(/*Jwks $jwks*/) : Response
59-
{
60-
$response = $this->response;
61-
$key = $this->config->getKeys()->getPublicKey();
62-
63-
$jwks = new Jwks($key);
64-
65-
return $this->createJsonResponse($response, $jwks);
49+
/**
50+
* @param Request $request
51+
*
52+
* @return Response
53+
*
54+
* @see https://openid.net/specs/openid-connect-registration-1_0.html
55+
*/
56+
final public function respondToDynamicClientRegistrationRequest(Request $request): Response {
57+
return $this->response->withStatus(501);
6658
}
6759

6860
final public function respondToAuthorizationRequest(
@@ -117,6 +109,27 @@ final public function respondToAuthorizationRequest(
117109
return $response;
118110
}
119111

112+
final public function respondToJwksMetadataRequest(/*Jwks $jwks*/) : Response
113+
{
114+
$response = $this->response;
115+
$key = $this->config->getKeys()->getPublicKey();
116+
117+
$jwks = new Jwks($key);
118+
119+
return $this->createJsonResponse($response, $jwks);
120+
}
121+
122+
final public function respondToOpenIdMetadataRequest() : Response
123+
{
124+
$response = $this->response;
125+
126+
$serverConfig = $this->config->getServer();
127+
128+
return $this->createJsonResponse($response, $serverConfig);
129+
}
130+
131+
final public function respondTo_Request(): Response {}
132+
120133
////////////////////////////// UTILITY METHODS \\\\\\\\\\\\\\\\\\\\\\\\\\\\\
121134

122135
private function createOauthServerExceptionResponse(Response $response, OAuthServerException $serverException): Response

tests/example.php

Lines changed: 5 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -89,9 +89,13 @@
8989
// Handle requests
9090
// -----------------------------------------------------------------------------
9191
switch ($request->getMethod() . $request->getRequestTarget()) {
92+
case 'GET/.well-known/jwks.json':
93+
$response = $server->respondToJwksMetadataRequest();
94+
break;
95+
9296
// @CHECKME: Do we also need 'GET/.well-known/oauth-authorization-server'?
9397
case 'GET/.well-known/openid-configuration':
94-
$response = $server->respondToWellKnownRequest();
98+
$response = $server->respondToOpenIdMetadataRequest();
9599
break;
96100

97101
case 'POST/access_token':

0 commit comments

Comments
 (0)