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 HTTPS + PATs in advanced SSH section #934

Merged
merged 2 commits into from
Jul 27, 2023

Conversation

Aariq
Copy link

@Aariq Aariq commented Jul 17, 2023

I fleshed out the section on HTTPS and PATs a bit. Something I didn't do that would probably be nice is add screenshots from GitHub. I could add some, but I thought it would be better for continuity if they matched the user and repo name from earlier in the lesson. I changed the bit about PATs being unique to GitHub in the beginning because they aren't. Bitbucket also has an option to use HTTPS + PAT and you'll find plenty of other non-git services that use PATs for authentication. Hopefully I've explained that succinctly in the language I added.

@Aariq Aariq changed the title fleshed out section on PATs Add HTTPS + PATs in advanced SSH section Jul 17, 2023
@Aariq
Copy link
Author

Aariq commented Jul 17, 2023

btw, this is in response to this comment: #895 (comment)

@@ -214,7 +264,9 @@ git config --global --unset credential.helper
FIXME: Windows 10: Credential Manager
FIXME: remove git credential.
Copy link
Author

Choose a reason for hiding this comment

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

I wasn't exactly sure what the existing notes meant, so I just left them for now. Happy to delete or flesh out if someone can provide a little more detail

Copy link
Contributor

@kekoziar kekoziar left a comment

Choose a reason for hiding this comment

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

Looks good. Thanks for updating and expanding this supplemental episode.

@kekoziar
Copy link
Contributor

@Aariq Thanks for expanding this episode. I have a couple of questions re the use of PATs. When GitHub first required the increased security, I could only find the term "Personal Access Token" in relation to Microsoft products (GitHub, Azure, Microsoft, but mostly GitHub). Now, the terminology seems to have migrated to other git/version control/cloud collaboration services (Atlassian, GitLab, and some smaller providers). Have you seen PATs in widespread use? Are their implementation in other services the same as in GitHub?

Thanks!

@Aariq
Copy link
Author

Aariq commented Jul 26, 2023

I have not seen the term "Personal Access Token" / PAT in widespread use, but the concept and terminology is closely related to other kinds of access tokens that are widely used for authentication via HTTPS—e.g. API tokens, bearer tokens, etc. To be honest, it's not clear to me how exactly PATs differ from any other kind of access token. From a user standpoint, they seem to function about the same and the skills gained by learning to use GitHub PATs are very transferrable, IMO. Even if "Personal Access Tokens" are not commonly used, the use of access tokens to authenticate via HTTPS is quite common (e.g. for some data APIs) and is a useful skill for researchers.

@Aariq
Copy link
Author

Aariq commented Jul 26, 2023

GitLab appears to have many more types of tokens including Personal Access Tokens, Group Access Tokens, Project Access Tokens, etc. But it doesn't actually look like they are used for authenticating with git. Bitbucket appears to use PATs the same way as GitHub—you can use them when prompted for a password by git

@kekoziar
Copy link
Contributor

@Aariq What do you think about changing the What is a PAT? section to add something along the lines of

Personal Access Tokens is the term GitHub uses to implement an access token, which is a common method to authenticate access via HTTPS

?

@Aariq
Copy link
Author

Aariq commented Jul 27, 2023

Yeah, that makes sense to me. How's this look now?

@kekoziar
Copy link
Contributor

Yeah, that makes sense to me. How's this look now?

Looks good. Thanks!

@kekoziar kekoziar merged commit 7fcaa28 into swcarpentry:ssh-adv Jul 27, 2023
1 of 2 checks passed
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.

2 participants