Commit 688a427
committed
docs(auth): document JWT-trust mode and complete validation gate
Extra scope beyond the docs sweep:
- Error-path wiring for the B.2 matrix disabled rows that B.3/B.12 did not
cover: password login/register/reset (401), SSO browser login (401, at the
browser callback; the cited service function stays live for the
default-funnel provisioning path), session-token refresh (401), user
management in the admin UI and admin API (403), invitations and team
membership writes for trust-only principals (403 via the new
LocalUserRecordRequiredError). tests/unit/mcpgateway/
test_trust_mode_disabled_surfaces.py covers every disabled matrix row and
asserts status plus message.
- Test-isolation fixes exposed by the two-mode gate: a conftest autouse
fixture clears the correlation-id contextvar per test (a sync test leaked
it into the worker root context and broke an A.9 audit lookup), a conftest
autouse fixture pins jwt_trust_mode=db per test so the suite is hermetic
when JWT_TRUST_MODE=jwt-trust is exported, test_jwt_trust_config
test_defaults now removes the ambient JWT_TRUST_MODE it claimed to
isolate, and the auth_cache key doctest asserts the mode segment against
settings instead of a hard-coded db.
- Pylint false-positive fix (pre-existing, gate-blocking): inline
not-callable disables on the two SQLAlchemy func.now() server defaults in
the external_group_mappings model.
- .secrets.baseline regenerated by make detect-secrets-scan (line drift
from the doc edits).
Signed-off-by: Jonathan Springer <jps@s390x.com>1 parent d6baa29 commit 688a427
19 files changed
Lines changed: 594 additions & 16 deletions
File tree
- docs
- docs
- architecture
- manage
- plans
- mcpgateway
- cache
- routers
- services
- tests
- unit/mcpgateway
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
168 | 168 | | |
169 | 169 | | |
170 | 170 | | |
171 | | - | |
| 171 | + | |
172 | 172 | | |
173 | 173 | | |
174 | 174 | | |
| |||
191 | 191 | | |
192 | 192 | | |
193 | 193 | | |
194 | | - | |
| 194 | + | |
195 | 195 | | |
196 | 196 | | |
197 | 197 | | |
| |||
203 | 203 | | |
204 | 204 | | |
205 | 205 | | |
| 206 | + | |
| 207 | + | |
| 208 | + | |
| 209 | + | |
206 | 210 | | |
207 | 211 | | |
208 | 212 | | |
| |||
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
97 | 97 | | |
98 | 98 | | |
99 | 99 | | |
| 100 | + | |
| 101 | + | |
| 102 | + | |
| 103 | + | |
100 | 104 | | |
101 | 105 | | |
102 | 106 | | |
| |||
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
328 | 328 | | |
329 | 329 | | |
330 | 330 | | |
| 331 | + | |
| 332 | + | |
| 333 | + | |
| 334 | + | |
| 335 | + | |
| 336 | + | |
| 337 | + | |
| 338 | + | |
| 339 | + | |
| 340 | + | |
| 341 | + | |
| 342 | + | |
| 343 | + | |
| 344 | + | |
| 345 | + | |
| 346 | + | |
| 347 | + | |
| 348 | + | |
| 349 | + | |
| 350 | + | |
| 351 | + | |
| 352 | + | |
| 353 | + | |
| 354 | + | |
| 355 | + | |
331 | 356 | | |
332 | 357 | | |
333 | 358 | | |
| |||
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
223 | 223 | | |
224 | 224 | | |
225 | 225 | | |
| 226 | + | |
| 227 | + | |
| 228 | + | |
| 229 | + | |
| 230 | + | |
| 231 | + | |
| 232 | + | |
| 233 | + | |
| 234 | + | |
| 235 | + | |
| 236 | + | |
226 | 237 | | |
227 | 238 | | |
228 | 239 | | |
| |||
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
687 | 687 | | |
688 | 688 | | |
689 | 689 | | |
| 690 | + | |
| 691 | + | |
| 692 | + | |
| 693 | + | |
| 694 | + | |
| 695 | + | |
| 696 | + | |
| 697 | + | |
| 698 | + | |
| 699 | + | |
| 700 | + | |
| 701 | + | |
690 | 702 | | |
691 | 703 | | |
692 | 704 | | |
| |||
0 commit comments