apisix-dashboard
apisix-dashboard copied to clipboard
fix(route): openapi import panic
fix openapi import panic when requestbody with empty content
Fixes #2477
Please answer these questions before submitting a pull request, or your PR will get closed.
Why submit this pull request?
- [x] Bugfix
- [ ] New feature provided
- [ ] Improve performance
- [ ] Backport patches
What changes will this PR take into?
Please update this section with detailed description.
Related issues
fix/resolve #2477
Checklist:
- [ ] Did you explain what problem does this PR solve? Or what new features have been added?
- [ ] Have you added corresponding test cases?
- [ ] Have you modified the corresponding document?
- [ ] Is this PR backward compatible? If it is not backward compatible, please discuss on the mailing list first
Codecov Report
Merging #2478 (6c62943) into master (3b2c371) will increase coverage by
0.12%
. The diff coverage isn/a
.
:exclamation: Current head 6c62943 differs from pull request most recent head 5e30928. Consider uploading reports for the commit 5e30928 to get more accurate results
@@ Coverage Diff @@
## master #2478 +/- ##
==========================================
+ Coverage 68.77% 68.90% +0.12%
==========================================
Files 58 131 +73
Lines 4042 3447 -595
Branches 0 842 +842
==========================================
- Hits 2780 2375 -405
- Misses 964 1072 +108
+ Partials 298 0 -298
Flag | Coverage Δ | |
---|---|---|
backend-e2e-test-ginkgo | ? |
|
backend-unit-test | ? |
|
frontend-e2e-test | 68.90% <ø> (?) |
Flags with carried forward coverage won't be shown. Click here to find out more.
Continue to review full report at Codecov.
Legend - Click here to learn more
Δ = absolute <relative> (impact)
,ø = not affected
,? = missing data
Powered by Codecov. Last update 3b2c371...5e30928. Read the comment docs.
@yjunsun LGTM, and you need to tweak the code to pass the CI test.
@yjunsun Hi, pls resolve the conflicts.