sostheim / krak8s

API Service for Kraken and Kubernetes Commands
Apache License 2.0
1 stars 5 forks source link

After Failed Deleting Get Namespaces Returns Panic Stack Dump in HTTP Response #6

Closed sostheim closed 7 years ago

sostheim commented 7 years ago

Initially reported by @jshimko

GET /v1/projects/5669723d/namespaces

"panic: runtime error: invalid memory address or nil pointer dereference\npanic(0x1434180, 0x200aeb0)\n\t/usr/local/go/src/runtime/panic.go:491 +0x283\nmain.MarshalNamespaceObject(0x0, 0x1)\n\t/go/src/krak8s/namespace.go:34 +0x39\nmain.(*NamespaceController).List(0xc42020e050, 0xc4201f22a0, 0xc4203d8800, 0xc420356ea0)\n\t/go/src/krak8s/namespace.go:107 +0x103\nkrak8s/app.MountNamespaceController.func4(0x2031b00, 0xc4201f2240, 0x202ac40, 0xc4203b7a40, 0xc4203d8800, 0xc4203bd3f8, 0xc4203bd400)\n\t/go/src/krak8s/app/controllers.go:335 +0xdd\nkrak8s/vendor/github.com/goadesign/goa.(*Controller).MuxHandler.func1.1(0x2031b00, 0xc4201f2240, 0x202ac40, 0xc4203b7a40, 0xc4203d8800, 0x2, 0x0)\n\t/go/src/krak8s/vendor/github.com/goadesign/goa/service.go:270 +0xaa\nkrak8s/vendor/github.com/goadesign/goa/middleware.Recover.func1.1(0x2031b00, 0xc4201f2240, 0x202ac40, 0xc4203b7a40, 0xc4203d8800, 0x0, 0x0)\n\t/go/src/krak8s/vendor/github.com/goadesign/goa/middleware/recover.go:37 +0xa0\nkrak8s/vendor/github.com/goadesign/goa/middleware.ErrorHandler.func1.1(0x2031b00, 0xc4201f2240, 0x202ac40, 0xc4203b7a40, 0xc4203d8800, 0x2, 0x2)\n\t/go/src/krak8s/vendor/github.com/goadesign/goa/middleware/error_handler.go:21 +0x90\nkrak8s/vendor/github.com/goadesign/goa/middleware.LogRequest.func1.1(0x2031b00, 0xc4201f2240, 0x202ac40, 0xc4203b7a40, 0xc4203d8800, 0xc420254080, 0x2031b00)\n\t/go/src/krak8s/vendor/github.com/goadesign/goa/middleware/log_request.go:75 +0xd48\nkrak8s/vendor/github.com/goadesign/goa/middleware.RequestIDWithHeaderAndLengthLimit.func1.1(0x2031b00, 0xc4201f21e0, 0x202ac40, 0xc4203b7a40, 0xc4203d8800, 0xc4201f2150, 0x2031b00)\n\t/go/src/krak8s/vendor/github.com/goadesign/goa/middleware/request_id.go:63 +0x144\nkrak8s/vendor/github.com/goadesign/goa.(*Controller).MuxHandler.func1(0x2030980, 0xc4201fe0e0, 0xc4203d8800, 0xc4201f2150)\n\t/go/src/krak8s/vendor/github.com/goadesign/goa/service.go:303 +0x336\nkrak8s/vendor/github.com/goadesign/goa.(*mux).Handle.func1(0x2030980, 0xc4201fe0e0, 0xc4203d8800, 0xc4201f20f0)\n\t/go/src/krak8s/vendor/github.com/goadesign/goa/mux.go:59 +0x1cb\nkrak8s/vendor/github.com/dimfeld/httptreemux.(*TreeMux).ServeLookupResult(0xc420292140, 0x2030980, 0xc4201fe0e0, 0xc4203d8800, 0xc8, 0xc42020e1b0, 0xc4201f20f0, 0x0)\n\t/go/src/krak8s/vendor/github.com/dimfeld/httptreemux/router.go:245 +0x133\nkrak8s/vendor/github.com/dimfeld/httptreemux.(*TreeMux).ServeHTTP(0xc420292140, 0x2030980, 0xc4201fe0e0, 0xc4203d8800)\n\t/go/src/krak8s/vendor/github.com/dimfeld/httptreemux/router.go:266 +0xdb\nkrak8s/vendor/github.com/goadesign/goa.(*mux).ServeHTTP(0xc420285a80, 0x2030980, 0xc4201fe0e0, 0xc4203d8800)\n\t/go/src/krak8s/vendor/github.com/goadesign/goa/mux.go:85 +0x4c\nnet/http.serverHandler.ServeHTTP(0xc42040fee0, 0x2030980, 0xc4201fe0e0, 0xc4203d8800)\n\t/usr/local/go/src/net/http/server.go:2619 +0xb4\nnet/http.(*conn).serve(0xc42018a1e0, 0x2031a40, 0xc4201b1040)\n\t/usr/local/go/src/net/http/server.go:1801 +0x71d\ncreated by net/http.(*Server).Serve\n\t/usr/local/go/src/net/http/server.go:2720 +0x288\n"
sostheim commented 7 years ago

Fixed in: https://github.com/samsung-cnct/krak8s/commit/a975e25084f74255d75f69de3e8c8a7ed3398ca3