question-mark
Stuck on an issue?

Lightrun Answers was designed to reduce the constant googling that comes with debugging 3rd party libraries. It collects links to all the places you might be looking at while hunting down a tough bug.

And, if you’re still stuck at the end, we’re happy to hop on a call to see how we can help out.

TAARDIS: Pass hashed client id to TAAR via AMO API request

See original GitHub issue

As TAAR moves away from being a shield study instead of receiving the client-id via a query string set by the add-on the client-id will be available in hashed form as a cookie a HTTP header sent by the browser.

Bug with the details of the Firefox implementation is here: https://bugzilla.mozilla.org/show_bug.cgi?id=1489531

This bug is to cover reading that cookie header and passing the hashed id down to TAAR via the AMO api.

For QA: this change will only work in the context of about:addons, so you have to update the discoverURL pref to https://discovery.addons-dev.allizom.org/%LOCALE%/firefox/discovery/pane/%VERSION%/%OS%/%COMPATIBILITY_MODE% (for -dev).

Then:

  • set your own client ID to a test ID that returns recommendations, e.g., 00000000-0000-0000-0000-000000000000. You can change the client ID in /path/to/current FF profile/datareporting/state.json:
    • open FF and navigate to about:profiles

    • find “This is the profile in use and it cannot be deleted.” and open the root directory

    • open datareporting/state.json and replace the content of this file with:

      {"clientID":"00000000-0000-0000-0000-000000000000"}
      
    • restart your browser

  • open about:addons and load the disco pane
  • you should get recommdations

If you repeat those steps with 00000000-aaaa-0000-0000-000000000000, you should not have the recommendations but a different list with editorial choices.

Issue Analytics

  • State:closed
  • Created 5 years ago
  • Comments:13 (11 by maintainers)

github_iconTop GitHub Comments

1reaction
willdurandcommented, Jan 21, 2019

@muffinresearch, do we still need a feature flag then if we have a check for disco?

I think it would be safer in case we need to disable it if something goes wrong.

1reaction
muffinresearchcommented, Jan 18, 2019

@rebmullin @willdurand This will need to go into prod next week to make the 29th January deadline so we’ll want to get this config’d on for stage and prod ahead of the Monday tag.

Read more comments on GitHub >

github_iconTop Results From Across the Web

OAuth2 - Sending a hash of your client_secret when using the ...
OAuth2 says that the method of client authentication is up to the authorisation server. It can be anything. Based on RFC 6749,.
Read more >
Untitled
Finance yahoo api json! Mcintosh ml-2c specs, Escape the titanic part 4! Drzalica za sekiru, Monnaie unique euro pays! Greenville falls park ohio, ......
Read more >
Untitled
Setas agosto 2015, Best feed through fly control horses, Budweiser girls costume, Severina ante karaoke, Full length mirrored wall cabinet!
Read more >
npmsearchfullcat_npm143.txt - GitHub
2012-10-31 0.4.0 api authorization 3scale web 3scale-senico Client for ... build nodejs asynchronous continuation-passing style akeebabackup NodeJS npm ...
Read more >
stopword.txt - GitHub Pages
n.m 2.n2 2.nbf 2.nf 2.nil 2.o 2.of 2.out1 2.q 2.r 2.requests 2.rl 2.rx 2.s5ks4k ... aphysician api-attack api-by-api api-id api-id-fbconnectserver-login-php ...
Read more >

github_iconTop Related Medium Post

No results found

github_iconTop Related StackOverflow Question

No results found

github_iconTroubleshoot Live Code

Lightrun enables developers to add logs, metrics and snapshots to live code - no restarts or redeploys required.
Start Free

github_iconTop Related Reddit Thread

No results found

github_iconTop Related Hackernoon Post

No results found

github_iconTop Related Tweet

No results found

github_iconTop Related Dev.to Post

No results found

github_iconTop Related Hashnode Post

No results found