-
Notifications
You must be signed in to change notification settings - Fork 13
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
Crowdstrike OAuth API: Feature: Support for Online-State handler - Get the online status for one or more hosts #45
base: next
Are you sure you want to change the base?
Conversation
Thank you for your submission! We have a total of 21 PRs open right now, and we are working hard on all of them! We will take a look as soon as we can. |
I am unsure of what the workfow is for testing, and I can't seem to find a way to install |
@JMeyer0101 Static Tests results for commit 59bb4ee - https://drive.google.com/file/d/1Hh23zgYQLHbDnCj25c_4pM5C9qWAT_zf/view?usp=drivesdk |
@JMeyer0101 Compile Tests results for commit 59bb4ee - https://drive.google.com/file/d/1aEHlooPfMzGZ3seSm7SMu8JQ3pllTPXe/view?usp=drivesdk |
@JMeyer0101 Static Tests results for commit d419a97 - https://drive.google.com/file/d/1buFHCzMfGobmp_vhTKuNxhqlD_t8ENQO/view?usp=drivesdk |
@JMeyer0101 Compile Tests results for commit d419a97 - https://drive.google.com/file/d/1jbRstkP2tSmnEm7jm6RWMNpZdVjX4r7r/view?usp=drivesdk |
Please ensure your pull request (PR) adheres to the following guidelines:
Pull Request Checklist
Please check if your PR fulfills the following requirements:
<App Name>: <PR Type> - <PR Description>
next
branch of the forked repo. Create separate feature branch for raising the PR.Pull Request Type
Please check the type of change your PR introduces:
Security Considerations (REQUIRED)
please document them in the
manual_readme_content.md
.manual_readme_content.md
all methods (eg, OAuth) used to authenticatewith the service that the connector is integrating with.
manual_readme_content.md
.the Vault API.
nosemgrep
?If yes, please provide justification in an additional comment next to the ignored code.
Release Notes (REQUIRED)
What is the current behavior? (OPTIONAL)
There does not currently exist a way to query for
online-state
of a device with the current app.What is the new behavior? (OPTIONAL)
Adds ability to call existing "/devices/entities/online-state/v1" endpoint
Other information (OPTIONAL)
Gathering the online state is required in specific scenarios. ex: crowdstrike by default will reset a quarantine attempt on a machine with online state 'unknown' after ~30min, resulting in infinite looping quarantine attempts via playbook on a machine that is misconfigured. Allowing playbooks to query the online state will prevent the issue.
Pay close attention to (OPTIONAL)
Screenshots (if relevant)