Repository navigation
Usage with multithread tokio runtime (axum). (dyn Permission + 'static) cannot be shared between threads safely.聽#186
Description
Hi,
Newbie here so apologize in advance if this is a stupid question 馃槄
I'm trying to use the SDK, specifically the grant_token, with axum on multithread runtime environment.
I run into the problem calling following example.
pubnub
.grant_token(usize)
.resources(Option<[Box<permissions::Permission>]>)
.patterns(Option<[Box<permissions::Permission>]>)
.authorized_user_id(Option<String>)
.meta(Option<HashMap<String, MetaValue>>)
.execute()
The error I get is
error[E0277]:
(dyn Permission + 'static)cannot be shared between threads safely
--> src/routes/player_route.rs:88:1
|
88 | #[debug_handler]
| ^^^^^^^^^^^^^^^^(dyn Permission + 'static)cannot be shared between threads safely
|
= help: the traitSyncis not implemented for(dyn Permission + 'static)
= note: required forstd::ptr::Unique<(dyn Permission + 'static)>to implementSync
note: required because it appears within the typeBox<(dyn Permission + 'static)>
Per my understanding, the cause is that .resources takes trait object Option<[Box<dyn Permission + 'static>] as a parameter whereas axum require Send for trait paremeter (tokio-rs/axum#2522) to ensure thread safety.
Anyone has successfully run the SDK on axum please enlighten me.
Thank you! 馃檹
Activity
Xavrax commentedon Jun 5, 2024
Hello @pongsakornsemsuwan !
First of all - I want to apologize for the late reply. We were a little bit busy and we've missed the notification about the new issue.
I've just tested our Pubnub SDK with
Axiumand it seems that some of our features work as expected (publish,subscribe) butgrant_tokenis also failing the build for me.That's very weird because
Permissionshould be handled and consumed within theexecutefunction and the lack of theSendtrait shouldn't be a problem in this case.Maybe we consume it too late and that's why compiler is not satisfied.
Anyway, I will take a closer look into that problem and fix it as soon as possible.
Xavrax commentedon Jun 5, 2024
Okay. I was right. We consume that too late and that's why compiler is not happy with that.
I found very simple solution that can be found in #188 .
I will let you know whenever we will this PR.
Until that you can use the branch
fix/permissionsin your project to let it compile.