iotaledger / iotaledger/identity

[Task] Enable workflow 'Build and run grpc tests' when grpc bindings receive more priority

Open
#1,507 0 comments 0 reactions 1 assignee Claimed by @chrisgitiota View on GitHub
gRPC No changelog
Dominant language
Rust
Stars
346
Forks
100
Avg merge
13h 41m
Merged PRs (30d)
2

Description

## Description

Currently the workflow [Build and run grpc tests](https://github.com/iotaledger/identity.rs/actions/workflows/build-and-test-grpc.yml) is disabled because of build issues similar to the following console log:

> 671.7 error: implementation of `std::marker::Send` is not general enough
> 671.7 --> src/services/document.rs:70:3
> 671.7 |
> 671.7 70 | / #[tracing::instrument(
> 671.7 71 | | name = "create_did_document",
> 671.7 72 | | skip_all,
> 671.7 73 | | fields(request = ?req.get_ref())
> 671.7 74 | | ret,
> 671.7 75 | | err,
> 671.7 76 | | )]
> 671.7 | |____^ implementation of `std::marker::Send` is not general enough
> 671.7 |
> 671.7 = note: `std::marker::Send` would have to be implemented for the type `&StrongholdStorage`
> 671.7 = note: ...but `std::marker::Send` is actually implemented for the type `&'0 StrongholdStorage`, for some specific lifetime `'0`
> 671.7 = note: this error originates in the attribute macro `tracing::instrument` (in Nightly builds, run with -Z macro-backtrace for more info)

This is probably related to the rust compiler bug [Tracking issue for incorrect lifetime bound errors in async](https://github.com/rust-lang/rust/issues/110338). As long as our focus is on typescript bindings and wasm32 platforms we postpone resolving this issue.

After an alpha release including typescript bindings, we need to re-enable the 'Build and run grpc tests' and fix/circumvent the above described issue.

Contributor guide

Open the contributing guide

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.