{"action":"created","comment":{"url":"https://api.github.com/repos/inverse-inc/packetfence/pulls/comments/93499078","pull_request_review_id":14028375,"id":93499078,"diff_hunk":"@@ -3747,45 +3747,15 @@ These sensors work by capturing the packets at the lowest level possible on your\n \n Microsoft DHCP sensor\n ^^^^^^^^^^^^^^^^^^^^^\n+DHCP-Forwarder is an optimized version of precedent udp-reflector, which installs easily and only copy DHCPREQUESTS and DHCPACK packets to the destination. \n \n-You will first need to download and install 'WinPcap' available from http://www.winpcap.org/install/\n+https://inverse.ca/downloads/PacketFence/windows-dhcp-forwarder/DHCP%20Forwarder%20Installer.exe[Download the installer here.]\n \n-You will also need to download and install 'Microsoft Visual C++ 2010 Redistributable' available from http://www.microsoft.com/download/details.aspx?id=5555\n+It will install WinPCAP, nssm, launch a configurator to ask for interface, IP and port, save the configuration, install and launch DHCP-Forwarder service.\n \n-NOTE: You absolutely need to install the 32-bit version of 'Microsoft Visual C++ 2010 Redistributable' even if you are using a 64-bit operating system.\n+When you will be asked for a host IP and port, specify PacketFence management IP and 767 as the UDP port.\n \n-Then get the remote sensor from Inverse's download website http://inverse.ca/downloads/PacketFence/udp-reflector/udp_reflector.exe\n-\n-Create the directory `C:\\udp-reflector` and move the downloaded file inside.\n-\n-Now we will create a service so the reflector starts on boot.\n-\n-First download and unzip nssm from https://nssm.cc/download\n-\n-Next create a batch file `udpreflector.bat` in `C:\\udp-reflector` that contain:\n-`C:\\udp-reflector\\udp_reflector.exe -s pcap0:67 -d 192.168.1.5:767 -b 25000`\n-\n-Where `pcap0` is the interface where your DHCP is listening (use `C:\\udp-reflector\\udp_reflector.exe -l` to list interfaces)\n-\n-Then run `nssm install udpreflector`\n-\n-In Application:\n-[options=\"compact\"]\n-* In 'Path' set it to `C:\\udp-reflector\\udpreflector.bat`\n-* In 'Startup directory' set it to `C:\\udp-reflector\\`\n-* In 'Arguments' set it to nothing\n-\n-In Details:\n-[options=\"compact\"]\n-* In 'Startup type' select Automatic\n-\n-In Log on:\n-[options=\"compact\"]\n-* In 'Log on as' select Local System account\n-\n-Then press 'Install service'\n-\n-The DHCP traffic should now be reflected on your PacketFence server.\n+The projet page is https://github.com/inverse-inc/packetfence-dhcp-forwarder[here].","path":"docs/PacketFence_Administration_Guide.asciidoc","position":47,"original_position":47,"commit_id":"d08f87b1915cc889979c87b828cd7196f748a87e","original_commit_id":"d08f87b1915cc889979c87b828cd7196f748a87e","user":{"login":"julsemaan","id":3857942,"avatar_url":"https://avatars.githubusercontent.com/u/3857942?v=3","gravatar_id":"","url":"https://api.github.com/users/julsemaan","html_url":"https://github.com/julsemaan","followers_url":"https://api.github.com/users/julsemaan/followers","following_url":"https://api.github.com/users/julsemaan/following{/other_user}","gists_url":"https://api.github.com/users/julsemaan/gists{/gist_id}","starred_url":"https://api.github.com/users/julsemaan/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/julsemaan/subscriptions","organizations_url":"https://api.github.com/users/julsemaan/orgs","repos_url":"https://api.github.com/users/julsemaan/repos","events_url":"https://api.github.com/users/julsemaan/events{/privacy}","received_events_url":"https://api.github.com/users/julsemaan/received_events","type":"User","site_admin":false},"body":"*project","created_at":"2016-12-21T19:03:05Z","updated_at":"2016-12-21T19:03:05Z","html_url":"https://github.com/inverse-inc/packetfence/pull/1864#discussion_r93499078","pull_request_url":"https://api.github.com/repos/inverse-inc/packetfence/pulls/1864","_links":{"self":{"href":"https://api.github.com/repos/inverse-inc/packetfence/pulls/comments/93499078"},"html":{"href":"https://github.com/inverse-inc/packetfence/pull/1864#discussion_r93499078"},"pull_request":{"href":"https://api.github.com/repos/inverse-inc/packetfence/pulls/1864"}}},"pull_request":{"url":"https://api.github.com/repos/inverse-inc/packetfence/pulls/1864","id":95352596,"html_url":"https://github.com/inverse-inc/packetfence/pull/1864","diff_url":"https://github.com/inverse-inc/packetfence/pull/1864.diff","patch_url":"https://github.com/inverse-inc/packetfence/pull/1864.patch","issue_url":"https://api.github.com/repos/inverse-inc/packetfence/issues/1864","number":1864,"state":"open","locked":false,"title":"Replacement of the Windows DHCP sensor section (UDP-Reflector) with DHCP-Forwarder.","user":{"login":"tlaurion","id":827570,"avatar_url":"https://avatars.githubusercontent.com/u/827570?v=3","gravatar_id":"","url":"https://api.github.com/users/tlaurion","html_url":"https://github.com/tlaurion","followers_url":"https://api.github.com/users/tlaurion/followers","following_url":"https://api.github.com/users/tlaurion/following{/other_user}","gists_url":"https://api.github.com/users/tlaurion/gists{/gist_id}","starred_url":"https://api.github.com/users/tlaurion/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/tlaurion/subscriptions","organizations_url":"https://api.github.com/users/tlaurion/orgs","repos_url":"https://api.github.com/users/tlaurion/repos","events_url":"https://api.github.com/users/tlaurion/events{/privacy}","received_events_url":"https://api.github.com/users/tlaurion/received_events","type":"User","site_admin":false},"body":"# Description\r\nReplaces udp-reflector documentation and launches DHCP-Forwarder.\r\n\r\n# Delete branch after merge\r\nYES\r\n\r\n# NEWS file entries\r\n* DHCP-Forwarder Installer replaces complicated udp-reflector deployement. IP helpers no more?\r\n\r\n## Enhancements\r\n* We could receive only 18% of the DHCP traffic we currently receive, thanks to DHCP-Forwarder. (see dhcp-forwarder/TESTING.md)\r\n\r\n# UPGRADE file entries\r\n* Should we create a uninstall guide for udp-reflector?\r\n","created_at":"2016-11-25T22:49:56Z","updated_at":"2016-12-21T19:03:05Z","closed_at":null,"merged_at":null,"merge_commit_sha":"6c8feee219f06adfbc80597ac79ae110bf8b49d8","assignee":null,"assignees":[],"milestone":null,"commits_url":"https://api.github.com/repos/inverse-inc/packetfence/pulls/1864/commits","review_comments_url":"https://api.github.com/repos/inverse-inc/packetfence/pulls/1864/comments","review_comment_url":"https://api.github.com/repos/inverse-inc/packetfence/pulls/comments{/number}","comments_url":"https://api.github.com/repos/inverse-inc/packetfence/issues/1864/comments","statuses_url":"https://api.github.com/repos/inverse-inc/packetfence/statuses/d08f87b1915cc889979c87b828cd7196f748a87e","head":{"label":"inverse-inc:feature/dhcp-forwarder-doc","ref":"feature/dhcp-forwarder-doc","sha":"d08f87b1915cc889979c87b828cd7196f748a87e","user":{"login":"inverse-inc","id":1203154,"avatar_url":"https://avatars.githubusercontent.com/u/1203154?v=3","gravatar_id":"","url":"https://api.github.com/users/inverse-inc","html_url":"https://github.com/inverse-inc","followers_url":"https://api.github.com/users/inverse-inc/followers","following_url":"https://api.github.com/users/inverse-inc/following{/other_user}","gists_url":"https://api.github.com/users/inverse-inc/gists{/gist_id}","starred_url":"https://api.github.com/users/inverse-inc/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/inverse-inc/subscriptions","organizations_url":"https://api.github.com/users/inverse-inc/orgs","repos_url":"https://api.github.com/users/inverse-inc/repos","events_url":"https://api.github.com/users/inverse-inc/events{/privacy}","received_events_url":"https://api.github.com/users/inverse-inc/received_events","type":"Organization","site_admin":false},"repo":{"id":3595658,"name":"packetfence","full_name":"inverse-inc/packetfence","owner":{"login":"inverse-inc","id":1203154,"avatar_url":"https://avatars.githubusercontent.com/u/1203154?v=3","gravatar_id":"","url":"https://api.github.com/users/inverse-inc","html_url":"https://github.com/inverse-inc","followers_url":"https://api.github.com/users/inverse-inc/followers","following_url":"https://api.github.com/users/inverse-inc/following{/other_user}","gists_url":"https://api.github.com/users/inverse-inc/gists{/gist_id}","starred_url":"https://api.github.com/users/inverse-inc/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/inverse-inc/subscriptions","organizations_url":"https://api.github.com/users/inverse-inc/orgs","repos_url":"https://api.github.com/users/inverse-inc/repos","events_url":"https://api.github.com/users/inverse-inc/events{/privacy}","received_events_url":"https://api.github.com/users/inverse-inc/received_events","type":"Organization","site_admin":false},"private":false,"html_url":"https://github.com/inverse-inc/packetfence","description":"PacketFence is a fully supported, trusted, Free and Open Source network access control (NAC) solution. Boasting an impressive feature set including a captive-portal for registration and remediation, centralized wired and wireless management, powerful BYOD management options, 802.1X support, layer-2 isolation of problematic devices; PacketFence can be used to effectively secure networks small to very large heterogeneous networks.","fork":false,"url":"https://api.github.com/repos/inverse-inc/packetfence","forks_url":"https://api.github.com/repos/inverse-inc/packetfence/forks","keys_url":"https://api.github.com/repos/inverse-inc/packetfence/keys{/key_id}","collaborators_url":"https://api.github.com/repos/inverse-inc/packetfence/collaborators{/collaborator}","teams_url":"https://api.github.com/repos/inverse-inc/packetfence/teams","hooks_url":"https://api.github.com/repos/inverse-inc/packetfence/hooks","issue_events_url":"https://api.github.com/repos/inverse-inc/packetfence/issues/events{/number}","events_url":"https://api.github.com/repos/inverse-inc/packetfence/events","assignees_url":"https://api.github.com/repos/inverse-inc/packetfence/assignees{/user}","branches_url":"https://api.github.com/repos/inverse-inc/packetfence/branches{/branch}","tags_url":"https://api.github.com/repos/inverse-inc/packetfence/tags","blobs_url":"https://api.github.com/repos/inverse-inc/packetfence/git/blobs{/sha}","git_tags_url":"https://api.github.com/repos/inverse-inc/packetfence/git/tags{/sha}","git_refs_url":"https://api.github.com/repos/inverse-inc/packetfence/git/refs{/sha}","trees_url":"https://api.github.com/repos/inverse-inc/packetfence/git/trees{/sha}","statuses_url":"https://api.github.com/repos/inverse-inc/packetfence/statuses/{sha}","languages_url":"https://api.github.com/repos/inverse-inc/packetfence/languages","stargazers_url":"https://api.github.com/repos/inverse-inc/packetfence/stargazers","contributors_url":"https://api.github.com/repos/inverse-inc/packetfence/contributors","subscribers_url":"https://api.github.com/repos/inverse-inc/packetfence/subscribers","subscription_url":"https://api.github.com/repos/inverse-inc/packetfence/subscription","commits_url":"https://api.github.com/repos/inverse-inc/packetfence/commits{/sha}","git_commits_url":"https://api.github.com/repos/inverse-inc/packetfence/git/commits{/sha}","comments_url":"https://api.github.com/repos/inverse-inc/packetfence/comments{/number}","issue_comment_url":"https://api.github.com/repos/inverse-inc/packetfence/issues/comments{/number}","contents_url":"https://api.github.com/repos/inverse-inc/packetfence/contents/{+path}","compare_url":"https://api.github.com/repos/inverse-inc/packetfence/compare/{base}...{head}","merges_url":"https://api.github.com/repos/inverse-inc/packetfence/merges","archive_url":"https://api.github.com/repos/inverse-inc/packetfence/{archive_format}{/ref}","downloads_url":"https://api.github.com/repos/inverse-inc/packetfence/downloads","issues_url":"https://api.github.com/repos/inverse-inc/packetfence/issues{/number}","pulls_url":"https://api.github.com/repos/inverse-inc/packetfence/pulls{/number}","milestones_url":"https://api.github.com/repos/inverse-inc/packetfence/milestones{/number}","notifications_url":"https://api.github.com/repos/inverse-inc/packetfence/notifications{?since,all,participating}","labels_url":"https://api.github.com/repos/inverse-inc/packetfence/labels{/name}","releases_url":"https://api.github.com/repos/inverse-inc/packetfence/releases{/id}","deployments_url":"https://api.github.com/repos/inverse-inc/packetfence/deployments","created_at":"2012-03-01T20:21:13Z","updated_at":"2016-12-14T19:58:30Z","pushed_at":"2016-12-21T18:54:39Z","git_url":"git://github.com/inverse-inc/packetfence.git","ssh_url":"git@github.com:inverse-inc/packetfence.git","clone_url":"https://github.com/inverse-inc/packetfence.git","svn_url":"https://github.com/inverse-inc/packetfence","homepage":"https://packetfence.org","size":184287,"stargazers_count":243,"watchers_count":243,"language":"Perl","has_issues":true,"has_downloads":true,"has_wiki":false,"has_pages":false,"forks_count":106,"mirror_url":null,"open_issues_count":261,"forks":106,"open_issues":261,"watchers":243,"default_branch":"devel"}},"base":{"label":"inverse-inc:devel","ref":"devel","sha":"a68abda497725dadb9d19e9a2d1affb6e657f523","user":{"login":"inverse-inc","id":1203154,"avatar_url":"https://avatars.githubusercontent.com/u/1203154?v=3","gravatar_id":"","url":"https://api.github.com/users/inverse-inc","html_url":"https://github.com/inverse-inc","followers_url":"https://api.github.com/users/inverse-inc/followers","following_url":"https://api.github.com/users/inverse-inc/following{/other_user}","gists_url":"https://api.github.com/users/inverse-inc/gists{/gist_id}","starred_url":"https://api.github.com/users/inverse-inc/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/inverse-inc/subscriptions","organizations_url":"https://api.github.com/users/inverse-inc/orgs","repos_url":"https://api.github.com/users/inverse-inc/repos","events_url":"https://api.github.com/users/inverse-inc/events{/privacy}","received_events_url":"https://api.github.com/users/inverse-inc/received_events","type":"Organization","site_admin":false},"repo":{"id":3595658,"name":"packetfence","full_name":"inverse-inc/packetfence","owner":{"login":"inverse-inc","id":1203154,"avatar_url":"https://avatars.githubusercontent.com/u/1203154?v=3","gravatar_id":"","url":"https://api.github.com/users/inverse-inc","html_url":"https://github.com/inverse-inc","followers_url":"https://api.github.com/users/inverse-inc/followers","following_url":"https://api.github.com/users/inverse-inc/following{/other_user}","gists_url":"https://api.github.com/users/inverse-inc/gists{/gist_id}","starred_url":"https://api.github.com/users/inverse-inc/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/inverse-inc/subscriptions","organizations_url":"https://api.github.com/users/inverse-inc/orgs","repos_url":"https://api.github.com/users/inverse-inc/repos","events_url":"https://api.github.com/users/inverse-inc/events{/privacy}","received_events_url":"https://api.github.com/users/inverse-inc/received_events","type":"Organization","site_admin":false},"private":false,"html_url":"https://github.com/inverse-inc/packetfence","description":"PacketFence is a fully supported, trusted, Free and Open Source network access control (NAC) solution. Boasting an impressive feature set including a captive-portal for registration and remediation, centralized wired and wireless management, powerful BYOD management options, 802.1X support, layer-2 isolation of problematic devices; PacketFence can be used to effectively secure networks small to very large heterogeneous networks.","fork":false,"url":"https://api.github.com/repos/inverse-inc/packetfence","forks_url":"https://api.github.com/repos/inverse-inc/packetfence/forks","keys_url":"https://api.github.com/repos/inverse-inc/packetfence/keys{/key_id}","collaborators_url":"https://api.github.com/repos/inverse-inc/packetfence/collaborators{/collaborator}","teams_url":"https://api.github.com/repos/inverse-inc/packetfence/teams","hooks_url":"https://api.github.com/repos/inverse-inc/packetfence/hooks","issue_events_url":"https://api.github.com/repos/inverse-inc/packetfence/issues/events{/number}","events_url":"https://api.github.com/repos/inverse-inc/packetfence/events","assignees_url":"https://api.github.com/repos/inverse-inc/packetfence/assignees{/user}","branches_url":"https://api.github.com/repos/inverse-inc/packetfence/branches{/branch}","tags_url":"https://api.github.com/repos/inverse-inc/packetfence/tags","blobs_url":"https://api.github.com/repos/inverse-inc/packetfence/git/blobs{/sha}","git_tags_url":"https://api.github.com/repos/inverse-inc/packetfence/git/tags{/sha}","git_refs_url":"https://api.github.com/repos/inverse-inc/packetfence/git/refs{/sha}","trees_url":"https://api.github.com/repos/inverse-inc/packetfence/git/trees{/sha}","statuses_url":"https://api.github.com/repos/inverse-inc/packetfence/statuses/{sha}","languages_url":"https://api.github.com/repos/inverse-inc/packetfence/languages","stargazers_url":"https://api.github.com/repos/inverse-inc/packetfence/stargazers","contributors_url":"https://api.github.com/repos/inverse-inc/packetfence/contributors","subscribers_url":"https://api.github.com/repos/inverse-inc/packetfence/subscribers","subscription_url":"https://api.github.com/repos/inverse-inc/packetfence/subscription","commits_url":"https://api.github.com/repos/inverse-inc/packetfence/commits{/sha}","git_commits_url":"https://api.github.com/repos/inverse-inc/packetfence/git/commits{/sha}","comments_url":"https://api.github.com/repos/inverse-inc/packetfence/comments{/number}","issue_comment_url":"https://api.github.com/repos/inverse-inc/packetfence/issues/comments{/number}","contents_url":"https://api.github.com/repos/inverse-inc/packetfence/contents/{+path}","compare_url":"https://api.github.com/repos/inverse-inc/packetfence/compare/{base}...{head}","merges_url":"https://api.github.com/repos/inverse-inc/packetfence/merges","archive_url":"https://api.github.com/repos/inverse-inc/packetfence/{archive_format}{/ref}","downloads_url":"https://api.github.com/repos/inverse-inc/packetfence/downloads","issues_url":"https://api.github.com/repos/inverse-inc/packetfence/issues{/number}","pulls_url":"https://api.github.com/repos/inverse-inc/packetfence/pulls{/number}","milestones_url":"https://api.github.com/repos/inverse-inc/packetfence/milestones{/number}","notifications_url":"https://api.github.com/repos/inverse-inc/packetfence/notifications{?since,all,participating}","labels_url":"https://api.github.com/repos/inverse-inc/packetfence/labels{/name}","releases_url":"https://api.github.com/repos/inverse-inc/packetfence/releases{/id}","deployments_url":"https://api.github.com/repos/inverse-inc/packetfence/deployments","created_at":"2012-03-01T20:21:13Z","updated_at":"2016-12-14T19:58:30Z","pushed_at":"2016-12-21T18:54:39Z","git_url":"git://github.com/inverse-inc/packetfence.git","ssh_url":"git@github.com:inverse-inc/packetfence.git","clone_url":"https://github.com/inverse-inc/packetfence.git","svn_url":"https://github.com/inverse-inc/packetfence","homepage":"https://packetfence.org","size":184287,"stargazers_count":243,"watchers_count":243,"language":"Perl","has_issues":true,"has_downloads":true,"has_wiki":false,"has_pages":false,"forks_count":106,"mirror_url":null,"open_issues_count":261,"forks":106,"open_issues":261,"watchers":243,"default_branch":"devel"}},"_links":{"self":{"href":"https://api.github.com/repos/inverse-inc/packetfence/pulls/1864"},"html":{"href":"https://github.com/inverse-inc/packetfence/pull/1864"},"issue":{"href":"https://api.github.com/repos/inverse-inc/packetfence/issues/1864"},"comments":{"href":"https://api.github.com/repos/inverse-inc/packetfence/issues/1864/comments"},"review_comments":{"href":"https://api.github.com/repos/inverse-inc/packetfence/pulls/1864/comments"},"review_comment":{"href":"https://api.github.com/repos/inverse-inc/packetfence/pulls/comments{/number}"},"commits":{"href":"https://api.github.com/repos/inverse-inc/packetfence/pulls/1864/commits"},"statuses":{"href":"https://api.github.com/repos/inverse-inc/packetfence/statuses/d08f87b1915cc889979c87b828cd7196f748a87e"}}}}
{"actor":{"display_login":"julsemaan"}}