lhumina_code/hero_compute
20
0
Fork 0
Code Issues 14 Pull requests 5 Projects Releases 7 Packages Wiki Activity Actions

fix: normalize RPC method names in SDK transport layer #82

Merged
mahmoud merged 2 commits from development_fix_rpc_case into development 2026-04-08 08:40:39 +00:00
Conversation 0 Commits 2 Files changed 6 +108 -108
mahmoud commented 2026-04-08 08:27:11 +00:00
Owner
Copy link

Summary

Fixes CamelCase RPC method names that cause "Method not found" errors with AxumRpcServer.

  • Add normalize_rpc_method() to SDK transport layer — automatically lowercases method names and strips cloud./explorer. prefixes in both http_rpc_unix() and http_rpc_tcp()
  • Remove lowercase_method() from UI server (now redundant)
  • Remove unused SERVER_DOMAIN constant
  • Fix console handler error mapping: 403 Forbidden → 502 Bad Gateway
  • Normalize all hardcoded method strings across heartbeat sender, explorer proxy, and integration tests

Test Results

  • cargo check --workspace — passed
  • cargo test --workspace — all passed
  • grep "method".*"[A-Z] — zero CamelCase method strings remaining

Closes #80

## Summary Fixes CamelCase RPC method names that cause "Method not found" errors with AxumRpcServer. - Add `normalize_rpc_method()` to SDK transport layer — automatically lowercases method names and strips `cloud.`/`explorer.` prefixes in both `http_rpc_unix()` and `http_rpc_tcp()` - Remove `lowercase_method()` from UI server (now redundant) - Remove unused `SERVER_DOMAIN` constant - Fix console handler error mapping: 403 Forbidden → 502 Bad Gateway - Normalize all hardcoded method strings across heartbeat sender, explorer proxy, and integration tests ## Test Results - `cargo check --workspace` — passed - `cargo test --workspace` — all passed - `grep "method".*"[A-Z]` — zero CamelCase method strings remaining Closes #80
fix: normalize RPC method names in SDK transport layer
Some checks failed
Test / test (push) Failing after 1m19s
Details
Test / test (pull_request) Failing after 1m19s
Details
c1032bb520 
Move method name normalization into http_rpc_unix() and http_rpc_tcp()
so ALL RPC calls are automatically lowercased with domain prefixes
stripped. This fixes "Method not found" errors from CamelCase method
names that AxumRpcServer doesn't recognize.

- Add normalize_rpc_method() to SDK transport layer
- Remove lowercase_method() from UI server (now redundant)
- Remove SERVER_DOMAIN constant (unused)
- Fix console handler error: 403 Forbidden → 502 Bad Gateway
- Normalize all hardcoded method strings across codebase

#80
style: apply cargo fmt
All checks were successful
Test / test (push) Successful in 1m25s
Details
Test / test (pull_request) Successful in 1m53s
Details
02798a1de4
mahmoud merged commit 949eb2267c into development 2026-04-08 08:40:39 +00:00
mahmoud deleted branch development_fix_rpc_case 2026-04-08 08:40:43 +00:00
Sign in to join this conversation.
Reviewers
No reviewers
Labels
Clear labels   
prio_critical

   
prio_low

   
type_bug

   
type_contact

   
type_issue

   
type_lead

   
type_question

   
type_story

   
type_task
No labels
prio_critical
prio_low
type_bug
type_contact
type_issue
type_lead
type_question
type_story
type_task
Milestone
Clear milestone
No items
No milestone
Projects
Clear projects
No items
No project
Assignees
Clear assignees
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference
lhumina_code/hero_compute!82
No description provided.