Skip to content

fix: allow curl fetcher to properly fallback to next mirror#15

Merged
Zeno-sole merged 1 commit intodeepin-community:masterfrom
myml:fix-mirror
Dec 8, 2025
Merged

fix: allow curl fetcher to properly fallback to next mirror#15
Zeno-sole merged 1 commit intodeepin-community:masterfrom
myml:fix-mirror

Conversation

@myml
Copy link
Copy Markdown
Contributor

@myml myml commented Nov 19, 2025

The current ostree curl fetcher has two issues that prevent proper fallback when using a mirrorlist:

  1. The max_size check does not account for non-2xx responses. When a server returns 3xx, 4xx, or 5xx, the response body may exceed max_size.
  2. Some state variables and temporary files are not reset when falling back to the next mirror.

@myml
Copy link
Copy Markdown
Contributor Author

myml commented Nov 19, 2025

upstream pull request ostreedev/ostree#3546

@github-actions
Copy link
Copy Markdown

github-actions Bot commented Dec 8, 2025

TAG Bot

TAG: 2024.6-1deepin11
EXISTED: no
DISTRIBUTION: unstable

Comment thread debian/patches/deepin-fix-curl-mirror.patch Outdated
Comment thread debian/patches/deepin-fix-curl-mirror.patch
Comment thread debian/patches/deepin-fix-curl-mirror.patch Outdated
The current ostree curl fetcher has two issues that prevent proper fallback when using a mirrorlist:
1. The max_size check does not account for non-2xx responses. When a server returns 3xx, 4xx, or 5xx, the response body may exceed max_size.
2. Some state variables and temporary files are not reset when falling back to the next mirror.
@deepin-ci-robot
Copy link
Copy Markdown
Contributor

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by: zhaohuiw42
Once this PR has been reviewed and has the lgtm label, please assign yukarichiba for approval. For more information see the Code Review Process.

The full list of commands accepted by this bot can be found here.

Details Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@Zeno-sole Zeno-sole merged commit eff97bc into deepin-community:master Dec 8, 2025
5 of 7 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.

4 participants