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

Issue 29587634: #3924 - Redirect Samsung Internet requests from CN to include video ads (Closed)

Created:
Oct. 24, 2017, 12:20 p.m. by f.nicolaisen
Modified:
Oct. 24, 2017, 4:50 p.m.
Reviewers:
mathias
CC:
arthur, paco, Fred
Base URL:
https://hg1/infrastructure
Visibility:
Public.

Description

#3924 - Redirect Samsung Internet requests from CN to include video ads

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+23 lines, -0 lines) Patch
M modules/filterserver/files/site.conf View 2 chunks +23 lines, -0 lines 0 comments Download

Messages

Total messages: 3
f.nicolaisen
Oct. 24, 2017, 12:20 p.m. (2017-10-24 12:20:03 UTC) #1
mathias
LGTM. Actually it doesn't look good to me (having a more powerful configuration language would), ...
Oct. 24, 2017, 3:11 p.m. (2017-10-24 15:11:27 UTC) #2
f.nicolaisen
Oct. 24, 2017, 4:48 p.m. (2017-10-24 16:48:39 UTC) #3
On 2017/10/24 15:11:27, mathias wrote:
> LGTM. Actually it doesn't look good to me (having a more powerful
configuration
> language would), but it's fair enough. Just wanted to point that out.

Couldn't agree more ;)

Powered by Google App Engine
This is Rietveld