Skip to content

fix: exclude overridable annotation values when parsing composed annotations#5179

Open
Mattias-Sehlstedt wants to merge 1 commit into
swagger-api:masterfrom
Mattias-Sehlstedt:exclude-override-meta-annotations
Open

fix: exclude overridable annotation values when parsing composed annotations#5179
Mattias-Sehlstedt wants to merge 1 commit into
swagger-api:masterfrom
Mattias-Sehlstedt:exclude-override-meta-annotations

Conversation

@Mattias-Sehlstedt
Copy link
Copy Markdown
Contributor

Pull Request

Thank you for contributing to swagger-core!

Please fill out the following information to help us review your PR efficiently.


Description

Change the meta-annotation parsing so that annotations that utilize @OverridesAttribute do not take the overridden annotation into consideration. This since the library currently does not support parsing the @OverridesAttribute value definition.

I know that the Hibernate validator most likely has code to manage this, but I do not believe that is a dependency that this project needs or wants as of now. Both swagger-core and springdoc-openapi both have extension possibilities for injecting interpretation logic of annotations, so that is how these annotations can be managed currently.

Fixes: #5178

Type of Change

  • 🐛 Bug fix
  • ✨ New feature
  • ♻️ Refactor (non-breaking change)
  • 🧪 Tests
  • 📝 Documentation
  • 🧹 Chore (build or tooling)

Checklist

  • I have added/updated tests as needed
  • I have added/updated documentation where applicable
  • The PR title is descriptive
  • The code builds and passes tests locally
  • I have linked related issues (if any)

Screenshots / Additional Context

@Mattias-Sehlstedt Mattias-Sehlstedt force-pushed the exclude-override-meta-annotations branch from 91852e9 to ad6b598 Compare May 12, 2026 19:11
@ewaostrowska ewaostrowska added the backlog label May 14, 2026 — with Claude
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Bug]: OverridesAttribute meta annotation takes the default value

2 participants