Skip to content
This repository was archived by the owner on Sep 30, 2024. It is now read-only.

Fix request blocking in Cody Gateway#62940

Merged
rafax merged 3 commits into
mainfrom
rg/fix_request_blocking
May 28, 2024
Merged

Fix request blocking in Cody Gateway#62940
rafax merged 3 commits into
mainfrom
rg/fix_request_blocking

Conversation

@rafax

@rafax rafax commented May 28, 2024

Copy link
Copy Markdown
Contributor

Fix request blocking in Cody Gateway by:

  • correcting the way we prefix env vars in loadFlaggingConfig (once instead of twice)
  • changing the order of operations in flagging/blocking logic (consider blocked prompts a reason for flagging)

Slack context.

Test plan

  • tested locally
  • existing unit tests

@rafax rafax requested a review from a team May 28, 2024 11:20
@cla-bot cla-bot Bot added the cla-signed label May 28, 2024
@rafax rafax changed the title Fix request blocking in Cody Gateway [WIP] Fix request blocking in Cody Gateway May 28, 2024
@rafax rafax requested a review from a team May 28, 2024 12:04
@rafax rafax changed the title [WIP] Fix request blocking in Cody Gateway Fix request blocking in Cody Gateway May 28, 2024
@rafax rafax merged commit 58e1169 into main May 28, 2024
@rafax rafax deleted the rg/fix_request_blocking branch May 28, 2024 12:23
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants