GitHub PullRequestReviewCommentEvent at 2014-03-31 13:33:40
Repository: metasploit-framework
Actor: zeroSteiner
Payload: {"comment":{"url":"https://api.github.com/repos/zeroSteiner/metasploit-framework/pulls/comments/11115114","id":11115114,"diff_hunk":"@@ -28,31 +28,34 @@ def initialize\n     register_options(\n       [\n         OptString.new('TARGET_URI', [ false, \"Single target URI\", nil]),\n-        OptPath.new('TARGET_URIS_FILE', [ false, \"Path to list of URIs to request\", File.join(Msf::Config.data_directory, \"wordlists\", \"http_owa_common.txt\")]),\n+        OptPath.new('TARGET_URIS_FILE', [ false, \"Path to list of URIs to request\", \n+          File.join(Msf::Config.data_directory, \"wordlists\", \"http_owa_common.txt\")]),\n       ], self.class)\n   end\n \n   def run_host(ip)\n-    if datastore['TARGET_URI']\n-      test_path = normalize_uri(datastore['TARGET_URI'])\n-      result = check_url(test_path)\n-      if result\n-        handle_result(test_path, result)\n-        return\n-      end\n+    test_uris = []\n+    turi = datastore['TARGET_URI']\n+    turis_file = datastore['TARGET_URIS_FILE']\n+    if !(turi && turis_file)","path":"modules/auxiliary/scanner/http/ntlm_info_enumeration.rb","position":30,"original_position":30,"commit_id":"2530fb9741c46ddc45683a89f6f332355b5a2735","original_commit_id":"2530fb9741c46ddc45683a89f6f332355b5a2735","user":{"login":"zeroSteiner","id":2058303,"avatar_url":"https://avatars.githubusercontent.com/u/2058303?","gravatar_id":"52eae3dc563ea7ac59e327e07ff63826","url":"https://api.github.com/users/zeroSteiner","html_url":"https://github.com/zeroSteiner","followers_url":"https://api.github.com/users/zeroSteiner/followers","following_url":"https://api.github.com/users/zeroSteiner/following{/other_user}","gists_url":"https://api.github.com/users/zeroSteiner/gists{/gist_id}","starred_url":"https://api.github.com/users/zeroSteiner/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/zeroSteiner/subscriptions","organizations_url":"https://api.github.com/users/zeroSteiner/orgs","repos_url":"https://api.github.com/users/zeroSteiner/repos","events_url":"https://api.github.com/users/zeroSteiner/events{/privacy}","received_events_url":"https://api.github.com/users/zeroSteiner/received_events","type":"User","site_admin":false},"body":"This is requiring that both turi and turis_file be set instead of just one or the other.","created_at":"2014-03-31T13:33:40Z","updated_at":"2014-03-31T13:33:40Z","html_url":"https://github.com/zeroSteiner/metasploit-framework/pull/6#discussion_r11115114","pull_request_url":"https://api.github.com/repos/zeroSteiner/metasploit-framework/pulls/6","_links":{"self":{"href":"https://api.github.com/repos/zeroSteiner/metasploit-framework/pulls/comments/11115114"},"html":{"href":"https://github.com/zeroSteiner/metasploit-framework/pull/6#discussion_r11115114"},"pull_request":{"href":"https://api.github.com/repos/zeroSteiner/metasploit-framework/pulls/6"}}}}