database icon indicating copy to clipboard operation
database copied to clipboard

The package is blocked by Roave/SecurityAdvisories based on CVE-2024-55586

Open jkavalik opened this issue 11 months ago • 13 comments

Advisory link - https://github.com/advisories/GHSA-f626-677r-j5vq What I suppose was the "source" - https://www.csirt.sk/nette-framework-vulnerability-permits-sql-injection.html

Result - all versions other than 4.0 RC are disallowed.

Imho the app allowing unfiltered user input into the DB library is the definition of app bug, not library bug, and the "vulnerability" designations seems nonsense to me, but I am no security expert.

jkavalik avatar Dec 10 '24 18:12 jkavalik

Wow, this is such a shitshow.

Now this, last week there was also a similar thing -> https://github.com/Roave/SecurityAdvisories/issues/137#issuecomment-2514567737

Granted, I'm not associated with either and the final verdict has not been spoken yet as to the validity of the claims, but being cautious here I too think the claim as presented on the external link page is not correct.

mfn avatar Dec 10 '24 19:12 mfn

I found emails that they wrote to me about this. So I replied to them that it’s not a bug, that this behavior is intentional and is also documented. However, I have no idea if it’s possible to revoke such a CVE.

dg avatar Dec 10 '24 22:12 dg

The real travesty is that it seems anyone can publish any CVE for any package, without maintainer acknowledgement, but then you need to proof they're not valid etc. Something isn't right here.

mfn avatar Dec 10 '24 23:12 mfn

@dg you can start by "improving" (their word, not mine) the GitHub Security Advisory (GHSA) here https://github.com/advisories/GHSA-f626-677r-j5vq/improve

Once/if the GHSA will be invalidated, Roave/SecurityAdvisories will/should stop blocking the installation (and the same for composer audit).

Then you can request a CVE rejection here https://cveform.mitre.org/ (Rejected CVE is an official term)

I wouldn't mind much if the CVE will not be rejected (heard there's a chance it won't, in general for various reasons, not this particular CVE) as long as the GHSA is.

I would "improve" the GHSA entry but I think it's better when the "improvement" comes from the maintainer, let's see.

spaze avatar Dec 11 '24 01:12 spaze

Great, so a "security expert" does not understand basic principles, says something about "spent time" but - THEIR nonsense stopped OUR CI in its tracks for days without any reason whatsoever...

jkavalik avatar Dec 13 '24 10:12 jkavalik

Any update on when this will be solved? Maybe @dg could release version 3.2.5, this way the block could be removed without waiting for the CVE rejection.

8ctopus avatar Dec 17 '24 07:12 8ctopus

HINT: Ignore this CVE in composer.json (temporary ignoring) for unlock

    "config": {
        "audit": {
            "ignore": {
                "CVE-2024-55586": "Visit https://github.com/nette/database/issues/314"
            }
        }
    },

otazniksk avatar Dec 17 '24 10:12 otazniksk

I have created a PR https://github.com/Roave/SecurityAdvisoriesBuilder/pull/759 to ignore the GHSA/CVE in roave/security-advisories.

spaze avatar Dec 17 '24 11:12 spaze

The PR was merged and roave/security-advisories doesn't block nette/database anymore https://github.com/Roave/SecurityAdvisories/commit/3076981ea708db9685cd16fa83f919cc0bd2cd65

spaze avatar Dec 17 '24 13:12 spaze

@spaze Thank you!

8ctopus avatar Dec 17 '24 15:12 8ctopus

It sucks that Composer itself started reporting it to me today, so I'll probably release a new version of the package.

dg avatar Dec 17 '24 15:12 dg

@dg you should be able to let Packagist ignore this advisory (see composer/packagist#1493 for such an example)

xabbuh avatar Dec 17 '24 16:12 xabbuh

To recap, new 3.2.5 version was released yesterday (thanks @dg!) so composer doesn't complain anymore when installing or running composer audit, roave/security-advisories has the GHSA/CVE excluded so CI pipelines should be green once again, mine are.

The CVE itself is still in "disputed" status, "Awaiting Analysis".

spaze avatar Dec 19 '24 12:12 spaze