Document how to use GH in the ring with 2FA.#115
Open
danielballan wants to merge 6 commits intoNSLS-II:masterfrom
Open
Document how to use GH in the ring with 2FA.#115danielballan wants to merge 6 commits intoNSLS-II:masterfrom
danielballan wants to merge 6 commits intoNSLS-II:masterfrom
Conversation
0b4b297 to
50e09be
Compare
mrakitin
reviewed
Jul 5, 2019
Member
mrakitin
left a comment
There was a problem hiding this comment.
I suggested some improvements of the text to avoid a possible leak of the token in a plain text form.
Revisions from @mrakitin Co-Authored-By: Maksim Rakitin <mrakitin@users.noreply.github.com>
mrakitin
reviewed
Jul 5, 2019
Member
mrakitin
left a comment
There was a problem hiding this comment.
Here is a couple of corrections. Otherwise looks good.
leofang
reviewed
Jul 5, 2019
|
|
||
| $ git remote set-url <NAME> <NEW_URL> | ||
|
|
||
| For use inside the ring, connecting via SSH does not work. (Consult ITC to ask |
Collaborator
There was a problem hiding this comment.
Not sure if this is intended?
Suggested change
| For use inside the ring, connecting via SSH does not work. (Consult ITC to ask | |
| For users inside the ring, connecting via SSH does not work. (Consult ITC to ask |
Contributor
Author
There was a problem hiding this comment.
Fair question. I meant "use". Maybe it could be phrase more clearly?
Member
There was a problem hiding this comment.
No, don't put users inside the ring, it's not safe there 🤣.
Maybe we can better say "For use in the controls network ..."?
leofang
reviewed
Jul 5, 2019
Member
|
In source/components/github.rst:
...
- For use inside the ring, connecting via SSH does not work. (Consult ITC to ask
- why.) Your best option is to use a personal access token, which you can do at
+ For use inside the ring:
+ * put the following in ~/.ssh/config:
+ Host github.com
+ ProxyCommand ssh -W %h:%p ssh01
+ User [your-github-username] (this line is an optional convenience)
+ * or set the GIT_SSH env var to ``'ssh -o "ProxyCommand ssh -W %h:%p ssh01"'`` in
+ your shell setup (eg. for bash, ``export GIT_SSH='ssh -o "ProxyCommand ssh -W %h:%p ssh01"'`` in your .bashrc)
+ * or just ``GIT_SSH='ssh -o "ProxyCommand ssh -W %h:%p ssh01"' git [whatever]``
+ * or you could use a personal access token, which you can do at
…-matt
|
Member
Won't this push all ssh traffic, including to other machines on the floor through the proxy? |
Member
|
doh! yup, scrub that one.
…On Sat, 6 Jul 2019, Thomas A Caswell wrote:
export GIT_SSH='ssh -o "ProxyCommand ssh -W %h:%p ssh01
Won't this push all ssh traffic, including to other machines on the floor through the proxy?
?
You are receiving this because you commented.
Reply to this email directly, view it on GitHub, or mute thethread.[ABJZ7Y6FOUIEPFG4YNJJMFLP572J7A5CNFSM4H6NNUFKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVX
HJKTDN5WW2ZLOORPWSZGODZKQKFI.gif]
|
Co-Authored-By: Maksim Rakitin <mrakitin@users.noreply.github.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
No description provided.