Skip to content
This repository has been archived by the owner on Apr 15, 2024. It is now read-only.

[V2.0] DID fragments instead of MethodID #266

Merged
merged 24 commits into from
Aug 29, 2022
Merged

[V2.0] DID fragments instead of MethodID #266

Show file tree
Hide file tree
Changes from 12 commits
Commits
Show all changes
24 commits
Select commit Hold shift + click to select a range
12e67ab
fds
Jun 29, 2022
b082114
Update identity imports
DyrellC Jun 29, 2022
50714d7
remove exchange keys mapping from user, wip
DyrellC Jun 30, 2022
f4164d6
fix signing key discrepancies
DyrellC Jun 30, 2022
9f059cb
remove ed key -> x key conversion in identifier
DyrellC Jun 30, 2022
179ea59
unused imports
DyrellC Jun 30, 2022
f37edc4
add subscribers via binary search pattern
DyrellC Jul 1, 2022
811b422
added resolver = 2
Jul 1, 2022
7f7e5f9
reorganised id, fmt and fixed imports
Jul 4, 2022
2f606e9
moved verify to DIDInfo to clean imports to package
Jul 5, 2022
a5dfbcf
Merge branch 'v2.0-dev' into v2.0/did-verification-method2
Jul 5, 2022
7b287b7
updated identity to latest rev
Jul 5, 2022
e766b84
Merge branch 'v2.0-dev' into v2.0/did-verification-method2
Jul 14, 2022
1b59fa3
Correct lifetime parameters in `Keyload::Wrap` to avoid unnecessary c…
Jul 15, 2022
96dc57d
Dyrell feedback
Jul 15, 2022
eec365b
restructured more of did
Jul 15, 2022
c11a7e6
Merge branch 'v2.0/did-verification-method2' into improve-lifetime-of…
kwek20 Jul 26, 2022
e9d941c
Merge pull request #4 from arnauorriols/improve-lifetime-of-subscribe…
kwek20 Jul 26, 2022
2179985
Arnau feedback
Jul 27, 2022
26ca4cb
Merge branch 'v2.0/did-verification-method2' of https://github.com/kw…
Jul 27, 2022
c47fc36
clippy
Aug 18, 2022
8584df8
Merge branch 'v2.0-dev' into v2.0/did-verification-method2
Aug 24, 2022
9f2655f
merge fix
Aug 24, 2022
0d13ad9
fixed permissions error
Aug 25, 2022
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
2 changes: 2 additions & 0 deletions Cargo.toml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -6,5 +6,7 @@ members = [
"streams",
]

resolver = "2"
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

required because of an identity feature (Identity also needs this now)


[profile.dev]
incremental = true
4 changes: 2 additions & 2 deletions lets/Cargo.toml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -20,7 +20,7 @@ tangle-client = ["iota-client/async", "futures", "iota-crypto/blake2b"]
# Enable the wasm-compatible IOTA-Tangle transport client (incompatile with `tangle-client` feature due to `iota-client/async` using `tokio`. Implies `std` feature)
tangle-client-wasm = ["iota-client/wasm", "futures"]
# Enable Iota Identity for use with Streams
did = ["identity", "serde"]
did = ["identity_iota", "serde"]
# # Enable synchronized transports using parking-lot
sync-parking-lot = ["parking_lot"]
# # Enable synchronized transports using spin
Expand All @@ -41,7 +41,7 @@ hex = {version = "0.4", default-features = false}

# Optional dependencies
futures = {version = "0.3.8", default-features = false, optional = true}
identity = {git = "https://github.com/iotaledger/identity.rs", rev = "86edaad", default-features = false, features = ["async"], optional = true}
identity_iota = {git = "https://github.com/iotaledger/identity.rs", rev = "d3920c2", default-features = false, optional = true}
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Seeing that this rev already takes the new package name used for releasing to crates.io, I wonder if we couldn't actually use version = "0.6.1" instead of using the git rev. We will have to do it anyway at some point

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This didnt work due to an identity error internally when compiling :P

error[E0432]: unresolved import `identity_did::revocation`
  --> /home/brord/.cargo/registry/src/github.com-1ecc6299db9ec823/identity_iota_client-0.6.1/src/credential/credential_validator.rs:11:19
   |
11 | use identity_did::revocation::RevocationBitmap;
   |                   ^^^^^^^^^^ could not find `revocation` in `identity_did`

iota-client = {version = "1.1.1", default-features = false, optional = true}
parking_lot = {version = "0.11.2", default-features = false, optional = true}
serde = {version = "1.0", default-features = false, features = ["derive"], optional = true}
Expand Down
8 changes: 6 additions & 2 deletions lets/src/address.rs
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -146,8 +146,12 @@ impl AppAddr {
let id_bytes = identifier.as_bytes();
// Create spongos to squeeze topic into final 8 bytes
let squeezed_topic: [u8; 8] = Spongos::<KeccakF1600>::init().sponge(base_topic);
assert_eq!(id_bytes.len(), 32, "identifier must be 32 bytes long");
addr[..32].copy_from_slice(id_bytes);
if id_bytes.len() > 32 {
let squeezed_id: [u8; 32] = Spongos::<KeccakF1600>::init().sponge(id_bytes);
addr[..32].copy_from_slice(&squeezed_id)
} else {
addr[..32].copy_from_slice(id_bytes);
}
kwek20 marked this conversation as resolved.
Show resolved Hide resolved
addr[32..].copy_from_slice(&squeezed_topic);
Self::new(addr)
}
Expand Down
282 changes: 0 additions & 282 deletions lets/src/id/did.rs
View file
Open in desktop

This file was deleted.

Loading