Skip to content

upgrade swagger-core from 2.2.53 to 2.2.54 - #3351

Merged
bnasslahsen merged 1 commit into
springdoc:mainfrom
Mattias-Sehlstedt:update-swagger-core
Sep 6, 2026
Merged

upgrade swagger-core from 2.2.53 to 2.2.54#3351
bnasslahsen merged 1 commit into
springdoc:mainfrom
Mattias-Sehlstedt:update-swagger-core

Conversation

@Mattias-Sehlstedt

@Mattias-Sehlstedt Mattias-Sehlstedt commented Sep 6, 2026

Copy link
Copy Markdown
Contributor

Swagger-core has implemented better handling for dates in swagger-api/swagger-core#5184. This means that some parts of ExtraSchemas from ec43ffb can be removed, since they are now handled upstream.

@Mattias-Sehlstedt
Mattias-Sehlstedt marked this pull request as draft September 6, 2026 10:53
@Mattias-Sehlstedt
Mattias-Sehlstedt force-pushed the update-swagger-core branch 2 times, most recently from 70999df to 33e56fd Compare September 6, 2026 11:07
@Mattias-Sehlstedt
Mattias-Sehlstedt marked this pull request as ready for review September 6, 2026 11:11
@bnasslahsen
bnasslahsen merged commit 2498ffb into springdoc:main Sep 6, 2026
1 check passed
bnasslahsen added a commit that referenced this pull request Sep 6, 2026
swagger-core now resolves java.time.Duration, LocalTime and OffsetTime
itself (swagger-api/swagger-core#5184), so the corresponding entries of
initExtraSchemas can be dropped.
@bnasslahsen

Copy link
Copy Markdown
Collaborator

Merged, thanks @Mattias-Sehlstedt. Full suite is green on both lines.

Backported to spring-boot-3 as well (2e7d271), so 2.9.1 ships swagger-core 2.2.54 too — the initExtraSchemas entries and the app115 / app243 fixtures were identical on that line, so the cherry-pick applied cleanly.

Recorded in the changelog under Changed, flagging the visible consequence for users: Duration and LocalTime now carry format: duration / format: partial-time instead of a bare string.

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.

2 participants