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

Safing.Portmaster version 0.7.14 #43095

Closed
wants to merge 1 commit into from
Closed

Safing.Portmaster version 0.7.14 #43095

wants to merge 1 commit into from

Conversation

derkrasseleo
Copy link
Contributor

@derkrasseleo derkrasseleo commented Jan 26, 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.0 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 Jan 26, 2022
@wingetbot wingetbot added Azure-Pipeline-Passed Validation pipeline passed. There may still be manual validation requirements. Validation-Executable-Error Needs-Attention This work item needs to be reviewed by a member of the core team. Possible-Duplicate labels Jan 26, 2022
@wingetbot
Copy link
Collaborator

Possible duplicate package entry.

Similar installer SHA256 hash found in manifest:

  • InstallerSha256: 2a7ea1568f83f5579c9921975a05aaf7e853bc29e5f5009d86ee8c1c27c5d336
    • manifests\s\Safing\Portmaster\0.7.0

@derkrasseleo
Copy link
Contributor Author

The hash is the same?!

@derkrasseleo
Copy link
Contributor Author

It's an update, so the hash should be the same right?

@ItzLevvie
Copy link
Contributor

It's an update, so the hash should be the same right?

Looks like 0.7.0 installs 0.7.14 so you should probably fix the version in the 0.7.0 package.

https://github.com/safing/portmaster/releases/tag/v0.7.14

@derkrasseleo
Copy link
Contributor Author

It's an update, so the hash should be the same right?

Looks like 0.7.0 installs 0.7.14 so you should probably fix the version in the 0.7.0 package.

https://github.com/safing/portmaster/releases/tag/v0.7.14

The 0.7.0 manifest's download URL is https://updates.safing.io/windows_amd64/packages/portmaster-installer_v0-7-0.exe
So it probably installs 0.7.0. But I downloaded both versions, and they seem to have the exact same size and hash, so maybe they're the same after all.
So should I update the version AND the installer URL of the 0.7.0 manifest?

@OfficialEsco
Copy link
Contributor

Because the Portmaster auto-updates, the installers we distribute are only minimal and usually only describe themselves, similar to all these "light or web installers" that only download the real deal when they are run.

Based on that, I have the following feedback:

  • Depending on the usage and behavior of winget, possibly use an identifier specific to the installer: Safing.Portmaster.Installer

    • While this would be more technically correct, just using Safing.Portmaster probably makes more sense for all practical use cases.
  • Use the version of the installer: v0.5.2

  • Use the direct link to the installer, not the "latest" redirect: https://updates.safing.io/windows_amd64/packages/portmaster-installer_v0-5-2.exe

    • Especially as you are providing a hash for it.
  • Mention to the user that the Portmaster auto-updates when doing a silent install, if possible.

Also, we will be issuing an update to the installer within the next month that will change the installation experience: The Portmaster service will not be started automatically after the install finished, but the user will be prompted to restart the device to complete installation. It would be interesting how this could be communicated in the install process here. If the installer is run interactively, then this should not be a problem, but I wonder how the silent installs would then work.

I'm not all too familiar with winget, so take my advice with a pinch of salt.

Originally posted by dhaavi in #11651 (comment)

@derkrasseleo
Copy link
Contributor Author

Thank you for clarifying! Didn't know the devs were already on it..

@derkrasseleo derkrasseleo deleted the autogenerated/Safing.Portmaster/804e6089-0353-4c4a-b329-c712deb638c9 branch January 26, 2022 15:16
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. Possible-Duplicate Validation-Executable-Error
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants