2048861 - Crash in [@ java.lang.IllegalStateException: at mozilla.components.feature.ipprotection.store.IPProtectionStore$1.invoke(IPProtectionStore.kt:396)]

Status: NEW

(Firefox for Android :: Privacy, defect)

Description

[Gabriele Svelto [:gsvelto]]

Crash report: https://crash-stats.mozilla.org/report/index/9a2a1ac8-5753-4721-bce5-28b9e0260618

Top 10 frames:

0  mozilla.components.feature.ipprotection.store.IPProtectionStore$1  invoke  IPProtectionStore.kt:396
1  mozilla.components.lib.state.Store$$ExternalSyntheticLambda0  invoke  R8$$SyntheticClass:16
2  org.mozilla.fenix.components.ipprotection.IPProtectionSnackbarMiddleware  invoke  IPProtectionSnackbarMiddleware.kt:22
3  mozilla.components.lib.state.Store$$ExternalSyntheticLambda1  invoke  R8$$SyntheticClass:14
4  org.mozilla.fenix.components.LogMiddleware  invoke  LogMiddleware.kt:22
5  mozilla.components.lib.state.Store$$ExternalSyntheticLambda1  invoke  R8$$SyntheticClass:14
6  mozilla.components.lib.state.Store  dispatch  Store.kt:53
7  org.mozilla.fenix.settings.IPProtectionFragment  onCreate  IPProtectionFragment.kt:30
8  androidx.fragment.app.FragmentStateManager  create  FragmentStateManager.java:56
9  androidx.fragment.app.FragmentStateManager  moveToExpectedState  FragmentStateManager.java:151

Same as with bug 2048858 this looks like a regression and the two are most likely related.

Comment 1

[BugBot [:suhaib / :marco/ :calixte]]

The severity field is not set for this bug.
:Gela, could you have a look please?

For more information, please visit BugBot documentation.

Comment 2

[Mike a [:mavduevskiy]]

There is an a IllegalStateException in the reducer.

It's probably fixed by the CheckAccount introduction - the screen would move the account state away from authenticated, wdyt?

Comment 3

[Jonathan Almeida [:jonalmeida]]

The exception in the trace says "VPN state machine is in a bad state" which is coming from that same throw, yes.

I don't see how it was fixed with the CheckAccount addition, but rather bug 2048858 as comment 0 suggested.

Let's leave this open for a bit longer to monitor the crash rates so that we don't see them rise again.

Comment 4

[BugBot [:suhaib / :marco/ :calixte]]

The severity field is not set for this bug.
:Gela, could you have a look please?

For more information, please visit BugBot documentation.

Comment 5

[BugBot [:suhaib / :marco/ :calixte]]

The severity field is not set for this bug.
:Gela, could you have a look please?

For more information, please visit BugBot documentation.