Seems reasonable to me.
What bothers me is this comment in shutdown's docstring:

        """Stops the serve_forever loop.

        Blocks until the loop has finished. This must be called while
        serve_forever() is running in another thread, or it will
        deadlock.
        """

I'm adding Giampaolo to the nosy list, to get his input on this.