Replace references to Session Manager with references to Session

looks great, any idea how to check that session manager should not be used?
Probably IS needs update with that

+++ b/core/lib/Drupal/Core/DrupalKernel.php
@@ -699,21 +699,21 @@ protected function getKernelParameters() {
+      // If there is a session, close and save it.
+      if ($this->container->initialized('session')) {
+        $session = $this->container->get('session');
+        if ($session->isStarted()) {
+          $session_started = TRUE;
+          $session->save();
         }
-        unset($session_manager);
+        unset($session);
       }

This itself is way better to understand. ... its about the session

So yeah is there anything left for the RTBC? I updated the issue summary with a beta evaluation and documented the user_logout() .

https://github.com/symfony/symfony/issues/12375 still open so user_logout() not the case, maybe we need separate issue for that...

Just a re-roll so RTBC

This looks good and an obvious followup from all the current work on sessions.

+++ b/core/modules/system/tests/modules/session_test/src/Controller/SessionTestController.php
@@ -36,8 +36,8 @@ public function get() {
    * @return string
    *   A notification message.
    */
-  public function getFromSessionObject() {
-    $value = \Drupal::request()->getSession()->get("session_test_key");
+  public function getFromSessionObject(Request $request) {
+    $value = $request->getSession()->get("session_test_key");

@@ -49,12 +49,12 @@ public function getFromSessionObject() {
    * @return string
    *   A notification message with session ID.
    */
-  public function getId() {
+  public function getId(Request $request) {

@@ -145,16 +145,16 @@ public function isLoggedIn() {
    * @return \Symfony\Component\HttpFoundation\JsonResponse
    *   The response.
    */
-  public function traceHandler() {
+  public function traceHandler(Request $request) {

The method document needs updating (sorry this seems quite minor).

Committed a108bf0 and pushed to 8.0.x. Thanks!

I agree that this is allowable under the beta followup to other work clause.