Optional `ctx` | Dagger | Page 1

opaque dome Nov 18, 2023, 3:54 PM

#

Possibly one for Helder Correia: Go docs suggest ctx is optional (https://docs.dagger.io/zenith/developer/go/191108/advanced-programming#miscellanea), but in Python I'm getting errors (both from LSP and if I ignore the LSP to run the code) if I don't provide it to a module.

I'm testing with the Infisical module and dagger.Infisical().get_secret() and the error is raised on instantiating the Infisical class

Advanced Programming | Dagger

{@include: ../../partials/_experimental.md}

#

Signature doesn't suggest it's optional either

cinder linden Nov 18, 2023, 4:36 PM

#

@opaque dome That's especially odd given that the Infisical module is written in Python...

#

#

Here are some calls from Golang using that module

opaque dome Nov 18, 2023, 4:42 PM

#

Well that's very odd indeed, clearly working as expected there. Looks like it'll be something I've not configured right?

cinder linden Nov 18, 2023, 4:42 PM

#

I should try to call it from a module written in python like yours.

#

I get scared when I see all of the self. in python 😆

opaque dome Nov 18, 2023, 4:43 PM

#

Same to be honest, I avoid classes like the plague

#

I can send you the repo link if that'll shortcut the test?

cinder linden Nov 18, 2023, 4:45 PM

#

opaque dome Nov 18, 2023, 4:45 PM

#

I've changed my commit of the Infisical module to the same commit used in the greetings-api module there, that hasn't solved it.

#Optional `ctx`