Replaced jdhcp with dhcp4java

Description

We have a licensing issues and I assume that the library is outdated as well as the Service Monitor itself.  We should either update the code are remove it from the code base.

Acceptance / Success Criteria

None

Lucidchart Diagrams

Activity

Show:

Ronny Trommer December 14, 2018 at 5:36 PM

Merged into release-23.0.2

Ronny Trommer December 14, 2018 at 4:06 PM
Edited

Works as expected. Tested extended mode true/false as well response as broadcast and specific IP address. Additionally tested two IPs in the same subnet against broadcast to see transaction IDs are taking in account. See comment in PR 2287.

Christian Pape December 12, 2018 at 2:36 PM
Edited

I updated the code to use another library (LGPL). This seems to work fine. I've created another 24.0.0-targeted issue  for a new Minion-capable version of the DhcpMonitor since this issue is targeted for 23.0.2.

 

See the following PR:

https://github.com/OpenNMS/opennms/pull/2287

Fixed

Details

Assignee

Reporter

Components

Sprint

Fix versions

Affects versions

Priority

PagerDuty

Created September 4, 2018 at 5:30 PM
Updated December 14, 2018 at 5:56 PM
Resolved December 14, 2018 at 5:36 PM