Source: Don't disconnect DCP client if it was never connected

Description

As a workaround for JDCP-237: Client.disconnect() is not idempotentNew, and the fact that the DCP client's `connect()` method calls `disconnect()` internally if the connection failed, we need to keep track of whether the connection was successful, so we don't end up calling `disconnect()` if the client has already been disconnected due to a failed connection attempt.

Environment

None

Gerrit Reviews

None

Release Notes Description

None

Activity

Show:
David Nault
changed the StatusMay 31, 2023 at 4:03 AM
Open
Closed
David Nault
updated the ResolutionMay 31, 2023 at 4:03 AM
None
Fixed
David Nault
changed the StatusMay 31, 2023 at 4:03 AM
New
Open
David Nault
updated the Linked IssuesMay 30, 2023 at 6:33 PM
None
This issue blocks CBSE-14292
David Nault
updated the Linked IssuesMay 30, 2023 at 6:33 PM
None
This issue relates to JDCP-237
David Nault
created the IssueMay 30, 2023 at 6:32 PM
Fixed
Pinned fields
Click on the next to a field label to start pinning.

Details

Assignee

Reporter

Story Points

Sprint

Fix versions

Priority

Instabug

Open Instabug

PagerDuty

Sentry

Zendesk Support

Created May 30, 2023 at 6:32 PM
Updated May 31, 2023 at 4:03 AM
Resolved May 31, 2023 at 4:03 AM
Instabug