vproxy: add icap tcp connect timeout

[DanielHaimanot]

Problem / Solution

Adds a configurable tcp_connect_timeout to the ICAP module's connection
establishment path. Previously socket.connect() was awaited with no bound,
leaving dials to the OS default.

Change

• IcapServiceConfig gains tcp_connect_timeout (default: 1s)
• connect() wraps the dial in tokio::time::timeout

Closes #66

[codecov]

Codecov Report

❌ Patch coverage is 35.00000% with 13 lines in your changes missing coverage. Please review.

Files with missing lines	Patch %	Lines
lib/vey-icap-client/src/service/connection.rs	33.33%	6 Missing ⚠️
lib/vey-icap-client/src/service/config/yaml.rs	20.00%	4 Missing ⚠️
lib/vey-icap-client/src/service/config/mod.rs	25.00%	3 Missing ⚠️

📢 Thoughts on this report? Let us know!

[zh-jq]

It seems we should also add a timeout limit to the ICAP OPTIONS request.

[DanielHaimanot]

It seems we should also add a timeout limit to the ICAP OPTIONS request.

right, I also prefer moving OPTIONS request out of the client and respect the TTL given by the ICAP server.