Skip to content

fix: stop treating a closed store as an invalid license - #40

Merged
TobbenTM merged 1 commit into
mainfrom
shanghai-v3/closed-store-response-update
Oct 9, 2026
Merged

TobbenTM merged 1 commit into
mainfrom
shanghai-v3/closed-store-response-update

Conversation

@TobbenTM

@TobbenTM TobbenTM commented Oct 8, 2026

Copy link
Copy Markdown
Member

store_closed_error derived from license_invalid_error, so a store that had closed its Moonbase account locked the license, and apps that delete the stored license on an invalid one dropped a license the buyer still owns. It now derives from moonbase_error and reads like being offline: validate_token_online keeps the license within the grace period, the unsigned answer relaxes nothing, and an offline license returned by a closed store's validation is stored and never re-validated, as for any offline license. The JUCE controller no longer locks on it, and deactivating on a closed store keeps the license and says why, since the device could not be activated again. Note for upgraders: code that caught license_invalid_error to handle a closed store must catch store_closed_error instead; new core and JUCE tests cover these paths, and the full suite passes on macOS with JUCE 8.0.4.

store_closed_error derived from license_invalid_error, so a store that had
closed its Moonbase account locked the license, and apps that delete the
stored license on an invalid one dropped a license the buyer still owns.

It now derives from moonbase_error and reads like being offline: within the
grace period validate_token_online keeps the license, and since the answer is
not signed it relaxes nothing. When a closed store answers a validation with
an offline license, the SDK stores it and never asks again, as for any
offline license.

The JUCE controller no longer locks on it, and deactivating on a closed store
keeps the license and says why, since the device could not be activated
again. Code that caught license_invalid_error to handle a closed store must
catch store_closed_error instead.
@argos-ci

argos-ci Bot commented Oct 8, 2026 •

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Argos notifications ↗︎

Build Status Details Updated (UTC)
default (Inspect) ✅ No changes detected - Oct 8, 2026, 4:12 PM

@TobbenTM
TobbenTM merged commit f9d8e09 into main Oct 9, 2026
17 checks passed
github-actions Bot pushed a commit that referenced this pull request Oct 9, 2026
## [4.9.1](v4.9.0...v4.9.1) (2026-10-09)

### Bug Fixes

* stop treating a closed store as an invalid license ([#40](#40)) ([f9d8e09](f9d8e09))
@github-actions

github-actions Bot commented Oct 9, 2026

Copy link
Copy Markdown

🎉 This PR is included in version 4.9.1 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant