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.

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

feat: webhook v2 spec #5202

Draft
wants to merge 9 commits into
base: master
Choose a base branch
from
Draft

feat: webhook v2 spec #5202

wants to merge 9 commits into from

Conversation

vinayteki95
Copy link

@vinayteki95 vinayteki95 commented Oct 15, 2024

Description

Rudder-server webhook is upgraded to communicate with transformer via a new contract (v2 spec) while also deprecating v1, v0 specs.

  • Server will panic if transformer is an older version with no support of v2 spec.
  • Server will find out if the transformer version is not compatible by looking for upgradedToSourceTransformV2 flag in transformer features.

Corresponding documentation

NOTE: This is a breaking change which expects transformer to support v2 version. Do not merge until transformer is released with v2 support.

Linear Ticket

Resolves INT-2802

Security

  • The code changed/added as part of this pull request won't create any security issues with how the software is being used.

@vinayteki95 vinayteki95 changed the title Feat.webhook v2 spec feat: webhook v2 spec Oct 15, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant