-
Notifications
You must be signed in to change notification settings - Fork 95
RAG-style attack test and related enhancements #67
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
Closed
Closed
Changes from 14 commits
Commits
Show all changes
18 commits
Select commit
Hold shift + click to select a range
67aa0fb
adding support for base_url with ollama and openai.
abutbul 27a99bb
refactored provider and model prompts
abutbul 30817ac
Disable bugged telemetry in RAG Poisoning test to prevent PostHog err…
abutbul 36ec7a6
restored comment
abutbul b7a8c4c
configuration error should also skip
abutbul 8c674a4
Improve error handling for RAG poisoning attack (these errors should…
abutbul efb8d63
Update ps_fuzz/attacks/rag_poisoning.py
davida-ps fb31d0c
Update ps_fuzz/attacks/rag_poisoning.py
davida-ps b3ac225
Update ps_fuzz/test_base.py
davida-ps bdbdd6e
Update ps_fuzz/attacks/rag_poisoning.py
davida-ps bbb026a
Update tests/test_chat_clients.py
davida-ps ed2562a
Update tests/test_chat_clients.py
davida-ps 0d9b6a8
Update tests/test_chat_clients.py
davida-ps 1c7faf4
Update ps_fuzz/attacks/rag_poisoning.py
davida-ps 64c7a9b
Add test for ClientLangChain with empty base URL
abutbul e9452bb
Update tests/test_chat_clients.py
davida-ps debcfce
agents and bandit configuration
davida-ps fba7413
version
davida-ps File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -1,6 +1,7 @@ | ||
| from .client_config import ClientConfig | ||
|
|
||
| class AttackConfig(object): | ||
| def __init__(self, attack_client: ClientConfig, attack_prompts_count: int): | ||
| def __init__(self, attack_client: ClientConfig, attack_prompts_count: int, embedding_config=None): | ||
| self.attack_client = attack_client | ||
| self.attack_prompts_count = attack_prompts_count | ||
| self.embedding_config = embedding_config |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
💡 Suggestion: The validation
if not value: raise ValueError(...)will also reject empty strings at assignment time, but the getter returns''as default. This is fine, but consider documenting this behavior - users might expect to be able to explicitly set empty string to "clear" the value.