Skip to content

Upgraded Microsoft.Extensions.ApiDescription.* to 8.0.14 - #5121

Merged
RicoSuter merged 2 commits into
RicoSuter:masterfrom
ntyrrell:bugfix/increase-M.E.ApiDescription-to-latest-GA
Apr 28, 2025
Merged

RicoSuter merged 2 commits into
RicoSuter:masterfrom
ntyrrell:bugfix/increase-M.E.ApiDescription-to-latest-GA

Conversation

@ntyrrell

Copy link
Copy Markdown
Contributor

Upgraded Microsoft.Extensions.ApiDescription.Server & Microsoft.Extensions.ApiDescription.Client to 9.0.3

This would update the runtime dependencies (which are still dependent on AspNetCore 6.0.3) as well as the following bugfixes for Microsoft.Extensions.ApiDescription.Server:
Fix up packaging for Extensions.ApiDescription.Server
Fixes for OpenAPI document generation with M.E.ApiDescription.Server
Upgrade to Microsoft.OpenApi 2.x and support OpenAPI v3.1
#59749 move TrimEnd after path normalization
And the following for Microsoft.Extensions.ApiDescription.Client:
#59749 move TrimEnd after path normalization
Fix duplicate add of items to Compile on Microsoft.Extensions.ApiDescription.Client.targets

@lahma

lahma commented Mar 17, 2025

Copy link
Copy Markdown
Collaborator

I think you can get these also by just referencing the latest versions in the consuming project?

@ntyrrell

Copy link
Copy Markdown
Contributor Author

I think you can get these also by just referencing the latest versions in the consuming project?

Sure, but I don't think that it's unreasonable for this project to reference the most up-to-date version.

It also puts more onus on the consumer of this library to have to maintain an additional dependency.

@RicoSuter

Copy link
Copy Markdown
Owner

Not sure whether this is a good idea, i think this might break e.g. .NET 8 users...
Isnt this package's version defined by the host application?

@lahma

lahma commented Mar 29, 2025

Copy link
Copy Markdown
Collaborator

My intuition would say that lowest non-vulnerable LTS would be a good approach, like 8.0.0. Some corporates might get confused if they are tied to say NET 8 LTS and 9.x dependencies start to show up.

@RicoSuter

Copy link
Copy Markdown
Owner

Thats probably a feasible approach.. (.net 8)
I think this apidescription stuff will be anyway deprecated longterm..

@ntyrrell

Copy link
Copy Markdown
Contributor Author

Thanks for the feedback. I'll adjust it to 8.0.* (not sure what the latest patch number is off hand)

@ntyrrell ntyrrell changed the title Upgraded Microsoft.Extensions.ApiDescription.* to 9.0.3 Upgraded Microsoft.Extensions.ApiDescription.* to 8.0.14 Apr 1, 2025
@RicoSuter
RicoSuter merged commit f878266 into RicoSuter:master Apr 28, 2025
@RicoSuter

Copy link
Copy Markdown
Owner

Thanks for the PR.

@ntyrrell
ntyrrell deleted the bugfix/increase-M.E.ApiDescription-to-latest-GA branch April 29, 2025 07:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants