ideally value should be taken from config