Unify sync error code mapping - Githubissues

realm / realm-java

Realm is a mobile database: a replacement for SQLite & ORMs

http://realm.io

Apache License 2.0

11.45k stars 1.75k forks source link

Unify sync error code mapping #7825

Closed cmelchior closed 1 year ago

cmelchior commented 1 year ago

Closes #7823

This PR accomplishes three things:

It fixes a few places where we send the wrong native error code to Java, resulting it being mapped to UNKNOWN.
It unifies our mapping of the native category to the Java equivalent. We had multiple places this was happening, and they were slightly divergent resulting in unexpected behavior.
Also, after the sync error refactors, most Protocol and session errors had ended up in the wrong category, resulting in them being reported as UNKNOWN.