{"action":"created","issue":{"url":"https://api.github.com/repos/hyperledger/fabric/issues/2286","repository_url":"https://api.github.com/repos/hyperledger/fabric","labels_url":"https://api.github.com/repos/hyperledger/fabric/issues/2286/labels{/name}","comments_url":"https://api.github.com/repos/hyperledger/fabric/issues/2286/comments","events_url":"https://api.github.com/repos/hyperledger/fabric/issues/2286/events","html_url":"https://github.com/hyperledger/fabric/pull/2286","id":166976177,"number":2286,"title":"[WIP] Fix Issue 2283","user":{"login":"christo4ferris","id":3630697,"avatar_url":"https://avatars.githubusercontent.com/u/3630697?v=3","gravatar_id":"","url":"https://api.github.com/users/christo4ferris","html_url":"https://github.com/christo4ferris","followers_url":"https://api.github.com/users/christo4ferris/followers","following_url":"https://api.github.com/users/christo4ferris/following{/other_user}","gists_url":"https://api.github.com/users/christo4ferris/gists{/gist_id}","starred_url":"https://api.github.com/users/christo4ferris/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/christo4ferris/subscriptions","organizations_url":"https://api.github.com/users/christo4ferris/orgs","repos_url":"https://api.github.com/users/christo4ferris/repos","events_url":"https://api.github.com/users/christo4ferris/events{/privacy}","received_events_url":"https://api.github.com/users/christo4ferris/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":11,"created_at":"2016-07-22T06:05:23Z","updated_at":"2016-07-25T10:58:22Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/hyperledger/fabric/pulls/2286","html_url":"https://github.com/hyperledger/fabric/pull/2286","diff_url":"https://github.com/hyperledger/fabric/pull/2286.diff","patch_url":"https://github.com/hyperledger/fabric/pull/2286.patch"},"body":"<!-- Provide a general summary of your changes in the Title above -->\r\n\r\n## Description\r\nchange peer REST port from 5000 to 7050 to avoid conflict with UPnP on Windows. Also reassigned all other ports as follows:\r\n\r\n7050 REST\r\n7051 peer gRPC\r\n7052 peer CLI\r\n7053 peer events\r\n7054 eCAP\r\n7055 eCAA\r\n7056 tCAP\r\n7057 tCAA\r\n7058 tlsCAP\r\n7059 tlsCAA\r\n\r\nNOTE: when this PR lands, it will require people to rebuild their Vagrant development environment(s), so we will want to make an announcement.\r\n\r\n## Motivation and Context\r\n<!-- Why is this change required? What problem does it solve? -->\r\n<!-- If it fixes an open issue, please link to the issue here. -->\r\nFixes #2283 \r\nAlso @bcbrock recommended that we eventually seek a block of ports from IANA. We should hold off on that until we sort out Consensus next.\r\n\r\n## How Has This Been Tested?\r\n<!-- If this PR does not contain a new test case, explain why. -->\r\n<!-- Describe in detail how you tested your changes. -->\r\nran all tests\r\n\r\n## Checklist:\r\n<!-- To check a box, and an 'x': [x] -->\r\n<!-- To uncheck box, add a space: [ ] -->\r\n<!-- If you're unsure about any of these, don't hesitate to ask. We're here to help! -->\r\n- [x] I have added a [Signed-off-by](https://github.com/hyperledger/fabric/blob/master/CONTRIBUTING.md#legal-stuff).\r\n- [x] I have either added documentation to cover my changes or this change requires no new documentation.\r\n- [x] I have either added unit tests to cover my changes or this change requires no new tests.\r\n- [x] I have run [golint](https://github.com/golang/lint) and have fixed valid warnings in code I have added or modified. This tool generates false positives so you may choose to ignore some warnings. The goal is clean, consistent, and readable code.\r\n\r\n<!-- The continuous integration build process will run [make checks](https://github.com/hyperledger/fabric/blob/master/Makefile#L22) to confirm that tests pass and that code quality meets minimum standards. You may optionally run this locally as PRs will not be accepted until they pass. -->\r\n\r\nSigned-off-by: Christopher Ferris <chrisfer@us.ibm.com>\r\n"},"comment":{"url":"https://api.github.com/repos/hyperledger/fabric/issues/comments/234923324","html_url":"https://github.com/hyperledger/fabric/pull/2286#issuecomment-234923324","issue_url":"https://api.github.com/repos/hyperledger/fabric/issues/2286","id":234923324,"user":{"login":"christo4ferris","id":3630697,"avatar_url":"https://avatars.githubusercontent.com/u/3630697?v=3","gravatar_id":"","url":"https://api.github.com/users/christo4ferris","html_url":"https://github.com/christo4ferris","followers_url":"https://api.github.com/users/christo4ferris/followers","following_url":"https://api.github.com/users/christo4ferris/following{/other_user}","gists_url":"https://api.github.com/users/christo4ferris/gists{/gist_id}","starred_url":"https://api.github.com/users/christo4ferris/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/christo4ferris/subscriptions","organizations_url":"https://api.github.com/users/christo4ferris/orgs","repos_url":"https://api.github.com/users/christo4ferris/repos","events_url":"https://api.github.com/users/christo4ferris/events{/privacy}","received_events_url":"https://api.github.com/users/christo4ferris/received_events","type":"User","site_admin":false},"created_at":"2016-07-25T10:58:22Z","updated_at":"2016-07-25T10:58:22Z","body":"I don't disagree but Think that should be an independent refactor"}}
{"actor":{"display_login":"christo4ferris"}}