bug, should pass action not msg