Remove GCP ping redirection for tagged pings.

[travis79]

This removes the redirection to GCP endpoint for all tagged pings

[travis79]

@Dexterp37 I know that this doesn't have any documentation additions yet, putting this up for discussion.

[Dexterp37]

I think we should only change the endpoint, let's not overload tagging for other reasons. Moreover, let's land this after Fenix MVP .

[codecov]

Codecov Report

Merging #3343 into master will increase coverage by 1.34%.
The diff coverage is n/a.

@@             Coverage Diff              @@
##             master    #3343      +/-   ##
============================================
+ Coverage     80.38%   81.72%   +1.34%     
- Complexity     2774     3217     +443     
============================================
  Files           382      405      +23     
  Lines         12544    13950    +1406     
  Branches       1871     2020     +149     
============================================
+ Hits          10083    11401    +1318     
- Misses         1700     1744      +44     
- Partials        761      805      +44

Impacted Files	Coverage Δ	Complexity Δ
...a/components/service/glean/net/HttpPingUploader.kt	95.45% <ø> (ø)	9 <0> (?)
...a/components/service/glean/config/Configuration.kt	100% <ø> (ø)	11 <0> (?)
...omponents/lib/publicsuffixlist/PublicSuffixList.kt	86.2% <0%> (-6.9%)	6% <0%> (ø)
...a/components/feature/app/links/AppLinksUseCases.kt	73.21% <0%> (-2.23%)	6% <0%> (+1%)
.../components/support/ktx/android/content/Context.kt	65.71% <0%> (-1.94%)	0% <0%> (ø)
...ts/concept/engine/manifest/WebAppManifestParser.kt	77.02% <0%> (-1.93%)	2% <0%> (-2%)
...ponents/feature/pwa/AbstractWebAppShellActivity.kt	20% <0%> (-1.74%)	3% <0%> (ø)
...mponents/browser/toolbar/display/DisplayToolbar.kt	92.14% <0%> (-0.88%)	45% <0%> (+3%)
...omponents/service/fxa/manager/FxaAccountManager.kt	87.3% <0%> (-0.82%)	34% <0%> (-3%)
...a/components/browser/search/SearchEngineManager.kt	89.18% <0%> (-0.56%)	17% <0%> (-1%)
... and 170 more

---

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 1bf1436...9f31591. Read the comment docs.

[Dexterp37]

This looks good to me now, thanks. Let's not merge this before 24th of June and, before doing that, let's check back with @pocmo and @akkomar . Can you please file a bug about doing the same for glean-core? :)

[travis79]

This looks good to me now, thanks. Let's not merge this before 24th of June and, before doing that, let's check back with @pocmo and @akkomar . Can you please file a bug about doing the same for glean-core? :)

Follow-up bug for glean-core filed here