Skip to content

Comments

orbic-network: Disable connection pooling#902

Draft
untitaker wants to merge 1 commit intomainfrom
orbic-network-connection-pooling
Draft

orbic-network: Disable connection pooling#902
untitaker wants to merge 1 commit intomainfrom
orbic-network-connection-pooling

Conversation

@untitaker
Copy link
Collaborator

@untitaker untitaker commented Feb 18, 2026

Every once in a while, I get the error posted in #901:

Caused by:
    0: Failed to send login request
    1: error sending request
    2: client error (SendRequest)
    3: connection closed before message completed

Either "failed to send login request" or "failed to start telnet" -- in either case there's a
request happening before it, and it's always "connection closed before
message completed".

This is on firmware version ORB400L_V1.1.5_BVZRT, stock orbic.

Disabling connection pooling seems to reduce the amount of flakiness.

Here is what I used to test this fix:

while echo | cargo run -p installer util orbic-shell --admin-password 96df5476 ; do true; done

Usually it would stop <100 iterations, now it can do 800+ iterations.

@untitaker untitaker force-pushed the orbic-network-connection-pooling branch from e531aff to a63078e Compare February 18, 2026 21:29
Every once in a while, I get the error posted in #901:

    Caused by:
        0: Failed to send login request
        1: error sending request
        2: client error (SendRequest)
        3: connection closed before message completed

(either this or "failed to start telnet" -- in either case there's a
request happening before it, and it's always "connection closed before
message completed")

Disabling connection pooling seems to reduce the amount of flakiness.

Here is what I used to test this fix:

    while echo | cargo run -p installer util orbic-shell --admin-password 96df5476 ; do true; done

Usually it would stop <100 iterations, now it can do 800+ iterations.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant