Join GitHub today
GitHub is home to over 50 million developers working together to host and review code, manage projects, and build software together.
Sign upRegular stub installer does not send download-complete and installer-run pings #2917
Comments
|
@btlechowski I would check if @aubrey can possibly pull the server logs for those requests, it could just be a hiccup in Fiddler. I only ever ran and tested with the regular (non-silent) installer, so it would be weird if it wasn't working (and I even downloaded the release candidate regular stub installer today and confirmed that it sent all three pings). |
|
@simonhong I'll investigate in the morning, but any thoughts why there could be a difference here between the regular stub installer and the silent one? |
|
Hmm, it's strange. need to check. |
|
Spoke with @simonhong and it seems to occur when performing an admin install. When you receive the UAC prompt, if you choose "Yes" it will do an admin install and if you choose "No" it will do a non-admin install. It seems that the admin install leads to this problem. |
|
Upon further investigation, it seems to be related to trying to read the app guid out of the registry when running in admin mode; the key isn't there. We don't need to store the app_guid in the registry, since we have access to the app_guid in the app bundle. So we can fix this problem. But, we also read the referral code from the registry and there is no clear way around that. @bbondy Just a heads up on this one. |
|
@btlechowski nice find @bbondy I don't think this is a blocker for tomorrow's release but let me know what you think. If we end up deciding on blocking, we'll need to release Monday morning. Unless we get this fixed today/tomorrow AM and release sometime Friday afternoon. |
|
let's not block |
|
We won't consult the registry at all in order to avoid this problem. For the app guid, we'll parse it out of the tag in the metainstaller and we'll access it via the app pointer. For the referral code, we'll parse it out of our filename as always and then append it to the tag that we forward to the installer (again accessing it via the app pointer, since Omaha already supports the concept of a referral id). |
|
@btlechowski I discovered an easier way to debug this, when the time comes. If you create a file named |
|
Verification passed on
Summery: [PASS - initiates pings] Windows 7 x64, BraveBrowserBetaSetup-OJV851.exe with whitelisted ref code
[PASS - initiates pings] Windows 7 x64, BraveBrowserSilentBetaSetup-ZTT758.exe with admin privileges, with whitelisted ref code
[PASS] Windows 10 x64, BraveBrowserBetaSetup.exe with admin privileges, with no ref code
[PASS] Windows 10 x64, BraveBrowserBetaSetup-BLE123.exe with admin privileges, with unwhitelisted ref code
[PASS] Windows 10 x64, BraveBrowserBetaSetup-MTS962.exe with admin privileges, with whitelisted ref code
[PASS] Windows 10 x64, BraveBrowserBetaSetup-MTS962.exe with no admin privileges, with whitelisted ref code
Note: after install, two windows were shown instead of one [PASS] Windows 10 x64, BraveBrowserSilentBetaSetup-BLE321.exe with unwhitelisted ref code
[PASS] Windows 10 x64, BraveBrowserSilentBetaSetup-MTS962.exe with whitelisted ref code
[PASS] Windows 10 x64, BraveBrowserBetaSetup32.exe with admin privileges, with no ref code
[PASS] Windows 10 x64, BraveBrowserBetaSetup32-BLE871.exe with admin privileges, with unwhitelisted ref code
[PASS] Windows 10 x64, BraveBrowserBetaSetup32-MTS962.exe with admin privileges, with whitelisted ref code
[PASS] Windows 10 x64, BraveBrowserBetaSetup32-MTS962.exe with no admin privileges, with whitelisted ref code
Note: after install, two windows were shown instead of one [PASS] Windows 10 x64, BraveBrowserSilentBetaSetup32-BLE631.exe with unwhitelisted ref code
[PASS] Windows 10 x64, BraveBrowserSilentBetaSetup32-MTS962.exe with whitelisted ref code
|
|
Awesome work @btlechowski |
Steps to Reproduce
Control PanelDownloadandInstallfromc:\Program Files (x86)\BraveSoftware\Update\Actual result:
Only

startupping is sentPings
download-completeandinstaller-runare not sentExpected result:
All 3 pings are sent
Reproduces how often:
always
Brave version (brave://version info)
Also reproduced on Windows 10 x64
cc @emerick @brave/legacy_qa