Skip to content

Client auth#119

Closed
enjeck wants to merge 6 commits intotorproject:masterfrom
enjeck:client-auth1
Closed

Client auth#119
enjeck wants to merge 6 commits intotorproject:masterfrom
enjeck:client-auth1

Conversation

@enjeck
Copy link
Copy Markdown
Contributor

@enjeck enjeck commented Mar 5, 2020

@emmapeel2
Copy link
Copy Markdown
Contributor

emmapeel2 commented Mar 13, 2020

The contents are very good!

But could you please name the folders as the slug of the article? so the URL makes more sense, and is better for CEO, and for working etc...

@enjeck
Copy link
Copy Markdown
Contributor Author

enjeck commented Apr 2, 2020

@emmapeel2 Edited :) I was worried about the slugs being too long

---
description:
If you see this, then you're trying to access an authenticated onion. An authenticated onion service is an onion service that requires you to provide an authentication token (in this case, a private key) before accessing the service.
[Learn more about onion authentication.](https://tb-manual.torproject.org/onion-services/#client-auth)
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

The way this paragraph reads, people might be confused that the private key is provided (i.e. transmitted) to the service. Perhaps it's worth clarifying in a sentence that "The private key is not transmitted to the service, and it's only used to decrypt its descriptor locally", or a variant of this.

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.

Fixed!

@gusgustavo
Copy link
Copy Markdown
Contributor

I merged all questions to just one, so it's easy to an user to click and learn. Thanks Cleo!

@gusgustavo gusgustavo closed this Jun 2, 2020
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