OWASP / www-project-top-10-for-large-language-model-applications

OWASP Foundation Web Respository
Other
506 stars 130 forks source link

Merge summary file into the Vulnerabilities files #225

Open Bobsimonoff opened 11 months ago

Bobsimonoff commented 11 months ago

Currently the summary of each risk is in a single file that is separate from the actual risk details. This causes a disconnect when the risk is updated. Above description in the template for each risk I would like to see the summary section. Then at production time all the summary sections can be grabbed and put into a single file for PDF generation.

GangGreenTemperTatum commented 11 months ago

Hey @Bobsimonoff can you highlight the specific area you are referring to please, or a screenshot? TYIA

Bobsimonoff commented 11 months ago

image

These summaries only exist together in a single file. The individual risk documents do not contain the summaries.

GangGreenTemperTatum commented 11 months ago

Understood, thanks. Adding @rossja but IMO the idea is to keep the vulnerabilities concise and duplicating data or adding too much additional context can cause confusion, lack of focus and ultimately not deliver our intention.

Bobsimonoff commented 11 months ago

Here is my thinking when you say, "the idea is to keep the vulnerabilities concise and duplicating data or adding too much additional context can cause confusion" -- I agree. adding a 1 sentence summary is not additional context and the risks should be concise. However, when we have risks that look like this: image

or this

image

I think an additional sentence at the top that says summary like the following greatly helps the reader:

Summary

Manipulating LLMs via crafted inputs can lead to unauthorized access, data breaches, and compromised decision-making.

Summary

Tampered training data can impair LLM models leading to responses that may compromise security, accuracy, or ethical behavior.

We do you have some very short risk descriptions and some long ones. Here are the word counts for the description sections of each LLM risk:

The longest summary we have is 20 words.

Up to everyone else, it is just a thought to make things easier for maintenance and the reader.

rossja commented 11 months ago

i agree that putting the summary into the entries is likely a good idea, i already had that on my own list of questions to raise for v2, so this issue is perfect timing.