Skip to content

Entitlement verification: Hide EntitlementVerificationMode.ENFORCED during beta and improve documentation#852

Merged
tonidero merged 2 commits into
entitlements-verificationfrom
toniricodiez/sdk-2923-hide-enforced-mode-for-sdk-security
Mar 14, 2023
Merged

Entitlement verification: Hide EntitlementVerificationMode.ENFORCED during beta and improve documentation#852
tonidero merged 2 commits into
entitlements-verificationfrom
toniricodiez/sdk-2923-hide-enforced-mode-for-sdk-security

Conversation

@tonidero

@tonidero tonidero commented Mar 8, 2023

Copy link
Copy Markdown
Contributor

Description

Completes SDK-2923.

We decided to pull the ENFORCED mode during beta of the entitlements verification feature. This PR also improves and adds some missing documentation.

Note that the internal SignatureVerificationMode.enforced still exists and tests remain, but it won't be available through the public API.

@tonidero tonidero requested a review from a team March 8, 2023 17:07
@tonidero tonidero marked this pull request as ready for review March 8, 2023 17:07
@tonidero tonidero force-pushed the toniricodiez/sdk-2923-hide-enforced-mode-for-sdk-security branch from f627da9 to 2f307b3 Compare March 8, 2023 17:11
@codecov

codecov Bot commented Mar 8, 2023

Copy link
Copy Markdown

Codecov Report

Merging #852 (2f307b3) into entitlements-verification (89a0e58) will increase coverage by 0.01%.
The diff coverage is 100.00%.

@@                      Coverage Diff                      @@
##           entitlements-verification     #852      +/-   ##
=============================================================
+ Coverage                      83.28%   83.29%   +0.01%     
=============================================================
  Files                            139      139              
  Lines                           4546     4544       -2     
  Branches                         600      599       -1     
=============================================================
- Hits                            3786     3785       -1     
  Misses                           540      540              
+ Partials                         220      219       -1     
Impacted Files Coverage Δ
...s/common/verification/SignatureVerificationMode.kt 94.44% <ø> (+4.97%) ⬆️
...n/java/com/revenuecat/purchases/EntitlementInfo.kt 48.35% <ø> (ø)
.../java/com/revenuecat/purchases/EntitlementInfos.kt 80.00% <ø> (ø)
...com/revenuecat/purchases/PurchasesConfiguration.kt 94.44% <ø> (ø)
...evenuecat/purchases/EntitlementVerificationMode.kt 100.00% <100.00%> (ø)

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

@tonidero tonidero merged commit ab06d01 into entitlements-verification Mar 14, 2023
@tonidero tonidero deleted the toniricodiez/sdk-2923-hide-enforced-mode-for-sdk-security branch March 14, 2023 09:01
tonidero added a commit that referenced this pull request Mar 15, 2023
…uring beta and improve documentation (#852)

### Description
Completes
[SDK-2923](https://linear.app/revenuecat/issue/SDK-2923/hide-enforced-mode-for-sdk-security).

We decided to pull the ENFORCED mode during beta of the entitlements
verification feature. This PR also improves and adds some missing
documentation.

Note that the internal `SignatureVerificationMode.enforced` still exists
and tests remain, but it won't be available through the public API.
@tonidero tonidero mentioned this pull request Mar 31, 2023
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