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

Updated Intel Search sample README #1194

Merged
merged 1 commit into from
Jul 1, 2024
Merged

Conversation

alhumaw
Copy link
Contributor

@alhumaw alhumaw commented Jun 28, 2024

Updated Intel Search sample README

Updated the README to reflect the debugging implementation. Included an example output.

  • Enhancement
  • Documentation
  • Code sample

Unit test coverage

UNIT TESTING NOT REQUIRED FOR SAMPLES

Bandit analysis

[main]	INFO	profile include tests: None
[main]	INFO	profile exclude tests: None
[main]	INFO	cli include tests: None
[main]	INFO	cli exclude tests: None
[main]	INFO	running on Python 3.12.3
Run started:2024-06-28 19:13:02.149532

Test results:
	No issues identified.

Code scanned:
	Total lines of code: 503
	Total lines skipped (#nosec): 0

Run metrics:
	Total issues (by severity):
		Undefined: 0
		Low: 0
		Medium: 0
		High: 0
	Total issues (by confidence):
		Undefined: 0
		Low: 0
		Medium: 0
		High: 0
Files skipped (0):

Added features and functionality

  • Added documentation onto the intel_search.py sample to cover the newest debugging implementation
    • falconpy/samples/intel/README.md

@jshcodes jshcodes assigned jshcodes and alhumaw and unassigned jshcodes Jun 28, 2024
@jshcodes jshcodes added documentation 📖 Improvements or additions to documentation code samples Functioning sample code to get you started labels Jun 28, 2024
@jshcodes jshcodes merged commit 83ceb6f into CrowdStrike:samples Jul 1, 2024
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
code samples Functioning sample code to get you started documentation 📖 Improvements or additions to documentation
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants