Skip to content

Commit ec5e11d

Browse files
lklimekclaude
andcommitted
fix: use explicit Encoding::Base58 for Identifier::to_string() calls
The Platform SDK changed Identifier::to_string() to require an explicit Encoding argument. Updated both production and test code to use to_string(Encoding::Base58) instead of the no-arg variant. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
1 parent d33490e commit ec5e11d

1 file changed

Lines changed: 3 additions & 2 deletions

File tree

src/backend_task/identity/add_key_to_identity.rs

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -15,6 +15,7 @@ use dash_sdk::dpp::identity::accessors::{IdentityGettersV0, IdentitySettersV0};
1515
use dash_sdk::dpp::identity::identity_public_key::accessors::v0::{
1616
IdentityPublicKeyGettersV0, IdentityPublicKeySettersV0,
1717
};
18+
use dash_sdk::dpp::platform_value::string_encoding::Encoding;
1819
use dash_sdk::dpp::prelude::UserFeeIncrease;
1920
use dash_sdk::dpp::state_transition::identity_update_transition::IdentityUpdateTransition;
2021
use dash_sdk::dpp::state_transition::identity_update_transition::methods::IdentityUpdateTransitionMethodsV0;
@@ -46,7 +47,7 @@ fn broadcast_error(error: &SdkError) -> TaskError {
4647
StateError::IdentityPublicKeyAlreadyExistsForUniqueContractBoundsError(e),
4748
) => {
4849
return TaskError::IdentityPublicKeyContractBoundsConflict {
49-
contract_id: format!("{}", e.contract_id()),
50+
contract_id: e.contract_id().to_string(Encoding::Base58),
5051
};
5152
}
5253
_ => {}
@@ -207,7 +208,7 @@ mod tests {
207208
);
208209
let sdk_err = SdkError::from(consensus);
209210
let err = broadcast_error(&sdk_err);
210-
let expected_contract_id = contract_id.to_string();
211+
let expected_contract_id = contract_id.to_string(Encoding::Base58);
211212
assert!(
212213
matches!(err, TaskError::IdentityPublicKeyContractBoundsConflict { ref contract_id } if *contract_id == expected_contract_id)
213214
);

0 commit comments

Comments
 (0)