Remove all hardcoded timeouts for ssh/docker daemon access #748

New Issue

decentral1se · 2026-01-11T09:11:22Z

decentral1se commented

2026-01-11 09:11:22 +00:00

I thought they were gone but they're still lurking!

connhelper.go

This affects alakazam when running secret generations which takes quite some time.

I thought they were gone but they're still lurking! `connhelper.go` This affects `alakazam` when running secret generations which takes quite some time.

decentral1se added the

bug

label 2026-01-11 09:11:22 +00:00

p4u1 commented

2026-01-11 11:03:05 +00:00

Did you actually use alkazam? 😃

😆 1

decentral1se commented

2026-01-11 21:46:41 +00:00

hahahaha, ok, i didn't but i did find some timeout code lurking still...

		pkg/client/client.go
		Lines 57 to 67 in 4e205cf13e
	
						helper, err := commandconnPkg.NewConnectionHelper(ctxEndpoint, conf.Timeout)

						if err != nil {

							return nil, err

						}

						httpClient := &http.Client{

							Transport: &http.Transport{

								DialContext:     helper.Dialer,

								IdleConnTimeout: 30 * time.Second,

							},

						}

I did run abra app run foo.bar.com app -d echo "going slow" && sleep 300 and it did NOT time out but it could be the SSH connection or the HTTP connection I believe, which is used to fire requests at the docker daemon.

alakazam is using the latest release afaict: 5a723221b9

@moritz can you help us with some --debug logs?

hahahaha, ok, i didn't but i did find some timeout code lurking still... https://git.coopcloud.tech/toolshed/abra/src/commit/4e205cf13e3878ef4c4194f71ec08f329a9d8c6c/pkg/client/client.go#L57-L67 I did run `abra app run foo.bar.com app -d echo "going slow" && sleep 300` and it did *NOT* time out but it could be the SSH connection or the HTTP connection I believe, which is used to fire requests at the docker daemon. alakazam is using the latest release afaict: https://git.coopcloud.tech/moritz/alakazam/commit/5a723221b95fe09941e2fe5452796437224aea60 @moritz can you help us with some `--debug` logs?

Sign in to join this conversation.

2 Participants

Notifications

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: toolshed/abra#748