Skip to content

[pull] master from josegonzalez:master#98

Merged
pull[bot] merged 3 commits intodumpmemory:masterfrom
josegonzalez:master
Feb 16, 2026
Merged

[pull] master from josegonzalez:master#98
pull[bot] merged 3 commits intodumpmemory:masterfrom
josegonzalez:master

Conversation

@pull
Copy link
Copy Markdown

@pull pull bot commented Feb 16, 2026

See Commits and Changes for more details.


Created by pull[bot] (v2.0.0-alpha.4)

Can you help keep this open source service alive? 💖 Please sponsor : )

Iamrodos and others added 3 commits February 16, 2026 10:16
The DMCA handling added in PR #454 had a bug: make_request_with_retry()
raises HTTPError before retrieve_data() could check the status code via
getcode(), making the case 451 handler dead code. This also affected
HTTP 403 TOS violations (e.g. jumoog/MagiskOnWSA).

Fix by catching HTTPError in retrieve_data() and converting 451 and
blocked 403 responses (identified by "block" key in response body) to
RepositoryUnavailableError. Non-block 403s (permissions, scopes) still
propagate as HTTPError. Also handle RepositoryUnavailableError in
retrieve_repositories() for the --repository case.

Rewrote tests to mock urlopen (not make_request_with_retry) to exercise
the real code path that was previously untested.

Closes #487
Fix HTTP 451 DMCA and 403 TOS handling regression (#487)
@pull pull bot locked and limited conversation to collaborators Feb 16, 2026
@pull pull bot added the ⤵️ pull label Feb 16, 2026
@pull pull bot merged commit 6006765 into dumpmemory:master Feb 16, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants