Refactor session to avoid duplicate calls to apis

[cesnietor]

Description:

Refactors the way we were calling /session api since previous implementaiton was making multiple requests (up to six) which could mean high latency if that requests take time to resolve. Now we call it only once and only at initial load.

Changes:

• created Session thunk to control Session API state
• Create new state in consoleSlice to store the session state (loading, done, etc)
• Updated Loging.tsx to default to /browser to avoid redirecting first to / and then to /browser when logging in.

Before:

After:

If navigating between tabs it won't call session api.

Test Steps:

Scenarios:

• With a minio running on port 9000, make assets and then make the binary make assets && make then run it on port 9090 like:

CONSOLE_ACCESS_KEY=minioconsole
CONSOLE_SECRET_KEY=minioconsole
CONSOLE_MINIO_SERVER=http://localhost:9000
CONSOLE_DEV_MODE=on ./console server

• Can login with valid user and the amount of calls to /session api are not more than one.
• Navigate between tabs (we should call /session only ONCE at initial load of whole page)
• Anonymous access works without issues (create bucket, set anonymous access to it [path /] and try to access to it without loging in). Everything should work just fine.

[prakashsvmx]

Tested. Changes look good to me. 👍

[dvaldivia]

Seeing a session call every time I navigate @cesnietor

[cesnietor]

@dvaldivia yes it was like that already before. I mentioned it in the description of the ticket as well.

[jinapurapu]

Session is being called twice on Bucket Details screen and on first load of Settings screen

[cesnietor]

@prakashsvmx @reivaj05 @dvaldivia @jinapurapu I've updated the PR addressing the comments and also doing the following -> We will no longer call /session api on every change path since it's a bit expensive (even if the api resolves quickly). More comments in the thread above with Daniel

[cesnietor]

rebased.