Swagger-ui: Circular Reference generate errors
On master
Circular reference generate errors.
Like "Node" contains a list of nodes for instance
Errors
Resolver error
too much recursion
Resolver error
too much recursion
Parameters view is also broken
Maybe relative to
Very slow loading times #1919
matthieugeerebaert
All 8 comments
Can you share a spec that reproduces it? We've tested circular reference support.
webron
on 10 Apr 2017
Here it is
matthieugeerebaert
on 10 Apr 2017
3.0.3 OK
3.0.5 and master KO
matthieugeerebaert
on 10 Apr 2017
Interesting. I see a different behavior if the file is embedded vs hosted. No major issues with the embedded version, but once hosted... :boom:.
webron
on 10 Apr 2017
Looks like it was introduced into the static build here: https://github.com/swagger-api/swagger-ui/commit/dd8fb58fcd50c0f0639e540d03a273557a66cf9e
shockey
on 25 Apr 2017
I see a different behavior if the file is embedded vs hosted.
@webron, thanks for the hint. You just saved me a couple additional hours of debugging 馃槃
The problem has been isolated to swagger-js, writing regression tests and looking for a fix now.
shockey
on 26 Apr 2017
Problem introduced here: https://github.com/swagger-api/swagger-js/commit/4e2dff33e75744e9
shockey
on 26 Apr 2017
Hi everyone - we just merged the PR that fixes this in swagger-js, and you can expect the fix to be in our Friday release tomorrow evening 馃槃
shockey
on 28 Apr 2017
Related issues
SangeetaGulia
路
4Comments
fehguy
路
3Comments
prabhat1790
路
3Comments
deepumi
路
3Comments
Deraen
路
4Comments
Most helpful comment
Interesting. I see a different behavior if the file is embedded vs hosted. No major issues with the embedded version, but once hosted... :boom:.