Merge pull request #254 from privacybydesign/supress-scheme-update-tr…

…ansport-errors

Chore: do not report transport errors to Sentry when updating schemes

CHANGELOG.md

@@ -8,6 +8,7 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 
 ### Changed
 - Include the cause of Android key store exceptions in Sentry reports
+- Do not report transport errors to Sentry when updating schemes
 
 ### Fixed
 - Stacktraces of native Android errors not parsed correctly

irmagobridge/event_handler.go

@@ -2,6 +2,7 @@ package irmagobridge
 
 import (
 	"fmt"
+
 	"github.com/go-errors/errors"
 	irma "github.com/privacybydesign/irmago"
 	irmaclient "github.com/privacybydesign/irmago/irmaclient"

irmagobridge/receive_event.go

@@ -5,6 +5,7 @@ import (
 	"fmt"
 
 	"github.com/go-errors/errors"
+	irma "github.com/privacybydesign/irmago"
 )
 
 // DispatchFromNative receives events from the Android / iOS native side
@@ -78,6 +79,10 @@ func DispatchFromNative(eventName, payloadString string) {
 		go func() {
 			defer recoverFromPanic("Handling UpdateSchemesEvent panicked")
 			err := bridgeEventHandler.updateSchemes()
+			// Ignore transport errors
+			if serr, ok := err.(*irma.SessionError); ok && serr.ErrorType == irma.ErrorTransport {
+				return
+			}
 			if err != nil {
 				reportError(errors.New(err), false)
 			}