-
Notifications
You must be signed in to change notification settings - Fork 8.3k
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
[SIEM] Final Shimming/Prep for Server NP Migration #56814
Merged
Merged
Changes from all commits
Commits
Show all changes
43 commits
Select commit
Hold shift + click to select a range
38dbce7
Route all our server setup through the plugin
rylnd b8a7000
Clean up server plugin exports
rylnd 3d23690
Move DE Routes to to conventional location
rylnd 2e284c5
Loosen up our RequestFacade
rylnd 7a35eac
Remove uses of 'src' alias
rylnd 5360a87
Fix types of our GraphQL requests
rylnd e41d618
Initialize our routes with SetupServices
rylnd faf8f77
Swap legacy spaces plugin for NP
rylnd 9418447
Remove unneeded typing
rylnd 80c5826
WIP: Converting our DE routes to use consolidated services
rylnd b0a7f93
Fix DE routing tests following refactor
rylnd aa7ae34
Fix incompatible request types
rylnd b54235d
Move our use of encryptedSavedObjects to NP
rylnd 3742670
Use NP elasticsearch client
rylnd 8ccaf4b
Update DE mocks with NP elasticsearch
rylnd 98d56b8
Remove savedObjects dependency from our legacy dependencies
rylnd 00d089d
WIP: Convert services to stateful object
rylnd 35bfc0b
WIP: Converting routes to use Services factory
rylnd 7e7cd30
WIP: Qualifying our Service to ClientsService
rylnd c97f190
Fix types on getIndex function
rylnd 101993f
Our ClientsService is a clients ... service
rylnd f7bb2fb
Clean up our server types
rylnd 2502698
Refactor mocks for new architecture
rylnd 36398bc
Simplify our routing mocks
rylnd 1e6f106
Loosen graphQL endpoint validations
rylnd bcff5e1
Remove unused graphql endpoint
rylnd f84a3d7
Reduce our dependence on PluginInitializerContext
rylnd 930b89a
Fix FrameworkAdapter type
rylnd c3c67f1
Update detections route tests
rylnd 79171ff
Remove unnecessary null checks
rylnd 156dd77
Remove unused type
rylnd e309001
Remove unused savedObjects client
rylnd 0816e62
Handle case where spaces is disabled
rylnd dfd81cd
Return default signals index if spaces are disabled
rylnd 11f3581
Remove unnecessary casting of our alerts client mock
rylnd 0e98dae
Return the 'default' space even when spaces are disabled
rylnd 8dc5622
Merge branch 'master' into final_backend_prep
rylnd 4e9f8d8
Merge branch 'master' into final_backend_prep
rylnd f53e6f7
Account for spaces being disabled in ClientsService
rylnd cb5d729
Merge branch 'master' into final_backend_prep
rylnd 668702b
Merge branch 'master' into final_backend_prep
rylnd 56fc3f5
Fix false positives in query signals routes tests
rylnd 7162528
style: Prefer null coalescing over ternary
rylnd 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 was deleted.
Oops, something went wrong.
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
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.
ouch on having to do ts-ignore-next-line. If we have to, we have to. Just some ouch. Any open tickets or other information about when and how these can be removed or is this is just temporary for a shim that will be altogether removed once we are out of the legacy folder?
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.
This is specific to the shim; this entire file will be removed on new platform.
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.
A few of these (CoreStart, CoreSetup) are properly typed, but because of the dynamic DI the plugins types are understandably loose. We could make this better, but we'd still have to cast parts of it, and this is all temporary code.