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

A web page is opened instead of showing the installer after clicking 'Install Now/Install on Close' #7684

Closed
v-xianya opened this issue Jan 18, 2024 · 8 comments
Assignees
Labels
🪲 regression Issue was working in a previous version 💻 windows Related to Windows platforms 🧪 testing Found through regular testing ✅ merged A fix for this issue has been merged
Milestone

Comments

@v-xianya
Copy link
Member

Storage Explorer Version: 1.33.0-dev (96)
Build Number: 20240117.7
Branch: main
Platform/OS: Windows 10
Architecture: x64
How Found: From running test cases
Regression From: Previous release (1.32.1)

Steps to Reproduce

  1. Close Storage Explorer if it is already open.
  2. Open a terminal.
  3. Navigate to where the Storage Explorer executable is located.
  4. Open Storage Explorer by running the command: .\StorageExplorer.exe --add-update-group=CTI.
  5. There is an update notification -> Click 'Install Now'.
    image
  6. Check whether Storage Explorer disappears, and the installer appears.

Expected Experience

Storage Explorer disappears, and the installer appears.
image

Actual Experience

  1. Storage Explorer disappears.
  2. A webpage is opened.
    image
@v-xianya v-xianya added 🧪 testing Found through regular testing 🪲 regression Issue was working in a previous version 💻 windows Related to Windows platforms labels Jan 18, 2024
@MRayermannMSFT MRayermannMSFT self-assigned this Jan 18, 2024
@MRayermannMSFT MRayermannMSFT added this to the 1.33.0 milestone Jan 18, 2024
@MRayermannMSFT
Copy link
Member

MRayermannMSFT commented Jan 18, 2024

@v-xianya can you try repro'ing this again tonight? I'm wondering if it was a transient issue, as I could not reproduce. If you do reproduce, please share your Storage Explorer log files to this issue. Thanks!

@v-xianya
Copy link
Member Author

Hi @MRayermannMSFT I can reproduce this issue on the main build 20240119.2. Here are the logs.
2024-01-18_174817.zip
2024-01-18_175917.zip

@MRayermannMSFT
Copy link
Member

MRayermannMSFT commented Jan 22, 2024

@v-xianya , while on a machine that repros this problem can you please do the following investigation:

  1. Uninstall any previously installed version of Storage Explorer
  2. Download this version of Storage Explorer: https://github.com/microsoft/AzureStorageExplorer/releases/download/v1.29.1/StorageExplorer-windows-ia32.exe
  3. Right click on the .exe > Properties
  4. Share a screenshot like the one below (you'll need to go to the "Digital Signatures" tab of the Properties dialog, then click "View Certificate" on the "Digital Signatures Details" dialog.
  5. Try installing that version of Storage Explorer - let us know if successful
  6. If successful, see if it offers to update. If not successful, please try the remaining steps from version 1.30.0.
  7. Click "Install Now"
  8. Let us know if the installer launches or a webpage is opened
    image

@v-xianya
Copy link
Member Author

Hi @MRayermannMSFT

  1. Below is the screenshot:
    image
  2. We successfully installed the provided version of storage explorer. After installing successfully, Click -> 'Help -> Check for Updates' -> There is no update available.
    Then we run command ".\StorageExplorer.exe --add-update-group=CTI", the Update notification prompts.

image
3. Click 'Install Now' -> Storage Explorer disappears and the below window prompt.
image

@MRayermannMSFT
Copy link
Member

I was finally able to reproduce this. Will give more updates if/when I have any.

@MRayermannMSFT
Copy link
Member

MRayermannMSFT commented Jan 26, 2024

b481e4a4 | Jan 25, 2024 | Reproduces
604cb6c1 | Jan 19, 2024 | Reproduces
ab99c0dd | Jan 17, 2024 | Unknown
223f9fe8 | Jan 16, 2024 | Unknown
5f0c6afe | Jan 12, 2024 | Unknown
1668eeba | Jan 12, 2024 | Unknown
35dcc616 | Jan 9, 2024 | Unknown
e2ceefac | Jan 8, 2024 | Reproduces
0ed461f1 | Jan 5, 2024 | Works

@MRayermannMSFT
Copy link
Member

I believe I have merged a fix. @v-xianya please verify. :)

@MRayermannMSFT MRayermannMSFT added the ✅ merged A fix for this issue has been merged label Jan 26, 2024
@v-xianya
Copy link
Member Author

Verified this issue on the main build 20240128.2. Fixed.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🪲 regression Issue was working in a previous version 💻 windows Related to Windows platforms 🧪 testing Found through regular testing ✅ merged A fix for this issue has been merged
Projects
None yet
Development

No branches or pull requests

2 participants