Back out "Manage RevproxyController lifetime with shared_ptr"

Reviewed By: jano Differential Revision: D49715876 fbshipit-source-id: 46ebcf7b24c54f8bd9370bd363d7d5e5bf710632
facebook · Sep 28, 2023 · 8eee8ac · 8eee8ac
1 parent d150845
commit 8eee8ac
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 5 deletions.
diff --git a/proxygen/lib/http/session/HTTPSessionAcceptor.cpp b/proxygen/lib/http/session/HTTPSessionAcceptor.cpp
@@ -30,7 +30,7 @@ HTTPSessionAcceptor::HTTPSessionAcceptor(
     std::shared_ptr<HTTPCodecFactory> codecFactory)
     : HTTPAcceptor(accConfig),
       codecFactory_(codecFactory),
-      simpleController_(std::make_shared<SimpleController>(this)) {
+      simpleController_(this) {
   if (!codecFactory_) {
     codecFactory_ =
         std::make_shared<HTTPDefaultSessionCodecFactory>(accConfig_);
@@ -98,7 +98,7 @@ void HTTPSessionAcceptor::onNewConnection(folly::AsyncTransport::UniquePtr sock,
                                 std::move(sock),
                                 localAddress,
                                 *peerAddress,
-                                controller.get(),
+                                controller,
                                 std::move(codec),
                                 tinfo,
                                 sessionInfoCb);

diff --git a/proxygen/lib/http/session/HTTPSessionAcceptor.h b/proxygen/lib/http/session/HTTPSessionAcceptor.h
@@ -100,8 +100,8 @@ class HTTPSessionAcceptor
    * override this function to provide their own more sophisticated
    * controller here.
    */
-  virtual std::shared_ptr<HTTPSessionController> getController() {
-    return simpleController_;
+  virtual HTTPSessionController* getController() {
+    return &simpleController_;
   }
 
   HTTPSessionStats* downstreamSessionStats_{nullptr};
@@ -136,7 +136,7 @@ class HTTPSessionAcceptor
 
   std::shared_ptr<HTTPCodecFactory> codecFactory_{};
 
-  std::shared_ptr<SimpleController> simpleController_;
+  SimpleController simpleController_;
 
   HTTPSession::InfoCallback* sessionInfoCb_{nullptr};