Skip to content

Add cert printing on failed derp handshake - #1906

Open
stalowyjez wants to merge 3 commits into
mainfrom
LLT-5911-add_more_logs_to_derp_connection
Open

Add cert printing on failed derp handshake#1906
stalowyjez wants to merge 3 commits into
mainfrom
LLT-5911-add_more_logs_to_derp_connection

Conversation

@stalowyjez

@stalowyjez stalowyjez commented Jul 23, 2026

Copy link
Copy Markdown
Contributor

Problem

We had a problem with debugging problems with DERP connections

Solution

Let's add printing certs on failing Derp connections, so we have some more info what happens inside

☑️ Definition of Done checklist

  • Commit history is clean (requirements)
  • README.md is updated
  • Functionality is covered by unit or integration tests

@stalowyjez
stalowyjez requested a review from a team as a code owner July 23, 2026 14:09
@stalowyjez
stalowyjez force-pushed the LLT-5911-add_more_logs_to_derp_connection branch from 1ab5c25 to e71e1c5 Compare July 23, 2026 14:14
@stalowyjez
stalowyjez force-pushed the LLT-5911-add_more_logs_to_derp_connection branch from e71e1c5 to 4ad3abc Compare July 23, 2026 15:22
@stalowyjez
stalowyjez force-pushed the LLT-5911-add_more_logs_to_derp_connection branch from 4ad3abc to c04f58b Compare July 23, 2026 15:48
gytsto
gytsto previously approved these changes Jul 24, 2026

@gytsto gytsto left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

few nits (can be ignored), but looks good +1

Comment thread crates/telio-relay/src/derp/http.rs Outdated
Comment thread Cargo.toml Outdated
Comment thread .unreleased/LLT-5911 Outdated
@stalowyjez
stalowyjez force-pushed the LLT-5911-add_more_logs_to_derp_connection branch from c04f58b to 6ca5e9f Compare July 24, 2026 11:39

@jjanowsk jjanowsk left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

+1

Comment thread .unreleased/LLT-5911 Outdated
@stalowyjez
stalowyjez force-pushed the LLT-5911-add_more_logs_to_derp_connection branch from 6ca5e9f to e81156f Compare August 3, 2026 15:04
@stalowyjez
stalowyjez force-pushed the LLT-5911-add_more_logs_to_derp_connection branch from e81156f to f9df9dd Compare August 3, 2026 15:09
Comment thread crates/telio-relay/src/derp/http.rs Outdated
Comment thread crates/telio-relay/src/derp/http.rs Outdated
@stalowyjez
stalowyjez force-pushed the LLT-5911-add_more_logs_to_derp_connection branch from f9df9dd to d7a69b8 Compare August 4, 2026 21:17
@stalowyjez
stalowyjez force-pushed the LLT-5911-add_more_logs_to_derp_connection branch from d7a69b8 to 3e7613c Compare August 5, 2026 14:33
@stalowyjez
stalowyjez force-pushed the LLT-5911-add_more_logs_to_derp_connection branch from 3e7613c to 6442bb4 Compare August 25, 2026 13:46
@stalowyjez
stalowyjez force-pushed the LLT-5911-add_more_logs_to_derp_connection branch from 6442bb4 to 6862950 Compare August 25, 2026 16:12
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.

4 participants