Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

fix(privacy-protocols): clear NSUserDefaults on load and reload #53

Merged
merged 2 commits into from
Jan 10, 2025

Conversation

jboileau99
Copy link
Contributor

Description of this change

This mirrors a change made in ketch-android to clear privacy protocol key-value pairs when loading/reloading the SDK.

USPrivacy string:
Screenshot 2025-01-09 at 9 14 12 PM

USPrivacy:
Screenshot 2025-01-09 at 9 14 39 PM

TCF + GPP + USPrivacy:
Screenshot 2025-01-09 at 9 15 45 PM

GPP + USPrivacy:
Screenshot 2025-01-09 at 9 16 21 PM

Why is this change being made?

  • Chore (non-functional changes)
  • Bug fix (non-breaking change that fixes an issue)
  • New feature (non-breaking change that adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)

How was this tested? How can the reviewer verify your testing?

  • Tested by changing between jurisdictions having the privacy protocol combinations shown above, and then ensuring that the old keys are deleted

Related issues

Checklist

  • I have added tests to cover my changes.
  • All new and existing tests passed.
  • I have evaluated the security impact of this change, and OWASP Secure Coding Practices have been observed.
  • I have informed stakeholders of my changes.

@jboileau99 jboileau99 requested review from a team as code owners January 10, 2025 05:17
@jboileau99 jboileau99 merged commit c002c22 into main Jan 10, 2025
1 check passed
@jboileau99 jboileau99 deleted the jboileau99/fix/clear-nsuserdefaults branch January 10, 2025 05:31
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants