Page MenuHomeDevCentral

Improve support for multiple remote addresses
ClosedPublic

Authored by dereckson on May 13 2018, 18:48.
Tags
None
Referenced Files
F2840904: D1638.id4181.diff
Mon, Apr 22, 21:37
Unknown Object (File)
Fri, Apr 19, 09:42
Unknown Object (File)
Fri, Apr 19, 01:41
Unknown Object (File)
Sun, Apr 14, 22:18
Unknown Object (File)
Wed, Apr 10, 09:41
Unknown Object (File)
Wed, Apr 10, 09:41
Unknown Object (File)
Tue, Apr 9, 12:04
Unknown Object (File)
Sat, Apr 6, 12:22
Subscribers
None

Details

Summary

Currently, if a proxy declares more than one remote address,
the separator needed to be a comma.

But Squid documentation provides a syntax with spaces.

To provide a better support, this change explodes the IP address
with several delimiters, comma and whitespaces (\r \n \t \f and space).

Test Plan

New tests provided

Diff Detail

Repository
rKOT Keruald OmniTools
Lint
Lint Passed
Unit
Tests Passed
Branch
fix-forwarded-multi-ip (branched from master)
Build Status
Buildable 2568
Build 2816: arc lint + arc unit

Event Timeline

dereckson created this revision.
This revision is now accepted and ready to land.May 13 2018, 19:22
This revision was automatically updated to reflect the committed changes.