Skip to content

Commit

Permalink
fix: add health check endpoint
Browse files Browse the repository at this point in the history
  • Loading branch information
wenxing-wang-ey committed Jan 14, 2025
1 parent e7a8da6 commit 326b67e
Show file tree
Hide file tree
Showing 4 changed files with 18 additions and 18 deletions.
2 changes: 1 addition & 1 deletion .github/workflows/release-update.yml
Original file line number Diff line number Diff line change
Expand Up @@ -3,7 +3,7 @@ name: Release updated
on:
push:
branches:
- wenxing/updateImages--
- wenxing/updateImages

jobs:
BuildDockerImage:
Expand Down
2 changes: 1 addition & 1 deletion .github/workflows/timber.yml
Original file line number Diff line number Diff line change
Expand Up @@ -3,7 +3,7 @@ name: Timber Development Workflow
on:
push:
branches:
- wenxing/updateImages
- wenxing/updateImages--
jobs:

SetupEnvironment:
Expand Down
14 changes: 7 additions & 7 deletions merkle-tree/src/filter-controller.js
Original file line number Diff line number Diff line change
Expand Up @@ -38,9 +38,9 @@ const newLeafResponseFunction = async (eventObject, args) => {
logger.debug(`eventname: ${eventName}`);

if (treeId === undefined || treeId === '') {
eventParams = config.contracts[contractName].events[eventName].parameters;
eventParams = config.contracts['default'].events[eventName].parameters;
} else {
eventParams = config.contracts[contractName].treeId[treeId].events[eventName].parameters;
eventParams = config.contracts['default'].treeId[treeId].events[eventName].parameters;
}

// Now some generic eventObject handling code:
Expand Down Expand Up @@ -90,9 +90,9 @@ const newLeavesResponseFunction = async (eventObject, args) => {
let eventParams;

if (treeId === undefined || treeId === '') {
eventParams = config.contracts[contractName].events[eventName].parameters;
eventParams = config.contracts['default'].events[eventName].parameters;
} else {
eventParams = config.contracts[contractName].treeId[treeId].events[eventName].parameters;
eventParams = config.contracts['default'].treeId[treeId].events[eventName].parameters;
}

// Now some generic eventObject handling code:
Expand Down Expand Up @@ -175,18 +175,18 @@ async function filterBlock(db, contractName, contractInstance, contractId, fromB

// TODO: if possible, make this easier to read and follow. Fewer 'if' statements. Perhaps use 'switch' statements instead?
if (treeId === undefined || treeId === '') {
eventNames = Object.keys(config.contracts[contractName].events);
eventNames = Object.keys(config.contracts['default'].events);
if (config.treeHeight !== undefined || config.treeHeight !== '') {
const { treeHeight } = config;
metadataService.insertTreeHeight({ treeHeight });
}
} else {
const { treeHeightDb } = await metadataService.getTreeHeight();
const { treeHeight } = config.contracts[contractName].treeId[treeId];
const { treeHeight } = config.contracts['default'].treeId[treeId];
if (treeHeightDb !== treeHeight && (treeHeight !== undefined || treeHeight !== '')) {
metadataService.insertTreeHeight({ treeHeight });
}
eventNames = Object.keys(config.contracts[contractName].treeId[treeId].events);
eventNames = Object.keys(config.contracts['default'].treeId[treeId].events);
}

const { treeHeight } = await metadataService.getTreeHeight();
Expand Down
18 changes: 9 additions & 9 deletions merkle-tree/src/routes/node.routes.js
Original file line number Diff line number Diff line change
Expand Up @@ -207,14 +207,14 @@ async function getRoot(req, res, next) {
}
}

// async function getHealthCheck(req, res, next) {
// try {
// res.data = "Ok";
// next();
// } catch (err) {
// next(err);
// }
// }
async function getHealthCheck(req, res, next) {
try {
res.data = "Ok";
next();
} catch (err) {
next(err);
}
}


// initializing routes
Expand Down Expand Up @@ -247,5 +247,5 @@ export default function(router) {
// ROOT ROUTES

router.route('/root').get(getRoot);
// router.route('/healthcheck').get(getHealthCheck);
router.route('/healthcheck').get(getHealthCheck);
}

0 comments on commit 326b67e

Please sign in to comment.