Accept session in addition to request

[ngbrown]

isAuthenticated and logout take a Request just to get the session. The page very well could have already gotten a session and if it is a storage based session, then it would be advantageous to not fetch from the database multiple times.

My suggestion is to change the signature like this:

  async logout(
    request: Request | Session,
    options: { redirectTo: string }
  ): Promise<void> {
    const session = isSession(request)
      ? request
      : await this._sessionStorage.getSession(request.headers.get("Cookie"));
// the rest is identical ...
}

isSession is provided by remix.

[sergiodxa]

This is a good idea, I think I could add this feature.