Skip to content

Commit 07b69d6

Browse files
bkoelmanbkoelman
committed
Include custom controllers and minimal API endpoints in OpenAPI; log warning for non-standard JSON:API action methods; fix broken content negotiation.
1 parent 1a9cfe3 commit 07b69d6

120 files changed

Lines changed: 5987 additions & 768 deletions

File tree

Some content is hidden

Large Commits have some content hidden by default. Use the searchbox below for content that may be hidden.

‎.config/dotnet-tools.json‎

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -31,11 +31,11 @@
3131
"rollForward": false
3232
},
3333
"microsoft.openapi.kiota": {
34-
"version": "1.27.0",
34+
"version": "1.28.0",
3535
"commands": [
3636
"kiota"
3737
],
3838
"rollForward": false
3939
}
4040
}
41-
}
41+
}

‎package-versions.props‎

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -20,6 +20,7 @@
2020
<KiotaVersion>1.*</KiotaVersion>
2121
<MicrosoftApiClientVersion>9.0.*</MicrosoftApiClientVersion>
2222
<MicrosoftApiServerVersion>9.0.*</MicrosoftApiServerVersion>
23+
<MiniValidationVersion>0.9.*</MiniValidationVersion>
2324
<NSwagApiClientVersion>14.4.*</NSwagApiClientVersion>
2425
<NewtonsoftJsonVersion>13.0.*</NewtonsoftJsonVersion>
2526
<ReadableExpressionsVersion>4.1.*</ReadableExpressionsVersion>

‎src/Examples/JsonApiDotNetCoreExample/Controllers/NonJsonApiController.cs‎

Lines changed: 18 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -3,9 +3,13 @@
33
namespace JsonApiDotNetCoreExample.Controllers;
44

55
[Route("[controller]")]
6+
[Tags("nonJsonApi")]
67
public sealed class NonJsonApiController : ControllerBase
78
{
8-
[HttpGet]
9+
[HttpGet(Name = "welcomeGet")]
10+
[HttpHead(Name = "welcomeHead")]
11+
[EndpointDescription("Returns a single-element JSON array.")]
12+
[ProducesResponseType<List<string>>(StatusCodes.Status200OK, "application/json")]
913
public IActionResult Get()
1014
{
1115
string[] result = ["Welcome!"];
@@ -14,12 +18,15 @@ public IActionResult Get()
1418
}
1519

1620
[HttpPost]
17-
public async Task<IActionResult> PostAsync()
21+
[EndpointDescription("Returns a greeting text, based on your name.")]
22+
[Consumes("application/json")]
23+
[ProducesResponseType<string>(StatusCodes.Status200OK, "text/plain")]
24+
[ProducesResponseType<string>(StatusCodes.Status400BadRequest, "text/plain")]
25+
public async Task<IActionResult> PostAsync([FromBody] string? name)
1826
{
19-
using var reader = new StreamReader(Request.Body, leaveOpen: true);
20-
string name = await reader.ReadToEndAsync();
27+
await Task.Yield();
2128

22-
if (string.IsNullOrEmpty(name))
29+
if (string.IsNullOrWhiteSpace(name))
2330
{
2431
return BadRequest("Please send your name.");
2532
}
@@ -29,14 +36,18 @@ public async Task<IActionResult> PostAsync()
2936
}
3037

3138
[HttpPut]
32-
public IActionResult Put([FromBody] string name)
39+
[EndpointDescription("Returns another greeting text.")]
40+
[ProducesResponseType<string>(StatusCodes.Status200OK, "text/plain")]
41+
public IActionResult Put([FromQuery] string? name)
3342
{
3443
string result = $"Hi, {name}";
3544
return Ok(result);
3645
}
3746

3847
[HttpPatch]
39-
public IActionResult Patch(string name)
48+
[EndpointDescription("Wishes you a good day.")]
49+
[ProducesResponseType<string>(StatusCodes.Status200OK, "text/plain")]
50+
public IActionResult Patch([FromHeader] string? name)
4051
{
4152
string result = $"Good day, {name}";
4253
return Ok(result);

‎src/Examples/JsonApiDotNetCoreExample/GeneratedSwagger/JsonApiDotNetCoreExample.json‎

Lines changed: 137 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -10,6 +10,143 @@
1010
}
1111
],
1212
"paths": {
13+
"/NonJsonApi": {
14+
"get": {
15+
"tags": [
16+
"nonJsonApi"
17+
],
18+
"description": "Returns a single-element JSON array.",
19+
"operationId": "welcomeGet",
20+
"responses": {
21+
"200": {
22+
"description": "OK",
23+
"content": {
24+
"application/json": {
25+
"schema": {
26+
"type": "array",
27+
"items": {
28+
"type": "string"
29+
}
30+
}
31+
}
32+
}
33+
}
34+
}
35+
},
36+
"head": {
37+
"tags": [
38+
"nonJsonApi"
39+
],
40+
"description": "Returns a single-element JSON array.",
41+
"operationId": "welcomeHead",
42+
"responses": {
43+
"200": {
44+
"description": "OK"
45+
}
46+
}
47+
},
48+
"post": {
49+
"tags": [
50+
"nonJsonApi"
51+
],
52+
"description": "Returns a greeting text, based on your name.",
53+
"requestBody": {
54+
"content": {
55+
"application/json": {
56+
"schema": {
57+
"type": "string"
58+
}
59+
}
60+
}
61+
},
62+
"responses": {
63+
"200": {
64+
"description": "OK",
65+
"content": {
66+
"text/plain": {
67+
"schema": {
68+
"type": "string"
69+
}
70+
}
71+
}
72+
},
73+
"400": {
74+
"description": "Bad Request",
75+
"content": {
76+
"text/plain": {
77+
"schema": {
78+
"type": "string"
79+
}
80+
}
81+
}
82+
}
83+
}
84+
},
85+
"put": {
86+
"tags": [
87+
"nonJsonApi"
88+
],
89+
"description": "Returns another greeting text.",
90+
"parameters": [
91+
{
92+
"name": "name",
93+
"in": "query",
94+
"schema": {
95+
"type": "string"
96+
}
97+
}
98+
],
99+
"responses": {
100+
"200": {
101+
"description": "OK",
102+
"content": {
103+
"text/plain": {
104+
"schema": {
105+
"type": "string"
106+
}
107+
}
108+
}
109+
}
110+
}
111+
},
112+
"patch": {
113+
"tags": [
114+
"nonJsonApi"
115+
],
116+
"description": "Wishes you a good day.",
117+
"parameters": [
118+
{
119+
"name": "name",
120+
"in": "header",
121+
"schema": {
122+
"type": "string"
123+
}
124+
}
125+
],
126+
"responses": {
127+
"200": {
128+
"description": "OK",
129+
"content": {
130+
"text/plain": {
131+
"schema": {
132+
"type": "string"
133+
}
134+
}
135+
}
136+
}
137+
}
138+
},
139+
"delete": {
140+
"tags": [
141+
"nonJsonApi"
142+
],
143+
"responses": {
144+
"200": {
145+
"description": "OK"
146+
}
147+
}
148+
}
149+
},
13150
"/api/operations": {
14151
"post": {
15152
"tags": [

‎src/Examples/OpenApiKiotaClientExample/GeneratedCode/ExampleApiClient.cs‎

Lines changed: 7 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -10,6 +10,7 @@
1010
using Microsoft.Kiota.Serialization.Multipart;
1111
using Microsoft.Kiota.Serialization.Text;
1212
using OpenApiKiotaClientExample.GeneratedCode.Api;
13+
using OpenApiKiotaClientExample.GeneratedCode.NonJsonApi;
1314
using System.Collections.Generic;
1415
using System.IO;
1516
using System.Threading.Tasks;
@@ -28,6 +29,12 @@ public partial class ExampleApiClient : BaseRequestBuilder
2829
get => new global::OpenApiKiotaClientExample.GeneratedCode.Api.ApiRequestBuilder(PathParameters, RequestAdapter);
2930
}
3031

32+
/// <summary>The NonJsonApi property</summary>
33+
public global::OpenApiKiotaClientExample.GeneratedCode.NonJsonApi.NonJsonApiRequestBuilder NonJsonApi
34+
{
35+
get => new global::OpenApiKiotaClientExample.GeneratedCode.NonJsonApi.NonJsonApiRequestBuilder(PathParameters, RequestAdapter);
36+
}
37+
3138
/// <summary>
3239
/// Instantiates a new <see cref="global::OpenApiKiotaClientExample.GeneratedCode.ExampleApiClient"/> and sets the default values.
3340
/// </summary>

0 commit comments

Comments
 (0)