This doesn’t feel to me like the best place to be joining up the index names into a single string, especially considering the `SearchServer` code splits it back up again to check the index names. Could this be refactored to use either a single index name or an array, then join the names together somewhere closer to the elasticsearch server?