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

fix: turn requiringSecureCoding on, adjust the available version #407

Merged
merged 1 commit into from
Jul 30, 2022

Conversation

liuyang1520
Copy link
Collaborator

Summary

  • fix: turn requiringSecureCoding on, adjust the available version

Noticed that we already switch to the recommended methods:

Enable the requiringSecureCoding as https://developer.apple.com/documentation/foundation/nskeyedarchiver/2962880-archiveddatawithrootobject recommended. Cannot find any usage of decodeObjectForKey or NSCoder or NSCoding in existing code base.

Checklist

  • Does your PR title have the correct title format?
  • Does your PR have a breaking change?: No

Sources/Amplitude/Amplitude.m Show resolved Hide resolved
@liuyang1520 liuyang1520 merged commit 53d7026 into main Jul 30, 2022
@liuyang1520 liuyang1520 deleted the fix-switch-to-secure-object-archiving branch July 30, 2022 03:26
github-actions bot pushed a commit that referenced this pull request Jul 31, 2022
## [8.11.1](v8.11.0...v8.11.1) (2022-07-31)

### Bug Fixes

* turn requiringSecureCoding on, adjust the available version ([#407](#407)) ([53d7026](53d7026))
@github-actions
Copy link

🎉 This PR is included in version 8.11.1 🎉

The release is available on:

Your semantic-release bot 📦🚀

junjie added a commit to junjie/Amplitude-iOS that referenced this pull request Sep 6, 2022
…entify-object

* commit 'f07749967fb8e628f9f2c743fab2ba6600574d85':
  chore(release): 8.12.0 [skip ci]
  feat: expose more function signature for logEvent (amplitude#409)
  chore: use the reusable workflow to create jira issue (amplitude#408)
  chore(release): 8.11.1 [skip ci]
  fix: turn requiringSecureCoding on, adjust the available version (amplitude#407)
  chore(release): 8.11.0 [skip ci]
  fix: deprecation fix (amplitude#404)
  feat: add EU SSL pinning cert, extent it to support multiple domains (amplitude#403)
  chore(release): 8.10.2 [skip ci]
  Revert "Compile with code coverage enabled (amplitude#398)" (amplitude#402)
  chore(release): 8.10.1 [skip ci]
  fix: wrong carrier info with deprecated method (amplitude#400)
  Compile with code coverage enabled (amplitude#398)
  chore(release): 8.10.0 [skip ci]
  feat: Add versionId to tracking plan data (amplitude#392)
  chore(release): 8.9.0 [skip ci]
  fix: fix release action
  Update Podfile.lock
  feat: Add support on Swift Package Manager for WatchOS (amplitude#381)
  docs: add Ampli SDK reference (amplitude#385)

# Conflicts:
#	Sources/Amplitude/AMPURLSession.m
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