-
Notifications
You must be signed in to change notification settings - Fork 17
[scenario] authz metadata discovery tests: compatibility with 2025-03-26 spec, prm root, and www-authenticate #28
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Merged
Merged
Changes from 21 commits
Commits
Show all changes
25 commits
Select commit
Hold shift + click to select a range
79cce6f
clean up constructor calls for server lifecycle
pcarleton 6326cc3
add back compat test
pcarleton f39bf10
fix dup logging, rm prm
pcarleton 70fc83e
avoid fallbacks
pcarleton 1fb3933
better error from example client
pcarleton 88de932
add final endpoint method
pcarleton c44a0b9
add test for OASM priority order
pcarleton c999337
s/prepack/prepare/ for npx
pcarleton 200cc57
try switching to .js
pcarleton a64fcc5
make tsdown version happy default
pcarleton 39086d1
fix lock
pcarleton 7bf6912
fix prmPath issue
pcarleton 9191e91
back to main package-lock
pcarleton 3f50f78
fix bin path
pcarleton f6035cd
Add comprehensive auth scenario test suite
pcarleton de0c6b7
consolidate tests into index.test.ts
pcarleton f357e77
rename metadata
pcarleton 2c6efa2
fix comments
pcarleton e9f657b
rework auth scenario listing
pcarleton 3aced3c
add 3rd variation of metadata
pcarleton a1fdeea
fix some type issues introduced by version bump
pcarleton f5e1452
add reusable spec references
pcarleton 7a0f39a
more refs
pcarleton 5c3a5c0
fix up spec references
pcarleton 030e97c
Merge branch 'main' into pcarleton/auth-back-compat
pcarleton File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.
Oops, something went wrong.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -26,7 +26,7 @@ | |
| "dist" | ||
| ], | ||
| "bin": { | ||
| "conformance": "dist/index.mjs" | ||
| "conformance": "dist/index.js" | ||
| }, | ||
| "devDependencies": { | ||
| "@eslint/js": "^9.8.0", | ||
|
|
@@ -43,7 +43,7 @@ | |
| "vitest": "^4.0.5" | ||
| }, | ||
| "dependencies": { | ||
| "@modelcontextprotocol/sdk": "^1.20.1", | ||
| "@modelcontextprotocol/sdk": "^1.22.0", | ||
|
Member
Author
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. this is to pick up the fix that makes these tests pass |
||
| "commander": "^14.0.2", | ||
| "express": "^5.1.0", | ||
| "lefthook": "^2.0.2", | ||
|
|
||
This file was deleted.
Oops, something went wrong.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
tsdown 0.15 creates
index.js, 0.16 createsindex.mjs, so since we're pinning to0.15for now (which seems to also be what pkg.pr.new uses), this needs to be.js