Skip to content

license-server: disable email-lookup activation for klipa#4

Merged
peterdsp merged 1 commit into
mainfrom
feat/klipa-file-only-activation
Jul 8, 2026
Merged

license-server: disable email-lookup activation for klipa#4
peterdsp merged 1 commit into
mainfrom
feat/klipa-file-only-activation

Conversation

@peterdsp

@peterdsp peterdsp commented Jul 8, 2026

Copy link
Copy Markdown
Owner

klipa's app now requires the signed license file to activate (not just an email), so the /activate email-lookup convenience is a freeload hole for it - anyone knowing a buyer's address could fetch their license.

  • Add a per-product email_activation flag. klipa → False: /activate returns 403 for klipa. PromptBar → True (unchanged, still activates by email).
  • Add inline_license (True for klipa): the signed license JSON is embedded in the purchase email body, not just attached, so buyers copy it straight from the message.

Verified locally: klipa /activate → 403; PromptBar /activate → 200 unchanged; klipa webhook still issues + emails with the inline license; both products list on /health.

klipa now requires the signed license file to activate, not just an
email, so the /activate email-lookup convenience is a freeload hole for
it (anyone knowing a buyer's address could fetch the license). Add a
per-product email_activation flag: klipa sets it False so /activate
returns 403 for klipa. PromptBar keeps email_activation=True, unchanged.

Also add inline_license (True for klipa): the signed license JSON is
embedded in the purchase email body, not just attached, so buyers can
copy it straight from the message and paste it into the app.
Copilot AI review requested due to automatic review settings July 8, 2026 14:33
@peterdsp peterdsp merged commit 60389bb into main Jul 8, 2026
1 check passed

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants