Reflection error | Dagger | Page 1

elfin viper Oct 30, 2023, 11:53 PM

#

🧵

#

(not urgent)

full crescent Oct 31, 2023, 2:59 PM

#

cc @bright sigil

elfin viper Oct 31, 2023, 4:31 PM

#

Here’s the (unfinished) PR if you want to repro: https://github.com/dagger/dagger/pull/6028

GitHub

dagger call: add support for service arguments by shykes · Pull Req...

bright sigil Oct 31, 2023, 5:24 PM

#

i'm heading off for the day (still recovering, sorry 🤒), but will make it priority for tomorrow ❤️

elfin viper Oct 31, 2023, 5:26 PM

#

No rush @bright sigil , feel better!

elfin viper Oct 31, 2023, 6:24 PM

#

Here's the line in sdk/go/querybuilder/marshal.go that panics:

if isOptional && IsZeroValue(fv.Interface()) {

Specifically fv.Interface() causes the panic. This is documented in the Go docs:

func (v Value) Interface() (i any)
Interface returns v's current value as an interface{}. It is equivalent to:
var i interface{} = (v's underlying value)

It panics if the Value was obtained by accessing unexported struct fields.

bright sigil Nov 2, 2023, 3:11 PM

#

pushed a fix, and left a comment in https://github.com/dagger/dagger/pull/6028#issuecomment-1790926110

GitHub

dagger call: add support for service arguments by shykes · Pull Req...

elfin viper Nov 2, 2023, 5:47 PM

#

Thanks @bright sigil ! By chance were you able to actually get it to work? I now get past the panic (yay!) but it says "service is not running" when in fact there is a service listening on that port on the host

#Reflection error