Skip to content

refactor!: switch to iroh headers for captive portal checks#3488

Merged
ramfox merged 3 commits intomainfrom
feat-iroh-header
Oct 7, 2025
Merged

refactor!: switch to iroh headers for captive portal checks#3488
ramfox merged 3 commits intomainfrom
feat-iroh-header

Conversation

@dignifiedquire
Copy link
Copy Markdown
Contributor

Description

Switches to use X-Iroh instead of X-Tailscale headers for the captive portal challenges.

Old headers are preserved on the iroh-relay side, until we do more breaking changes.

Breaking Changes

The challenges for captive portals need to be sent to the latest version of iroh-relay

@github-actions
Copy link
Copy Markdown

github-actions bot commented Sep 30, 2025

Documentation for this PR has been generated and is available at: https://n0-computer.github.io/iroh/pr/3488/docs/iroh/

Last updated: 2025-10-07T14:28:49Z

@n0bot n0bot bot added this to iroh Sep 30, 2025
@github-project-automation github-project-automation bot moved this to 🏗 In progress in iroh Sep 30, 2025
@github-actions
Copy link
Copy Markdown

github-actions bot commented Sep 30, 2025

Netsim report & logs for this PR have been generated and is available at: LOGS
This report will remain available for 3 days.

Last updated for commit: 9fcceff

@ramfox ramfox added this to the v0.93.0 milestone Oct 2, 2025

const NO_CONTENT_CHALLENGE_HEADER: &str = "X-Tailscale-Challenge";
const NO_CONTENT_RESPONSE_HEADER: &str = "X-Tailscale-Response";
// TODO: remove before 1.0
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.

Please link to an issue linked to the milestone.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

const NO_CONTENT_RESPONSE_HEADER: &str = "X-Tailscale-Response";
// TODO: remove before 1.0
const NO_CONTENT_CHALLENGE_HEADER_LEGACY: &str = "X-Tailscale-Challenge";
const NO_CONTENT_CHALLENGE_HEADER: &str = "X-Iroh-Challenge";
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.

Seeing "Iroh" capitalised is jarring 😉

@dignifiedquire dignifiedquire added this pull request to the merge queue Oct 7, 2025
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to failed status checks Oct 7, 2025
@ramfox ramfox added this pull request to the merge queue Oct 7, 2025
Merged via the queue into main with commit d6f33f9 Oct 7, 2025
29 checks passed
@github-project-automation github-project-automation bot moved this from 🏗 In progress to ✅ Done in iroh Oct 7, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: ✅ Done

Development

Successfully merging this pull request may close these issues.

3 participants