Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

add autotls blog post #137

Merged
merged 24 commits into from
Feb 13, 2025
Merged

add autotls blog post #137

merged 24 commits into from
Feb 13, 2025

Conversation

2color
Copy link
Contributor

@2color 2color commented Dec 20, 2024

@2color 2color requested review from cewood and lidel December 20, 2024 13:24
@2color 2color assigned aschmahmann and unassigned aschmahmann Dec 20, 2024
@2color 2color requested a review from aschmahmann December 20, 2024 13:24
src/_blog/aut-tls.md Outdated Show resolved Hide resolved
src/_blog/aut-tls.md Outdated Show resolved Hide resolved
src/_blog/aut-tls.md Outdated Show resolved Hide resolved
src/_blog/aut-tls.md Outdated Show resolved Hide resolved
src/_blog/aut-tls.md Outdated Show resolved Hide resolved
src/_blog/aut-tls.md Outdated Show resolved Hide resolved
src/_blog/aut-tls.md Outdated Show resolved Hide resolved
@2color 2color requested a review from guillaumemichel January 8, 2025 14:22
Copy link
Contributor

@guillaumemichel guillaumemichel left a comment

Choose a reason for hiding this comment

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

Great post! A few nits:

src/_blog/aut-tls.md Outdated Show resolved Hide resolved
src/_blog/aut-tls.md Outdated Show resolved Hide resolved
src/_blog/aut-tls.md Outdated Show resolved Hide resolved
src/_blog/aut-tls.md Outdated Show resolved Hide resolved
src/_blog/aut-tls.md Outdated Show resolved Hide resolved
Copy link
Contributor

@mishmosh mishmosh left a comment

Choose a reason for hiding this comment

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

Excited for this to go live. Added a few comments with a lens towards using this for browser advocacy and drawing in new users, and 2 small typos.

src/_blog/aut-tls.md Outdated Show resolved Hide resolved
src/_blog/aut-tls.md Outdated Show resolved Hide resolved
src/_blog/aut-tls.md Outdated Show resolved Hide resolved
src/_blog/aut-tls.md Outdated Show resolved Hide resolved
src/_blog/aut-tls.md Outdated Show resolved Hide resolved
src/_blog/aut-tls.md Outdated Show resolved Hide resolved
src/_blog/aut-tls.md Outdated Show resolved Hide resolved
src/_blog/aut-tls.md Outdated Show resolved Hide resolved
Copy link
Member

@lidel lidel left a comment

Choose a reason for hiding this comment

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

Some asks/nits inline, but once addressed in the context of Thursday 13th, I think this is mostly ready, as soon we merge example from libp2p/go-libp2p#3103

src/_blog/aut-tls.md Outdated Show resolved Hide resolved
src/_blog/aut-tls.md Outdated Show resolved Hide resolved
src/_blog/aut-tls.md Outdated Show resolved Hide resolved
src/_blog/aut-tls.md Outdated Show resolved Hide resolved

> **Note:** Another valid shorter representation of the multiaddr is `/dns4/147-75-63-129.k51qzi5uqu5dht5qyglpp8q4qldzx6d094lqdffp5n80zj5u6vfxk7n4pmutoo.libp2p.direct/tcp/4002/tls/ws`, but it requires a DNS lookup to resolve the IP address. In theory, avoiding the DNS lookup is a performance win, but in practice, browsers don't let you manually set the SNI hostname for a WebSocket connection, so the browser WebSocket API will always need the DNS name for the Secure WebSocket connection.

This multiaddr can be dialed from any browser with `js-libp2p`. You can see this in action with the[Helia Identify Tool](https://helia-identify.on.fleek.co/?peer-or-maddr=%2Fip4%2F147.75.63.129%2Ftcp%2F4002%2Ftls%2Fsni%2F147-75-63-129.k51qzi5uqu5dht5qyglpp8q4qldzx6d094lqdffp5n80zj5u6vfxk7n4pmutoo.libp2p.direct%2Fws)
Copy link
Member

Choose a reason for hiding this comment

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

nit: missing space after the

src/_blog/aut-tls.md Outdated Show resolved Hide resolved
src/_blog/aut-tls.md Outdated Show resolved Hide resolved
src/_blog/aut-tls.md Outdated Show resolved Hide resolved
src/_blog/aut-tls.md Outdated Show resolved Hide resolved
src/_blog/aut-tls.md Outdated Show resolved Hide resolved
Copy link
Contributor

@aschmahmann aschmahmann left a comment

Choose a reason for hiding this comment

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

Added a few comments, but looks great.

src/_blog/aut-tls.md Outdated Show resolved Hide resolved
src/_blog/aut-tls.md Outdated Show resolved Hide resolved
src/_blog/aut-tls.md Outdated Show resolved Hide resolved
src/_blog/aut-tls.md Outdated Show resolved Hide resolved
src/_blog/aut-tls.md Outdated Show resolved Hide resolved
src/_blog/auto-tls.md Outdated Show resolved Hide resolved
src/_blog/auto-tls.md Outdated Show resolved Hide resolved
@2color 2color merged commit b4305b0 into main Feb 13, 2025
2 checks passed
@2color 2color deleted the autotls-blog branch February 13, 2025 15:13
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.

6 participants