GoogleCloudPlatform / cloud-sql-proxy

A utility for connecting securely to your Cloud SQL instances
Apache License 2.0
1.25k stars 345 forks source link

Ignore context in go-fuse Lookup method #2260

Open enocom opened 1 week ago

enocom commented 1 week ago

On heavy use, the go-fuse library will cancel a context (perhaps due to an interrupt). Rather than handle interrupts, we should ignore them and let the underlying Dialer finish its job.

https://github.com/GoogleCloudPlatform/cloud-sql-proxy/blob/main/internal/proxy/proxy_other.go#L102

This would remove the need for https://github.com/GoogleCloudPlatform/cloud-sql-go-connector/issues/831.

Cf. https://github.com/hanwen/go-fuse/issues/452