refactor(server.go) No lock is needed when reading server variables in a internal function

Our server is a single routine for loop. We have a lock to avoid "data-race" between the external function calls(read/write) and internal operations(write). So we can drop the extra lock when reading variables internally.