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

fix: add product scope to keygen provider #6975

Conversation

ezekg
Copy link
Contributor

@ezekg ezekg commented Jun 30, 2022

This PR adds a product qualifier to artifact download requests. Resolving an issue where, in the case of a Keygen account distributing multiple products using electron-builder, a conflict can arise due to a shared filename across products. In most cases, this happens with stable.yml and its yml friends.

This issue will present itself under the following circumstances:

  1. One or more products have an OPEN distribution strategy, meaning no license is required to view or download the artifacts. For example, Product A has a LICENSED distribution strategy, while Product B has an OPEN distribution strategy. This may result in Product A's updater requesting the latest.yml artifact without a product qualifier, causing our API to return the artifact of Product B, given it was released at a later date or has a higher version number.
  2. The licensee has access to more than 1 product via their licenses. For example, when authenticating as a Keygen user who has licenses for both Product A and Product B. This may, again, result in an artifact mix up.

Both of these scenarios result in a failed update.

@changeset-bot
Copy link

changeset-bot bot commented Jun 30, 2022

🦋 Changeset detected

Latest commit: 9a7f5ab

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 1 package
Name Type
electron-updater Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

@ezekg ezekg marked this pull request as ready for review June 30, 2022 19:38
@netlify
Copy link

netlify bot commented Jun 30, 2022

Deploy Preview for car-park-attendant-cleat-11576 ready!

Name Link
🔨 Latest commit 9a7f5ab
🔍 Latest deploy log https://app.netlify.com/sites/car-park-attendant-cleat-11576/deploys/62bdfb97ff783900081fe761
😎 Deploy Preview https://deploy-preview-6975--car-park-attendant-cleat-11576.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site settings.

@mmaietta mmaietta merged commit 8279d05 into electron-userland:master Jul 4, 2022
@github-actions github-actions bot mentioned this pull request Jul 4, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants