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

Fixed the ability to override the API URL for Socket Requests #64

Merged
merged 27 commits into from
Apr 1, 2025

Conversation

dacoburn
Copy link
Collaborator

API URL could not be overridden correctly

Root Cause

API Url could be set via an Environment variable but it wasn't being passed to the underlying SDK

Fix

Passed the captured override to the SDK

Public Changelog

N/A

@dacoburn dacoburn requested a review from a team as a code owner March 25, 2025 18:54
@dacoburn dacoburn requested review from bmeck and alxhotel and removed request for a team March 25, 2025 18:54
@socket-security-staging
Copy link

socket-security-staging bot commented Apr 1, 2025

New and removed dependencies detected. Learn more about Socket for GitHub ↗︎

Package New capabilities Transitives Size Publisher
pypi/[email protected] environment, eval, filesystem, network, shell Transitive: unsafe +709 780 MB Ofekmeister

View full report↗︎

Copy link

socket-security bot commented Apr 1, 2025

New and removed dependencies detected. Learn more about Socket for GitHub ↗︎

Package New capabilities Transitives Size Publisher
pypi/[email protected] environment, eval, filesystem, network, shell Transitive: unsafe +771 1.93 GB Ofekmeister

View full report↗︎

Copy link

github-actions bot commented Apr 1, 2025

🚀 Preview package published!

Install with:

pip install --index-url https://test.pypi.org/simple/ --extra-index-url https://pypi.org/simple socketsecurity==2.0.32.dev1

Docker image: socketdev/cli:pr-64

@dacoburn dacoburn merged commit b17e947 into main Apr 1, 2025
4 checks passed
@dacoburn dacoburn deleted the doug/fix-api-override-url branch April 1, 2025 03:37
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.

3 participants