Skip to content

[8.0] MOD-7864: Obfuscation API#5176

Merged
kei-nan merged 1 commit into8.0from
backport-5060-to-8.0
Nov 6, 2024
Merged

[8.0] MOD-7864: Obfuscation API#5176
kei-nan merged 1 commit into8.0from
backport-5060-to-8.0

Conversation

@redisearch-backport-pull-request
Copy link
Contributor

Description

Backport of #5060 to 8.0.

* MOD-7864: initial commit

* * modify object obfuscate api

* * compilation fixes

* * fix tests

* * add query node obfuscation test

* * Code Review - Round #1

* * add field path obfuscation
* update tests

* * Code Review - Round #2

* * Code Review - Round #3

* * Code Review - Round #4

* * add MAX_UNIQUE_ID_TEXT_LENGTH_UPPER_BOUND, compute it based on sizeof t_uniqueId

* * add copyright

(cherry picked from commit 9235579)
@kei-nan kei-nan enabled auto-merge November 6, 2024 15:25
@kei-nan kei-nan added this pull request to the merge queue Nov 6, 2024
@codecov
Copy link

codecov bot commented Nov 6, 2024

Codecov Report

Attention: Patch coverage is 98.41270% with 1 line in your changes missing coverage. Please review.

Project coverage is 86.34%. Comparing base (1b0d90c) to head (4b44155).
Report is 1 commits behind head on 8.0.

Files with missing lines Patch % Lines
src/obfuscation/obfuscation_api.c 98.41% 1 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##              8.0    #5176      +/-   ##
==========================================
+ Coverage   86.33%   86.34%   +0.01%     
==========================================
  Files         189      190       +1     
  Lines       34413    34476      +63     
==========================================
+ Hits        29709    29767      +58     
- Misses       4704     4709       +5     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

Merged via the queue into 8.0 with commit b070459 Nov 6, 2024
@kei-nan kei-nan deleted the backport-5060-to-8.0 branch November 6, 2024 17:03
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.

1 participant