Skip to content

chore(ci): bump org.bouncycastle:bc-fips from 2.1.2 to 2.1.3 - #407

Open
dependabot[bot] wants to merge 1 commit into
mainfrom
dependabot/maven/org.bouncycastle-bc-fips-2.1.3
Open

dependabot[bot] wants to merge 1 commit into
mainfrom
dependabot/maven/org.bouncycastle-bc-fips-2.1.3

Conversation

@dependabot

@dependabot dependabot Bot commented on behalf of github Sep 18, 2026 •

Copy link
Copy Markdown
Contributor

Bumps org.bouncycastle:bc-fips from 2.1.2 to 2.1.3.

@dependabot dependabot Bot added dependencies Pull requests that update a dependency file java Pull requests that update java code labels Sep 18, 2026
@dependabot
dependabot Bot requested review from a team as code owners September 18, 2026 18:26
@dependabot dependabot Bot added the java Pull requests that update java code label Sep 18, 2026
@dependabot
dependabot Bot requested a review from a team as a code owner September 18, 2026 18:26
@coderabbitai

coderabbitai Bot commented Sep 18, 2026 •

Copy link
Copy Markdown
Contributor

Important

Review skipped

Bot user detected.

To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration
  • Configuration used: Organization UI
  • Review profile: CHILL
  • Plan: Advanced
  • Run ID: eaf7a701-4879-4628-9534-479eec058304

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • 🔍 Trigger review
  • Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@github-actions

Copy link
Copy Markdown
Contributor

@dependabot
dependabot Bot force-pushed the dependabot/maven/org.bouncycastle-bc-fips-2.1.3 branch from 14d2c94 to 8d1dcd7 Compare September 25, 2026 18:23
@github-actions

Copy link
Copy Markdown
Contributor

@dependabot
dependabot Bot force-pushed the dependabot/maven/org.bouncycastle-bc-fips-2.1.3 branch from 8d1dcd7 to 085bf56 Compare October 1, 2026 17:06
@github-actions

github-actions Bot commented Oct 1, 2026

Copy link
Copy Markdown
Contributor

@dependabot
dependabot Bot force-pushed the dependabot/maven/org.bouncycastle-bc-fips-2.1.3 branch from 085bf56 to c0be26b Compare October 15, 2026 18:35
@github-actions

Copy link
Copy Markdown
Contributor

dmihalcik-virtru added a commit that referenced this pull request Oct 21, 2026
## Summary

- **checks.yaml / mavenverify:** pass `-Dsonar.skip=true` when
`SONAR_TOKEN` is empty. Pushes to `main` and same-repo PRs still run the
Sonar analysis.
- **codeql.yaml:** pass `github_token` to `buf-setup-action`, as
`checks.yaml` already does.

## Rationale

Dependabot and fork PRs can't read Actions secrets. On #408, #407 and
#387, `mavenverify` gets through the build and all tests, then fails in
the Sonar goal:

> [ERROR] Not authorized. Please check the 'sonar.token' property or the
'SONAR_TOKEN' environment variable ...

`sonar-maven-plugin` checks `sonar.skip` before it starts the scanner
([source](https://github.com/SonarSource/sonar-scanner-maven/blob/master/sonar-maven-plugin/src/main/java/org/sonarsource/scanner/maven/SonarQubeMojo.java)),
so the job now passes or fails on the build and tests alone. I put the
change in the workflow instead of `pom.xml` because it depends on the CI
environment.

CodeQL on #407 failed with `API rate limit exceeded` inside the buf
setup step. Without a token, that step calls the GitHub API
unauthenticated and shares the runner IP's rate limit.

## Test plan

- [x] YAML parses; checked the skip logic for empty and non-empty
`SONAR_TOKEN`
- [ ] This PR's own `mavenverify` still runs Sonar (token available)
- [ ] After merge, the recreated Dependabot PRs show `sonar.skip = true:
Skipping analysis` and `mavenverify` passes

Companion PR: #425 (Dependabot PR titles).

Signed-off-by: Dave Mihalcik <dmihalcik@virtru.com>
dmihalcik-virtru added a commit that referenced this pull request Oct 21, 2026
## Summary

Adds `.github/dependabot.yml` so Dependabot PR titles pass the
conventional commit check.

- Runtime dependency bumps → `fix(main): bump ...` (release-please cuts
a patch release)
- Development/test-scoped bumps → `chore(ci): bump ...`
- `open-pull-requests-limit: 0` keeps version updates off. Dependabot
keeps opening security-update PRs only, as it does today, and those now
use the prefixes too.

## Rationale

All three open Dependabot PRs (#408, #407, #387) fail **Validate PR
title**:

> Unknown scope "deps" found in pull request title "chore(deps): bump
org.bouncycastle:bcprov-jdk18on from 1.84 to 1.85". Scope must match one
of: ci, cmdline, docs, main, sdk, tests, examples, release/v0\.\d+.

Dependabot adds the `:` after a prefix ending in `)`
([docs](https://docs.github.com/code-security/dependabot/working-with-dependabot/dependabot-options-reference#commit-message--)).

## Caveat

Dependabot decides what counts as a development dependency. It labeled
`bc-fips` as `deps-dev` (it's `test` scope in `sdk/pom.xml`) even though
`sdk-fips-bc` depends on it at compile scope. So `bc-fips` bumps will be
titled `chore(ci)`, and release-please won't cut a release for them.
Retitle those PRs to `fix(main)` before merging.

## Test plan

- [x] YAML parses
- [ ] After merge, comment `@dependabot recreate` on #408, #407, #387
and check the new titles pass the PR-title check

Companion PR: skip the Sonar scan when `SONAR_TOKEN` is unavailable.

<!-- This is an auto-generated comment: release notes by coderabbit.ai
-->

<!-- coderabbit-ai-generated:v1 -->

## Summary by CodeRabbit

- **Chores**
- Added automated weekly dependency update checks across Maven-based
project areas.
- Security updates remain enabled, while routine non-security version
updates are not opened automatically.
- Automated dependency changes now follow consistent commit message
conventions.

<!-- end of auto-generated comment: release notes by coderabbit.ai -->

Signed-off-by: Dave Mihalcik <dmihalcik@virtru.com>
@dmihalcik-virtru

Copy link
Copy Markdown
Member

@dependabot recreate

Bumps org.bouncycastle:bc-fips from 2.1.2 to 2.1.3.

---
updated-dependencies:
- dependency-name: org.bouncycastle:bc-fips
  dependency-version: 2.1.3
  dependency-type: direct:development
...

Signed-off-by: dependabot[bot] <support@github.com>
@dependabot dependabot Bot changed the title chore(deps-dev): bump org.bouncycastle:bc-fips from 2.1.2 to 2.1.3 chore(ci): bump org.bouncycastle:bc-fips from 2.1.2 to 2.1.3 Oct 21, 2026
@dependabot
dependabot Bot force-pushed the dependabot/maven/org.bouncycastle-bc-fips-2.1.3 branch from c0be26b to 52db219 Compare October 21, 2026 00:46
@github-actions

Copy link
Copy Markdown
Contributor

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

dependencies Pull requests that update a dependency file java Pull requests that update java code

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants