Add getBestBlockId RPC method

Author: kakao-jun-e

rpc/src/v1/impls/chain.rs

@@ -28,7 +28,7 @@ use jsonrpc_core::Result;
 
 use super::super::errors;
 use super::super::traits::Chain;
-use super::super::types::{Block, Bytes, Parcel};
+use super::super::types::{Block, BlockNumberAndHash, Bytes, Parcel};
 
 pub struct ChainClient {
     client: Arc<Client>,
@@ -122,6 +122,13 @@ impl Chain for ChainClient {
         Ok(self.client.chain_info().best_block_number)
     }
 
+    fn get_best_block_id(&self) -> Result<BlockNumberAndHash> {
+        Ok(BlockNumberAndHash {
+            number: self.client.chain_info().best_block_number,
+            hash: self.client.chain_info().best_block_hash,
+        })
+    }
+
     fn get_block_hash(&self, block_number: u64) -> Result<Option<H256>> {
         Ok(self.client.block_hash(BlockId::Number(block_number)))
     }

rpc/src/v1/traits/chain.rs

@@ -19,7 +19,7 @@ use ctypes::{H160, H256, Public, U256};
 
 use jsonrpc_core::Result;
 
-use super::super::types::{Block, Bytes, Parcel};
+use super::super::types::{Block, BlockNumberAndHash, Bytes, Parcel};
 
 build_rpc_trait! {
     pub trait Chain {
@@ -71,6 +71,10 @@ build_rpc_trait! {
         # [rpc(name = "chain_getBestBlockNumber")]
         fn get_best_block_number(&self) -> Result<BlockNumber>;
 
+        /// Gets the number and the hash of the best block.
+        # [rpc(name = "chain_getBestBlockId")]
+        fn get_best_block_id(&self) -> Result<BlockNumberAndHash>;
+
         /// Gets the hash of the block with given number.
         # [rpc(name = "chain_getBlockHash")]
         fn get_block_hash(&self, u64) -> Result<Option<H256>>;

rpc/src/v1/types/block.rs

@@ -14,7 +14,7 @@
 // You should have received a copy of the GNU Affero General Public License
 // along with this program.  If not, see <https://www.gnu.org/licenses/>.
 
-use ccore::Block as CoreBlock;
+use ccore::{Block as CoreBlock, BlockNumber};
 use ctypes::{H160, H256, U256};
 
 use super::Parcel;
@@ -84,3 +84,10 @@ impl From<CoreBlock> for Block {
         }
     }
 }
+
+#[derive(Debug, Serialize)]
+#[serde(rename_all = "camelCase")]
+pub struct BlockNumberAndHash {
+    pub number: BlockNumber,
+    pub hash: H256,
+}

rpc/src/v1/types/mod.rs

@@ -20,6 +20,7 @@ mod parcel;
 mod work;
 
 pub use self::block::Block;
+pub use self::block::BlockNumberAndHash;
 pub use self::bytes::Bytes;
 pub use self::parcel::Parcel;
 pub use self::work::Work;

spec/JSON-RPC.md

@@ -82,7 +82,7 @@ In the current version, it's only supported through HTTP.
  * [version](#version)
 ***
  * [chain_getBestBlockNumber](#chain_getbestblocknumber)
- * [chain_getBestBlockId](#chain_getbestblockid) (not implemented yet)
+ * [chain_getBestBlockId](#chain_getbestblockid)
  * [chain_getBlockHash](#chain_getblockhash)
  * [chain_getBlockByHash](#chain_getblockbyhash)
  * [chain_sendSignedParcel](#chain_sendsignedparcel)
@@ -173,7 +173,7 @@ Response Example
 ```
 
 ## chain_getBestBlockId
-(not implemented yet) Gets the number and the hash of the best block.
+Gets the number and the hash of the best block.
 
 Params: No parameters
 
@@ -189,7 +189,7 @@ Request Example
 
 Response Example
 ```
-{"jsonrpc":"2.0","result":{"number":1,"hash":"0x56642f04d519ae3262c7ba6facf1c5b11450ebaeb7955337cfbc45420d573077"},"id":null}
+{"jsonrpc":"2.0","result":{"hash":"0x56642f04d519ae3262c7ba6facf1c5b11450ebaeb7955337cfbc45420d573077","number":1},"id":null}
 ```
 
 ## chain_getBlockHash