element-hq / element-ios

A glossy Matrix collaboration client for iOS
https://element.io
Apache License 2.0
1.72k stars 479 forks source link

Sharing a URL via the sharesheet results in an encryption error since the rust upgrade #7621

Closed JMoVS closed 1 year ago

JMoVS commented 1 year ago

Steps to reproduce

  1. Go to your phone's browser
  2. Share a link via the sharesheet to Element
  3. Then open element and observe an encryption error

Outcome

What did you expect?

No encryption error

What happened instead?

one message stating an encryption error and one that is the real message

Your phone model

No response

Operating system version

iOS 16.5.1a

Application version

No response

Homeserver

No response

Will you send logs?

Yes

pixlwave commented 1 year ago

Duplicate of #7499.

JMoVS commented 1 year ago

@pixlwave doesn't seem to be the same as the recipient can decrypt the message

pixlwave commented 1 year ago

Sadly I believe its all related as the Rust crytpo isn't multi-process safe. We're disabling all extensions that write to the crypto db in #7618.

JMoVS commented 1 year ago

ok, thx

jacotec commented 1 year ago

Sadly I believe its all related as the Rust crytpo isn't multi-process safe. We're disabling all extensions that write to the crypto db in #7618.

@pixlwave I believe a better way would be to move back to the old crypto unless Rust is fixed instead of removing elementary features which are part of Element since years (and any other messaging app).

How should we explain to our users that we've chosen the only platform where you can' share from other apps after we hardly worked to get people away from walled garden mainstream?

pixlwave commented 1 year ago

@jacotec I would suggest taking this to https://github.com/vector-im/element-meta/discussions there's not much I can personally do about the decision.

jacotec commented 1 year ago

@pixlwave Is there already a discussion, or should I open a new one?