Skip to content

feat(amqp-webhook): add AMQP deliveryMode as a configuration#58

Open
rudwna wants to merge 2 commits intovymalo:mainfrom
rudwna:rudwna/add-amqp-delivery-mode
Open

feat(amqp-webhook): add AMQP deliveryMode as a configuration#58
rudwna wants to merge 2 commits intovymalo:mainfrom
rudwna:rudwna/add-amqp-delivery-mode

Conversation

@rudwna
Copy link
Contributor

@rudwna rudwna commented Nov 25, 2025

This pull request introduces support for configuring the AMQP delivery mode via environment variables

AMQP Delivery Mode Configuration:

  • Added WEBHOOK_AMQP_DELIVERY_MODE environment variable configuration with a default of 1 (non-persistent)
  • Introduce deliveryMode in AmqpWebhookHandler, using it inside getMessageProps
    • Move getMessageProps from companion object method to an instance method

@rudwna rudwna changed the title feat: add AMQP deliveryMode as a configuration feat(AMQP webhook): add AMQP deliveryMode as a configuration Nov 25, 2025
@rudwna rudwna changed the title feat(AMQP webhook): add AMQP deliveryMode as a configuration feat(amqp-webhook): add AMQP deliveryMode as a configuration Nov 25, 2025
…figuration with default of 1 (non-persistent)
@rudwna rudwna force-pushed the rudwna/add-amqp-delivery-mode branch from 656fb03 to 85a7527 Compare November 25, 2025 03:35
@stephane-segning
Copy link
Contributor

@codex review this please

@chatgpt-codex-connector
Copy link

You have reached your Codex usage limits for code reviews. You can see your limits in the Codex usage dashboard.
To continue using code reviews, you can upgrade your account or add credits to your account and enable them for code reviews in your settings.

@moritz89
Copy link

FYI, sourcery is free for OSS projects

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.

3 participants