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

Update artifact file download #138

Merged
merged 1 commit into from
Nov 2, 2022

Conversation

sanfern
Copy link
Contributor

@sanfern sanfern commented Nov 1, 2022

Signed-off-by: sanfern [email protected]

kf/bpf.go Outdated
@@ -548,17 +553,11 @@ func (b *BPF) GetArtifacts(conf *config.Config) error {
}
}

URL.Path = path.Join(URL.Path, b.Program.Name, b.Program.Version, platform, b.Program.Artifact)
log.Info().Msgf("Downloading - %s", URL)
Copy link
Contributor

Choose a reason for hiding this comment

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

This is probably only relevant for http(s)

@sanfern sanfern force-pushed the sanfern-dev-file-download branch from de953b2 to e80cfe5 Compare November 1, 2022 19:30
Copy link
Contributor

@jniesz jniesz left a comment

Choose a reason for hiding this comment

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

LGTM

Copy link

@charleskbliu0 charleskbliu0 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

Copy link
Contributor

@jaysheth2 jaysheth2 left a comment

Choose a reason for hiding this comment

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

LGTM

@sanfern sanfern merged commit 1ae31b6 into l3af-project:main Nov 2, 2022
@sanfern sanfern deleted the sanfern-dev-file-download branch November 2, 2022 05:20
sanfern added a commit to sanfern/l3afd that referenced this pull request Mar 6, 2023
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.

4 participants