You are viewing a plain text version of this content. The canonical link for it is here.

Posted to dev@spot.apache.org by brandon-edwards <gi...@git.apache.org> on 2016/12/17 07:31:02 UTC

[GitHub] incubator-spot pull request #5: User domain hard coded as "intel" for DNS Su...

GitHub user brandon-edwards opened a pull request:

    https://github.com/apache/incubator-spot/pull/5

    User domain hard coded as "intel" for DNS Suspicious Connects Analysis

    On line 67 of https://github.com/Open-Network-Insight/open-network-insight/blob/spot/spot-ml/src/main/scala/org/apache/spot/utilities/DomainProcessor.scala the code tests the domain string variable against the string "intel" in order to determine whether the domain should be considered to be in the users domain. For non intel solution-users this is a problem, I will allow this string to be passed from spot.conf. See Jira issue: https://issues.apache.org/jira/browse/SPOT-24 .

You can merge this pull request into a Git repository by running:

    $ git pull https://github.com/brandon-edwards/incubator-spot User_Domain_Designation_Fix

Alternatively you can review and apply these changes as the patch at:

    https://github.com/apache/incubator-spot/pull/5.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

    This closes #5
    
----
commit bc5744f6f5e81521a3905ae072177218fdd6b05c
Author: Brandon Edwards <br...@intel.com>
Date:   2016-12-05T22:59:38Z

     Eddited DNSWordCreation.scala DomainProcessor.scala DNSSuspiciousConnectsModel.scala, and SuspicousConnectsArgumentParser to all allow in a variable user domain designation. More work to do in order to have this information come in from the spot.conf.

commit 41ffbc3c3744eadc7baac4633f160188bcad731a
Author: Brandon Edwards <br...@intel.com>
Date:   2016-12-06T17:03:02Z

    Inserted lines into spot.conf and ml_ops.sh for input of user domain string.

commit d7d6ae07344a9a4308067963cb9b03e493b995d0
Author: Brandon Edwards <br...@intel.com>
Date:   2016-12-06T22:45:38Z

    More modifications in order to pass the user domain info around to the places that are needed.

commit f3b3652673713fd36152605c64df9a95e140e818
Author: Brandon Edwards <br...@intel.com>
Date:   2016-12-08T00:39:18Z

    More editing on SuspiciousConnectsArgumentParser to eliminate old (dropped) arguments.

commit f7596ca8f13629e09d88a4d257ecb6f8d10f33de
Author: Brandon Edwards <br...@intel.com>
Date:   2016-12-08T00:46:41Z

    And yet more editing of SuspiciousConnectsArgumentParser.scala to remove old argument references.

commit 8f0988ae205a54e3d3d67c56aa7dab7f8b5e5584
Author: Brandon Edwards <br...@intel.com>
Date:   2016-12-08T22:52:59Z

    Updating spot branch from upstream (spot repo)
    Merge branch 'spot' of https://github.com/Open-Network-Insight/open-network-insight into spot

commit f6c2b2ba754832a3aff04baec6af791e77b53221
Author: Brandon Edwards <br...@intel.com>
Date:   2016-12-08T22:55:12Z

    Trying to resolve conflicts by merging in an updated version of spot branch
    Merge branch 'spot' into Shouldbe_good_domain_fix

commit 55b0497bf3a6dbe87e659cadffcb70784cb9c1af
Author: Brandon Edwards <br...@intel.com>
Date:   2016-12-09T21:56:44Z

    Changed the usderDomain input to be optional in the argument parser and changed the default value of this armgument to be the empty string in spot.conf.

commit 13fc71863558d8d27512ca7c710fda585812a63a
Author: Brandon Edwards <br...@intel.com>
Date:   2016-12-10T01:02:50Z

    Added a test to DomainProcessorTest.scala to insure that an empty string for userDomain would not be taken as the legitimate user domain in the case of an empty string domain.

commit b9cc67d1ea4579781a95d86e1d34ea49cf069b68
Author: Brandon Edwards <br...@intel.com>
Date:   2016-12-13T20:37:48Z

    Modified spot.conf to contain logic that only tries to pass the userdomain option when the spot.conf variable USER_DOMAIN is non-empty.

commit 3bf290dd0a33368486abea1d803163bbc7cd7736
Author: Brandon Edwards <br...@intel.com>
Date:   2016-12-14T00:21:47Z

    Changed the position of the optional parameter to be at the end of all other parameters

commit 452fca35bd33c291b88cd4a664f531db1ba7d7e2
Author: Brandon Edwards <br...@intel.com>
Date:   2016-12-14T16:25:30Z

    Omitted an extra space after one of the jar parameter entry lines that was causing an error.

commit 1a0269f2e0f9a1297e1084c01b1991362f0a74ae
Author: Brandon Edwards <br...@intel.com>
Date:   2016-12-14T16:35:14Z

    Changed USER_DOMAIN_PARSER_CMD to USER_DOMAIN_CMD

----


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] incubator-spot pull request #5: User domain hard coded as "intel" for DNS Su...

Posted by brandon-edwards <gi...@git.apache.org>.

Github user brandon-edwards closed the pull request at:

    https://github.com/apache/incubator-spot/pull/5


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---