Skip to content

Commit

Permalink
RestBackend: remove adding context to query
Browse files Browse the repository at this point in the history
This part of the code was broken and adding the topic string again as a
context query parameter, which was confusing to the user as well as the
client implementations.

Signed-off-by: Alexander Krimm <[email protected]>
  • Loading branch information
wirew0rm committed Nov 12, 2024
1 parent b1ae46c commit 5b1c53f
Showing 1 changed file with 1 addition and 2 deletions.
3 changes: 1 addition & 2 deletions src/majordomo/include/majordomo/RestBackend.hpp
Original file line number Diff line number Diff line change
Expand Up @@ -766,8 +766,7 @@ struct RestBackend<Mode, VirtualFS, Roles...>::RestWorker {
bool respondWithLongPollRedirect(const httplib::Request &request, httplib::Response &response, const mdp::Topic &subscription, detail::PollingIndex redirectLongPollingIdx) {
auto uri = URI<>::factory()
.path(request.path)
.addQueryParameter("LongPollingIdx", std::to_string(redirectLongPollingIdx))
.addQueryParameter("SubscriptionContext", subscription.toMdpTopic().str());
.addQueryParameter("LongPollingIdx", std::to_string(redirectLongPollingIdx));

// copy over the original query parameters
addParameters(request, uri);
Expand Down

0 comments on commit 5b1c53f

Please sign in to comment.