Skip to content

fix: [DevOps] Temporary for NVD, use datafeed instead of API#921

Open
CharlesDuboisSAP wants to merge 3 commits into
mainfrom
fix-nvd
Open

fix: [DevOps] Temporary for NVD, use datafeed instead of API#921
CharlesDuboisSAP wants to merge 3 commits into
mainfrom
fix-nvd

Conversation

@CharlesDuboisSAP

Copy link
Copy Markdown
Contributor

Context

AI/ai-sdk-java-backlog#448.

If the current dependency vulnerability scan fails, then we move temporarily to a data feed, higher bandwidth but higher reliability.

@CharlesDuboisSAP CharlesDuboisSAP self-assigned this Jun 30, 2026
@CharlesDuboisSAP CharlesDuboisSAP added the please-review Request to review a pull-request label Jun 30, 2026
@CharlesDuboisSAP CharlesDuboisSAP enabled auto-merge (squash) June 30, 2026 11:50

@Jonas-Isr Jonas-Isr 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.

Can we just run it from the branch and see if it works?

Comment thread pom.xml
<version>12.1.0</version>
<configuration>
<connectionTimeout>60000</connectionTimeout>
<nvdDatafeedUrl>https://dependency-check.github.io/DependencyCheck_Builder/nvd_cache/nvdcve-{0}.json.gz</nvdDatafeedUrl>

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.

(Question)

This automatically jumps in if the call to the usual server fails?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

It replaces the server

@CharlesDuboisSAP

Copy link
Copy Markdown
Contributor Author

Can we just run it from the branch and see if it works?

It works (ish) https://github.com/SAP/ai-sdk-java/actions/runs/28446859432/job/84298387124

@Jonas-Isr

Copy link
Copy Markdown
Member

It works (ish) https://github.com/SAP/ai-sdk-java/actions/runs/28446859432/job/84298387124

Why the "ish"? It looks like it is working properly right? Or do you mean because it found a violation?

@CharlesDuboisSAP

CharlesDuboisSAP commented Jul 1, 2026

Copy link
Copy Markdown
Contributor Author

Why the "ish"? It looks like it is working properly right? Or do you mean because it found a violation?

I just fixed the violation

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

Labels

please-review Request to review a pull-request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants