Thanks to visit codestin.com
Credit goes to github.com

Skip to content

feat: [WARNING: behavior change] migrate to httpx with explicit TLS/proxy configuration#824

Merged
yeisonvargasf merged 1 commit intomainfrom
feat/new-tls-proxy-configs
Dec 2, 2025
Merged

feat: [WARNING: behavior change] migrate to httpx with explicit TLS/proxy configuration#824
yeisonvargasf merged 1 commit intomainfrom
feat/new-tls-proxy-configs

Conversation

@yeisonvargasf
Copy link
Member

Migrate HTTP client from requests to httpx with consolidated configuration. Remove unofficial support for requests/httpx environment variables in favor of explicit Safety CLI configuration options.

  • Migrate HTTP client from requests to httpx
  • Remove proxy_timeout and proxy_required options (deprecated workarounds)
  • Add deprecation warnings for removed proxy parameters
  • Consolidate TLS/proxy configuration into explicit config modules
  • Add new SafetyPlatformClient with unified HTTP handling
  • Stop reading unofficial REQUESTS_* and HTTPX_* environment variables

Note: Users relying on unofficial environment variables or importing Safety CLI as a library may experience connection failures and should migrate to official configuration methods.

…roxy configuration

Migrate HTTP client from requests to httpx with consolidated configuration.
Remove unofficial support for requests/httpx environment variables in favor
of explicit Safety CLI configuration options.

- Migrate HTTP client from requests to httpx
- Remove proxy_timeout and proxy_required options (deprecated workarounds)
- Add deprecation warnings for removed proxy parameters
- Consolidate TLS/proxy configuration into explicit config modules
- Add new SafetyPlatformClient with unified HTTP handling
- Stop reading unofficial REQUESTS_* and HTTPX_* environment variables

Note: Users relying on unofficial environment variables or importing Safety CLI
as a library may experience connection failures and should migrate to official
configuration methods.
@yeisonvargasf yeisonvargasf self-assigned this Dec 2, 2025
@yeisonvargasf yeisonvargasf added enhancement Suggests an improvement or new feature. Critical Issues that break core functionality or pose significant security vulnerabilities. labels Dec 2, 2025
@coderabbitai
Copy link

coderabbitai bot commented Dec 2, 2025

Important

Review skipped

Auto reviews are disabled on this repository.

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

✨ Finishing touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch feat/new-tls-proxy-configs

Comment @coderabbitai help to get the list of available commands and usage tips.

@yeisonvargasf yeisonvargasf removed the Critical Issues that break core functionality or pose significant security vulnerabilities. label Dec 2, 2025
@safety-bot
Copy link
Contributor

🚀 Artifacts — PR #824 by @yeisonvargasf

Security notice: You are viewing pre-release CI artifacts from PR #824 by @yeisonvargasf. These commands may execute code on your machine. Do NOT run them unless you have reviewed the PR diff and trust the source. The snippets include a confirmation prompt.

Download the wheel file and binaries with gh CLI or from the workflow artifacts.

📦 Install & Run

Pre-requisites

# Install uv if needed
curl -LsSf https://astral.sh/uv/install.sh | sh

# Create and enter artifacts directory
mkdir artifacts && cd artifacts

Quick Test with Python Package

bash -c 'set -euo pipefail; echo; echo "WARNING: You are about to download and execute CI artifacts from PR #824 by @yeisonvargasf. Do NOT proceed unless you have reviewed the PR diff and trust the source."; echo; read -rp "Type I understand to continue: " C; [ "$C" = "I understand" ] || { echo "Aborted."; exit 1; }; gh run download 19871521456 -n dist -R pyupio/safety; uvx safety-*-py3-none-any.whl --version'

Run other Safety commands as follows

uvx safety-*-py3-none-any.whl auth status
uvx safety-*-py3-none-any.whl auth login
uvx safety-*-py3-none-any.whl scan

Note: You need to be logged in to GitHub to access the artifacts.

@WanderingLuca WanderingLuca self-requested a review December 2, 2025 20:09
@yeisonvargasf yeisonvargasf merged commit c561027 into main Dec 2, 2025
39 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement Suggests an improvement or new feature.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants