Skip to content

CI: confirm webseed redirects + versioned hashes - #80

Merged
plowsof merged 1 commit into
mainfrom
webseed_ci
Nov 23, 2025
Merged

CI: confirm webseed redirects + versioned hashes#80
plowsof merged 1 commit into
mainfrom
webseed_ci

Conversation

@plowsof

@plowsof plowsof commented Nov 23, 2025

Copy link
Copy Markdown
Owner

No description provided.

@librootorg

Copy link
Copy Markdown
Contributor

I'm not quite sure if I understand this:

      CDN_URL="https://downloads.getmonero.org"
      if ! wget --spider -q "$CDN_URL/hashes-$VERSION"; then

These versioned hashes are created atm in create_monero_torrent.sh, but in the downloads.getmonero.org the versioned hash files don't exist (e.g. https://downloads.getmonero.org/hashes-v0.18.4.4).

@plowsof

plowsof commented Nov 23, 2025

Copy link
Copy Markdown
Owner Author

yes, the create_torrent script downloads "hashes.txt" (no version) from : https://www.getmonero.org/downloads/hashes.txt and adds the version inside the torrent, release maintainer simply forgot to add hashes-v0.18.4.4.txt - this is new as weve never had a versioned release there. he did it for http://downloads.getmonero.org/hashes-v0.18.4.3.txt so this is a good workflow to have @ release

you can see the warnings if you scroll to the bottom of https://github.com/plowsof/monero-torrent/actions/runs/19611368152

@librootorg

librootorg commented Nov 23, 2025

Copy link
Copy Markdown
Contributor

Ooh I see, makes sense! Neat.

Btw isn't the .txt extension is missing?

https://downloads.getmonero.org/hashes-v0.18.4.4
vs
https://downloads.getmonero.org/hashes-v0.18.4.4.txt

if ! wget --spider -q "$CDN_URL/hashes-$VERSION"; then <-- no .txt 

@plowsof

plowsof commented Nov 23, 2025

Copy link
Copy Markdown
Owner Author

Good spot! thats a bug

@plowsof
plowsof merged commit 761738f into main Nov 23, 2025
2 checks passed
@plowsof
plowsof deleted the webseed_ci branch November 23, 2025 22:08
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.

2 participants