Skip to content

Commit dc06443

Browse files
committed
Merge branch 'mr/cardao/fix-maven-download-call' into 'master'
Fix Maven download URL and improve logs See merge request it/e3-core!115
2 parents 6837150 + 7932d1a commit dc06443

File tree

1 file changed

+5
-2
lines changed

1 file changed

+5
-2
lines changed

src/e3/maven.py

Lines changed: 5 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -21,7 +21,10 @@ def __init__(self, group: str, name: str, version: str) -> None:
2121
self.package_group = group
2222
self.package_name = name
2323
self.version = version
24-
self.url = f"https://repo1.maven.org/maven2/{group}/{name}/{version}/{filename}"
24+
self.url = (
25+
f"https://repo1.maven.org/maven2/{group.replace('.', '/')}/{name}/{version}"
26+
f"/{filename}"
27+
)
2528

2629
# To get the expected checksum of the current file, we need to make an
2730
# additonnal HEAD request. This is because maven send the checksum directly on
@@ -32,7 +35,7 @@ def __init__(self, group: str, name: str, version: str) -> None:
3235
sha1_checksum = hdrs.get("x-checksum-sha1")
3336
md5_checksum = hdrs.get("x-checksum-md5")
3437
if not md5_checksum and not sha1_checksum:
35-
raise RuntimeError("No checksum provided")
38+
raise RuntimeError(f"No checksum provided for {group}/{self.filename}")
3639

3740
# No 'elif' because maven can send both together into HTTP headers.
3841
if md5_checksum:

0 commit comments

Comments
 (0)