Skip to content

feat(config): add TLS and proxy configuration modules#823

Merged
yeisonvargasf merged 1 commit intomainfrom
feat/tls-proxy-config-modules
Nov 27, 2025
Merged

feat(config): add TLS and proxy configuration modules#823
yeisonvargasf merged 1 commit intomainfrom
feat/tls-proxy-config-modules

Conversation

@yeisonvargasf
Copy link
Member

@yeisonvargasf yeisonvargasf commented Nov 27, 2025

  • Add TLS Config with SAFETY_CA_BUNDLE and Truststore support.
  • Refactor proxy logic into a dedicated module.

@yeisonvargasf yeisonvargasf self-assigned this Nov 27, 2025
@yeisonvargasf yeisonvargasf added the enhancement Suggests an improvement or new feature. label Nov 27, 2025
@coderabbitai
Copy link

coderabbitai bot commented Nov 27, 2025 •
edited
Loading

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/tls-proxy-config-modules

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

@yeisonvargasf yeisonvargasf force-pushed the feat/tls-proxy-config-modules branch from 70bb07f to bd886c2 Compare November 27, 2025 23:15
@safety-bot
Copy link
Contributor

safety-bot commented Nov 27, 2025 •
edited
Loading

🚀 Artifacts — PR #823 by @yeisonvargasf

Security notice: You are viewing pre-release CI artifacts from PR #823 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 #823 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 19749860462 -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.

@yeisonvargasf yeisonvargasf force-pushed the feat/tls-proxy-config-modules branch from bd886c2 to 3435d4a Compare November 27, 2025 23:21
@yeisonvargasf
feat(config): add TLS and proxy configuration modules
ede1b38 
- Add TLS Config with SAFETY_CA_BUNDLE and Truststore support.
- Refactor proxy logic into a dedicated module.
@yeisonvargasf yeisonvargasf force-pushed the feat/tls-proxy-config-modules branch from 3435d4a to ede1b38 Compare November 27, 2025 23:25
@yeisonvargasf yeisonvargasf merged commit 6a4515d into main Nov 27, 2025
39 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@felipekamakura felipekamakura felipekamakura approved these changes

Assignees

@yeisonvargasf yeisonvargasf

Labels

enhancement Suggests an improvement or new feature.

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@yeisonvargasf @safety-bot @felipekamakura