Move UserSubscription to composition
This allows for two very important simplifying changes. First, we can
use the default copy operators on the UserSubscription class, which is
far less error prone than writing it manually, which we have two copies
of in code already.
Second, it allows the Subscription class to move to using values rather
than shared_ptr everywhere, which cleans up a significant amount of
code.
Tested:
Ran Redfish-Event-Listener, subscription created and destroyed
correctly.
Calling POST SubmitTestEvent showed events propagating to server.
Change-Id: I6d258cfe3594edddf3960ae2d4559d70acca1bf8
Signed-off-by: Ed Tanous <ed@tanous.net>
diff --git a/redfish-core/lib/eventservice_sse.hpp b/redfish-core/lib/eventservice_sse.hpp
index c0a9527..80c0a3a 100644
--- a/redfish-core/lib/eventservice_sse.hpp
+++ b/redfish-core/lib/eventservice_sse.hpp
@@ -44,16 +44,15 @@
std::string lastEventId(req.getHeaderValue("Last-Event-Id"));
- std::shared_ptr<redfish::Subscription> subValue =
- std::make_shared<redfish::Subscription>(conn);
+ std::shared_ptr<Subscription> subValue =
+ std::make_shared<Subscription>(conn);
// GET on this URI means, Its SSE subscriptionType.
- subValue->subscriptionType = redfish::subscriptionTypeSSE;
+ subValue->userSub.subscriptionType = redfish::subscriptionTypeSSE;
- subValue->protocol = "Redfish";
- subValue->retryPolicy = "TerminateAfterRetries";
- subValue->eventFormatType = "Event";
- subValue->filter = filter;
+ subValue->userSub.protocol = "Redfish";
+ subValue->userSub.retryPolicy = "TerminateAfterRetries";
+ subValue->userSub.eventFormatType = "Event";
std::string id = manager.addSSESubscription(subValue, lastEventId);
if (id.empty())