Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Update UISI Auto-Reporting to match Web/iOS implementation #5596

Closed
langleyd opened this issue Mar 21, 2022 · 0 comments · Fixed by #5639
Closed

Update UISI Auto-Reporting to match Web/iOS implementation #5596

langleyd opened this issue Mar 21, 2022 · 0 comments · Fixed by #5639
Assignees
Labels
A-E2EE T-Task Refactoring, enabling or disabling functionality, other engineering tasks Z-parity-with-ios

Comments

@langleyd
Copy link
Member

Bring the UISI auto-reporting implementation in line with web and iOS( recently added in element-hq/element-ios#5743)

More Specifically:

  • Implementation of android differed slightly with web in just monitoring decryption attempts after initial sync.
  • Current implementation may be sending false positives as is checking isEncrypted but not clear event/error here
  • Web also allows 60 a minimum of seconds between reports. This is what was also set on iOS and seems like another reasonable measure to avoid user impact and sending to many reports. At the moment on android I think it is 2 seconds.
@langleyd langleyd self-assigned this Mar 21, 2022
@ouchadam ouchadam added A-E2EE T-Task Refactoring, enabling or disabling functionality, other engineering tasks Z-parity-with-ios labels Mar 21, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A-E2EE T-Task Refactoring, enabling or disabling functionality, other engineering tasks Z-parity-with-ios
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants