Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Sign up
Appearance settings

Added support for polling full scans endpoint #84

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

Merged
dacoburn merged 1 commit into main from doug/move-to-polling-full-scan
Jun 9, 2025

Conversation

@dacoburn
Copy link
Collaborator

@dacoburn dacoburn commented Jun 9, 2025

Improving the CLI behavior to use polling for the Full Scans endpoint. Also switching to always using the diff endpoint with temporary head full scans if one doesn't exist.

Public Changelog

  • Added an improvement to poll for if the full scans are ready before doing the diff endpoint. This is to solve for network configurations that can view long running connections as idle and terminate them.

@dacoburn dacoburn requested a review from a team as a code owner June 9, 2025 22:18
@dacoburn dacoburn requested review from Planeshifter and jghebre and removed request for a team June 9, 2025 22:18
@dacoburn dacoburn added the Product Changelog New features for the public changelog label Jun 9, 2025
Copy link

github-actions bot commented Jun 9, 2025
edited
Loading

🚀 Preview package published!

Install with:

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

Docker image: socketdev/cli:pr-84

@dacoburn dacoburn merged commit 1ad4995 into main Jun 9, 2025
6 of 7 checks passed
@dacoburn dacoburn deleted the doug/move-to-polling-full-scan branch June 9, 2025 22:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@bcomnes bcomnes bcomnes approved these changes

@reberhardt7 reberhardt7 reberhardt7 approved these changes

@Planeshifter Planeshifter Awaiting requested review from Planeshifter Planeshifter is a code owner automatically assigned from SocketDev/eng

@jghebre jghebre Awaiting requested review from jghebre jghebre is a code owner automatically assigned from SocketDev/eng

Assignees

No one assigned

Labels

Product Changelog New features for the public changelog

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

AltStyle によって変換されたページ (->オリジナル) /