Skip to content

fix(query): added support for a new case in "elasticsearch domain not encrypted" query#7680

Merged
cx-miguel-dasilva merged 8 commits into
masterfrom
AST-40688
Sep 3, 2025
Merged

fix(query): added support for a new case in "elasticsearch domain not encrypted" query#7680
cx-miguel-dasilva merged 8 commits into
masterfrom
AST-40688

Conversation

@cx-ricardo-jesus

Copy link
Copy Markdown
Contributor

Closes #

Reason for Proposed Changes

  • The current implementation does not support the cases when the field Enabled is not defined inside the NodeToNodeEncryptionOptions block.

Proposed Changes

  • Added two more cases on the helper function called node_to_node_block_not_defined for the case that was not taken into account.
  • Added samples for the case mentioned above

I submit this contribution under the Apache-2.0 license.

@cx-ricardo-jesus cx-ricardo-jesus requested a review from a team as a code owner August 28, 2025 10:31
@github-actions github-actions Bot added the query New query feature label Aug 28, 2025
@github-actions

Copy link
Copy Markdown
Contributor

kics-logo

KICS version: v2.1.11

Category Results
CRITICAL CRITICAL 0
HIGH HIGH 0
MEDIUM MEDIUM 0
LOW LOW 0
INFO INFO 0
TRACE TRACE 0
TOTAL TOTAL 0
Metric Values
Files scanned placeholder 1
Files parsed placeholder 1
Files failed to scan placeholder 0
Total executed queries placeholder 47
Queries failed to execute placeholder 0
Execution time placeholder 0

@cx-miguel-dasilva cx-miguel-dasilva merged commit 30f53ab into master Sep 3, 2025
27 checks passed
@cx-miguel-dasilva cx-miguel-dasilva deleted the AST-40688 branch September 3, 2025 15:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

query New query feature

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants