Revert: Reset joinSession cache only on connect_document_error (#20784)

[pragya91]

Reverts #19974 PR which was applied as a performance improvement. In the original PR, we checked for the kind of error received on the socket to decide whether to clear joinSession cache or not and reuse the existing session whenever possible. However a bug was identified where one section of the code was not update to stamp the errorFrom property which is used to perform such check which resulted in the cache never clearing for and the code getting stuck in a loop to continue connection retries.

Impact of revert: There is no compatibility-issue/regression that this revert would introduce, as it was only a perf improvement.

Follow up: Captured in
AB#7833

[msfluid-bot]

⯆ @fluid-example/bundle-size-tests: -66 Bytes

Metric Name	Baseline Size	Compare Size	Size Diff
aqueduct.js	452.29 KB	452.29 KB	■ No change
azureClient.js	545.36 KB	545.35 KB	⯆ -12 Bytes
connectionState.js	680 Bytes	680 Bytes	■ No change
containerRuntime.js	255.17 KB	255.17 KB	■ No change
fluidFramework.js	339.39 KB	339.39 KB	■ No change
loader.js	129.89 KB	129.89 KB	■ No change
map.js	41.35 KB	41.35 KB	■ No change
matrix.js	143.62 KB	143.62 KB	■ No change
odspClient.js	513.82 KB	513.82 KB	■ No change
odspDriver.js	97.52 KB	97.52 KB	■ No change
odspPrefetchSnapshot.js	42.39 KB	42.39 KB	■ No change
sharedString.js	161.39 KB	161.39 KB	■ No change
sharedTree.js	339.39 KB	339.39 KB	■ No change
Total Size	3.14 MB	3.14 MB	⯆ -66 Bytes

---

Baseline commit: 75a9de2

Generated by 🚫 dangerJS against 53aeb4e