Skip to content

APPENG-4467- rpm analyzier mile stone one#222

Open
RedTanny wants to merge 45 commits into
RHEcosystemAppEng:mainfrom
RedTanny:APPENG-4467-Rpm-Checker
Open

APPENG-4467- rpm analyzier mile stone one#222
RedTanny wants to merge 45 commits into
RHEcosystemAppEng:mainfrom
RedTanny:APPENG-4467-Rpm-Checker

Conversation

@RedTanny
Copy link
Copy Markdown
Collaborator

No description provided.

@RedTanny RedTanny force-pushed the APPENG-4467-Rpm-Checker branch from cde60ba to 50638ee Compare April 27, 2026 13:30
@batzionb
Copy link
Copy Markdown

batzionb commented May 6, 2026

Can the API payload be simplified somehow?
Will this more simplified structure work:

{
  "scan": {
    "vulns": [
      { "vuln_id": "CVE-2023-0464" }
    ]
  },
  "rpm": {
      "name": "openssl",
      "version": "1.1.1k",
      "release": "8.el9_9",
      "arch": "x86_64"
    }
}

without the additional ecosystem option
more intuitive name than "target_package"
without all other fields being required and without pipeline_mode
if rpm is present, use the rpm pipleline mode
and not under the image field, which is confusing as it's not an image

@batzionb
Copy link
Copy Markdown

batzionb commented May 6, 2026

In addition - can the API changes appear in the openapi spec

@batzionb
Copy link
Copy Markdown

batzionb commented May 6, 2026

Follow up on #222 (comment)
Using the API as is, requires the client to define a dummy repo URL so request won't be rejected
See here

@zvigrinberg zvigrinberg self-requested a review May 13, 2026 11:56
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