Commit 15c83d9
fix(tap-agent): resolve V2 actor system and collection ID parsing issues
This commit addresses multiple issues that were preventing the V2 TAP actor
system from functioning properly:
1. **Actor Race Condition Prevention**: Added existence check in SenderAccount
to prevent duplicate actor creation attempts that caused "Actor already
registered" errors during concurrent receipt processing.
2. **Collection ID Parsing**: Fixed parsing of collection_id strings from
database during migration period:
- Handle both 20-byte addresses and 32-byte collection IDs
- Proper whitespace trimming and error reporting
- Consistent 0x prefix handling for both Address and CollectionId parsing
3. **Attestation Error Messages**: Updated error messages to be factual about
what is observed rather than speculative about causes when key derivation
fails after testing multiple combinations.
4. **BIP32 Path Length Handling**: Maintained existing v1/v2 fallback system
for handling long deployment IDs that exceed BIP32 path limits.
The V2 TAP actor system now starts successfully and can process receipts.
RAV generation is pending allocation closure in the test environment.
🤖 Generated with [Claude Code](https://claude.ai/code)
Co-Authored-By: Claude <[email protected]>1 parent d662e43 commit 15c83d9
File tree
8 files changed
+450
-11
lines changed- contrib/indexer-service
- crates
- attestation
- src
- monitor/src
- service/src/tap
- tap-agent/src/agent
8 files changed
+450
-11
lines changedSome generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
11 | 11 | | |
12 | 12 | | |
13 | 13 | | |
14 | | - | |
15 | 14 | | |
16 | 15 | | |
17 | 16 | | |
| |||
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
6 | 6 | | |
7 | 7 | | |
8 | 8 | | |
| 9 | + | |
9 | 10 | | |
| 11 | + | |
10 | 12 | | |
11 | 13 | | |
12 | 14 | | |
| |||
0 commit comments