Page MenuHomeSolus

Update ipscan to 3.7.2
ClosedPublic

Authored by serebit on Jun 6 2020, 9:59 PM.
Tags
None
Referenced Files
F11047880: D9011.id21800.diff
Thu, Aug 10, 3:54 PM
F11047879: D9011.id.diff
Thu, Aug 10, 3:54 PM
F11047878: D9011.id21685.diff
Thu, Aug 10, 3:54 PM
F11033415: D9011.diff
Wed, Aug 9, 3:40 PM
F11016707: D9011.diff
Sat, Aug 5, 7:23 PM
F10916882: D9011.id.diff
Jul 8 2023, 2:29 PM
F10866657: D9011.diff
Jun 17 2023, 4:42 PM
F10850834: D9011.id21685.diff
Jun 12 2023, 7:31 AM

Details

Summary

Additionally, use JDK11 for both build and runtime.

Notable changes between this version and 3.7.0:

  • Add a new Java Built-in pinger by default (InetAddress.isReachable()), which uses ICMP if enough privileges or falls back to TCP port 7
  • Fix handling of escape sequences in HTTP Sender
  • Enable ICMP Echo pinger for Linux

Full release notes are available here.

Test Plan

Scan some IPs!

Diff Detail

Repository
R1482 ipscan
Branch
master
Lint
No Lint Coverage
Unit
No Test Coverage

Event Timeline

livingsilver94 added inline comments.
files/ipscan
3–5

Hm is this actually a good idea? Better to ask @DataDrake, but it is my understanding that we want to allow users to specify JAVA_HOME only for packages aimed at building applications (gradle, ant, apache, sbt. Maybe even scala but idk).

files/ipscan
3–5

I figured it couldn't hurt, but if core would rather not allow it then I can change the diff.

files/ipscan
3–5

With "apache" I meant "maven" but yeah you get the point.

LGTM. Thanks!

I don't really think it hurts to have the ability to override JAVA_HOME, though I don't think everything needs it either.

This revision is now accepted and ready to land.Jun 13 2020, 3:41 PM
This revision was automatically updated to reflect the committed changes.