Skip to content

fix: validate required properties of security scheme before serialization#2952

Merged
baywet merged 2 commits into
mainfrom
fix/required-security-scheme-properties
Jul 14, 2026
Merged

fix: validate required properties of security scheme before serialization#2952
baywet merged 2 commits into
mainfrom
fix/required-security-scheme-properties

Conversation

@baywet

@baywet baywet commented Jul 14, 2026

Copy link
Copy Markdown
Member

fixes #1843

@baywet
fix: validate required properties of security scheme before serializa…
0dd9a75 
…tion

Signed-off-by: Vincent Biret <vibiret@microsoft.com>
@baywet baywet requested a review from a team as a code owner July 14, 2026 18:08
@baywet baywet self-assigned this Jul 14, 2026
@baywet baywet enabled auto-merge July 14, 2026 18:08
@baywet baywet mentioned this pull request Jul 14, 2026
Closed
@github-code-quality

github-code-quality Bot commented Jul 14, 2026 •
edited
Loading

Copy link
Copy Markdown

Code Coverage Overview

Languages: C#

C# / code-coverage/dotnet

The overall coverage in the branch remains at 89%, unchanged from the branch.

Show a code coverage summary of the most impacted files.
File 63fc55d 16826b0 +/-
/home/runner/wo...onNodeHelper.cs 94% 90% -4%
/home/runner/wo...rce.Designer.cs 45% 43% -2%
/home/runner/wo...Deserializer.cs 69% 68% -1%
/home/runner/wo...Deserializer.cs 69% 68% -1%
/home/runner/wo...Deserializer.cs 79% 78% -1%
/home/runner/wo...curityScheme.cs 71% 80% +9%

Updated July 14, 2026 18:46 UTC
Code Coverage is in Public Preview. Learn more and provide us with your feedback.

@baywet
chore: fixes mock setup
16826b0 
Signed-off-by: Vincent Biret <vibiret@microsoft.com>
@baywet baywet force-pushed the fix/required-security-scheme-properties branch from cd1cdd4 to 16826b0 Compare July 14, 2026 18:21
@baywet baywet mentioned this pull request Jul 14, 2026
Merged
@baywet

baywet commented Jul 14, 2026

Copy link
Copy Markdown
Member Author

@microsoft-github-policy-service rerun

1 similar comment
@baywet

baywet commented Jul 14, 2026

Copy link
Copy Markdown
Member Author

@microsoft-github-policy-service rerun

@baywet baywet closed this Jul 14, 2026
auto-merge was automatically disabled July 14, 2026 18:44

Pull request was closed

@baywet baywet reopened this Jul 14, 2026
@baywet baywet enabled auto-merge (squash) July 14, 2026 18:44
@baywet

baywet commented Jul 14, 2026

Copy link
Copy Markdown
Member Author

@microsoft-github-policy-service rerun

@baywet baywet merged commit f31b192 into main Jul 14, 2026
23 of 26 checks passed
@baywet baywet deleted the fix/required-security-scheme-properties branch July 14, 2026 18:47
@release-please-token-provider release-please-token-provider Bot mentioned this pull request Jul 14, 2026
Open
@sonarqubecloud

sonarqubecloud Bot commented Jul 14, 2026

Copy link
Copy Markdown

Quality Gate Passed Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
100.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud

baywet added a commit that referenced this pull request Jul 14, 2026
@baywet
[support/v2] port PR #2952 (#2954)
26de615 
* Merge 16826b0 into 3ef2b9e

* test(library): remove invalid OpenAPI 3.2 inline data

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

Copilot-Session: b7a44557-39ee-4285-886c-7918d3caec2f
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@peombwa peombwa peombwa approved these changes

Assignees

@baywet baywet

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Incorrect default value for OpenApiSecurityScheme.In when using "http" type with "bearer" scheme

2 participants

@baywet @peombwa