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 advancedFraudSignalsEnabled property #1560

Merged
merged 1 commit into from
Apr 28, 2020

Conversation

davidme-stripe
Copy link
Contributor

@davidme-stripe davidme-stripe commented Apr 28, 2020

This allows you to control whether the Stripe SDK is collecting additional device data for fraud prevention.
For more details on the information we collect, visit https://stripe.com/docs/disputes/prevention/advanced-fraud-detection

@@ -81,6 +82,19 @@ + (NSString *)defaultPublishableKey {
return _defaultPublishableKey;
}

+ (void)setAdvancedFraudSignalsEnabled:(BOOL)enabled {
[self advancedFraudSignalsEnabled];
Copy link
Contributor

Choose a reason for hiding this comment

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

analyzer might complain about unused result here

This allows you to control whether the Stripe SDK is collecting additional device data for fraud prevention.
For more details on the information we collect, visit https://stripe.com/docs/disputes/prevention/advanced-fraud-detection
@davidme-stripe
Copy link
Contributor Author

Travis-ci.org seems to be down. 😢 I ran this build on a separate CI instance, and it passed, so I'm going to merge this.

@davidme-stripe davidme-stripe merged commit 9320104 into master Apr 28, 2020
@davidme-stripe davidme-stripe deleted the davidme/advanced-fraud-signals branch April 28, 2020 20:20
kgaidis-stripe added a commit that referenced this pull request Dec 2, 2022
Financial Connections: added support for public_token returned from oauth results call.
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.

3 participants