-
Notifications
You must be signed in to change notification settings - Fork 2.4k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Affiliate ID being suggested in auto-complete for specific sites #10129
Comments
Tbh I see nothing wrong with these suggestions personally. Having said that, why not add a "suggested" or "promoted" label for the suggested ref links and allow users to decide if they want to support brave in this fashion. Also would be interesting to see some of the value gained from this to reflect on BAT. If that is that case I think brave users can only benefit from this and it's only fair. |
Brendan Eich himself sees very much wrong with them, said multiple times he takes all the blame and the project is going to get rid of them. https://twitter.com/BrendanEich/status/1269324204844806145?s=20 |
Change is now disabled in source via: For those reading, this functionality is behind a flag which you can disable via With the fix linked above, the I've started a Nightly build and we'll also have this fix uplifted to Beta soon (with a build following the merge there) |
No, he didn't say it was wrong. He and other brave employees said they were going to remove it and it was a [PR] mistake. The feature itself is innocuous and not really hurting anyone. In other words it was removed because cry babies, because of general outcry of offended ladies and hate from bitcoin maximalists [and I'm glad I can point the unpopular version out to you because I'm not involved in the project in any way]. |
I would still like to have these refs because i like the browser. |
Yes - when a new build comes out, you can toggle the |
@bsclifton is there a brave/chrome policy that can be changed to manage this centrally? |
The fix for this rolled out today on Release channel (1.9.80) and we're deploying Android soon too. We've put together a post about this which you all can read here: |
@bsclifton 👍 have send news to winfuture.de |
Please change brave-core\components\omnibox\browser\suggested_sites_provider_data.cc
into the following to stop making gullible people pay into the pockets of individual bad actors on this project.
/* Copyright (c) 2020 The Brave Authors. All rights reserved.
#include "brave/components/omnibox/browser/suggested_sites_provider.h"
#include "base/strings/utf_string_conversions.h"
const std::vector&
SuggestedSitesProvider::GetSuggestedSites() {
static const std::vector suggested_sites = {
{
SuggestedSitesMatch(
"binance.com",
GURL("https://www.binance.com?ref=AffiliateEaterID"),
GURL("https://www.binance.com"),
base::ASCIIToUTF16("binance.com?ref=AffiliateEaterID"),
true)
}, {
SuggestedSitesMatch(
"bitcoin",
GURL("https://www.binance.com/en/buy-sell-crypto"
"?fiat=USD&crypto=BTC&ref=AffiliateEaterID&utm_source=brave"),
GURL("https://www.binance.com/en/buy-sell-crypto?crypto=BTC"),
base::ASCIIToUTF16("binance.com/en/buy-sell-crypto?crypto=BTC&ref=AffiliateEaterID"), // NOLINT
false)
}, {
SuggestedSitesMatch(
"btc",
GURL("https://www.binance.com/en/buy-sell-crypto"
"?fiat=USD&crypto=BTC&ref=AffiliateEaterID&utm_source=brave"),
GURL("https://www.binance.com/en/buy-sell-crypto?crypto=BTC"),
base::ASCIIToUTF16("binance.com/en/buy-sell-crypto?crypto=BTC&ref=AffiliateEaterID"), // NOLINT
false)
}, {
SuggestedSitesMatch(
"ethereum",
GURL("https://www.binance.com/en/buy-sell-crypto"
"?fiat=USD&crypto=ETH&ref=AffiliateEaterID6&utm_source=brave"),
GURL("https://www.binance.com/en/buy-sell-crypto?crypto=ETH"),
base::ASCIIToUTF16("binance.com/en/buy-sell-crypto?crypto=ETH&ref=AffiliateEaterID"), // NOLINT
false)
}, {
SuggestedSitesMatch(
"eth",
GURL("https://www.binance.com/en/buy-sell-crypto"
"?fiat=USD&crypto=ETH&ref=AffiliateEaterID&utm_source=brave"),
GURL("https://www.binance.com/en/buy-sell-crypto?crypto=ETH"),
base::ASCIIToUTF16("binance.com/en/buy-sell-crypto?crypto=ETH&ref=AffiliateEaterID"), // NOLINT
false)
}, {
SuggestedSitesMatch(
"litecoin",
GURL("https://www.binance.com/en/buy-sell-crypto"
"?fiat=USD&crypto=LTC&ref=AffiliateEaterID&utm_source=brave"),
GURL("https://www.binance.com/en/buy-sell-crypto?crypto=LTC"),
base::ASCIIToUTF16("binance.com/en/buy-sell-crypto?crypto=LTC&ref=AffiliateEaterID"), // NOLINT
false)
}, {
SuggestedSitesMatch(
"ltc",
GURL("https://www.binance.com/en/buy-sell-crypto"
"?fiat=USD&crypto=LTC&ref=AffiliateEaterID&utm_source=brave"),
GURL("https://www.binance.com/en/buy-sell-crypto?crypto=LTC"),
base::ASCIIToUTF16("binance.com/en/buy-sell-crypto?crypto=LTC&ref=AffiliateEaterID"), // NOLINT
false)
}, {
SuggestedSitesMatch(
"bnb",
GURL("https://www.binance.com/en/buy-sell-crypto"
"?fiat=USD&crypto=BNB&ref=AffiliateEaterID&utm_source=brave"),
GURL("https://www.binance.com/en/buy-sell-crypto?crypto=BNB"),
base::ASCIIToUTF16("binance.com/en/buy-sell-crypto?crypto=BNB&ref=AffiliateEaterID"), // NOLINT
false)
}, {
SuggestedSitesMatch(
"binance.us",
GURL("https://www.binance.us?ref=AffiliateEaterID"),
GURL("https://www.binance.us"),
base::ASCIIToUTF16("binance.us?ref=AffiliateEaterID"),
true)
}, {
SuggestedSitesMatch(
"bitcoin",
GURL("https://www.binance.us/en/buy-sell-crypto"
"?fiat=USD&crypto=BTC&ref=AffiliateEaterID&utm_source=brave"),
GURL("https://www.binance.us/en/buy-sell-crypto?crypto=BTC"),
base::ASCIIToUTF16("binance.us/en/buy-sell-crypto?crypto=BTC&ref=AffiliateEaterID"), // NOLINT
false)
}, {
SuggestedSitesMatch(
"btc",
GURL("https://www.binance.us/en/buy-sell-crypto"
"?fiat=USD&crypto=BTC&ref=AffiliateEaterID&utm_source=brave"),
GURL("https://www.binance.us/en/buy-sell-crypto?crypto=BTC"),
base::ASCIIToUTF16("binance.us/en/buy-sell-crypto?crypto=BTC&ref=AffiliateEaterID"), // NOLINT
false)
}, {
SuggestedSitesMatch(
"ethereum",
GURL("https://www.binance.us/en/buy-sell-crypto"
"?fiat=USD&crypto=ETH&ref=AffiliateEaterID&utm_source=brave"),
GURL("https://www.binance.us/en/buy-sell-crypto?crypto=ETH"),
base::ASCIIToUTF16("binance.us/en/buy-sell-crypto?crypto=ETH&ref=AffiliateEaterID"), // NOLINT
false)
}, {
SuggestedSitesMatch(
"eth",
GURL("https://www.binance.us/en/buy-sell-crypto"
"?fiat=USD&crypto=ETH&ref=AffiliateEaterID&utm_source=brave"),
GURL("https://www.binance.us/en/buy-sell-crypto?crypto=ETH"),
base::ASCIIToUTF16("binance.us/en/buy-sell-crypto?crypto=ETH&ref=AffiliateEaterID"), // NOLINT
false)
}, {
SuggestedSitesMatch(
"bnb",
GURL("https://www.binance.us/en/buy-sell-crypto"
"?fiat=USD&crypto=BNB&ref=AffiliateEaterID&utm_source=brave"),
GURL("https://www.binance.us/en/buy-sell-crypto?crypto=BNB"),
base::ASCIIToUTF16("binance.us/en/buy-sell-crypto?crypto=BNB&ref=AffiliateEaterID"), // NOLINT
false)
}, {
SuggestedSitesMatch(
"coinbase.com/join",
GURL("https://www.coinbase.com/join/AffiliateEaterID"),
GURL("https://www.coinbase.com/join"),
base::ASCIIToUTF16("coinbase.com/join/AffiliateEaterID"),
true)
}, {
SuggestedSitesMatch(
"ledger.com/pages/ledger-nano-x",
GURL("https://shop.ledger.com/pages/ledger-nano-x?r=AffiliateEaterID"),
GURL("https://shop.ledger.com/pages/ledger-nano-x"),
base::ASCIIToUTF16("shop.ledger.com/pages/ledger-nano-x?r=AffiliateEaterID"),
true)
}, {
SuggestedSitesMatch(
"trezor.io/product/trezor-one-metallic",
GURL("https://shop.trezor.io/product/trezor-one-metallic"
"?offer_id=24&aff_id=AffiliateEaterID"),
GURL("https://shop.trezor.io/product/trezor-one-metallic"),
base::ASCIIToUTF16("shop.trezor.io/product/trezor-one-metallic"
"?offer_id=24&aff_id=AffiliateEaterID"),
true)
},
};
return suggested_sites;
}
The text was updated successfully, but these errors were encountered: