-
Notifications
You must be signed in to change notification settings - Fork 105
feat: Add mTLS and custom HMAC support to webhooks #736
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
Conversation
|
New dependencies detected. Learn more about Socket for GitHub ↗︎
|
| vi.clearAllMocks(); | ||
| }); | ||
|
|
||
| it("should return the chainId from chainOverrides if it exists by slug", async () => { |
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.
Unrelated but fixing some broken unit tests
| const path = process.env.NODE_ENV === "test" ? ".env.test" : ".env"; | ||
| dotenv.config({ path }); | ||
|
|
||
| export const JsonSchema = z.string().refine( |
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.
Unused
|
This PR is stale because it has been open for 7 days with no activity. Remove stale label or comment or this PR will be closed in 3 days. |
PR-Codex overview
This PR focuses on adding mTLS (Mutual TLS) support for secure communication, updating the configuration management, and enhancing webhook functionalities. It introduces new fields for storing encrypted certificates and keys, along with related validation and request handling improvements.
Detailed summary
mtlsCertificateEncryptedandmtlsPrivateKeyEncryptedcolumns to theconfigurationtable.updateConfigurationfunction to usePrisma.ConfigurationUpdateInput.updateAuthConfiguration.generateSecretHmac256for generating HMAC signatures for webhook requests.sendWebhookRequestto utilize mTLS if configured.generateRequestHeadersto include HMAC authorization..env.testfor AWS and GCP configurations.testWebhookRoutefor testing webhook functionality.crypto.tsfor improved decryption handling.