1282080 - Update logging of AUS URL for new SYSTEM_CAPABILITIES parameter

Status: RESOLVED FIXED

(Testing :: Firefox UI Tests, defect)

Description

[Henrik Skupin [:whimboo][⌚️UTC+2]]

As seen here we have a new capabilities parameter in the AUS URL:

https://firefox-ui-tests.s3.amazonaws.com/108aea0e-77ea-403c-b497-7081e97f6b27/log_info.log

https://aus5.mozilla.org/update/6/Firefox/48.0/20160623122823/WINNT_x86-msvc/en-US/beta-cdntest/Windows_NT%206.1/%SYSTEM_CAPABILITIES%/default/default/update.xml?force=1

We have to get it replaced similar to the other replacements for this string.

Comment 1

[Henrik Skupin [:whimboo][⌚️UTC+2]]

Digging in the code changes on DXR showed me that this change came in for Firefox 49.0 via bug 1271761.

I'm taking this so we can get it working again.

Comment 2

[Henrik Skupin [:whimboo][⌚️UTC+2]]

We can use UpdateUtils.jsm and let it do the formatting. There is no need anymore that we have to do it on our own. Hurray!

The appropriate placeholder replacement for SYSTEM_CAPABILITIES has been added with bug 1271761.

Comment 3

[Henrik Skupin [:whimboo][⌚️UTC+2]]

Attachment: Bug 1282080 - Fix logging of AUS server URL for new SYSTEM_CAPABILITIES parameter.

Review commit: https://reviewboard.mozilla.org/r/61796/diff/#index_header
See other reviews: https://reviewboard.mozilla.org/r/61796/

Comment 4

[Maja Frydrychowicz :impossibus (was :maja_zf) (needinfo me)]

Comment on attachment 8767121 [details]
Bug 1282080 - Fix logging of AUS server URL for new SYSTEM_CAPABILITIES parameter.

https://reviewboard.mozilla.org/r/61796/#review59208

Comment 5

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/6c52fa3a1a9c
Fix logging of AUS server URL for new SYSTEM_CAPABILITIES parameter. r=maja_zf

Comment 6

[Carsten Book [:Tomcat]]

https://hg.mozilla.org/mozilla-central/rev/6c52fa3a1a9c

Comment 7

[Henrik Skupin [:whimboo][⌚️UTC+2]]

It's a test-only change which we also need for aurora. Thanks.

Comment 8

[Henrik Skupin [:whimboo][⌚️UTC+2]]

We also need this for beta.

Comment 9

[Carsten Book [:Tomcat]]

https://hg.mozilla.org/releases/mozilla-aurora/rev/9212b9883204

Comment 10

[Carsten Book [:Tomcat]]

https://hg.mozilla.org/releases/mozilla-beta/rev/92ec0fc8583e