Skip to content

Move owid-rust to the commit that refuses redirects when fetching a key - #37

Merged
jwrosewell merged 1 commit into
mainfrom
chore/owid-refuse-redirects
Sep 6, 2026
Merged

Move owid-rust to the commit that refuses redirects when fetching a key#37
jwrosewell merged 1 commit into
mainfrom
chore/owid-refuse-redirects

Conversation

@jwrosewell

@jwrosewell jwrosewell commented Sep 6, 2026

Copy link
Copy Markdown
Contributor

Moves the owid-rust submodule from 8aa5eee8 to ed63d58c, the 51Degrees fork main after syncing SWAN-community/owid-rust#7 (no redirects, status checked before the body is read) plus #8 and #9, the rustfmt and clippy tidies of it. Without #7 a creator whose domain answered 302 could have some other host's key trusted as its own. The build time copy into fodid/src/owid picks the new source up unchanged. fodid tests against the refreshed copy: 81 passed. Produced with AI assistance and needs human review.

@jwrosewell
jwrosewell force-pushed the chore/owid-refuse-redirects branch from 7374505 to 5359943 Compare September 6, 2026 15:56
…ing a key

The 51Degrees fork of owid-rust now carries SWAN-community/owid-rust#7,
which builds the ureq agent with no redirects and checks the status before
reading the body, so a creator whose domain answered 302 could no longer
have some other host's key trusted as its own, nor be reported as serving
an unreadable key when it served none, plus #8 and #9, the rustfmt and
clippy tidies of that change. The pin moves from 8aa5eee8 to ed63d58c, the
fork's main after the sync. The copy into fodid/src/owid that
ci/copy-owid-source.ps1 makes picks the new source up unchanged.

fodid tests, run against the refreshed copy: 81 passed.
@jwrosewell
jwrosewell force-pushed the chore/owid-refuse-redirects branch from 5359943 to 1c78345 Compare September 6, 2026 16:00
@jwrosewell
jwrosewell merged commit 93ad350 into main Sep 6, 2026
9 checks passed
@jwrosewell
jwrosewell deleted the chore/owid-refuse-redirects branch September 6, 2026 16: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.

1 participant