Skip to content

Comments

Add functional test for Orion Kafka custom notifications using extra macros (service, servicePath, authToken, attributes) #4732#4733

Merged
fgalan merged 2 commits intomasterfrom
feature/4732-Add-functional-test-for-Orion-Kafka-custom-notifications-using-extra-macros
Nov 28, 2025
Merged

Add functional test for Orion Kafka custom notifications using extra macros (service, servicePath, authToken, attributes) #4732#4733
fgalan merged 2 commits intomasterfrom
feature/4732-Add-functional-test-for-Orion-Kafka-custom-notifications-using-extra-macros

Conversation

@orianar
Copy link
Collaborator

@orianar orianar commented Nov 28, 2025

Issue #4732

Note: The ${servicePath} macro was not used in Kafka topics because it contain the / character, which is not allowed in Kafka topic names. Using it would cause error

 Topic name is invalid: 'orion_s1_/subservA_fgm' contains one or more characters other than ASCII alphanumerics, '.', '_' and '-'

This PR is based on PR #4726.
After merging the base PR, the only changes introduced by this PR will be the addition of the new test directory 4732_custom_notif_extra_macros_kafka

…-Kafka-custom-notifications-using-extra-macros
Copy link
Member

@fgalan fgalan left a comment

Choose a reason for hiding this comment

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

LGTM

@fgalan fgalan merged commit 2fadf6b into master Nov 28, 2025
17 checks passed
@fgalan fgalan deleted the feature/4732-Add-functional-test-for-Orion-Kafka-custom-notifications-using-extra-macros branch November 28, 2025 16:13
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.

2 participants