feat: usage base billed - license key check - move to private api

[sean-brydon]

Not to be merged until private api has been deployed and migrations have been applied and we are ready to switch

More info here https://app.campsite.co/cal/posts/xdwtm9bcavqq

[github-actions]

Hey there and thank you for opening this pull request! 👋🏼

We require pull request titles to follow the Conventional Commits specification and it looks like your proposed title needs to be adjusted.

Details:

Unknown release type "Feat" found in pull request title "Feat: usage base billed - license key check - move to private api". 

Available types:
 - feat: A new feature
 - fix: A bug fix
 - docs: Documentation only changes
 - style: Changes that do not affect the meaning of the code (white-space, formatting, missing semi-colons, etc)
 - refactor: A code change that neither fixes a bug nor adds a feature
 - perf: A code change that improves performance
 - test: Adding missing tests or correcting existing tests
 - build: Changes that affect the build system or external dependencies (example scopes: gulp, broccoli, npm)
 - ci: Changes to our CI configuration files and scripts (example scopes: Travis, Circle, BrowserStack, SauceLabs)
 - chore: Other changes that don't modify src or test files
 - revert: Reverts a previous commit

[vercel]

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name	Status	Preview	Comments	Updated (UTC)
ai	✅ Ready (Inspect)	Visit Preview	💬 Add feedback	Jul 25, 2024 5:36pm

2 Skipped Deployments

Name	Status	Preview	Comments	Updated (UTC)
cal	⬜️ Ignored (Inspect)	Visit Preview		Jul 25, 2024 5:36pm
calcom-web-canary	⬜️ Ignored (Inspect)	Visit Preview		Jul 25, 2024 5:36pm

[sean-brydon]

It makes more sense to early return here if its found in .env. Saves a redundant db call

[zomars]

I think the DB is the preferred way to check/save the key. The env one is only for backwards compatibility. If a self hosted uses the wizard to set update the key, then it will never reach it unless it manually removes it from env. Although it makes sense to alert the user that it might have differing keys in the DB/env.

[sean-brydon]

I get this - but to set the license key in the wizard it needs to be set in the .env.

https://app.campsite.co/cal/posts/zxcq11vnt6tg#comment-l2toxbc9dr55

[sean-brydon]

Restores this to what it is in main (this is a based branch off the frontend changes for this api feature).
This file won't get used in newer self hosted instances. But will leave it here for now so we can track what happens with old self hosted instances.

[zomars]

@sean-brydon I need more context on CAL_SIGNATURE_TOKEN. Why is needed? Where in the code can I find more info about how to set it up?. What happens if a self hoster doesn't set it up properly?

[zomars]

Pushed a fix that hopefully get unit tests working.

[sean-brydon]

@sean-brydon I need more context on CAL_SIGNATURE_TOKEN. Why is needed? Where in the code can I find more info about how to set it up?. What happens if a self hoster doesn't set it up properly?

Hi zomars - i have DM'd you a bit more info but TLDR.

Its use to ensure self hosters can only increase billing ƒor their instance when signature tokens match