Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

gitrepo: bump git packages #959

Merged
merged 1 commit into from
Nov 16, 2022
Merged

gitrepo: bump git packages #959

merged 1 commit into from
Nov 16, 2022

Conversation

aryan9600
Copy link
Member

  • Bump fluxcd/pkg/git to v0.7.0
  • Bump fluxcd/pkg/git/gogit to v0.2.0
  • Bump fluxcd/pkg/git/libgit2 to v0.3.0

Signed-off-by: Sanskar Jaiswal jaiswalsanskar078@gmail.com

Signed-off-by: Sanskar Jaiswal <jaiswalsanskar078@gmail.com>
gitReader, err = gogit.NewClient(dir, authOpts)
clientOpts := []gogit.ClientOption{gogit.WithDiskStorage()}
if authOpts.Transport == git.HTTP {
clientOpts = append(clientOpts, gogit.WithInsecureCredentialsOverHTTP())
Copy link
Member

Choose a reason for hiding this comment

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

As a follow-up PR, the use of WithInsecureCredentialsOverHTTP() needs to be guarded by the --insecure-allow-http flag, as per RFC0004.

Copy link
Member

@pjbgf pjbgf left a comment

Choose a reason for hiding this comment

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

LGTM

thanks @aryan9600! 🙇

@pjbgf pjbgf added the area/ci CI related issues and pull requests label Nov 16, 2022
@pjbgf pjbgf merged commit 16fed89 into fluxcd:main Nov 16, 2022
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
area/ci CI related issues and pull requests
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants