Feature/node 3500 support of dapp id in sdk bm and bk#246
Open
sauin wants to merge 17 commits into
Open
Conversation
…blockchain.message
…nt wire by server version
…app_id to dapp_id
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Adds dapp_id support to the
tvm_clientSDK andtvm-clifor the v3 BM/BK REST API (/v2/account,/v2/messages) while keeping compatibility with pre-1.0.0 servers. The wire format is selected at runtime from the server's GraphQLinfo.version:version >= "1.0.0"→ new fieldsaccount_idanddapp_idare sent and returned.version < "1.0.0"→ legacyaddress/dst_dapp_idshape is used.The selection happens transparently inside the SDK after a one-time GraphQL endpoint resolution; callers always work with the new types.
Breaking changes
ParamsOfGetAccount:address→account_id(strict 64-char hex, no0x, no workchain). New requireddapp_idfield.ResultOfGetAccount:dapp_idis nowString(notOption). Newaccount_idfield.ParamsOfSendMessage:dst_dapp_id: Option<String>→dapp_id: String(empty allowed only against pre-1.0.0 servers).ResultOfSendMessage: now exposesaccount_idanddapp_id, always populated (server response on v>=1.0.0, locally derived from request on legacy servers).ParamsOfProcessMessage:dst_dapp_id→dapp_id: String.The
tvm-cliflag--dst-dapp-idis preserved — only the SDK field name changes.tvm-cli behavior
The
accountcommand now requires a dapp_id when connected to a v>=1.0.0 server. Provide it as part of the address:tvm-cli account ::
tvm-cli account :
On older servers the legacy single-address form continues to work; the CLI auto-detects which mode applies based on the server's reported version.