Skip to content

feat: add confluence page id to secret extra details and to sarif properties#357

Merged
cx-leonardo-fontes merged 3 commits intomasterfrom
add-pageId-sarif
Dec 12, 2025
Merged

feat: add confluence page id to secret extra details and to sarif properties#357
cx-leonardo-fontes merged 3 commits intomasterfrom
add-pageId-sarif

Conversation

@cx-leonardo-fontes
Copy link
Copy Markdown
Contributor

No description provided.

@github-actions
Copy link
Copy Markdown

kics-logo

KICS version: v1.7.13

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

@cx-leonardo-fontes cx-leonardo-fontes changed the title feat: add confluence page id to secret extra details (yaml/json) and to sarif properties feat: add confluence page id to secret extra details and to sarif properties Dec 12, 2025
@github-actions
Copy link
Copy Markdown

github-actions Bot commented Dec 12, 2025

Logo
Checkmarx One – Scan Summary & Details410437b9-0a0b-423e-89e4-4c05b5d5886d

Great job! No new security vulnerabilities introduced in this pull request

@cx-leonardo-fontes cx-leonardo-fontes marked this pull request as ready for review December 12, 2025 16:24
@cx-leonardo-fontes cx-leonardo-fontes requested a review from a team as a code owner December 12, 2025 16:24
Comment thread lib/reporting/report_test.go Outdated
},
},
{
name: "confluence.pageId from secret extra detail should be in result properties",
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.

The name of the test is confusing, should we update it?

@cx-leonardo-fontes cx-leonardo-fontes merged commit f606f03 into master Dec 12, 2025
12 checks passed
@cx-leonardo-fontes cx-leonardo-fontes deleted the add-pageId-sarif branch December 12, 2025 17:25
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.

2 participants