Skip to content

Add shared Quarkus server security provider contracts - #67

Merged
kdubb merged 2 commits into
mainfrom
codex/issue-255-quarkus-security
Oct 7, 2026
Merged

kdubb merged 2 commits into
mainfrom
codex/issue-255-quarkus-security

Conversation

@kdubb

@kdubb kdubb commented Oct 7, 2026

Copy link
Copy Markdown
Contributor

Generated Quarkus server packages currently expose separate nested security types, forcing applications to repeat a CDI producer for each package. Add a runtime-owned ServerSecurityProvider SPI and ServerSecuritySubjectSelector so a shared integration library can validate credentials, provide declared permissions, and select subjects without importing generated service classes.

The SPI retains native Quarkus request and identity types; generated glue adapts contract metadata at the package boundary. Issuer/audience trust and credentials remain application configuration.

Validation: :sunday-jaxrs-quarkus:check passed. The companion generator changes compile every supported frontend and boot two independently packaged contracts against one shared provider, including signed JWT rejection and an explicit producer override.

Companion to outfoxx/sunday-generator#255. Merge and publish this runtime before pinning/releasing the generator support.

Expose runtime-owned credential validation, scheme metadata, permissions, and subject selection APIs for independently generated service packages. Export the Quarkus identity and request-context dependencies used by the public SPI.
@greptile-apps

greptile-apps Bot commented Oct 7, 2026 •

Copy link
Copy Markdown

RetriggerConfidence Score: 5/5

[High risk] Adds security framework contracts to the server library.

The PR appears safe to merge; no actionable defects were found in the new contracts or dependency declarations.

Summary

Adds shared Quarkus server security contracts so application integrations can avoid depending on generated service classes.

  • ServerSecurityProvider defines credential checks, scheme metadata, and optional permissions.
  • ServerSecuritySubjectSelector chooses the subject鈥檚 scheme for an AND requirement.
  • Adds public Quarkus dependencies for the native request and identity types.
  • No actionable issues were found.

Reviews (1) 路 Last reviewed commit: "Add shared Quarkus server security provi..." 路 Reviewed by Greptile

Exercise native request and metadata forwarding, trusted identity and permissions, absent credentials, optional permissions, and validation failure propagation. Cover the SPI in the runtime repository so its SonarCloud new-code coverage includes these contracts.
@sonarqubecloud

sonarqubecloud Bot commented Oct 7, 2026

Copy link
Copy Markdown

@kdubb
kdubb merged commit 4aa6ae8 into main Oct 7, 2026
2 checks passed
@kdubb
kdubb deleted the codex/issue-255-quarkus-security branch October 7, 2026 13:56
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