Clearing the client state using 'resetStore'

[oavasilcai]

Not sure if this is actually a bug but more of behaviour issue, we use resetStore to clear the state on user logout. However, active queries on that page are being sent once resetStore is called.

The queries made require authentication on the backend, so once logout is made they respond with a 401.

I've already submitted a PR with a possible solution, just not sure how the apollo team wants to go about it: #1487

[Poincare]

This is currently intended behavior as far as I know.

If resetStore didn't refetch queries that are active on your page, then your cache would be left in state that may display information pre-resetStore. This is obviously bad (e.g. if you resetStore on logout).

[oavasilcai]

There is no information in pre-resetStore - and I would prefer that over refetching queries and getting 403's because the token/cookies are invalid.

The usecase makes it more difficult for us because this piece of our app is kind of a 'microservice' and logouts navigates away from it's own 'micro-domain'

[oavasilcai]

@Poincare also, in the PR I've added an option to resetStore to be able to not refetch, default behaviour is still to run active queries. PR here: #1487

[helfer]

I think this should be fixed in react-apollo now, which was keeping the observable queries alive even when they weren't used any more.

[oavasilcai]

hi helfer, got a link to the PR ?

[helfer]

@oavasilcai #1636 and the PR that references it in react-apollo. It should be shipped in react-apollo@^1.2.0.