[GitHub] drill pull request #999: DRILL-5881:Java Client: [Threat Modeling] Drillbit ...

[sohami <gi...@git.apache.org>]

GitHub user sohami opened a pull request:

    https://github.com/apache/drill/pull/999

    DRILL-5881:Java Client: [Threat Modeling] Drillbit may be spoofed by …

    …an attacker and this may lead to data being written to the attacker's target instead of Drillbit

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

    $ git pull https://github.com/sohami/drill DRILL-5881

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

    https://github.com/apache/drill/pull/999.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 #999
    
----

----


---

[GitHub] drill issue #999: DRILL-5881:Java Client: [Threat Modeling] Drillbit may be ...

[parthchandra <gi...@git.apache.org>]

Github user parthchandra commented on the issue:

    https://github.com/apache/drill/pull/999
  
    +1. 
    You need a newline at the end :)


---

[GitHub] drill pull request #999: DRILL-5881:Java Client: [Threat Modeling] Drillbit ...

[asfgit <gi...@git.apache.org>]

Github user asfgit closed the pull request at:

    https://github.com/apache/drill/pull/999


---

[GitHub] drill issue #999: DRILL-5881:Java Client: [Threat Modeling] Drillbit may be ...

[sohami <gi...@git.apache.org>]

Github user sohami commented on the issue:

    https://github.com/apache/drill/pull/999
  
    @parthchandra - Please help to review this PR. I have added new unit tests for the change and made sure all the existing test's are also passing.


---

[GitHub] drill issue #999: DRILL-5881:Java Client: [Threat Modeling] Drillbit may be ...

[parthchandra <gi...@git.apache.org>]

Github user parthchandra commented on the issue:

    https://github.com/apache/drill/pull/999
  
    +1 to the second commit as well. This addresses the problem of the unit tests failing intermittently.


---