Skip to content

Supporting more properties for rabbitmq #3806

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
Open
wants to merge 9 commits into
base: main
Choose a base branch
from
Open

Supporting more properties for rabbitmq #3806

wants to merge 9 commits into from
+635 −53

Conversation

abossard
Copy link
Contributor

Description

This pull request introduces several enhancements to the RabbitMQ bindings and metadata utilities. The most important changes involve adding new utility functions to extract specific metadata fields and updating the RabbitMQ publishing functions to utilize these new utilities.

  • Enhancements to metadata utilities
  • Updates to RabbitMQ bindings

Issue reference

Please reference the issue this PR will close: #3650

Duplicate of

#3661
I messed up the DCO and didn't figure out how to fix it.

Checklist

Please make sure you've completed the relevant tasks for this PR, out of the following list:

  • Code compiles correctly
  • Created/updated tests
  • Extended the documentation / Created issue in the https://github.com/dapr/docs/ repo: dapr/docs#[issue number]

@abossard abossard requested review from a team as code owners April 29, 2025 10:06
Andre Bossard added 2 commits April 29, 2025 12:06
Add support for messageId, correlationId, and type in RabbitMQ bindings
ed37e34 
Signed-off-by: Andre Bossard <[email protected]>
Squashed commit of the following:
76ad9bb 
commit 3549843
Author: Josh van Leeuwen <[email protected]>
Date:   Mon Mar 24 17:50:50 2025 -0300

    go.mod: CVE updates (dapr#3713)

    Signed-off-by: joshvanl <[email protected]>
    Signed-off-by: Andre Bossard <[email protected]>

commit 9a11502
Author: Cassie Coyle <[email protected]>
Date:   Wed Apr 23 07:33:24 2025 -0500

    Add baggage header support to http binding (dapr#3723)

    Signed-off-by: Cassandra Coyle <[email protected]>
    Signed-off-by: Andre Bossard <[email protected]>

commit 8014a1d
Author: Josh van Leeuwen <[email protected]>
Date:   Wed Mar 26 19:55:55 2025 -0300

    CVE go mod dep update (dapr#3716)

    Signed-off-by: joshvanl <[email protected]>
    Signed-off-by: Andre Bossard <[email protected]>

commit 72ffcd1
Author: Josh van Leeuwen <[email protected]>
Date:   Mon Mar 24 17:50:50 2025 -0300

    go.mod: CVE updates (dapr#3713)

    Signed-off-by: joshvanl <[email protected]>
    Signed-off-by: Andre Bossard <[email protected]>

commit 12be96f
Author: Yaron Schneider <[email protected]>
Date:   Wed Jan 29 13:41:28 2025 -0800

    Add deepseek support, update Go to 1.23.5 (dapr#3659)

    Signed-off-by: yaron2 <[email protected]>
    Signed-off-by: Andre Bossard <[email protected]>

commit a00c8f3
Merge: a395f74 397766a
Author: Andre Bossard <[email protected]>
Date:   Tue Apr 29 11:41:27 2025 +0200

    Merge branch 'dapr:main' into supporting_more_properties_for_rabbitmq_on_main

commit a395f74
Author: Andre Bossard <[email protected]>
Date:   Mon Apr 28 10:33:59 2025 +0200

    Refactor tryGetProperty to use strings.EqualFold for case-insensitive key matching

    Signed-off-by: Andre Bossard <[email protected]>

commit 1d4f3a1
Author: Anton Troshin <[email protected]>
Date:   Thu Apr 24 15:19:04 2025 -0500

    Support Oracle Connect Descriptors (dapr#3800)

    Signed-off-by: Anton Troshin <[email protected]>
    Signed-off-by: Andre Bossard <[email protected]>

commit 4630f5c
Author: Anton Troshin <[email protected]>
Date:   Wed Apr 23 07:34:59 2025 -0500

    Solace pubsub conformance test fix (dapr#3802)

    Signed-off-by: Anton Troshin <[email protected]>
    Co-authored-by: Yaron Schneider <[email protected]>
    Signed-off-by: Andre Bossard <[email protected]>

commit ed3f7a8
Author: Cassie Coyle <[email protected]>
Date:   Wed Apr 23 07:33:24 2025 -0500

    Add baggage header support to http binding (dapr#3723)

    Signed-off-by: Cassandra Coyle <[email protected]>
    Signed-off-by: Andre Bossard <[email protected]>

commit af31fae
Author: MikelRev <[email protected]>
Date:   Thu Apr 17 11:54:52 2025 -0400

    Updated sqlserver auth to utilize default scope. (dapr#3698)

    Co-authored-by: Yaron Schneider <[email protected]>
    Signed-off-by: Andre Bossard <[email protected]>

commit c5a0407
Author: Andre Bossard <[email protected]>
Date:   Wed Apr 16 13:39:26 2025 +0200

    Fix metadata handling by removing unnecessary value escaping in RabbitMQ message processing

    Signed-off-by: Andre Bossard <[email protected]>

commit 5bcb186
Author: Andre Bossard <[email protected]>
Date:   Wed Apr 16 13:13:24 2025 +0200

    Refactor RabbitMQ metadata handling to use case-insensitive key retrieval

    Signed-off-by: Andre Bossard <[email protected]>

commit 4f17a7f
Author: Emmanuel Auffray <[email protected]>
Date:   Thu Apr 10 10:06:37 2025 +1200

    Adding GoogleAI models too (dapr#3689)

    Signed-off-by: Emmanuel Auffray <[email protected]>
    Co-authored-by: Josh van Leeuwen <[email protected]>
    Co-authored-by: Yaron Schneider <[email protected]>
    Co-authored-by: Cassie Coyle <[email protected]>
    Signed-off-by: Andre Bossard <[email protected]>

commit 652c733
Author: Josh van Leeuwen <[email protected]>
Date:   Wed Mar 26 19:55:55 2025 -0300

    CVE go mod dep update (dapr#3716)

    Signed-off-by: joshvanl <[email protected]>
    Signed-off-by: Andre Bossard <[email protected]>

commit 71e9a4b
Author: Josh van Leeuwen <[email protected]>
Date:   Mon Mar 24 17:50:50 2025 -0300

    go.mod: CVE updates (dapr#3713)

    Signed-off-by: joshvanl <[email protected]>
    Signed-off-by: Andre Bossard <[email protected]>

commit 621f7ef
Author: Emmanuel Auffray <[email protected]>
Date:   Mon Mar 24 05:38:17 2025 +1300

    Adding Ollama as a conversation component for local dev/running of LLMs (dapr#3688)

    Signed-off-by: Emmanuel Auffray <[email protected]>
    Co-authored-by: Mike Nguyen <[email protected]>
    Co-authored-by: Yaron Schneider <[email protected]>
    Signed-off-by: Andre Bossard <[email protected]>

commit 229524f
Author: Emmanuel Auffray <[email protected]>
Date:   Sat Mar 22 00:52:15 2025 +1300

    Fix reference links of conversation components (dapr#3690)

    Signed-off-by: Emmanuel Auffray <[email protected]>
    Signed-off-by: Andre Bossard <[email protected]>

commit 9dbe6e3
Author: Andre Bossard <[email protected]>
Date:   Wed Apr 16 12:19:53 2025 +0200

    Add metadata handling for RabbitMQ messages and corresponding test

    Signed-off-by: Andre Bossard <[email protected]>

commit 3f0ec0b
Author: Andre Bossard <[email protected]>
Date:   Thu Mar 20 03:25:29 2025 +0100

    Add RabbitMQ metadata properties test and configuration

    Signed-off-by: Andre Bossard <[email protected]>

commit b6acda8
Author: Andre Bossard <[email protected]>
Date:   Tue Mar 18 12:25:31 2025 +0100

    Add integration test for publishing metadata properties to RabbitMQ

    Signed-off-by: Andre Bossard <[email protected]>

commit f726110
Author: Andre Bossard <[email protected]>
Date:   Tue Mar 18 09:26:49 2025 +0100

    Refactor RabbitMQ publishing: consolidate metadata application into a single function

    Signed-off-by: Andre Bossard <[email protected]>

commit 4cb5600
Author: Andre Bossard <[email protected]>
Date:   Tue Mar 18 09:16:14 2025 +0100

    Refactor RabbitMQ metadata handling: consolidate metadata retrieval into common package

    Signed-off-by: Andre Bossard <[email protected]>

commit 069b12f
Author: Andre Bossard <[email protected]>
Date:   Tue Mar 18 08:12:25 2025 +0100

    Refactor metadata function names for consistency: update messageId and correlationId to messageID and correlationID

commit d508479
Merge: af3fca0 d8ac01b
Author: Andre Bossard <[email protected]>
Date:   Tue Mar 18 07:58:36 2025 +0100

    Merge branch 'dapr:main' into supporting_more_properties_for_rabbitmq_on_main

commit af3fca0
Merge: 57ae111 637d18d
Author: Andre Bossard <[email protected]>
Date:   Wed Mar 12 03:28:26 2025 -0700

    Merge branch 'main' into supporting_more_properties_for_rabbitmq_on_main

commit 57ae111
Merge: d0c75f6 2997e47
Author: Andre Bossard <[email protected]>
Date:   Tue Mar 4 08:49:52 2025 +0100

    Merge branch 'dapr:main' into supporting_more_properties_for_rabbitmq_on_main

commit d0c75f6
Author: Andre Bossard <[email protected]>
Date:   Wed Feb 5 11:43:03 2025 +0100

    Add support for messageId, correlationId, and type in RabbitMQ bindings

    Signed-off-by: Andre Bossard <[email protected]>

Signed-off-by: Andre Bossard <[email protected]>
@abossard abossard force-pushed the supporting_more_properties_for_rabbitmq_on_main branch from 5642742 to 76ad9bb Compare April 29, 2025 10:06
abossard and others added 7 commits May 2, 2025 08:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

RabbitMQ Messages should have some cloud event property fields populated, e.g. message id and correlation id
2 participants
@abossard @cicoyle