FusionAuth / go-client

FusionAuth Go Client Library!

https://fusionauth.io/

Apache License 2.0

29 stars 32 forks source link

Compilation error: pkg/fusionauth/Domain.go:2858:16: undefined: double #57

Open rschilham opened 2 years ago

rschilham commented 2 years ago

After upgrading to the latest version I get the following compilation errors:

github.com/FusionAuth/go-client/pkg/fusionauth

../../../go/pkg/mod/github.com/!fusion!auth/go-client@v0.0.0-20210810031959-aff96aeebda7/pkg/fusionauth/Domain.go:2858:16: undefined: double ../../../go/pkg/mod/github.com/!fusion!auth/go-client@v0.0.0-20210810031959-aff96aeebda7/pkg/fusionauth/Domain.go:2859:16: undefined: double ../../../go/pkg/mod/github.com/!fusion!auth/go-client@v0.0.0-20210810031959-aff96aeebda7/pkg/fusionauth/Domain.go:4453:16: undefined: double

robotdan commented 2 years ago

I think the go client pulls from master which isn't always actually released.

However, it looks like we are using double instead of float64 and float instead of float32 for Go in our builder.

It looks like our builder is incorrect for float and double. https://github.com/FusionAuth/fusionauth-client-builder/blob/e44bf2ed46096d5411b9da43fc79bc5c73f0a9fc/src/main/client/_macros.ftl#L44

@mooreds not sure if there is much we can do about the Go client pulling from master - assuming that is just the pattern Go uses?

robotdan commented 2 years ago

Thanks for reporting @rschilham - much appreciated!!
The domain should be fixed under https://github.com/FusionAuth/go-client/commit/10e9f82a5b9d58f3f3f183cf2f47c6a0c5ba932e