Skip to content

Commit

Permalink
Merge pull request #872 from ayeshLK/main
Browse files Browse the repository at this point in the history
[Main] [Fix] Additional parameters sent in the subscription request is not properly propagated into the `websubhub:VerifiedSubscription` message
  • Loading branch information
ayeshLK authored Sep 15, 2023
2 parents 00d347d + d6ce50b commit 340d46d
Show file tree
Hide file tree
Showing 3 changed files with 8 additions and 11 deletions.
2 changes: 1 addition & 1 deletion ballerina/Dependencies.toml
Original file line number Diff line number Diff line change
Expand Up @@ -5,7 +5,7 @@

[ballerina]
dependencies-toml-version = "2"
distribution-version = "2201.8.0-20230726-145300-b2bdf796"
distribution-version = "2201.8.0-20230908-135700-74a59dff"

[[package]]
org = "ballerina"
Expand Down
12 changes: 2 additions & 10 deletions ballerina/subscription.bal
Original file line number Diff line number Diff line change
Expand Up @@ -99,12 +99,7 @@ isolated function processSubscriptionVerification(http:Headers headers, HttpToWe
string respStringPayload = check subscriberResponse.getTextPayload();
if (respStringPayload == challenge) {
VerifiedSubscription verifiedMessage = {
hub: message.hub,
hubMode: message.hubMode,
hubCallback: message.hubCallback,
hubTopic: message.hubTopic,
hubLeaseSeconds: message.hubLeaseSeconds,
hubSecret: message.hubSecret
...message
};
check adaptor.callOnSubscriptionIntentVerifiedMethod(verifiedMessage, headers);
}
Expand Down Expand Up @@ -186,10 +181,7 @@ isolated function processUnSubscriptionVerification(http:Headers headers, HttpTo
string respStringPayload = check subscriberResponse.getTextPayload();
if (respStringPayload == challenge) {
VerifiedUnsubscription verifiedMessage = {
hubMode: message.hubMode,
hubCallback: message.hubCallback,
hubTopic: message.hubTopic,
hubSecret: message.hubSecret
...message
};
check adaptor.callOnUnsubscriptionIntentVerifiedMethod(verifiedMessage, headers);
}
Expand Down
5 changes: 5 additions & 0 deletions changelog.md
Original file line number Diff line number Diff line change
Expand Up @@ -6,6 +6,11 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0

## [Unreleased]

## [1.9.1] - 2023-09-14

### Fixed
- [Additional parameters sent in the subscription request is not properly propagated into the `websubhub:VerifiedSubscription` message](https://github.com/ballerina-platform/ballerina-standard-library/issues/4800)

## [1.5.1] - 2023-03-16

### Fixed
Expand Down

0 comments on commit 340d46d

Please sign in to comment.