IOS-8008 Sign hashes with different sizes #382
Merged
Conversation
This file contains 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
tureck1y
commented
Nov 1, 2024
| @@ -125,10 +125,7 @@ public enum TangemSdkError: Error, LocalizedError, Encodable { | |||
|
|
|||
| /// This error is returned when a `SignCommand` receives only empty hashes for signature. | |||
| case emptyHashes | |||
|
|
|||
| /// This error is returned when a `SignCommand` receives hashes of different lengths for signature. | |||
| case hashSizeMustBeEqual | |||
There was a problem hiding this comment.
больше не актуально, карта конечно все равно вернет свою ошибку, но посредством сдк эту ошибку получить более невозможно
tureck1y
commented
Nov 1, 2024
| @@ -0,0 +1,56 @@ | |||
| // | |||
| // ChunkHashesUtil.swift | |||
There was a problem hiding this comment.
Перенес старую логику чанкования из команды в отдельную утилиту ради тестируемости и добавил сюда новый чанкинг
tureck1y
commented
Nov 1, 2024
| // ChunkedHashesContainer.swift | ||
| // TangemSdk | ||
| // | ||
| // Created by Alexander Osokin on 31.10.2024. |
There was a problem hiding this comment.
Сюда перенес стейт менеджмент чанков из команды опять же ради тестируемости. Логика чуть поменялась. Раньше я оперировал ренджами, вычисляя чанки на лету, а теперь сразу чанками
tureck1y
commented
Nov 1, 2024
| do { | ||
| let signatures = try self.processSignatures(with: session.environment) | ||
|
|
||
| if let remainingSignatures = session.environment.card?.wallets[self.walletPublicKey]?.remainingSignatures { |
There was a problem hiding this comment.
просто перенос пониже, чтобы был доступ к количеству подписей
tureck1y
commented
Nov 1, 2024
| @@ -20,45 +20,25 @@ public struct SignResponse: JSONStringConvertible { | |||
|
|
|||
| /// Signs transaction hashes using a wallet private key, stored on the card. | |||
| class SignCommand: Command { | |||
| typealias Response = SignResponse | |||
There was a problem hiding this comment.
пришлось поделить, чтобы не делать лишнего в функции десериализации
tureck1y
requested review from
skibinalexander,
Andoran90,
fedorov-d and
AlekseiMuraveinik1
skibinalexander
approved these changes
Nov 1, 2024
AlekseiMuraveinik1
approved these changes
Nov 1, 2024
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.
У текущей реализации команды подписи было ограничение, она ожидала хеши одного размера, иначе кидала ошибку. Это связано с техничеакой реализацией подписи - мы передаем на карту конкатенированную подпись и размер хеша, а карта уже сама нарезает, подпись возвращается в таком же виде.
Из дополнительных ограничений - макс 10 хешей за операцию и макс 2 хеша для старых девайсов
Я перерработал чанкование, чтобы хеши объединялись в группы по размерам, с учетом ограничений выше и с сохранением порядка. Порядок респонса с подписями соответствуют порядку переданных хешей. У тому же может быть несколько одинаковых хешей с разными подписями