Skip to content

fix(query): small fixes on the query "Azure App Service Client Certificate Disabled" for Terraform#7634

Merged
cx-artur-ribeiro merged 1 commit into
masterfrom
AST-98823
Aug 7, 2025
Merged

fix(query): small fixes on the query "Azure App Service Client Certificate Disabled" for Terraform#7634
cx-artur-ribeiro merged 1 commit into
masterfrom
AST-98823

Conversation

@cx-ricardo-jesus

@cx-ricardo-jesus cx-ricardo-jesus commented Aug 7, 2025

Copy link
Copy Markdown
Contributor

Closes #

Reason for Proposed Changes

  • This query was flagging an incorrect value for search_line on the results and stating MISSING on the results of one of the policies.

Proposed Changes

  • Removed the negative sample that should not be there
  • Fixed the sprintf arguments on the last policy.
  • Fixed buid_search_line arguments in order to return the correct value.

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 7, 2025 17:01
@github-actions github-actions Bot added query New query feature terraform Terraform query azure PR related with Azure Cloud labels Aug 7, 2025
@github-actions

github-actions Bot commented Aug 7, 2025

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-artur-ribeiro
cx-artur-ribeiro approved these changes Aug 7, 2025
View reviewed changes

@cx-artur-ribeiro cx-artur-ribeiro left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM

@cx-artur-ribeiro cx-artur-ribeiro merged commit 1691b0a into master Aug 7, 2025
27 checks passed
@cx-artur-ribeiro cx-artur-ribeiro deleted the AST-98823 branch August 7, 2025 18:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

1 more reviewer

@cx-artur-ribeiro cx-artur-ribeiro cx-artur-ribeiro approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

@cx-ricardo-jesus cx-ricardo-jesus

Labels

azure PR related with Azure Cloud query New query feature terraform Terraform query

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@cx-ricardo-jesus @cx-artur-ribeiro