-
Notifications
You must be signed in to change notification settings - Fork 306
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
How to handle conflicting primary sources #108
Comments
I'm not sure to understand what is not covered in the referenced PR. Sorry I'm not familiar with Magento, so I would need a dump of something you would want to achieve and that is not yet possible. |
Thats not really magento specific, just a special case. Timeline:
So they conflict in what they say, and I dont know what the best way to handle this would be. |
I would say in that case somebody has to try to reproduce the sec. flaw for the affected versions. If not possible I would recommend to take the wider range to be sure. |
Follow what the vendor says? If in doubt just ask the vendor or look at what they've announced? Unless magneto are saying two things? But that repository you linked to even says this in their readme.md:
|
Hi, me again,
so now we have directly a case, where the first primary source describes a bigger affected version area then a review from another source I would also call as primary.
Cotya/magento-security-advisories#1
So I think another optional list for linking to reviews would be wise, while I would not change the affected range yet, if not at least two reviews come to the same result.
Do you think this makes sense?
The text was updated successfully, but these errors were encountered: