Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix: Disregard curl configuration #1571

Merged
merged 1 commit into from
Jan 29, 2025

Conversation

lj3954
Copy link
Member

@lj3954 lj3954 commented Jan 28, 2025

Description

Quickget relies upon specific parameters passed to curl in order to find or download various operating systems. User defined configuration could break this, such as by sending undesired headers or cookies, which could cause requests to be blocked, or as in the case of #1570, to set output flags which cause curl to attempt to download files to invalid or incorrect directories. This issue can be resolved by disregarding the curl configuration file for any and all invocations within quickget, achieved by passing the --disable command line flag.

Type of change

  • Bug fix (non-breaking change which fixes an issue)

Checklist:

  • I have performed a self-review of my code
  • I have tested my code in common scenarios and confirmed there are no regressions

TuxVinyards added a commit to TuxVinyards/qqX that referenced this pull request Jan 28, 2025
ensure consistent download targets

quickemu-project/quickemu#1571
TuxVinyards added a commit to TuxVinyards/qqX that referenced this pull request Jan 29, 2025
ensure consistent download targets

aligns with changes to 1.11.05 and upstream

f7ac8ad

quickemu-project/quickemu#1571
@philclifford philclifford merged commit eb0fd03 into quickemu-project:master Jan 29, 2025
6 checks passed
@lj3954 lj3954 deleted the curl_configuration branch January 29, 2025 15:22
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.

bug: Quickget failed to download with curl
2 participants