-
Notifications
You must be signed in to change notification settings - Fork 10
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
reporter api #137
Merged
MaartendeKruijf
merged 40 commits into
development
from
feature/136-reporter-api-definition
May 15, 2024
Merged
reporter api #137
Changes from 4 commits
Commits
Show all changes
40 commits
Select commit
Hold shift + click to select a range
233191e
add first reporter api documentation
lucamrgs 0baa488
create api reporter instantiation and links. must test
lucamrgs 0c8f3b3
add model for api response
lucamrgs b2c9373
add parsing from cache entry to reporter api results
lucamrgs 607dcb8
enroll soarca-level cache object in both api and reporting
lucamrgs de79172
add template report api testing
lucamrgs 73467f8
rename report model to cache model
lucamrgs 4b0a21a
add get executions base test
lucamrgs 35ad402
add reporter api testing
lucamrgs 1610440
split unittest and integration test on reporter api
lucamrgs efe5afd
trying to solve lint error cannot parse response comment
lucamrgs df5e1e2
solving lint error due to scheme specification
lucamrgs 6d25758
remove schemes specification
lucamrgs 9ef4e67
remove scheme from success specification
lucamrgs b650e83
add right scheme to response
lucamrgs de82654
fix lint error check in controller and recorder body bytes in tests
lucamrgs 824b88c
change reporter documentation according to review
lucamrgs 7f58da3
change get executions to return full executions information
lucamrgs 9e3bbc0
fix commented test check
lucamrgs b27bd8b
remove unused function for cache
lucamrgs e8266ed
inject cache size from controller
lucamrgs b14a27f
add comment ref to cyentific workflow status repo in reporter api model
lucamrgs 3d38c23
move reporter api init setup to reporter api file
lucamrgs bda81e2
use error soarca package in reporter api
lucamrgs b92772a
change mustparse to parse in reporter api
lucamrgs 6af48b8
change default request interval in reporter api to const var
lucamrgs b89e7c3
fix playbook error var use in reporter api
lucamrgs c2da441
change uuid parse to mustparse in cache tests
lucamrgs bc7aa3a
separate mock cache from mock downstram reporter
lucamrgs bb688c1
add file mock cache
lucamrgs 0d16403
remove unused fcns from cache mock
lucamrgs fee2f3a
move report api dynamic tests to unittest
lucamrgs e0e5f33
align reporter api model with cyentific workflow status model
lucamrgs c3cfbcf
implement status text
lucamrgs 8d6197e
fix lint error check
lucamrgs 867f444
add command types to cacao model
lucamrgs 841c51a
change reporter api resutls model
lucamrgs 9990771
remove deep copy of cache entries now passed by reference to api
lucamrgs 4c0c736
fix lint bool comparison complaint
lucamrgs 98b95c1
change reporting status int to status type in tests
lucamrgs File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Creating an empty object and then setting according to the its status might make more sense you won't need so many temp variables