Remove spurious while loop

[Yossi]

ap.active is a method (obviously, since its used on the previous line) so it always evaluates to not False. It is NOT a flag indicating an active connection or not. This becomes clear when you try to print(ap.active)