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

Kiwix.KiwixJS version 2.0.9.0 #64990

Conversation

Jaifroid
Copy link
Contributor

@Jaifroid Jaifroid commented Jul 2, 2022

  • Have you signed the Contributor License Agreement?
  • Have you checked that there aren't other open pull requests for the same manifest update/change?
  • Have you validated your manifest locally with winget validate --manifest <path>?
  • Have you tested your manifest locally with winget install --manifest <path>?
  • Does your manifest conform to the 1.1 schema?

Note: <path> is the name of the directory containing the manifest you're submitting.


Microsoft Reviewers: Open in CodeFlow

@wingetbot
Copy link
Collaborator

Service Badge  Service Badge  

@wingetbot
Copy link
Collaborator

/AzurePipelines run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@ghost ghost added the Moderator-Approved One of the Moderators has reviewed and approved this PR label Jul 2, 2022
@wingetbot wingetbot added the Validation-Installation-Error Error installing application label Jul 2, 2022
@ghost
Copy link

ghost commented Jul 2, 2022

Hello @Jaifroid,
The package manager bot determined there was an issue with installing the application correctly. Please check the application installs correctly. Once repaired, please push an update to your pull request.

@ghost ghost added the Needs-Author-Feedback This needs a response from the author. label Jul 2, 2022
@Jaifroid
Copy link
Contributor Author

Jaifroid commented Jul 2, 2022

@ItzLevvie I rebuilt this app and signed it again, because validation was failing before. It's still failing, yet it installs fine locally (see screenshot below). I cannot find anything wrong with the package. I can only assume now that there is a problem in the winget validation pipeline. It wouldn't be the first time I've had this issue with appxbundle packages, and it has needed developer attention to resolve it. I believe the winget devs don't test regularly on appxbundles that contain HTML/JS UWP apps. Can this PR be flagged for developer attention, please?

image

@ghost ghost added Needs-Attention This work item needs to be reviewed by a member of the core team. and removed Needs-Author-Feedback This needs a response from the author. labels Jul 2, 2022
@ItzLevvie
Copy link
Contributor

Monday, 4th of July 2022 is a holiday in the United States so the WinGet engineering team will check back on Tuesday, 5th of July 2022.

cc @denelon: This pull request needs an engineer's attention.

@AmelBawa-msft
Copy link
Contributor

/azp run

@ghost ghost removed Validation-Installation-Error Error installing application Needs-Attention This work item needs to be reviewed by a member of the core team. labels Jul 5, 2022
@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@wingetbot wingetbot added the Validation-Installation-Error Error installing application label Jul 5, 2022
@ghost
Copy link

ghost commented Jul 5, 2022

Hello @Jaifroid,
The package manager bot determined there was an issue with installing the application correctly. Please check the application installs correctly. Once repaired, please push an update to your pull request.

@ghost ghost added the Needs-Author-Feedback This needs a response from the author. label Jul 5, 2022
@Trenly
Copy link
Contributor

Trenly commented Jul 5, 2022

2022-07-05 09:32:09.365 [CORE] Deployment operation #0: error 0x800B0110: Opening the package from location KiwixWebApp_2.0.9.0_AnyCPU.appxbundle failed.
2022-07-05 09:32:09.367 [FAIL] D:\a\_work\1\s\external\pkg\src\AppInstallerCommonCore\Deployment.cpp(159)\WindowsPackageManager.dll!00007FFECACF7684: (caller: 00007FFECAC895E5) Exception(1) tid(cec) 800B0110 The certificate is not valid for the requested usage.

I get this error even when running locally

@Jaifroid
Copy link
Contributor Author

Jaifroid commented Jul 5, 2022

Installing on Windows 11, standard mode (not developer), the file referenced by the manifest is this one: https://github.com/kiwix/kiwix-js-windows/releases/download/v2.0.9/KiwixWebApp_2.0.9.0_AnyCPU.appxbundle

Download it, double click it, app installer loads:

image

Installs correctly.

@ghost ghost added Needs-Attention This work item needs to be reviewed by a member of the core team. and removed Needs-Author-Feedback This needs a response from the author. labels Jul 5, 2022
@Jaifroid
Copy link
Contributor Author

Jaifroid commented Jul 5, 2022

Uninstall above install and now try using winget:

image

@Trenly could you kindly explain the steps necessary to reproduce the error you're seeing, because I can't see it locally in Windows 11, either with the package installer or with winget.

@Trenly
Copy link
Contributor

Trenly commented Jul 5, 2022

Windows 11, either with the package installer or with winget.

Clone the repo; Open powershell; CD to the folder and run ./Tools/PRTest.ps1 64990

@Jaifroid
Copy link
Contributor Author

Jaifroid commented Jul 5, 2022

I've just tested installation of the package on a friend's PC. Windows 11 Pro, not in developer mode. Her windows is Spanish language. Installs correctly, says "Kiwix JS is ready":

image

@Jaifroid
Copy link
Contributor Author

Jaifroid commented Jul 5, 2022

But the package installs fine with winget. How can I find out what the "requested usage" is that the certificate is not valid for? Clearly it is valid for installation of the package, or winget would fail to install it.

@Trenly
Copy link
Contributor

Trenly commented Jul 5, 2022

I found this related issue

I also found this image, showing how sometimes your system setup restricts the certificate -
image

This article on Extended Key Usage may also be helpful.

@Jaifroid
Copy link
Contributor Author

Jaifroid commented Jul 5, 2022

Thank you very much @Trenly for the interesting research that may help to identify the issue as being a problem with Windows 11 keychain checking.

The thing is, no matter how much I adjust the certificate settings on my local machine, it will not change the fact that the error is manifesting in the PR validation script running on Azure, and that the error has manifested only this month when I first submitted KiwixJS 2.0.9.0 -- all the previous versions of Kiwix JS prior to July have validated fine, using the same certificate and build chain.

Having said that, the current package is built with our new org certificate (the previous one runs out end of August). New certificate, same error, only on the .appxbundle package. At the same time I (originally) submitted 2.0.9.0, I also submitted an .exe Electron package built with the same (previous) certificate, which passed with no problems.

So: a certificate is valid when submitting an .exe package, but fails when submitting an .appxbundle. But the appxbundle installs fine with winget anyway. This suggests an inconsistency in the PR Testing script to me, though I'm happy to try to test other possibilities.

@wingetbot wingetbot added Validation-Completed Validation passed and removed Validation-Completed Validation passed labels Jul 11, 2022
@denelon denelon added Validation-Completed Validation passed and removed Validation-Completed Validation passed labels Jul 12, 2022
@Jaifroid
Copy link
Contributor Author

@amirms Thank you for adding a waiver for this app. @ItzLevvie Would you be able to mark this as reviewed again, as merging is blocked?

@ghost ghost added Needs-Attention This work item needs to be reviewed by a member of the core team. and removed Needs-Author-Feedback This needs a response from the author. labels Jul 13, 2022
@Jaifroid
Copy link
Contributor Author

@denelon This has passed validation, has the required review, but the bot still thinks it is blocked due to missing review...

Copy link
Contributor

@AmelBawa-msft AmelBawa-msft left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Manually approving since bot was not triggered post "validation-completed" label
CC: @denelon

@ghost ghost merged commit d29f88b into microsoft:master Jul 13, 2022
@Jaifroid Jaifroid mentioned this pull request Jul 18, 2022
5 tasks
This pull request was closed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Azure-Pipeline-Passed Validation pipeline passed. There may still be manual validation requirements. Moderator-Approved One of the Moderators has reviewed and approved this PR Needs-Attention This work item needs to be reviewed by a member of the core team. Validation-Completed Validation passed Waived-Validation-Installation-Error
Projects
None yet
Development

Successfully merging this pull request may close these issues.

7 participants