Hi Tyler, Thanks for working on this, it's a great feature to have and I'm glad it's getting worked on.
I'll let other people hop in if they disagree (given that there's more work to be done), but the way I'd get feedback on code like this is just to open a pull request on GitHub. It's easier than using the files on the ticket for a variety of reasons, including making it easier to get and see updates and fixes to the code. It wouldn't surprise me if there's a bit of discussion on the PR, given that this is a pretty nice to have feature that has a bit of nuance that we'll want to discuss as things come together, so having it in one place would be great. I would just make sure to mention that it's not ready for merging because you're looking for help and feedback on things like tests. Justin On Fri, Feb 3, 2017 at 12:22 PM, Tyler Moore <[email protected]> wrote: > Hey All, > > I attached some work on the JIRA to METRON-571 issue about stellar external > functions. > Could someone please review and maybe give me some insights on test > integration? > > These are the details on the issue and link is provided as well: > > Add stellar keywords for executing local commands > > ------------------------------------------------- > > > > Key: METRON-571 > > URL: https://issues.apache.org/jira/browse/METRON-571 > > Project: Metron > > Issue Type: Improvement > > Reporter: Jon Zeolla > > Priority: Minor > > Attachments: ExternalFunctions.java, ExternalFunctionsTest.java > > > > > > Stellar should have the ability to execute scripts on a server in order > to undertake automated mitigation or alerting. Perhaps SHELL_EXEC? > > Regards, > > Tyler Moore > Software Engineer > Phone: 248-909-2769 > Email: [email protected] <[email protected]> >
