Simon Bos

Results 9 comments of Simon Bos

Ok, I don't have any more questions; thanks for all the useful comments :smile: Merge?

Yes, I forgot to run the tests locally after changing the `newServer` output signature. With the latest change, `go test -v` (in the `cmd/serve/dlna` directory) runs succesfully.

I am experiencing a similar issue in (Jenkins) CI, although it's flaky - only on some runs there is a connection refused error as follows. ``` [2024-05-02T09:39:21.954Z] 2024/05/02 09:39:12 github.com/testcontainers/testcontainers-go...

> Do you know what docker engine/version is running in your CI? It's running "Docker Engine - Community" version 20.10.23. There is some extra info in the logs above.

Thanks for the effort @emetsger ! > what docker engine/version are you running on your local machine? Docker Desktop for Mac v4.29.0.

Hey @emetsger, applying this fix seems to work for me - I haven't seen any `connect: connection refused` since.

Hi @rodaine, thanks for the reply! I tried out the expressions using the following: ``` google.protobuf.Timestamp input_time = 2 [(buf.validate.field).cel = { id: "echo_request.input_time", message: "Timestamp should be valid", expression:...

Ok, that's clear, I'll have a look for implementing this (first in Go).

This seems to be resolved with https://github.com/cloudevents/sdk-go/pull/1095. However, there is no release tagged yet for this submodule (see [pkg.go.dev](https://pkg.go.dev/github.com/cloudevents/sdk-go/protocol/nats_jetstream/v3)). Is that deliberate @embano1 ?