Skip to content

feat: add support for BR-07.01 and BR-07.02 - #127

Merged
trumant merged 4 commits into
mainfrom
add-support-for-BR-07
Sep 21, 2025
Merged

trumant merged 4 commits into
mainfrom
add-support-for-BR-07

Conversation

@trumant

@trumant trumant commented Sep 5, 2025

Copy link
Copy Markdown
Contributor

This change adds support for the newly proposed
controls for secrets management within the project.

BR-07.01 is fully implemented and BR-07.02 is stubbed out

This change relates to the work in ossf/security-baseline#373

@trumant
trumant force-pushed the add-support-for-BR-07 branch 2 times, most recently from 0350d50 to 78fe205 Compare September 5, 2025 13:37
@trumant
trumant marked this pull request as ready for review September 5, 2025 13:39
@trumant
trumant requested a review from a team as a code owner September 5, 2025 13:39
Comment thread evaluation_plans/osps/build_release/evaluations.go Outdated
jmeridth
jmeridth previously approved these changes Sep 6, 2025

@jmeridth jmeridth left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Couple suggestions. Non-blocking.

Comment thread evaluation_plans/osps/build_release/evaluations.go Outdated
Comment thread evaluation_plans/osps/build_release/steps.go
Comment thread data/security-posture.go
@jmeridth

jmeridth commented Sep 6, 2025

Copy link
Copy Markdown
Member

@trumant Not 100% sure on the suggestion code alignments. They are rendering weird, so please double check them if used.

Comment thread evaluation_plans/osps/build_release/evaluations.go Outdated
jmeridth
jmeridth previously approved these changes Sep 6, 2025
Comment thread evaluation_plans/osps/build_release/evaluations.go Outdated
@jmeridth

Copy link
Copy Markdown
Member

@trumant ossf/security-baseline#373 merged. We good to move forward here? Do we need a dependency update on this PR?

@trumant

trumant commented Sep 18, 2025

Copy link
Copy Markdown
Contributor Author

@trumant ossf/security-baseline#373 merged. We good to move forward here? Do we need a dependency update on this PR?

Will take a look tomorrow to see where this stands.

trumant and others added 2 commits September 19, 2025 07:54
This change adds support for the newly proposed
controls for secrets management within the project.

BR-07.01 is fully implemented and BR-07.02 is stubbed out

This change relates to the work in ossf/security-baseline#373

Signed-off-by: Travis Truman <trumant@gmail.com>
Co-authored-by: Jason Meridth <35014+jmeridth@users.noreply.github.com>
@trumant
trumant force-pushed the add-support-for-BR-07 branch from 9ceb5ec to 8734307 Compare September 19, 2025 11:54
@kusari-inspector

kusari-inspector Bot commented Sep 19, 2025 •

Copy link
Copy Markdown

Kusari Inspector

Kusari Analysis Results:

Proceed with these changes

✅ No Flagged Issues Detected
All values appear to be within acceptable risk parameters.

Combined security analysis shows this PR is safe to merge. The dependency analysis confirms the addition of github.com/google/go-github/v74@v74.0.0 is a well-maintained Google library with excellent security scores (maintenance: 10/10, code review: 9/10) and zero known vulnerabilities. The code analysis reinforces this with completely clean results across all security categories - no code vulnerabilities, exposed secrets, workflow issues, or dependency problems. All 11 files passed security checks and govulncheck found no issues. This represents a routine maintenance update from a reputable source with minimal security risk and no identified threats.

Note

View full detailed analysis result for more information on the output and the checks that were run.


@kusari-inspector rerun - Trigger a re-analysis of this PR
@kusari-inspector feedback [your message] - Send feedback to our AI and team
See Kusari's documentation for setup and configuration.
Commit: 457d044, performed at: 2025-09-19T11:58:24Z

Found this helpful? Give it a 👍 or 👎 reaction!

@kusari-inspector

Copy link
Copy Markdown

Kusari PR Analysis rerun based on - 8734307 performed at: 2025-09-19T11:55:34Z - link to updated analysis

Co-authored-by: Jason Meridth <35014+jmeridth@users.noreply.github.com>
@trumant
trumant force-pushed the add-support-for-BR-07 branch from 8734307 to 8639a7a Compare September 19, 2025 11:58
@kusari-inspector

Copy link
Copy Markdown

Kusari PR Analysis rerun based on - 8639a7a performed at: 2025-09-19T11:59:08Z - link to updated analysis

@trumant
trumant enabled auto-merge (squash) September 19, 2025 12:00
@kusari-inspector

Copy link
Copy Markdown

Kusari PR Analysis rerun based on - 457d044 performed at: 2025-09-21T15:51:54Z - link to updated analysis

@trumant
trumant dismissed eddie-knight’s stale review September 21, 2025 16:00

Dismissing as the requested changes were made and the PR has since received review

@trumant
trumant merged commit e91fedc into main Sep 21, 2025
7 checks passed
@trumant
trumant deleted the add-support-for-BR-07 branch September 21, 2025 16:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants