-
Notifications
You must be signed in to change notification settings - Fork 1.2k
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
generated embedded_spec.go removing refs #1816
Comments
Weird. Will investigate this one |
This the culprit: go-openapi/loads@0d17535 The new spec cloning using gob instead of json fails to properly clone spec.Ref structures. |
Added a custom gob encoder/decoder for Ref. Looks like it works. Will finish the testing later. |
@kenjones-cisco there are pending PR in go-openapi/spec (the fixer) and go-openapi/loads (the verifier) to solve this issue. I'll update go-swagger vendor as soon as those are merged. Thanks. |
* Added test to make sure regression go-swagger/go-swagger#1816 is fixed by gob decoder in go-openapi/spec Signed-off-by: Frederic BIDON <fredbi@yahoo.com>
* Contributes go-swagger/go-swagger#1816 Signed-off-by: Frederic BIDON <fredbi@yahoo.com>
@fredbi Thanks for the update!!! |
* update vendor: go-openapi/spec, go-openapi/loads > NOTE: this is WIP and would require go-openapi/spec#87 and go-openapi/loads#22 to be merged. Signed-off-by: Frederic BIDON <fredbi@yahoo.com>
* update vendor: go-openapi/spec, go-openapi/loads > NOTE: this is WIP and would require go-openapi/spec#87 and go-openapi/loads#22 to be merged. Signed-off-by: Frederic BIDON <fredbi@yahoo.com>
Acknowledge fixed go-swagger/go-swagger#1816
* update vendor: go-openapi/spec, go-openapi/loads Signed-off-by: Frederic BIDON <fredbi@yahoo.com>
* update vendor: go-openapi/spec, go-openapi/loads Signed-off-by: Frederic BIDON <fredbi@yahoo.com>
* update vendor: go-openapi/spec, go-openapi/loads Signed-off-by: Frederic BIDON <fredbi@yahoo.com>
Acknowledge fix #1816 with go-openapi updates
* Includes fix for go-swagger#1824 * fixes go-swagger#1816 * completes and supersedes PR go-swagger#1817 Credit: @kenjones-cisco for original PR go-swagger#1817 Signed-off-by: Frederic BIDON <fredbi@yahoo.com>
Problem statement
Starting around the end of the week of 11/16 when generating code, I'm noticing that the embedded_spec.go is no longer include the ref paths, they are being set to
#
.Examples:
Swagger specification
Steps to reproduce
Environment
swagger version: master
go version: 1.11.1
OS: alpine
The text was updated successfully, but these errors were encountered: