add todo about error handler