Add new blog for password authentication#317
Conversation
pratyakshsharma
force-pushed
the
authentication-blog
branch
from
3729fcf to
ad0f59f
Compare
pratyakshsharma
force-pushed
the
authentication-blog
branch
from
ad0f59f to
63e5d7f
Compare
|
@steveburnett I have updated the blog based on your suggestions. Please have a look again. |
steveburnett
left a comment
steveburnett
left a comment
There was a problem hiding this comment.
This looks great! I made a lot of comments, mostly to make sentences smoother and easier to read, some minor punctuation and capitalizing, but I found no serious problems, and the structure of the topics flows smoothly.
I really liked your including each problem in the order you encountered them, and how you fixed them. Nice work!
|
|
||
| Authentication and authorization are two main pillars for data security. While it is ok to have a presto cluster running | ||
| without authentication for development purposes, production clusters need to be secured at all times. Setting up secure clusters | ||
| comes with its own challenges in terms of the involved setup, configuration changes. In this blog, we try to set up a secure presto |
There was a problem hiding this comment.
| comes with its own challenges in terms of the involved setup, configuration changes. In this blog, we try to set up a secure presto | |
| comes with its own challenges in terms of the involved setup and configuration changes. In this blog, we try to set up a secure Presto |
I'll try to comment on any use of Presto (in sentences, not in code) that should be capitalized but I might miss one, so if you could help by checking this yourself in your revision I would appreciate it. Thanks!
There was a problem hiding this comment.
sure thing, will take care.
pratyakshsharma
force-pushed
the
authentication-blog
branch
2 times, most recently
from
684d2a6 to
5fc973a
Compare
steveburnett
left a comment
steveburnett
left a comment
There was a problem hiding this comment.
Thanks for the update! I did another beginning-to-end review and found a few more nits of phrasing.
pratyakshsharma
force-pushed
the
authentication-blog
branch
from
5fc973a to
70bca07
Compare
|
@steveburnett please have a look again. |
steveburnett
left a comment
steveburnett
left a comment
There was a problem hiding this comment.
LGTM! (docs)
Thanks for the revisions. Looks good!
2 participants
Current presto docs around setting up password authentication is not very helpful. There are good amount of chances an end-user will end up facing some error or the other when trying to set up on local. This blog provides a hands-on guide for users for setting up password-file based authentication on local, incrementally solving different errors as they are encountered.
Additional context - prestodb/presto#27485 (comment)