Skip to content

Commit

Permalink
Merge pull request #4836 from brave/fix/ads-private-registries
Browse files Browse the repository at this point in the history
Set INCLUDE_PRIVATE_REGISTRIES instead of EXCLUDE_PRIVATE_REGISTRIES
  • Loading branch information
tmancey authored Mar 5, 2020
2 parents 4785c87 + 0292001 commit 9c1f108
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 2 deletions.
2 changes: 1 addition & 1 deletion vendor/bat-native-ads/src/bat/ads/internal/uri_helper.cc
Original file line number Diff line number Diff line change
Expand Up @@ -57,7 +57,7 @@ bool Uri::MatchesDomainOrHost(
const std::string& url1,
const std::string& url2) {
return SameDomainOrHost(GURL(url1), GURL(url2),
net::registry_controlled_domains::EXCLUDE_PRIVATE_REGISTRIES);
net::registry_controlled_domains::INCLUDE_PRIVATE_REGISTRIES);
}

} // namespace helper
Original file line number Diff line number Diff line change
Expand Up @@ -151,7 +151,7 @@ void RedeemToken::CreateConfirmation(
confirmation_info.id = base::GenerateGUID();
confirmation_info.creative_instance_id = ad_info.creative_instance_id;
const std::string tld_plus_1 = GetDomainAndRegistry(GURL(ad_info.target_url),
net::registry_controlled_domains::EXCLUDE_PRIVATE_REGISTRIES);
net::registry_controlled_domains::INCLUDE_PRIVATE_REGISTRIES);
confirmation_info.channel_id = tld_plus_1;
confirmation_info.type = confirmation_type;
confirmation_info.token_info = token_info;
Expand Down

0 comments on commit 9c1f108

Please sign in to comment.