Skip to content

fix(pypi-mapping): fetch parselmouth compressed mapping from conda-mapping.prefix.dev#6172

Merged
ruben-arts merged 3 commits into
prefix-dev:mainfrom
nichmor:fix/compressed-mapping-r2-url
May 22, 2026
Merged

fix(pypi-mapping): fetch parselmouth compressed mapping from conda-mapping.prefix.dev#6172
ruben-arts merged 3 commits into
prefix-dev:mainfrom
nichmor:fix/compressed-mapping-r2-url

Conversation

@nichmor
Copy link
Copy Markdown
Contributor

@nichmor nichmor commented May 21, 2026

Description

Fixes: #5980

Now we will fetch the mapping from our official CDN instead of github.

How Has This Been Tested?

Running a pixi lock file on a project with pypi deps

Checklist:

  • I have performed a self-review of my own code
  • I have made corresponding changes to the documentation (crate-level docs updated)
  • Snapshot tests updated to reflect new behavior

…pping.prefix.dev

The previous URL pointed at `raw.githubusercontent.com`, which is
blocked on some corporate networks (prefix-dev#5980). Parselmouth now publishes
the same file to its existing public R2 bucket at
`https://conda-mapping.prefix.dev/compressed-v0/compressed_mapping.json`,
so we can fetch from the trusted `prefix.dev` domain instead.

The Git-committed file in the parselmouth repo is unchanged, so the
old URL keeps working for now; this PR only swaps the default that
pixi ships with.

Closes prefix-dev#5980.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
@nichmor nichmor marked this pull request as draft May 21, 2026 09:40
@nichmor nichmor marked this pull request as ready for review May 22, 2026 12:41
@ruben-arts
Copy link
Copy Markdown
Contributor

Would it make sense to support both domains? And fall back to the one that is accessible?

@nichmor
Copy link
Copy Markdown
Contributor Author

nichmor commented May 22, 2026

Would it make sense to support both domains? And fall back to the one that is accessible?

tbh I'm not convinced to support both of them, given the fact that githubusercontent can be inaccessible for some of our users. I see that Cloudflare promises 99.9% monthly uptime so I think we should be good?

@ruben-arts
Copy link
Copy Markdown
Contributor

I'm more worried about users that do have access to github but not to prefix.dev. Merging this would break there setup. For opt-in network setup this might be very annoying.

@baszalmstra
Copy link
Copy Markdown
Contributor

Wont that be the case anyway? We currently also only host the sha lookup on prefix.dev.

@ruben-arts
Copy link
Copy Markdown
Contributor

Wont that be the case anyway? We currently also only host the sha lookup on prefix.dev.

Ah I forgot about that 👍

Copy link
Copy Markdown
Contributor

@ruben-arts ruben-arts left a comment

Choose a reason for hiding this comment

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

Thanks

@ruben-arts ruben-arts merged commit 1cd9118 into prefix-dev:main May 22, 2026
38 checks passed
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.

Fetch parcelmouth mapping from published trusted source, rather than raw GitHub source

3 participants