ok with logger