caivega / ipfslib

Other
2 stars 1 forks source link

S4 -jt_getTransactionByIndex,当参数index为负数时,会自动转成很大的整数。 #102

Closed foreso-GitHub closed 3 years ago

foreso-GitHub commented 4 years ago

jt_getTransactionByIndex,当参数index为负数时,会自动转成很大的整数。应该抛错。

请求:

{
  "jsonrpc":"2.0",
  "method":"jt_getTransactionByIndex",
  "params": [
    "j43x4vLAy6hW6AE3r9QqFqudmH2ajBzYm9", -1
  ],
  "id":1
}

结果:

{
    "id": 1,
    "jsonrpc": "2.0",
    "message": "can't find transaction j43x4vLAy6hW6AE3r9QqFqudmH2ajBzYm9, 4294967295",
    "result": "",
    "status": 140
}
foreso-GitHub commented 3 years ago

版本:20201114-v0.5.2-dev,没有修复,结果仍然是:

{
    "id": 1,
    "jsonrpc": "2.0",
    "message": "can't find transaction j43x4vLAy6hW6AE3r9QqFqudmH2ajBzYm9, 4294967295",
    "result": "",
    "status": 140
}
caivega commented 3 years ago

已经修复,可以在下一个build验证

返回错误信息如下:

{
"id": 1,
"jsonrpc": "2.0",
"message": "index or sequence should be >= 0",
"result": "",
"status": -269
}
foreso-GitHub commented 3 years ago

版本:20201130-v0.5.3-dev已经修复。

foreso-GitHub commented 3 years ago

版本:20201130-v0.5.3-dev已经修复。