Rietveld Code Review Tool
Help | Bug tracker | Discussion group | Source code

Issue 5067678511267840: Noissue - Ensure statsmaster is aware of all filter servers (Closed)

Created:
Feb. 13, 2015, 3:29 p.m. by mathias
Modified:
Feb. 13, 2015, 5:21 p.m.
Reviewers:
Wladimir Palant
Visibility:
Public.

Description

Noissue - Ensure statsmaster is aware of all filter servers

Patch Set 1 #

Total comments: 2

Patch Set 2 : Noissue - Extend statsmaster known_hosts by IPv4 addresses #

Unified diffs Side-by-side diffs Delta from patch set Stats (+40 lines, -0 lines) Patch
M modules/statsmaster/files/known_hosts View 1 1 chunk +12 lines, -0 lines 0 comments Download
M modules/statsmaster/files/sitescripts.ini View 2 chunks +28 lines, -0 lines 0 comments Download

Messages

Total messages: 5
mathias
Feb. 13, 2015, 3:29 p.m. (2015-02-13 15:29:53 UTC) #1
Wladimir Palant
http://codereview.adblockplus.org/5067678511267840/diff/5629499534213120/modules/statsmaster/files/known_hosts File modules/statsmaster/files/known_hosts (right): http://codereview.adblockplus.org/5067678511267840/diff/5629499534213120/modules/statsmaster/files/known_hosts#newcode21 modules/statsmaster/files/known_hosts:21: filter7.adblockplus.org,78.46.32.104 ecdsa-sha2-nistp256 AAAAE2VjZHNhLXNoYTItbmlzdHAyNTYAAAAIbmlzdHAyNTYAAABBBLlfPzJA+ssEPR4+DhOzBk0K79PMvamytZc4GPllue25qc63tcF9Ze+Dd7WdeZdyfq0ifu/L/paQ25ApEGbYCa4= What about the IPv6 address for ...
Feb. 13, 2015, 4:39 p.m. (2015-02-13 16:39:10 UTC) #2
mathias
Feb. 13, 2015, 5:02 p.m. (2015-02-13 17:02:56 UTC) #3
mathias
http://codereview.adblockplus.org/5067678511267840/diff/5629499534213120/modules/statsmaster/files/known_hosts File modules/statsmaster/files/known_hosts (right): http://codereview.adblockplus.org/5067678511267840/diff/5629499534213120/modules/statsmaster/files/known_hosts#newcode21 modules/statsmaster/files/known_hosts:21: filter7.adblockplus.org,78.46.32.104 ecdsa-sha2-nistp256 AAAAE2VjZHNhLXNoYTItbmlzdHAyNTYAAAAIbmlzdHAyNTYAAABBBLlfPzJA+ssEPR4+DhOzBk0K79PMvamytZc4GPllue25qc63tcF9Ze+Dd7WdeZdyfq0ifu/L/paQ25ApEGbYCa4= On 2015/02/13 16:39:11, Wladimir Palant wrote: ...
Feb. 13, 2015, 5:03 p.m. (2015-02-13 17:03:42 UTC) #4
Wladimir Palant
Feb. 13, 2015, 5:07 p.m. (2015-02-13 17:07:44 UTC) #5
LGTM

Powered by Google App Engine
This is Rietveld