Path issues and unit tests resolved

Per slack, this is being investigated for re-introduction into 1.4.1. Putting this on pause for now.

Confirmed this is happening with current mainline and 1.4, but not 1.2,1.3 or 1.5.

Per slack conversation.

This should be fixed by

@sdwilsh Are you good to close this out? The change in ipv6 should be handled by the updated version now in main

Ready for review.

I've got a fix for this in the works.

I'm back home now, feel free to send them to me and I'll take a look at it today (I have a few minutes now, but likely will need to look at it in about 12 hours).

was that after rebasing it onto our current main branch or applying changes directly? Or something else?

Yeah, changes are pretty small and also pretty isolated. I’m guessing the merge might be fast, unless there are significant conflicts. However if there are conflicts probably faster to re-apply the change.

Let’s get tests at least for the changes here before we check it in. It should be trivial to test, especially since it looks like there is no version difference to take into account.