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

Configure git to use https instead of ssh. #799

Merged
merged 1 commit into from
Oct 25, 2019
Merged

Configure git to use https instead of ssh. #799

merged 1 commit into from
Oct 25, 2019

Conversation

andrewjeffree
Copy link

Pretty straightforward. Means that when write-git-creds is used it'll work for ssh and https sources without any changes needed. This was discussed in #711

@codecov
Copy link

codecov bot commented Oct 3, 2019

Codecov Report

Merging #799 into master will decrease coverage by 0.01%.
The diff coverage is 50%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #799      +/-   ##
==========================================
- Coverage   72.77%   72.75%   -0.02%     
==========================================
  Files          63       63              
  Lines        4812     4816       +4     
==========================================
+ Hits         3502     3504       +2     
- Misses       1051     1052       +1     
- Partials      259      260       +1
Impacted Files Coverage Δ
cmd/server.go 79.42% <ø> (ø) ⬆️
server/events/git_cred_writer.go 83.33% <50%> (-6.67%) ⬇️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update ecda302...a9c843d. Read the comment docs.

Copy link
Member

@lkysow lkysow left a comment

Choose a reason for hiding this comment

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

🎉

Copy link
Member

@lkysow lkysow left a comment

Choose a reason for hiding this comment

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

Oops, one thing I realized we missed. We should update the docs for the flag:

      Write out a .git-credentials file with the provider user and token to allow
      authentication with git over HTTPS. This does write secrets to disk and should
      only be enabled in a secure environment.

to

      Write out a .git-credentials file with the provider user and token to allow
      cloning private modules over HTTPS or SSH. This does write secrets to disk and should
      only be enabled in a secure environment.

@andrewjeffree
Copy link
Author

Ah yep. I've just updated the documentation. So should be all good to go now.

@andrewjeffree
Copy link
Author

@lkysow it won't let me mark your requested changes as fixed/completed for some reason.

@lkysow
Copy link
Member

lkysow commented Oct 24, 2019

@andrewjeffree
Copy link
Author

@lkysow done :)

@lkysow lkysow merged commit 27b9977 into runatlantis:master Oct 25, 2019
@lkysow
Copy link
Member

lkysow commented Oct 25, 2019

🎉 awesome work! I'll try to get a release out soon. :D

@Jenishk56
Copy link

Jenishk56 commented Aug 10, 2021

Hello there,

Big confused, Do i need to add .git-credentials file when use write-git-creds ?

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.

5 participants