Per bug 1344670, it looks like using example.com doesn't quite work for self-support code. :mythmon, why did we change the URL for self-support as well, in bug 1326225? What does the %server(s) stuff even do (does it do *something*? Or just break the fetch but nothing else?). We'll need to update this to make sure 54 going to beta won't hit bug 1344670 again.
Hrmpf, no tracking flags for the SHIELD product. :-(
Component: Repair → General
Product: Shield → Firefox
As I understand it, there is a test server running on localhost that can be accessed via a hostname that is available in prefs as |%(server)s|. That test server is not accessible over https. Both the old self-repair  and the new recipe-client  require their urls to be https, and won't accept http. I was told that the easy way to get an https url was to use example.com, which would point to the same server as |%(server)s|. That's all I know about that. I'm not sure what the right thing to do is here. To me it sounds like the WPT tests from bug 1344670 should probably support example.com, like mochitests, or disable self-support and the recipe-client. : https://dxr.mozilla.org/mozilla-central/rev/58753259bfeb3b818eac7870871b0aae1f8de64a/browser/modules/SelfSupportBackend.jsm#247-250 : https://dxr.mozilla.org/mozilla-central/rev/58753259bfeb3b818eac7870871b0aae1f8de64a/browser/extensions/shield-recipe-client/lib/RecipeRunner.jsm#53-56
(In reply to Mike Cooper [:mythmon] from comment #2) > That's all I know about that. I'm not sure what the right thing to do is > here. To me it sounds like the WPT tests from bug 1344670 should probably > support example.com, like mochitests, or disable self-support and the > recipe-client. James, would anything stop us from disabling both the self-support and recipe-client? I don't think that any of the tests are dependent on those.
No, afaik that should be fine (I assume these are both frontend features that don't affect the platform).
I'd think a considerably better idea would be to set the pref for extensions.shield-recipe-client.api_url to a working value in browser_RecipeRunner.js, the single place which currently cares anything about what the URL is or whether it works at all, rather than disable something we ship on a considerable chunk of our tests.
And we're out of time yet again for actual reviewed patches written by people who understand what's going on and how things should be done.
Commits pushed to master at https://github.com/mozilla/normandy https://github.com/mozilla/normandy/commit/f71d6c41c2eee4e5af80bef4a413d7a7dff3aa46 Bug 1345650 - use https://%(server)s/ for shield/self-support URLs to not break web-platform-tests on beta, and set shield-recipe-client.api_url to a working value in browser_RecipeRunner.js https://github.com/mozilla/normandy/commit/0386d6ae4d7c1ba4d5d7cb27b24181f2dcd5a3ea Merge pull request #694 from mythmon/bug-1345650-backport Bug 1345650 - Backport preference fix
You need to log in before you can comment on or make changes to this bug.