Skip to content
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

Create Azure OpenAI detector #2347

Merged
merged 1 commit into from
Nov 20, 2024
Merged

Create Azure OpenAI detector #2347

merged 1 commit into from
Nov 20, 2024

Conversation

rgmz
Copy link
Contributor

@rgmz rgmz commented Jan 27, 2024

Description:

This fixes #1939.

Checklist:

  • Tests passing (make test-community)?
  • Lint passing (make lint this requires golangci-lint)?

@rgmz rgmz requested a review from a team as a code owner January 27, 2024 22:12
@rgmz rgmz force-pushed the feat/openai-azure branch from a8cbd1b to 0983638 Compare January 27, 2024 22:13
rgmz
rgmz commented Jan 27, 2024
View reviewed changes
pkg/detectors/azure_openai/azure_openai.go Outdated Show resolved Hide resolved
pkg/engine/ahocorasick/ahocorasickcore.go Outdated Show resolved Hide resolved
@rgmz rgmz force-pushed the feat/openai-azure branch 2 times, most recently from 439af11 to b0ad7a4 Compare January 30, 2024 13:21
@dustin-decker
Copy link
Contributor

Thank you. I've submitted a request for using Azure Open AI so we can make test credentials.

@rgmz rgmz force-pushed the feat/openai-azure branch from b0ad7a4 to 59200a6 Compare February 3, 2024 19:23
@rgmz rgmz force-pushed the feat/openai-azure branch from 59200a6 to d7c8680 Compare February 29, 2024 10:33
@rgmz rgmz force-pushed the feat/openai-azure branch from d7c8680 to 770d7a3 Compare April 14, 2024 14:23
@rgmz rgmz force-pushed the feat/openai-azure branch 2 times, most recently from 40c2990 to 695cc50 Compare May 22, 2024 19:58
@ahrav
Copy link
Collaborator

ahrav commented May 23, 2024

Thank you. I've submitted a request for using Azure Open AI so we can make test credentials.

@dustin-decker do you know if these have been setup?

@rgmz rgmz force-pushed the feat/openai-azure branch 2 times, most recently from bffff72 to e236353 Compare June 5, 2024 00:41
@rgmz rgmz force-pushed the feat/openai-azure branch 3 times, most recently from da533e9 to 194c740 Compare June 21, 2024 02:55
@rgmz rgmz force-pushed the feat/openai-azure branch 2 times, most recently from 822f2a5 to 86c6d63 Compare July 1, 2024 18:39
@rgmz rgmz force-pushed the feat/openai-azure branch from 86c6d63 to 2313629 Compare September 13, 2024 11:50
@rgmz rgmz force-pushed the feat/openai-azure branch from 2313629 to 6bb1263 Compare November 3, 2024 15:41
@rgmz rgmz requested a review from a team as a code owner November 3, 2024 15:41
@rgmz rgmz force-pushed the feat/openai-azure branch 2 times, most recently from 2bbf8ad to aa639f1 Compare November 11, 2024 19:22
@rgmz rgmz force-pushed the feat/openai-azure branch from aa639f1 to 8894adb Compare November 20, 2024 03:11
@rgmz rgmz requested a review from a team as a code owner November 20, 2024 03:11
@rgmz rgmz force-pushed the feat/openai-azure branch 2 times, most recently from 0e003bb to 47035fb Compare November 20, 2024 03:29
ahrav
ahrav approved these changes Nov 20, 2024
View reviewed changes
Copy link
Collaborator

@ahrav ahrav left a comment

Choose a reason for hiding this comment

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

Thanks for adding this!

pkg/detectors/azure_openai/azure_openai.go Show resolved Hide resolved
pkg/detectors/azure_openai/azure_openai.go Outdated Show resolved Hide resolved
@rgmz rgmz force-pushed the feat/openai-azure branch from 47035fb to 7f4f6de Compare November 20, 2024 20:06
@ahrav ahrav merged commit c9780de into trufflesecurity:main Nov 20, 2024
13 checks passed
@rgmz rgmz deleted the feat/openai-azure branch November 20, 2024 20:50
@blsaccess blsaccess mentioned this pull request Nov 21, 2024
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@dustin-decker dustin-decker dustin-decker left review comments

@ahrav ahrav ahrav approved these changes

Assignees
No one assigned
Labels
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Add OpenAI Azure Detector
3 participants
@rgmz @dustin-decker @ahrav