episerver / foundation-mvc-cms

Apache License 2.0
17 stars 27 forks source link

Remove DNT header check #114

Open marisks opened 3 years ago

marisks commented 3 years ago

According to MDN, the DNT header use is not recommended: https://developer.mozilla.org/en-US/docs/Web/HTTP/Headers/DNT

But it is used in the SearchService: https://github.com/episerver/foundation-mvc-cms/blob/net5/src/Foundation/Features/Search/SearchService.cs#L62

And this check actually doesn't work so that tracking is never enabled.