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.

Sign up for GitHub

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

Jump to bottom

Scala Common Enrich: fix unnecessarily-created JSON object as a result of the PII Enrichment #3636

Closed
BenFradet opened this issue Feb 23, 2018 · 1 comment
Closed

Scala Common Enrich: fix unnecessarily-created JSON object as a result of the PII Enrichment #3636

BenFradet opened this issue Feb 23, 2018 · 1 comment
Assignees
@knservis
Milestone
R106 Acropolis (P...

Comments

@BenFradet
Copy link
Contributor

For a specific invalid configuration scenario the code may create a new key pointing to an empty object. See: json-path/JsonPath#438

if someone finds a better title, I'm 👍
@BenFradet BenFradet added the bug label Feb 23, 2018
@BenFradet BenFradet added this to the R10x [STR] PII Enrichment pt.2 milestone Feb 23, 2018
@BenFradet BenFradet mentioned this issue Feb 23, 2018
Merged
7 tasks
@alexanderdean alexanderdean changed the title Scala Common Enrich: fix unnecessarily-created json object as a result of the pii enrichment Scala Common Enrich: fix unnecessarily-created JSON object as a result of the PII Enrichment Mar 2, 2018
knservis added a commit that referenced this issue May 31, 2018
@knservis
Failing: Scala Common Enrich: fix unnecessarily-created JSON object a…
e9b4722 
…s a result of the PII Enrichment #3636
@knservis
Copy link
Contributor

@BenFradet Added failing case.

knservis added a commit that referenced this issue Jun 1, 2018
@knservis
Scala Common Enrich: fix unnecessarily-created JSON object as a resul…
e040738 
…t of the PII Enrichment (closes #3636)
knservis added a commit that referenced this issue Jun 1, 2018
@knservis
Scala Common Enrich: fix unnecessarily-created JSON object as a resul…
b43e0a4 
…t of the PII Enrichment (closes #3636)
knservis added a commit that referenced this issue Jun 1, 2018
@knservis
Scala Common Enrich: fix unnecessarily-created JSON object as a resul…
63b8d2c 
…t of the PII Enrichment (closes #3636)
knservis added a commit that referenced this issue Jun 6, 2018
@knservis
Scala Common Enrich: fix unnecessarily-created JSON object as a resul…
c42dbfd 
…t of the PII Enrichment (closes #3636)
knservis added a commit that referenced this issue Jun 6, 2018
@knservis
Scala Common Enrich: fix unnecessarily-created JSON object as a resul…
ec6647f 
…t of the PII Enrichment (closes #3636)
knservis added a commit that referenced this issue Jun 14, 2018
@knservis
Scala Common Enrich: fix unnecessarily-created JSON object as a resul…
e0f37cd 
…t of the PII Enrichment (closes #3636)
knservis added a commit that referenced this issue Jun 14, 2018
@knservis
Scala Common Enrich: fix unnecessarily-created JSON object as a resul…
1e0f543 
…t of the PII Enrichment (closes #3636)
BenFradet pushed a commit that referenced this issue Jun 14, 2018
@knservis @BenFradet
Scala Common Enrich: fix unnecessarily-created JSON object as a resul…
e9ec8b0 
…t of the PII Enrichment (closes #3636)
oguzhanunlu pushed a commit to snowplow/common-enrich that referenced this issue May 29, 2020
@knservis @oguzhanunlu
Fix unnecessarily-created JSON object as a result of the PII Enrichme…
8169d82 
…nt (closes snowplow/snowplow#3636)
@chuwy chuwy mentioned this issue Sep 16, 2020
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@knservis knservis

Labels
None yet
Projects
None yet
Milestone
R106 Acropolis (PII Enrichment pt.2)
Development

No branches or pull requests
2 participants
@BenFradet @knservis