← Back to issue list

cli: parse http-proxy and https-proxy

View original Github issue

Metadata

Project
snapcraft
Number
#3884
Type
pull request
State
merged
Author
mr-cal
Labels
Created
Updated
Closed

Current evaluation

Merged into hotfix/7.1.2. Adds CLI parsing for --http-proxy and --https-proxy to resolve CRAFT-1217. Approved, passed CI with 100% diff coverage, and merged despite a minor unresolved comment regarding core22 proxy support.

Suggested action:

No scores available.

Issue body

- [X] Have you followed the [guidelines for contributing](https://github.com/snapcore/snapcraft/blob/master/CONTRIBUTING.md)? - [X] Have you signed the [CLA](http://www.ubuntu.com/legal/contributors/)? - [X] Have you successfully run `make lint`? - [X] Have you successfully run `pytest tests/unit`? ----- Parse the command line arguments `--http-proxy` and `--https-proxy`. source: https://forum.snapcraft.io/t/error-unrecognized-arguments-http-proxy/30967 (CRAFT-1217)

Evaluation history

Date Model Scores Action Summary
qwen/qwen3.6-35b-a3b Merged into hotfix/7.1.2. Adds CLI parsing for --http-proxy and --https-proxy to resolve CRAFT-1217. Approved, passed CI with 100% diff coverage, and merged despite a minor unresolved comment regarding core22 proxy support.
qwen3.6-35b-a3b-mtp-q6 Merged into hotfix/7.1.2. Adds CLI argument parsing for --http-proxy and --https-proxy to enable HTTP proxy pass-through. Achieved 100% test coverage. Minor discussion regarding core22 proxy support did not block the merge.
qwen3.6-35b-a3b-mtp-q6 Merged into hotfix/7.1.2, implementing CLI parsing for --http-proxy and --https-proxy to enable proxy passthrough. Resolves CRAFT-1217.

Update history

No update history recorded yet.

Related issues

Issue Project State Summary Similarity
#3906 providers: pass http-proxy and https-proxy to craft-providers snapcraft merged Merged. Passes http_proxy and https_proxy environment variables to craft-providers, enabling proxy support in core22. Approved by reviewer, passed CI, and increased code coverage.
70%