fix OptionalLiveReloadServer create bean

[akaGelo]

The call of the method optionalLiveReloadServer() is realised in the method LocalDevToolsAutoConfiguration.onClassPathChanged(ClassPathChangedEvent) which implies the address to Bean, which is not really so. Every time a new instance of OptionalLiveReloadServer is created.

OptionalLiveReloadServer uses @PostConstruct annotation for the starting of LiveReloadServer while this method is not executed during the direct call. This brings to not executed this.server = null OptionalLiveReloadServer:64 as well.

This breaches the logic of realization of the method OptionalLiveReloadServer.triggerReload() which delegates the call to a not started server.

P.S. Should I attach the youtube links with the demonstration of the problem in the debugger?

[wilkinsona]

Thanks for the PR. I think this could be fixed more easily by adding the missing @Configuration to LiveReloadConfiguration. Its absence is why calling optionalLiveReloadServer() creates a new instance every time. @Configuration will ensure that the class is proxied such that only a single instance of the optional live reload server is created.

[akaGelo]

I've made some changes according to the recommendation