Infinit loop with more than 255 services

[cleemullins]

In the BotConfigurationBase class, the following code is present:

       let found = false;
        do {
            found = false;
            service.id = Math.floor((Math.random() * 255)).toString();
            for (const existingService of this.services) {
                if (existingService.id === service.id) {
                    found = true;
                    break;
                }
            }
	} while (found);

Problem 1:
This code results in an infinite loop if 255 or more services are specified.

Problem 2:
This code DOES NOT check to see if the Service exists the way the C# code does. Sync with @tomlm to figure out the "Why?" of this behavior, along with the reason for the ID Reassignment.

This is similar to C# Bug:
microsoft/botbuilder-dotnet#1910