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

Add support for Identity-Based Pricing #1995

Merged
merged 5 commits into from
Jun 25, 2024
Merged

Conversation

dangtony98
Copy link
Collaborator

@dangtony98 dangtony98 commented Jun 19, 2024

Description 📣

This PR adds support for identity-based pricing.

Logic: In order to implement identity-based pricing, we create a new price in Stripe with seat_type metadata (can be either user or identity). When Infisical fetches back the feature set from the license server, the license server computes identitiesUsed and has identityLimit based on the Stripe product + metadata on the price. The Infisical platform then imposes the necessary constraints at the frontend UI-level and backend.

Included in this PR:

  • If identityLimit is specified in the customer's feature set, then checks are imposed for example when inviting a new user to an organization or when a new user logs into an organization via SAML / LDAP.
  • When users are added/removed from an organization, Infisical updates the license server with the new member and identity counts.

Note: Existing users of Infisical are not affected by the price change; only new users that will be bound to the identity-based prices on Stripe.

Type ✨

  • Bug fix
  • New feature
  • Breaking change
  • Documentation

@dangtony98 dangtony98 marked this pull request as draft June 19, 2024 21:01
@dangtony98 dangtony98 marked this pull request as ready for review June 19, 2024 21:20
@dangtony98 dangtony98 requested review from maidul98 and akhilmhdh and removed request for maidul98 June 19, 2024 21:20
akhilmhdh
akhilmhdh previously approved these changes Jun 25, 2024
@dangtony98 dangtony98 merged commit 514df55 into main Jun 25, 2024
6 checks passed
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.

None yet

2 participants