Skip to content

Update ASI06_Memory_and_Context_Poisoning .md #718

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.

Sign up for GitHub

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

Jump to bottom
Open
wants to merge 6 commits into
base: main
Choose a base branch
from
Open

Update ASI06_Memory_and_Context_Poisoning .md #718

wants to merge 6 commits into from

Conversation

@Josh-Beck
Copy link

@Josh-Beck Josh-Beck commented Sep 17, 2025

[ASI06 V1]

Main work done in Google Doc, transferring to here for review and comments.

itskerenkatz
itskerenkatz requested changes Sep 21, 2025
View reviewed changes
Copy link
Collaborator

@itskerenkatz itskerenkatz left a comment

Choose a reason for hiding this comment

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

Hi team!
Great great work,
Added a few comments
Do not forget to add a comparison to our existing frameworks

itskerenkatz
itskerenkatz requested changes Sep 21, 2025
View reviewed changes
Copy link
Collaborator

@itskerenkatz itskerenkatz left a comment

Choose a reason for hiding this comment

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

Hi team!
Great great work,
Added a few comments
Do not forget to add a comparison to our existing frameworks

@Josh-Beck
Copy link
Author

Josh-Beck commented Sep 22, 2025

@itskerenkatz - I have added changes based on your feedback. Can we get this PR through and then go back to add references to other OWASP works? I am not sure I can do that change now, I would like to separate that PR.

@Josh-Beck
added T&M
bce1ce7 
Signed-off-by: Joshua Beck <[email protected]>
@Josh-Beck
Copy link
Author

Josh-Beck commented Sep 25, 2025

Changes have been added, this is in a good initial state. I would like to get the PR through so comments can be separated into additional PRs rather than cluttering this one. If we want a different approach that's fine, please let me know!

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

Reviewers

@guerilla7 guerilla7 Awaiting requested review from guerilla7 guerilla7 is a code owner

@hoeg hoeg Awaiting requested review from hoeg hoeg is a code owner

@itskerenkatz itskerenkatz Awaiting requested review from itskerenkatz itskerenkatz is a code owner

2 more reviewers

@thismohsin thismohsin thismohsin left review comments

@kerenkatzapex kerenkatzapex kerenkatzapex approved these changes

Reviewers whose approvals may not affect merge requirements

Requested changes must be addressed to merge this pull request.

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@Josh-Beck @thismohsin @kerenkatzapex @itskerenkatz