Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[Tools] Swagger #107

Closed
fffej opened this issue Nov 23, 2018 · 9 comments
Closed

[Tools] Swagger #107

fffej opened this issue Nov 23, 2018 · 9 comments

Comments

@fffej
Copy link
Contributor

fffej commented Nov 23, 2018

We use Swagger to document some of our web APIs.

I believe it should be in Adopt.

@fffej
Copy link
Contributor Author

fffej commented Nov 27, 2018

@Michael-Upton Am I right in thinking that the out of process work is using this? Any reason why it shouldn't be in adopt?

@ChrisHurley
Copy link
Contributor

A subtle nuance here - there's the OpenAPI specification (backed by Swagger) and there's OpenAPI/Swagger generation libraries like Swashbuckle. Are we referring to both?

(Also worth noting that if you really want your generated documentation to be useful you need to be using the correct attributes and XML doc strings - otherwise you'll end up with a relatively bare list of endpoints).

@fffej
Copy link
Contributor Author

fffej commented Nov 29, 2018

Good point, let's say we adopt the Open API spec and our default tool of choice is ?

@tugberkugurlu
Copy link
Contributor

FWIW, we currently use NSwag for reasons described here:

@ChrisHurley
Copy link
Contributor

ChrisHurley commented Nov 29, 2018

NSwag looks pretty good to me, and I wasn't aware of its existence - although I'm interested in where things are with API versioning support, since we use that with WebApi now, and their WebApi support is deprecated.

(Maybe we should look into retiring WebApi in favour of ASP.NET Core...)

@tugberkugurlu
Copy link
Contributor

(Maybe we should look into retiring WebApi in favour of ASP.NET Core...)

👍 as there will be less and less support for Web API as time progresses.

@tugberkugurlu
Copy link
Contributor

@PeterGerrard
Copy link
Contributor

@fffej
Copy link
Contributor Author

fffej commented Dec 6, 2018

To try and get this one closed out here's where I think we are.

Adopt:

  • Open API specification - our APIs are built using the Open API specification

Explore

  • NSwag - An implementation of the Open API specification currently being explored by the Data Catalog team.
  • Swashbuckle - An implementation of the OpenAPI specification currently being explored by the Versioning teams for providing the out-of-process APIs.

👍 if you're happy and I'll create a PR.

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

No branches or pull requests

4 participants