Add deep links redaction

[fathyb]

Ref: LIB-686

[f2prateek]

given the example fb123456789://authorize#access_token=secretsecretsecretsecret&some=data, wouldn't the desired output be fb123456789://authorize#access_token=((redacted/fb-auth-token))&some=data? (i.e. &some=data at the end.

[fathyb]

Facebook sends other private query parameters to the deep link that could be considered sensible, so we remove everything until we get a space ([^ ]).

[f2prateek]

Got it - sounds good 👍

[f2prateek]

Not a blocker, but we should consider instrument logging around when information is replaced with this mechanism. Otherwise folks may be confused about this behaviour.

[gerhard-segment]

Good point.

What is the risk of adding this ?

[f2prateek]

I just realized there is logging already - https://github.com/segmentio/analytics-ios/pull/798/files#diff-ecadcecb2e3287f62402c496069b01a2R85. This looks reasonable to me!

[codecov-io]

Codecov Report

Merging #798 into master will increase coverage by 0.84%.
The diff coverage is 96.11%.

@@            Coverage Diff            @@
##           master    #798      +/-   ##
=========================================
+ Coverage   85.66%   86.5%   +0.84%     
=========================================
  Files          52      52              
  Lines        2671    2772     +101     
=========================================
+ Hits         2288    2398     +110     
+ Misses        383     374       -9