fix(oidc): remove unnecessary e2e-encryption imports

[kilimnik]

I noticed that the crate won't build without the e2e-encryption feature enabled. This does not make it build rather it is a step towards it.
There is still https://github.com/neodyme-labs/matrix-rust-sdk/blob/oidc_e2e/crates/matrix-sdk/src/oidc/mod.rs#L319 missing.

• Public API changes documented in changelogs (optional)

Signed-off-by: Daniel Kilimnik [email protected]

[kilimnik]

I have no Idea why the test failed, it works locally on my machine :)

[kilimnik]

It seems like the whole cross_process is dependent on the e2e encryption, so it's currently not possible to use oidc without e2e

[bnjbvr]

Yeah, OIDC requires the cross-process lock because it requires one, in case a refresh happened in another process (can happen on iOS because the process receiving notifications (NSE) runs concurrently to the app's process). The "plan" to fix this properly would be to have a single cross-process lock throughout the entire app, which has been somewhat sketched in this comment. In the meanwhile, I'm not sure there's much we can do; maybe document the requirement would be a nice win, to not confuse other future readers?

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 85.38%. Comparing base (e37ad11) to head (2e16021).
Report is 306 commits behind head on main.

Additional details and impacted files

@@            Coverage Diff             @@
##             main    #4553      +/-   ##
==========================================
+ Coverage   85.36%   85.38%   +0.02%     
==========================================
  Files         284      284              
  Lines       31886    31886              
==========================================
+ Hits        27219    27227       +8     
+ Misses       4667     4659       -8     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[poljar]

Thanks, looks good.