Skip to content

Conversation

@philipphofmann
Copy link
Member

@philipphofmann philipphofmann commented Jan 29, 2021

📜 Description

Add sendDefaultPii to SentryOptions. When enabled, the SDK sends personal identifiable along with
events. Currently, that only applies to the IP address of the user.

💡 Motivation and Context

Fixes GH-910

💚 How did you test it?

📝 Checklist

  • I reviewed the submitted code
  • I added tests to verify the changes
  • I updated the CHANGELOG if needed
  • I updated the docs if needed
  • Review from the native team if needed
  • No breaking changes

🔮 Next steps

Update the docs.

Add sendDefaultPii to SentryOptions. When enabled, the SDK sends personal identifiable along with
events. Currently, that only applies to the IP address of the user.

Fixes GH-910
@philipphofmann philipphofmann self-assigned this Jan 29, 2021
@philipphofmann philipphofmann requested a review from a team January 29, 2021 10:35
@codecov-io
Copy link

codecov-io commented Jan 29, 2021

Codecov Report

Merging #923 (e1724e0) into master (e8d9357) will increase coverage by 0.06%.
The diff coverage is 100.00%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #923      +/-   ##
==========================================
+ Coverage   94.68%   94.74%   +0.06%     
==========================================
  Files          76       76              
  Lines        3497     3504       +7     
==========================================
+ Hits         3311     3320       +9     
+ Misses        186      184       -2     
Impacted Files Coverage Δ
Sources/Sentry/SentryClient.m 93.53% <100.00%> (+0.13%) ⬆️
Sources/Sentry/SentryOptions.m 100.00% <100.00%> (ø)
Sources/Sentry/SentryInstallation.m 95.45% <0.00%> (+9.09%) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update e8d9357...e1724e0. Read the comment docs.

Copy link
Member

@bruno-garcia bruno-garcia left a comment

Choose a reason for hiding this comment

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

LGTM 🚀

@philipphofmann philipphofmann enabled auto-merge (squash) February 1, 2021 10:10
philipphofmann added a commit to getsentry/sentry-docs that referenced this pull request Feb 1, 2021
We added this for Apple recently, see
getsentry/sentry-cocoa#923.
@philipphofmann philipphofmann merged commit 7654326 into master Feb 1, 2021
@philipphofmann philipphofmann deleted the feat/send-default-pii branch February 1, 2021 10:22
philipphofmann added a commit to getsentry/sentry-docs that referenced this pull request Feb 22, 2021
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.

Set user.ip_address to {{auto}} when sendDefaultPii=true

5 participants