Skip to content

Commit 5a0dbc8

Browse files
committed
refactor: simplify resource URL creation in auth.ts across multiple exercises
1 parent 3fa08d7 commit 5a0dbc8

File tree

21 files changed

+21
-42
lines changed

21 files changed

+21
-42
lines changed

exercises/01.discovery/03.solution.pr/src/auth.ts

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1,8 +1,7 @@
11
import { EPIC_ME_AUTH_SERVER_URL } from './client.ts'
22

33
export async function handleOAuthProtectedResourceRequest(request: Request) {
4-
const resourceServerUrl = new URL(request.url)
5-
resourceServerUrl.pathname = '/mcp'
4+
const resourceServerUrl = new URL('/mcp', request.url)
65

76
return Response.json({
87
resource: resourceServerUrl.toString(),

exercises/02.init/01.problem.authenticate/src/auth.ts

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -9,8 +9,7 @@ import { EPIC_ME_AUTH_SERVER_URL } from './client.ts'
99
* This is how the client knows where to request authorization from.
1010
*/
1111
export async function handleOAuthProtectedResourceRequest(request: Request) {
12-
const resourceServerUrl = new URL(request.url)
13-
resourceServerUrl.pathname = '/mcp'
12+
const resourceServerUrl = new URL('/mcp', request.url)
1413

1514
return Response.json({
1615
resource: resourceServerUrl.toString(),

exercises/02.init/01.solution.authenticate/src/auth.ts

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -14,8 +14,7 @@ export function handleUnauthorized() {
1414
* This is how the client knows where to request authorization from.
1515
*/
1616
export async function handleOAuthProtectedResourceRequest(request: Request) {
17-
const resourceServerUrl = new URL(request.url)
18-
resourceServerUrl.pathname = '/mcp'
17+
const resourceServerUrl = new URL('/mcp', request.url)
1918

2019
return Response.json({
2120
resource: resourceServerUrl.toString(),

exercises/02.init/02.problem.params/src/auth.ts

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -20,8 +20,7 @@ export function handleUnauthorized() {
2020
* This is how the client knows where to request authorization from.
2121
*/
2222
export async function handleOAuthProtectedResourceRequest(request: Request) {
23-
const resourceServerUrl = new URL(request.url)
24-
resourceServerUrl.pathname = '/mcp'
23+
const resourceServerUrl = new URL('/mcp', request.url)
2524

2625
return Response.json({
2726
resource: resourceServerUrl.toString(),

exercises/02.init/02.solution.params/src/auth.ts

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -17,8 +17,7 @@ export function handleUnauthorized(request: Request) {
1717
* This is how the client knows where to request authorization from.
1818
*/
1919
export async function handleOAuthProtectedResourceRequest(request: Request) {
20-
const resourceServerUrl = new URL(request.url)
21-
resourceServerUrl.pathname = '/mcp'
20+
const resourceServerUrl = new URL('/mcp', request.url)
2221

2322
return Response.json({
2423
resource: resourceServerUrl.toString(),

exercises/03.auth-info/01.problem.introspect/src/auth.ts

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -41,8 +41,7 @@ export function handleUnauthorized(request: Request) {
4141
* This is how the client knows where to request authorization from.
4242
*/
4343
export async function handleOAuthProtectedResourceRequest(request: Request) {
44-
const resourceServerUrl = new URL(request.url)
45-
resourceServerUrl.pathname = '/mcp'
44+
const resourceServerUrl = new URL('/mcp', request.url)
4645

4746
return Response.json({
4847
resource: resourceServerUrl.toString(),

exercises/03.auth-info/01.solution.introspect/src/auth.ts

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -58,8 +58,7 @@ export function handleUnauthorized(request: Request) {
5858
* This is how the client knows where to request authorization from.
5959
*/
6060
export async function handleOAuthProtectedResourceRequest(request: Request) {
61-
const resourceServerUrl = new URL(request.url)
62-
resourceServerUrl.pathname = '/mcp'
61+
const resourceServerUrl = new URL('/mcp', request.url)
6362

6463
return Response.json({
6564
resource: resourceServerUrl.toString(),

exercises/03.auth-info/02.problem.error/src/auth.ts

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -64,8 +64,7 @@ export function handleUnauthorized(request: Request) {
6464
* This is how the client knows where to request authorization from.
6565
*/
6666
export async function handleOAuthProtectedResourceRequest(request: Request) {
67-
const resourceServerUrl = new URL(request.url)
68-
resourceServerUrl.pathname = '/mcp'
67+
const resourceServerUrl = new URL('/mcp', request.url)
6968

7069
return Response.json({
7170
resource: resourceServerUrl.toString(),

exercises/03.auth-info/02.solution.error/src/auth.ts

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -69,8 +69,7 @@ export function handleUnauthorized(request: Request) {
6969
* This is how the client knows where to request authorization from.
7070
*/
7171
export async function handleOAuthProtectedResourceRequest(request: Request) {
72-
const resourceServerUrl = new URL(request.url)
73-
resourceServerUrl.pathname = '/mcp'
72+
const resourceServerUrl = new URL('/mcp', request.url)
7473

7574
return Response.json({
7675
resource: resourceServerUrl.toString(),

exercises/03.auth-info/03.problem.active/src/auth.ts

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -72,8 +72,7 @@ export function handleUnauthorized(request: Request) {
7272
* This is how the client knows where to request authorization from.
7373
*/
7474
export async function handleOAuthProtectedResourceRequest(request: Request) {
75-
const resourceServerUrl = new URL(request.url)
76-
resourceServerUrl.pathname = '/mcp'
75+
const resourceServerUrl = new URL('/mcp', request.url)
7776

7877
return Response.json({
7978
resource: resourceServerUrl.toString(),

0 commit comments

Comments
 (0)