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.

Sign up for GitHub

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

Jump to bottom

NullReferenceExcetion when generating operation table for PR #5837

Closed
heaths opened this issue Mar 27, 2023 · 2 comments
Closed

NullReferenceExcetion when generating operation table for PR #5837

heaths opened this issue Mar 27, 2023 · 2 comments
Assignees
@chidozieononiwu
Labels
APIView Central-EngSys This issue is owned by the Engineering System team. Swagger Issues for APIView support of Swagger

Comments

@heaths
Copy link
Member

heaths commented Mar 27, 2023

Checks on PR Azure/azure-rest-api-specs#23290 failed with:

Unhandled exception: System.NullReferenceException: Object reference not set to an instance of an object.
   at SwaggerApiParser.BaseSchema.GetTypeFormat() in /mnt/vss/_work/1/s/tools/apiview/parsers/swagger-api-parser/SwaggerApiParser/SwaggerSpec/SwaggerTypes.cs:line 134
   at SwaggerApiParser.SwaggerApiViewParameter.GetTypeFormat() in /mnt/vss/_work/1/s/tools/apiview/parsers/swagger-api-parser/SwaggerApiParser/SwaggerApiView/SwaggerApiViewParameter.cs:line 40
   at SwaggerApiParser.SwaggerApiViewOperationParameters.TokenSerializeTableRows(SerializeContext context) in /mnt/vss/_work/1/s/tools/apiview/parsers/swagger-api-parser/SwaggerApiParser/SwaggerApiView/SwaggerApiViewParameter.cs:line 148
   at SwaggerApiParser.SwaggerApiViewOperationParameters.TokenSerialize(SerializeContext context) in /mnt/vss/_work/1/s/tools/apiview/parsers/swagger-api-parser/SwaggerApiParser/SwaggerApiView/SwaggerApiViewParameter.cs:line 136
   at SwaggerApiParser.SwaggerApiViewOperation.TokenSerialize(SerializeContext context) in /mnt/vss/_work/1/s/tools/apiview/parsers/swagger-api-parser/SwaggerApiParser/SwaggerApiView/SwaggerApiViewOperation.cs:line 104
   at SwaggerApiParser.SwaggerApiViewPaths.TokenSerialize(SerializeContext context) in /mnt/vss/_work/1/s/tools/apiview/parsers/swagger-api-parser/SwaggerApiParser/SwaggerApiView/SwaggerApiViewPaths.cs:line 101
   at SwaggerApiParser.SwaggerApiViewSpec.TokenSerialize(SerializeContext context) in /mnt/vss/_work/1/s/tools/apiview/parsers/swagger-api-parser/SwaggerApiParser/SwaggerApiView/SwaggerApiViewSpec.cs:line 47
   at SwaggerApiParser.SwaggerApiViewRoot.TokenSerialize(SerializeContext context) in /mnt/vss/_work/1/s/tools/apiview/parsers/swagger-api-parser/SwaggerApiParser/SwaggerApiView/SwaggerApiViewRoot.cs:line 66
   at SwaggerApiParser.SwaggerApiViewRoot.GenerateCodeFile() in /mnt/vss/_work/1/s/tools/apiview/parsers/swagger-api-parser/SwaggerApiParser/SwaggerApiView/SwaggerApiViewRoot.cs:line 40
   at SwaggerApiParser.Program.HandleGenerateCodeFile(IEnumerable`1 swaggers, String output, String packageName, List`1 swaggerLinks, String readmeFile, String readmeTag) in /mnt/vss/_work/1/s/tools/apiview/parsers/swagger-api-parser/SwaggerApiParser/Program.cs:line 109
   at SwaggerApiParser.Program.<>c.<<Main>b__0_3>d.MoveNext() in /mnt/vss/_work/1/s/tools/apiview/parsers/swagger-api-parser/SwaggerApiParser/Program.cs:line 57
@ghost ghost added the needs-triage Workflow: This is a new issue that needs to be triaged to the appropriate team. label Mar 27, 2023
@kurtzeborn kurtzeborn removed the needs-triage Workflow: This is a new issue that needs to be triaged to the appropriate team. label Apr 3, 2023
@kurtzeborn kurtzeborn added Central-EngSys This issue is owned by the Engineering System team. Spec PR Tools Tooling that runs in azure-rest-api-specs repo. labels Apr 3, 2023
@kurtzeborn kurtzeborn moved this from 🤔Triage to 📋Backlog in Azure SDK EngSys 🚢🎉 Apr 3, 2023
@kurtzeborn kurtzeborn added APIView and removed Spec PR Tools Tooling that runs in azure-rest-api-specs repo. labels Apr 3, 2023
@github-project-automation github-project-automation bot moved this to 🆕 New in ApiView Apr 3, 2023
@mikekistler mikekistler added the Swagger Issues for APIView support of Swagger label Apr 17, 2023
@mikekistler mikekistler moved this from 🆕 New to 📋 Backlog in ApiView Apr 17, 2023
@mikekistler
Copy link
Member

Making this high priority at least to get debug done and understand the scenario. May lower later.

@chidozieononiwu chidozieononiwu mentioned this issue Apr 19, 2023
Merged
@chidozieononiwu
Copy link
Member

Resolved by #6009

@github-project-automation github-project-automation bot moved this from 📋 Backlog to 🎊 Closed in Azure SDK EngSys 🚢🎉 Apr 26, 2023
@github-project-automation github-project-automation bot moved this from 📋 Backlog to ✅ Done in ApiView Apr 26, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@chidozieononiwu chidozieononiwu

Labels
APIView Central-EngSys This issue is owned by the Engineering System team. Swagger Issues for APIView support of Swagger
Projects
ApiView
Status: ✅ Done
Milestone
No milestone
Development

No branches or pull requests
4 participants
@heaths @kurtzeborn @chidozieononiwu @mikekistler