Skip to content

Remove pii data from headers#9

Merged
inbaltako merged 4 commits intomasterfrom
dev
Nov 30, 2020
Merged

Remove pii data from headers#9
inbaltako merged 4 commits intomasterfrom
dev

Conversation

@inbaltako
Copy link
Contributor

No description provided.

@inbaltako inbaltako requested review from AmitBu and alexivsn November 24, 2020 09:26
@codecov-io
Copy link

codecov-io commented Nov 29, 2020

Codecov Report

Merging #9 (ff7026e) into master (6e529f6) will increase coverage by 11.35%.
The diff coverage is 76.19%.

Impacted file tree graph

@@              Coverage Diff              @@
##             master       #9       +/-   ##
=============================================
+ Coverage          0   11.35%   +11.35%     
- Complexity        0      250      +250     
=============================================
  Files             0       15       +15     
  Lines             0      555      +555     
=============================================
+ Hits              0       63       +63     
- Misses            0      492      +492     
Impacted Files Coverage Δ Complexity Δ
src/ConfigurationManager.php 0.00% <0.00%> (ø) 23.00 <0.00> (?)
src/SecureNativeContext.php 0.00% <0.00%> (ø) 17.00 <0.00> (?)
src/Utils.php 42.16% <86.36%> (ø) 44.00 <17.00> (?)
src/SecureNativeOptions.php 32.55% <86.66%> (ø) 50.00 <17.00> (?)
src/SecureNative.php 0.00% <0.00%> (ø) 23.00% <0.00%> (?%)
src/Models/SecurenativeEvent.php 0.00% <0.00%> (ø) 1.00% <0.00%> (?%)
src/Models/Context.php 0.00% <0.00%> (ø) 1.00% <0.00%> (?%)
src/HttpClient.php 0.00% <0.00%> (ø) 3.00% <0.00%> (?%)
src/Models/UserTraits.php 0.00% <0.00%> (ø) 1.00% <0.00%> (?%)
... and 9 more

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 6e529f6...ff7026e. Read the comment docs.

Copy link
Contributor

@AmitBu AmitBu left a comment

Choose a reason for hiding this comment

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

Looks good, approved 🙂

@inbaltako inbaltako merged commit e0023fb into master Nov 30, 2020
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.

3 participants