Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Windows Standalone installer: Two browser windows open when installing as non-admin #22179

Closed
mherrmann opened this issue Apr 8, 2022 · 3 comments · Fixed by brave/brave-core#13049
Assignees
Labels
installer/omaha Issues for windows omaha installer OS/Desktop OS/Windows priority/P3 The next thing for us to work on. It'll ride the trains. QA Pass-Win64 QA/Yes release-notes/include

Comments

@mherrmann
Copy link

Steps to Reproduce

  1. Run Brave's StandaloneSetup.exe.
  2. Reject the UAC dialog asking for admin privileges.
  3. Accept the follow-up dialog "do you want to install as normal user?" with yes. This installs Brave for just the current user into %LOCALAPPDATA% instead of C:\Program Files.

Actual result:

Two browser windows open, in addition to the installer dialog:

2022-04-08 19_05_41-Window

Expected result:

Just one browser window should open.

Reproduces how often:

Always.

Brave version (brave://version info)

Release 1.37.111.

Version/Channel Information:

  • Can you reproduce this issue with the current release? Yes
  • Can you reproduce this issue with the beta channel? Probably
  • Can you reproduce this issue with the nightly channel? Yes

Miscellaneous Information:

A very similar bug #1833 affected Brave's online installer.

@mherrmann mherrmann added OS/Windows installer/omaha Issues for windows omaha installer OS/Desktop labels Apr 8, 2022
@mherrmann mherrmann self-assigned this Apr 8, 2022
@rebron rebron added the priority/P3 The next thing for us to work on. It'll ride the trains. label Apr 11, 2022
@mherrmann
Copy link
Author

Just ran into #22122 while working on this.

@LaurenWags
Copy link
Member

@brave/legacy_qa we should check Win 10 and 11 at a minimum. Win 8 and 7 would be helpful, but not strictly necessary.

@MadhaviSeelam
Copy link

MadhaviSeelam commented Apr 28, 2022

Verification PASSED using

Brave 1.39.83 Chromium: 101.0.4951.41 (Official Build) beta (64-bit)
Revision 93c720db8323b3ec10d056025ab95c23a31997c9-refs/branch-heads/4951@{#904}
OS Windows 11 Version 21H2 (Build 22000.613)
  • Downloaded BraveBrowserStandaloneBetaSetup.exe from https://github.com/brave/brave-browser/releases/tag/v1.39.83
  • Double-clicked to install
  • Click No on the User Access Control prompt
  • Clicked Yes when prompted Brave-Browser-Beta can be installed without administrator privileges. Continue?
  • Let installation finish
  • Checked # of launched Brave windows, post-install
  • Verified only one (1) browser window was launched, post-install when installing as non-admin.
ex1 ex2 ex3 ex4 ex5 ex6 ex7
downloads admin ss1 ss4 ss2 downloads version

Verified Regression case by selecting Yes on UAC prompt to install Brave as admin


Verification PASSED on

Brave | 1.39.96 Chromium: 101.0.4951.54 (Official Build) beta (64-bit)
-- | --
Revision | 67da1aeb32cedd27634ca6634fb79cbd85d3f0ab-refs/branch-heads/4951@{#1126}
OS | Windows 10 Version 21H2 (Build 19044.1645)

  • Verified the test plan from Fix: Windows standalone installer opened 2 windows (WIP) brave-core#13049 and STR from the description
  • Issue is reproduced in the Build version 1.37.111
    1 37 111 - non admin install
  • Installed standalone installer 1.39.x without admin rights and ensured that two browser windows are NOT opened after the installtion
    1 39 x - non Admin rights install
  • Confirmed that brave is just installed for the current user and the folder Application gets created under the %LOCALAPPDATA% instead of C:\Program Files
  • Confirmed that Brave-Browser-Beta folder gets created under the registry path Computer\HKEY_CURRENT_USER\SOFTWARE\BraveSoftware\
    image
  • Confirmed that the new Omaha updater version 1.3.361.113 is shown in the updater in the registry
  • Detailed testing on the installers and updater will be covered in the issue Bump Brave's Windows updater (Omaha) to v1.3.361.113 #22060

avinassh pushed a commit to avinassh/brave-browser-hardening that referenced this issue May 29, 2022
 - Added Solana support for account creation, sending SOL and sending SPL tokens with Brave Wallet. ([#22348](brave/brave-browser#22348))
 - Added the ability to buy with Ramp using Brave Wallet. ([#21639](brave/brave-browser#21639))
 - Added JSONSanitizer to API helper requests for Brave Wallet. ([#21831](brave/brave-browser#21831))
 - Added Dapp UI for requesting a public key and for decrypting ciphers using Brave Wallet. ([#21177](brave/brave-browser#21177))
 - Added web3_clientVersion support for Brave Wallet. ([#19278](brave/brave-browser#19278))
 - Added the ability to allow users to search sites for RSS feeds for Brave News. ([#21768](brave/brave-browser#21768))
 - Added support for blob partitioning. ([#21746](brave/brave-browser#21746))
 - Added minimum macOS version for Sparkle update process. ([#22918](brave/brave-browser#22918))
 - [Security] Blocked "window.ethereum" completely in third party iframes. ([#22686](brave/brave-browser#22686))
 - [Security] Updated Brave Wallet panel to prominently display eTLD+1 as reported on HackerOne by renekroka. ([#21787](brave/brave-browser#21787))
 - [Security] Fixed incorrect origin being displayed in Brave Wallet when a spend approval is pending. ([#19557](brave/brave-browser#19557))
 - Implemented eth_getEncryptionPublicKey for Brave Wallet. ([#19276](brave/brave-browser#19276))
 - Implemented account discovery when restoring Brave Wallet. ([#18104](brave/brave-browser#18104))
 - Updated Omaha installer version for Windows to v1.3.36.113. ([#22060](brave/brave-browser#22060))
 - Updated default IPFS configuration values. ([#22068](brave/brave-browser#22068))
 - Updated Gas Limit validation and error messaging for unapproved transactions with Brave Wallet. ([#21714](brave/brave-browser#21714))
 - Updated Brave Wallet to automatically add swap taker asset to the visible asset list. ([#21428](brave/brave-browser#21428))
 - Updated Brave Wallet portfolio network filter for multichain support. ([#20780](brave/brave-browser#20780))
 - Reduced adblock filter memory usage by optimizing unused regex rules. ([#21970](brave/brave-browser#21970))
 - Removed known Dialog Insight user tracking parameters from URLs. ([#22082](brave/brave-browser#22082))
 - Removed ability to swap ERC721 tokens with Brave Wallet. ([#21550](brave/brave-browser#21550))
 - Fixed crash which occurred when opening Brave Shields while using Google Meet. ([#22814](brave/brave-browser#22814))
 - Fixed inability to rename Solana account in Brave Wallet after it has been created. ([#22958](brave/brave-browser#22958))
 - Fixed incorrectly computed insufficient funds errors in Brave Wallet. ([#22877](brave/brave-browser#22877))
 - Fixed ERC20 and ERC721 transfers being incorrectly displayed as ETH transfers in the Brave Wallet transactions panel. ([#22044](brave/brave-browser#22044))
 - Fixed text alignment issues under the Brave Wallet "Recent transactions" panel when using long account names. ([#21216](brave/brave-browser#21216))
 - Fixed breakage in webpack build caused by OpenSSL 3.0. ([#22305](brave/brave-browser#22305))
 - Fixed two windows being opened on launch when the browser was installed without administrator privileges on Windows. ([#22179](brave/brave-browser#22179))
 - Upgraded Chromium to 102.0.5005.61. ([#22923](brave/brave-browser#22923)) ([Changelog for 102.0.5005.61](https://chromium.googlesource.com/chromium/src/+log/101.0.4951.67..102.0.5005.61?pretty=fuller&n=1000))
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
installer/omaha Issues for windows omaha installer OS/Desktop OS/Windows priority/P3 The next thing for us to work on. It'll ride the trains. QA Pass-Win64 QA/Yes release-notes/include
Projects
None yet
Development

Successfully merging a pull request may close this issue.

5 participants