{"action":"created","comment":{"url":"https://api.github.com/repos/spechub/Hets/pulls/comments/65440503","id":65440503,"diff_hunk":"@@ -45,20 +48,87 @@\n \tprivate static OWLOntologyIRIMapperImpl mapper =\n \t\tnew OWLOntologyIRIMapperImpl();\n \tprivate static URI uri = null;\n+\tprivate static int maxRedirects = 0;\n+\n+\tprivate static void checkMaxRedirects() {\n+\t\tString t = System.getProperty(\"http.maxRedirects\", null);\n+\t\tif (t == null || t.isEmpty()) {\n+\t\t\tmaxRedirects = 20;\n+\t\t\treturn;\n+\t\t}\n+\t\ttry {\n+\t\t\tint n = Integer.parseInt(t, 10);\n+\t\t\tif (n > 0) {\n+\t\t\t\tmaxRedirects = n;\n+\t\t\t}\n+\t\t} catch (NumberFormatException nfe) {\n+\t\t\tSystem.err.println(\"Ignoring invalid system property \"\n+\t\t\t\t+ \"'http.maxRedirects' - invalid number '\" + t + \"'\");\n+\t\t}\n+\t}\n+\n+\tprivate static URLConnection getConnection(URL url, int redirCount)\n+\t\tthrows IOException\n+\t{\n+\t\tURLConnection c = url.openConnection();\n+\t\tif (!(c instanceof HttpURLConnection)) {\n+\t\t\treturn c;\n+\t\t}\n+\t\tHttpURLConnection con = (HttpURLConnection) c;\n+\t\tcon.addRequestProperty(\"Accept\", \"text/plain\");\n+\t\tcon.setConnectTimeout(300 * 1000);\t// 5 min timeout","path":"OWL2/java/de/unibremen/informatik/OWL2Parser.java","position":404,"original_position":43,"commit_id":"47bb42d5a2ea6356bfa21924a004637803bb440f","original_commit_id":"47bb42d5a2ea6356bfa21924a004637803bb440f","user":{"login":"jelmd","id":4813414,"avatar_url":"https://avatars.githubusercontent.com/u/4813414?v=3","gravatar_id":"","url":"https://api.github.com/users/jelmd","html_url":"https://github.com/jelmd","followers_url":"https://api.github.com/users/jelmd/followers","following_url":"https://api.github.com/users/jelmd/following{/other_user}","gists_url":"https://api.github.com/users/jelmd/gists{/gist_id}","starred_url":"https://api.github.com/users/jelmd/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/jelmd/subscriptions","organizations_url":"https://api.github.com/users/jelmd/orgs","repos_url":"https://api.github.com/users/jelmd/repos","events_url":"https://api.github.com/users/jelmd/events{/privacy}","received_events_url":"https://api.github.com/users/jelmd/received_events","type":"User","site_admin":false},"body":"No.\r\n\r\nThis is not related to \"download time\". It is related to the tcp connection (initial handshake) setup and basically says: if a connection can't be established within 5 min, stop waiting for an answer and fail. Therefore I think 5 min are more than sufficient.\r\n\r\nAnother tweak would be the readTimeout setting, i.e. how long max. a single read on the related socket may block. This would go a little bit more in the \"server too busy to answer\" direction (IIRC the default is \"wait forever\"). So setting both to lower values is more or less a resource optimization, higher values usually make not much sense.\r\n\r\nAnyway, therefore I think, \"download size/time\" is kind of orthogonal to these settings.","created_at":"2016-06-01T20:54:11Z","updated_at":"2016-06-01T20:54:11Z","html_url":"https://github.com/spechub/Hets/pull/1642#discussion_r65440503","pull_request_url":"https://api.github.com/repos/spechub/Hets/pulls/1642","_links":{"self":{"href":"https://api.github.com/repos/spechub/Hets/pulls/comments/65440503"},"html":{"href":"https://github.com/spechub/Hets/pull/1642#discussion_r65440503"},"pull_request":{"href":"https://api.github.com/repos/spechub/Hets/pulls/1642"}}},"pull_request":{"url":"https://api.github.com/repos/spechub/Hets/pulls/1642","id":72228016,"html_url":"https://github.com/spechub/Hets/pull/1642","diff_url":"https://github.com/spechub/Hets/pull/1642.diff","patch_url":"https://github.com/spechub/Hets/pull/1642.patch","issue_url":"https://api.github.com/repos/spechub/Hets/issues/1642","number":1642,"state":"open","locked":false,"title":"1620 owl http redirects","user":{"login":"jelmd","id":4813414,"avatar_url":"https://avatars.githubusercontent.com/u/4813414?v=3","gravatar_id":"","url":"https://api.github.com/users/jelmd","html_url":"https://github.com/jelmd","followers_url":"https://api.github.com/users/jelmd/followers","following_url":"https://api.github.com/users/jelmd/following{/other_user}","gists_url":"https://api.github.com/users/jelmd/gists{/gist_id}","starred_url":"https://api.github.com/users/jelmd/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/jelmd/subscriptions","organizations_url":"https://api.github.com/users/jelmd/orgs","repos_url":"https://api.github.com/users/jelmd/repos","events_url":"https://api.github.com/users/jelmd/events{/privacy}","received_events_url":"https://api.github.com/users/jelmd/received_events","type":"User","site_admin":false},"body":"fixes #1620 ","created_at":"2016-06-01T18:19:25Z","updated_at":"2016-06-01T20:54:12Z","closed_at":null,"merged_at":null,"merge_commit_sha":"55e7dba6a58c55bb3779b03fb7b6cc614644785e","assignee":null,"milestone":null,"commits_url":"https://api.github.com/repos/spechub/Hets/pulls/1642/commits","review_comments_url":"https://api.github.com/repos/spechub/Hets/pulls/1642/comments","review_comment_url":"https://api.github.com/repos/spechub/Hets/pulls/comments{/number}","comments_url":"https://api.github.com/repos/spechub/Hets/issues/1642/comments","statuses_url":"https://api.github.com/repos/spechub/Hets/statuses/47bb42d5a2ea6356bfa21924a004637803bb440f","head":{"label":"spechub:1620_owl_http_redirects","ref":"1620_owl_http_redirects","sha":"47bb42d5a2ea6356bfa21924a004637803bb440f","user":{"login":"spechub","id":6953185,"avatar_url":"https://avatars.githubusercontent.com/u/6953185?v=3","gravatar_id":"","url":"https://api.github.com/users/spechub","html_url":"https://github.com/spechub","followers_url":"https://api.github.com/users/spechub/followers","following_url":"https://api.github.com/users/spechub/following{/other_user}","gists_url":"https://api.github.com/users/spechub/gists{/gist_id}","starred_url":"https://api.github.com/users/spechub/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/spechub/subscriptions","organizations_url":"https://api.github.com/users/spechub/orgs","repos_url":"https://api.github.com/users/spechub/repos","events_url":"https://api.github.com/users/spechub/events{/privacy}","received_events_url":"https://api.github.com/users/spechub/received_events","type":"Organization","site_admin":false},"repo":{"id":20492711,"name":"Hets","full_name":"spechub/Hets","owner":{"login":"spechub","id":6953185,"avatar_url":"https://avatars.githubusercontent.com/u/6953185?v=3","gravatar_id":"","url":"https://api.github.com/users/spechub","html_url":"https://github.com/spechub","followers_url":"https://api.github.com/users/spechub/followers","following_url":"https://api.github.com/users/spechub/following{/other_user}","gists_url":"https://api.github.com/users/spechub/gists{/gist_id}","starred_url":"https://api.github.com/users/spechub/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/spechub/subscriptions","organizations_url":"https://api.github.com/users/spechub/orgs","repos_url":"https://api.github.com/users/spechub/repos","events_url":"https://api.github.com/users/spechub/events{/privacy}","received_events_url":"https://api.github.com/users/spechub/received_events","type":"Organization","site_admin":false},"private":false,"html_url":"https://github.com/spechub/Hets","description":"The Heterogeneous Tool Set","fork":false,"url":"https://api.github.com/repos/spechub/Hets","forks_url":"https://api.github.com/repos/spechub/Hets/forks","keys_url":"https://api.github.com/repos/spechub/Hets/keys{/key_id}","collaborators_url":"https://api.github.com/repos/spechub/Hets/collaborators{/collaborator}","teams_url":"https://api.github.com/repos/spechub/Hets/teams","hooks_url":"https://api.github.com/repos/spechub/Hets/hooks","issue_events_url":"https://api.github.com/repos/spechub/Hets/issues/events{/number}","events_url":"https://api.github.com/repos/spechub/Hets/events","assignees_url":"https://api.github.com/repos/spechub/Hets/assignees{/user}","branches_url":"https://api.github.com/repos/spechub/Hets/branches{/branch}","tags_url":"https://api.github.com/repos/spechub/Hets/tags","blobs_url":"https://api.github.com/repos/spechub/Hets/git/blobs{/sha}","git_tags_url":"https://api.github.com/repos/spechub/Hets/git/tags{/sha}","git_refs_url":"https://api.github.com/repos/spechub/Hets/git/refs{/sha}","trees_url":"https://api.github.com/repos/spechub/Hets/git/trees{/sha}","statuses_url":"https://api.github.com/repos/spechub/Hets/statuses/{sha}","languages_url":"https://api.github.com/repos/spechub/Hets/languages","stargazers_url":"https://api.github.com/repos/spechub/Hets/stargazers","contributors_url":"https://api.github.com/repos/spechub/Hets/contributors","subscribers_url":"https://api.github.com/repos/spechub/Hets/subscribers","subscription_url":"https://api.github.com/repos/spechub/Hets/subscription","commits_url":"https://api.github.com/repos/spechub/Hets/commits{/sha}","git_commits_url":"https://api.github.com/repos/spechub/Hets/git/commits{/sha}","comments_url":"https://api.github.com/repos/spechub/Hets/comments{/number}","issue_comment_url":"https://api.github.com/repos/spechub/Hets/issues/comments{/number}","contents_url":"https://api.github.com/repos/spechub/Hets/contents/{+path}","compare_url":"https://api.github.com/repos/spechub/Hets/compare/{base}...{head}","merges_url":"https://api.github.com/repos/spechub/Hets/merges","archive_url":"https://api.github.com/repos/spechub/Hets/{archive_format}{/ref}","downloads_url":"https://api.github.com/repos/spechub/Hets/downloads","issues_url":"https://api.github.com/repos/spechub/Hets/issues{/number}","pulls_url":"https://api.github.com/repos/spechub/Hets/pulls{/number}","milestones_url":"https://api.github.com/repos/spechub/Hets/milestones{/number}","notifications_url":"https://api.github.com/repos/spechub/Hets/notifications{?since,all,participating}","labels_url":"https://api.github.com/repos/spechub/Hets/labels{/name}","releases_url":"https://api.github.com/repos/spechub/Hets/releases{/id}","deployments_url":"https://api.github.com/repos/spechub/Hets/deployments","created_at":"2014-06-04T17:20:48Z","updated_at":"2016-02-11T12:30:52Z","pushed_at":"2016-06-01T18:19:26Z","git_url":"git://github.com/spechub/Hets.git","ssh_url":"git@github.com:spechub/Hets.git","clone_url":"https://github.com/spechub/Hets.git","svn_url":"https://github.com/spechub/Hets","homepage":"http://hets.eu","size":261319,"stargazers_count":7,"watchers_count":7,"language":"Haskell","has_issues":true,"has_downloads":true,"has_wiki":true,"has_pages":false,"forks_count":8,"mirror_url":null,"open_issues_count":512,"forks":8,"open_issues":512,"watchers":7,"default_branch":"master"}},"base":{"label":"spechub:master","ref":"master","sha":"c3ac0c457b0ae912441cc7cde79a2dcd6fc5114d","user":{"login":"spechub","id":6953185,"avatar_url":"https://avatars.githubusercontent.com/u/6953185?v=3","gravatar_id":"","url":"https://api.github.com/users/spechub","html_url":"https://github.com/spechub","followers_url":"https://api.github.com/users/spechub/followers","following_url":"https://api.github.com/users/spechub/following{/other_user}","gists_url":"https://api.github.com/users/spechub/gists{/gist_id}","starred_url":"https://api.github.com/users/spechub/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/spechub/subscriptions","organizations_url":"https://api.github.com/users/spechub/orgs","repos_url":"https://api.github.com/users/spechub/repos","events_url":"https://api.github.com/users/spechub/events{/privacy}","received_events_url":"https://api.github.com/users/spechub/received_events","type":"Organization","site_admin":false},"repo":{"id":20492711,"name":"Hets","full_name":"spechub/Hets","owner":{"login":"spechub","id":6953185,"avatar_url":"https://avatars.githubusercontent.com/u/6953185?v=3","gravatar_id":"","url":"https://api.github.com/users/spechub","html_url":"https://github.com/spechub","followers_url":"https://api.github.com/users/spechub/followers","following_url":"https://api.github.com/users/spechub/following{/other_user}","gists_url":"https://api.github.com/users/spechub/gists{/gist_id}","starred_url":"https://api.github.com/users/spechub/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/spechub/subscriptions","organizations_url":"https://api.github.com/users/spechub/orgs","repos_url":"https://api.github.com/users/spechub/repos","events_url":"https://api.github.com/users/spechub/events{/privacy}","received_events_url":"https://api.github.com/users/spechub/received_events","type":"Organization","site_admin":false},"private":false,"html_url":"https://github.com/spechub/Hets","description":"The Heterogeneous Tool Set","fork":false,"url":"https://api.github.com/repos/spechub/Hets","forks_url":"https://api.github.com/repos/spechub/Hets/forks","keys_url":"https://api.github.com/repos/spechub/Hets/keys{/key_id}","collaborators_url":"https://api.github.com/repos/spechub/Hets/collaborators{/collaborator}","teams_url":"https://api.github.com/repos/spechub/Hets/teams","hooks_url":"https://api.github.com/repos/spechub/Hets/hooks","issue_events_url":"https://api.github.com/repos/spechub/Hets/issues/events{/number}","events_url":"https://api.github.com/repos/spechub/Hets/events","assignees_url":"https://api.github.com/repos/spechub/Hets/assignees{/user}","branches_url":"https://api.github.com/repos/spechub/Hets/branches{/branch}","tags_url":"https://api.github.com/repos/spechub/Hets/tags","blobs_url":"https://api.github.com/repos/spechub/Hets/git/blobs{/sha}","git_tags_url":"https://api.github.com/repos/spechub/Hets/git/tags{/sha}","git_refs_url":"https://api.github.com/repos/spechub/Hets/git/refs{/sha}","trees_url":"https://api.github.com/repos/spechub/Hets/git/trees{/sha}","statuses_url":"https://api.github.com/repos/spechub/Hets/statuses/{sha}","languages_url":"https://api.github.com/repos/spechub/Hets/languages","stargazers_url":"https://api.github.com/repos/spechub/Hets/stargazers","contributors_url":"https://api.github.com/repos/spechub/Hets/contributors","subscribers_url":"https://api.github.com/repos/spechub/Hets/subscribers","subscription_url":"https://api.github.com/repos/spechub/Hets/subscription","commits_url":"https://api.github.com/repos/spechub/Hets/commits{/sha}","git_commits_url":"https://api.github.com/repos/spechub/Hets/git/commits{/sha}","comments_url":"https://api.github.com/repos/spechub/Hets/comments{/number}","issue_comment_url":"https://api.github.com/repos/spechub/Hets/issues/comments{/number}","contents_url":"https://api.github.com/repos/spechub/Hets/contents/{+path}","compare_url":"https://api.github.com/repos/spechub/Hets/compare/{base}...{head}","merges_url":"https://api.github.com/repos/spechub/Hets/merges","archive_url":"https://api.github.com/repos/spechub/Hets/{archive_format}{/ref}","downloads_url":"https://api.github.com/repos/spechub/Hets/downloads","issues_url":"https://api.github.com/repos/spechub/Hets/issues{/number}","pulls_url":"https://api.github.com/repos/spechub/Hets/pulls{/number}","milestones_url":"https://api.github.com/repos/spechub/Hets/milestones{/number}","notifications_url":"https://api.github.com/repos/spechub/Hets/notifications{?since,all,participating}","labels_url":"https://api.github.com/repos/spechub/Hets/labels{/name}","releases_url":"https://api.github.com/repos/spechub/Hets/releases{/id}","deployments_url":"https://api.github.com/repos/spechub/Hets/deployments","created_at":"2014-06-04T17:20:48Z","updated_at":"2016-02-11T12:30:52Z","pushed_at":"2016-06-01T18:19:26Z","git_url":"git://github.com/spechub/Hets.git","ssh_url":"git@github.com:spechub/Hets.git","clone_url":"https://github.com/spechub/Hets.git","svn_url":"https://github.com/spechub/Hets","homepage":"http://hets.eu","size":261319,"stargazers_count":7,"watchers_count":7,"language":"Haskell","has_issues":true,"has_downloads":true,"has_wiki":true,"has_pages":false,"forks_count":8,"mirror_url":null,"open_issues_count":512,"forks":8,"open_issues":512,"watchers":7,"default_branch":"master"}},"_links":{"self":{"href":"https://api.github.com/repos/spechub/Hets/pulls/1642"},"html":{"href":"https://github.com/spechub/Hets/pull/1642"},"issue":{"href":"https://api.github.com/repos/spechub/Hets/issues/1642"},"comments":{"href":"https://api.github.com/repos/spechub/Hets/issues/1642/comments"},"review_comments":{"href":"https://api.github.com/repos/spechub/Hets/pulls/1642/comments"},"review_comment":{"href":"https://api.github.com/repos/spechub/Hets/pulls/comments{/number}"},"commits":{"href":"https://api.github.com/repos/spechub/Hets/pulls/1642/commits"},"statuses":{"href":"https://api.github.com/repos/spechub/Hets/statuses/47bb42d5a2ea6356bfa21924a004637803bb440f"}}}}
{"actor":{"display_login":"jelmd"}}