Skip to content

feat(introspection): return structured failure details - #8

Merged
shreemaan-abhishek merged 1 commit into
release/1.8from
feat/introspection-failure-details-1.8
Sep 10, 2026
Merged

feat(introspection): return structured failure details#8
shreemaan-abhishek merged 1 commit into
release/1.8from
feat/introspection-failure-details-1.8

Conversation

@shreemaan-abhishek

@shreemaan-abhishek shreemaan-abhishek commented Sep 10, 2026

Copy link
Copy Markdown

Summary

Returns additive structured failure metadata as the fourth result from introspect, while preserving the existing result, error, and endpoint-status values. The metadata distinguishes invalid requests and tokens from endpoint, response, coordination, and configuration failures and includes a recommended downstream HTTP status.

Strictly validates the required boolean active claim and carries failure metadata through single-flight result sharing.

This is stacked on #7 and should be merged after it.

Checklist

  • Tests pass with docker build -f tests/Dockerfile . -t lua-resty-openidc/test
  • Tests pass with docker run -t --rm lua-resty-openidc/test:latest
  • ChangeLog is updated for behavior changes, bug fixes, features, or user-visible documentation changes

All 79 focused introspection tests pass. Both complete CI runs pass after rerunning one unrelated transient client_secret_jwt test failure.

@coderabbitai

coderabbitai Bot commented Sep 10, 2026

Copy link
Copy Markdown

Important

Review skipped

Auto reviews are disabled on base/target branches other than the default branch.

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Essentials

Run ID: b1862e9d-6788-416d-8d79-b59d68cd854e

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • 🔍 Trigger review

Comment @coderabbitai help to get the list of available commands.

@shreemaan-abhishek shreemaan-abhishek mentioned this pull request Sep 10, 2026
3 tasks
@shreemaan-abhishek
shreemaan-abhishek force-pushed the fix/introspection-owner-1.8 branch from fab4be0 to ecf53fc Compare September 10, 2026 07:33
@shreemaan-abhishek
shreemaan-abhishek force-pushed the feat/introspection-failure-details-1.8 branch from 75b9cde to 4df1f08 Compare September 10, 2026 07:33
@shreemaan-abhishek
shreemaan-abhishek changed the base branch from fix/introspection-owner-1.8 to release/1.8 September 10, 2026 08:46
@shreemaan-abhishek
shreemaan-abhishek merged commit cabeb20 into release/1.8 Sep 10, 2026
4 of 5 checks passed
@shreemaan-abhishek shreemaan-abhishek self-assigned this Sep 10, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant