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

StackOverflowException during kiota update --clean-output -o Generated #5391

Closed
HikmetCilan opened this issue Sep 11, 2024 · 2 comments
Closed
Labels
fixed Status: No Recent Activity status:waiting-for-author-feedback Issue that we've responded but needs author feedback to close type:bug A broken experience

Comments

@HikmetCilan
Copy link

HikmetCilan commented Sep 11, 2024

What are you generating using Kiota, clients or plugins?

API Client/SDK

In what context or format are you using Kiota?

Nuget tool

Client library/SDK language

None

Describe the bug

I encountered a StackOverflowException while attempting to update the generated output using the kiota update --clean-output -o Generated command.

Expected behavior

The kiota update --clean-output command should successfully update the generated files without throwing a StackOverflowException.

How to reproduce

dotnet tool install --global Microsoft.OpenApi.Kiota
dotnet tool update --global Microsoft.OpenApi.Kiota
kiota update --clean-output -o Generated

Received a StackOverflowException with the following stack trace:
`Stack overflow.
Repeat 1053 times:

at System.Linq.Enumerable.SequenceEqual[[System.Collections.Generic.KeyValuePair...
at Kiota.Builder.Validation.OpenApiSchemaComparer.EqualsInternal(Microsoft.OpenApi.Models.OpenApiSchema...
at System.Linq.Enumerable.SequenceEqual...
at Kiota.Builder.Validation.DivergentResponseSchema...
at Microsoft.OpenApi.Validations.OpenApiValidator.Validate...
at Kiota.Builder.KiotaBuilder.CreateOpenApiDocumentAsync...
at Kiota.Builder.Caching.DocumentCachingProvider+d__13.MoveNext()...`

Open API description file

No response

Kiota Version

1.18.0

Latest Kiota version known to work for scenario above?(Not required)

No response

Known Workarounds

dotnet tool uninstall --global Microsoft.OpenApi.Kiota
dotnet tool install --global Microsoft.OpenApi.Kiota --version 1.17.0

kiota update --clean-output -o Generated

Configuration

No response

Debug output

Click to expand log ```
</details>


### Other information

_No response_
@HikmetCilan HikmetCilan added status:waiting-for-triage An issue that is yet to be reviewed or assigned type:bug A broken experience labels Sep 11, 2024
@github-project-automation github-project-automation bot moved this to Needs Triage 🔍 in Kiota Sep 11, 2024
@andrueastman
Copy link
Member

Thanks for raising this @HikmetCilan

I believe this is similar to the issue closed yesterday at #5369 via #5370.

Any chance you'd be willing to validate this by building the source on the main branch? Alternatively, you should be able to try out a preview release of the nuget tool before the end of the week.

@andrueastman andrueastman added fixed status:waiting-for-author-feedback Issue that we've responded but needs author feedback to close and removed status:waiting-for-triage An issue that is yet to be reviewed or assigned labels Sep 11, 2024
@andrueastman andrueastman moved this from Needs Triage 🔍 to In Review 💭 in Kiota Sep 11, 2024
Copy link
Contributor

This issue has been automatically marked as stale because it has been marked as requiring author feedback but has not had any activity for 4 days. It will be closed if no further activity occurs within 3 days of this comment.

@github-project-automation github-project-automation bot moved this from In Review 💭 to Done ✔️ in Kiota Sep 19, 2024
@baywet baywet closed this as not planned Won't fix, can't repro, duplicate, stale Sep 30, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
fixed Status: No Recent Activity status:waiting-for-author-feedback Issue that we've responded but needs author feedback to close type:bug A broken experience
Projects
Archived in project
Development

No branches or pull requests

3 participants