tree 9e4619598485a172da0ba833a957f4e292f2301d
parent 59b98b2222fddbea3d6f678d9e94006521f0c381
author Ed Tanous <edtanous@google.com> 1629920251 -0700
committer Ed Tanous <ed@tanous.net> 1631154173 +0000

Remove unused variables in connection class

Both of these variables are leftover from when middlewares existed in
this codebase, and are essentially unused.  The only use is to buffer
cleanupTempSession, which in practice, doesn't require the variable, and
can be run in all cases.

Because this code is "cleaning up" the basic auth session that got
created when the request started, in theory, if the request failed, it's
possible we didn't create a session, but that can happen through the
golden path too, if we access an unprotected resource, or access with
non basic auth, so there's no reason to hide this behind the "did the
middlewares fail" check.  This got stuck under the check because of the
way the middlewares used to be ordered, to keep "identical" code paths.

Tested:
curl -vvvv --insecure --auth root:0penBmc"https://192.168.7.2:443/redfish/v1/Systems"

Succeeded with and without the --auth flag.  This tests basic auth,
which is the only thing that could've been potentially  changed as part
of this commit.

Signed-off-by: Ed Tanous <edtanous@google.com>
Change-Id: I8d2500fd2abaab3b23abed51a9a9f55e8d171b76