Alan-Jowett / CoPilot-For-Consensus / 21489082244

78%

Build Type

push

github

Committed by web-flow

Commit Message

Add diagnostic context to JWKS fetch error logs (#1077)

* Initial plan

* Add diagnostic context to JWKS fetch error logs

- Include target URL and timeout in all JWKS fetch error messages
- Helps diagnose network/DNS/firewall issues in production
- Add test to verify diagnostic context is present in logs
- All existing tests pass

Co-authored-by: Alan-Jowett <20480683+Alan-Jowett@users.noreply.github.com>

* Fix linting issues in middleware.py

- Fix line length violations
- Split long docstring lines
- All tests still pass

Co-authored-by: Alan-Jowett <20480683+Alan-Jowett@users.noreply.github.com>

* Address code review feedback for consistency

- Move jwks_url definition outside loop for consistency
- Use jwks_url variable in all error messages
- Add diagnostic context to non-5xx HTTP error message
- All tests pass, linting passes

Co-authored-by: Alan-Jowett <20480683+Alan-Jowett@users.noreply.github.com>

* Complete diagnostic context coverage in all error paths

- Add diagnostic context to fallback error message in _fetch_jwks_with_retry
- Use jwks_url variable consistently in _fetch_jwks error message
- All error paths now include target URL and timeout for diagnosis
- All tests pass, linting passes

Co-authored-by: Alan-Jowett <20480683+Alan-Jowett@users.noreply.github.com>

* Fix UnboundLocalError in _fetch_jwks error handling

Move jwks_url definition before try block to prevent UnboundLocalError
if exception occurs before assignment completes. This matches the pattern
used in _fetch_jwks_with_retry() at line 133.

Fixes code review comment 2742396303

Co-authored-by: Alan-Jowett <20480683+Alan-Jowett@users.noreply.github.com>

---------

Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com>
Co-authored-by: Alan-Jowett <20480683+Alan-Jowett@users.noreply.github.com>

Run Details

5 of 9 new or added lines in 1 file covered. (55.56%)

2 existing lines in 1 file now uncovered.

8707 of 11190 relevant lines covered (77.81%)

0.87 hits per line