Merge pull request #2052 from mempool/nymkappa/bugfix/blocks-list-pagination-error
Fix pagination issue in blocks list
This commit is contained in:
		
						commit
						e58b71fd4f
					
				@ -579,17 +579,13 @@ class Blocks {
 | 
			
		||||
  }
 | 
			
		||||
 | 
			
		||||
  public async $getBlocks(fromHeight?: number, limit: number = 15): Promise<BlockExtended[]> {
 | 
			
		||||
    let currentHeight = fromHeight !== undefined ? fromHeight : this.getCurrentBlockHeight();
 | 
			
		||||
    let currentHeight = fromHeight !== undefined ? fromHeight : await blocksRepository.$mostRecentBlockHeight();
 | 
			
		||||
    const returnBlocks: BlockExtended[] = [];
 | 
			
		||||
 | 
			
		||||
    if (currentHeight < 0) {
 | 
			
		||||
      return returnBlocks;
 | 
			
		||||
    }
 | 
			
		||||
 | 
			
		||||
    if (currentHeight === 0 && Common.indexingEnabled()) {
 | 
			
		||||
      currentHeight = await blocksRepository.$mostRecentBlockHeight();
 | 
			
		||||
    }
 | 
			
		||||
 | 
			
		||||
    // Check if block height exist in local cache to skip the hash lookup
 | 
			
		||||
    const blockByHeight = this.getBlocks().find((b) => b.height === currentHeight);
 | 
			
		||||
    let startFromHash: string | null = null;
 | 
			
		||||
 | 
			
		||||
@ -790,7 +790,7 @@ class Routes {
 | 
			
		||||
 | 
			
		||||
  public async getBlocks(req: Request, res: Response) {
 | 
			
		||||
    try {
 | 
			
		||||
      if (['mainnet', 'testnet', 'signet', 'regtest'].includes(config.MEMPOOL.NETWORK)) { // Bitcoin
 | 
			
		||||
      if (['mainnet', 'testnet', 'signet'].includes(config.MEMPOOL.NETWORK)) { // Bitcoin
 | 
			
		||||
        const height = req.params.height === undefined ? undefined : parseInt(req.params.height, 10);
 | 
			
		||||
        res.setHeader('Expires', new Date(Date.now() + 1000 * 60).toUTCString());
 | 
			
		||||
        res.json(await blocks.$getBlocks(height, 15));
 | 
			
		||||
 | 
			
		||||
		Loading…
	
	
			
			x
			
			
		
	
		Reference in New Issue
	
	Block a user