Skip to content

Commit

Permalink
🐛 strip mvn:// prefix. This is a short-term workaround. (#98)
Browse files Browse the repository at this point in the history
Short-term workaround.
The UI merged a PR to prefixes the Application.Binary with "mvn://".
This PR is a fall-back until #97 can be merged which requires an
analyzer patch to fix the builtin provider.

Signed-off-by: Jeff Ortel <jortel@redhat.com>
  • Loading branch information
jortel committed Jun 20, 2024
1 parent 78acf14 commit bda9527
Showing 1 changed file with 2 additions and 1 deletion.
3 changes: 2 additions & 1 deletion cmd/mode.go
Original file line number Diff line number Diff line change
Expand Up @@ -115,7 +115,8 @@ func (r *Mode) getArtifact() (err error) {

// mavenArtifact get maven artifact.
func (r *Mode) mavenArtifact(application *api.Application, maven *repository.Maven) (err error) {
err = maven.FetchArtifact(application.Binary)
artifact := strings.TrimPrefix(application.Binary, "mvn://")
err = maven.FetchArtifact(artifact)
if err != nil {
return
}
Expand Down

0 comments on commit bda9527

Please sign in to comment.