cover.Store.GetAll() return a map will potentially cause panic(a map without protection)

[gavwu]

// memoryStore holds the registered services only into memory
type memoryStore struct {
	mu          sync.RWMutex
	servicesMap map[string][]string
}

// Get returns all the registered service information
func (l *memoryStore) GetAll() map[string][]string {
	l.mu.RLock()
	defer l.mu.RUnlock()
	return l.servicesMap
}

Proposal:

1. Clone a map and return
2. Support access the map under protection

[CarlJi]

Great suggestion 👍

[gavwu]

To preserve the meaning of GetAll, I prefer to clone a map and return it, potentially leave the overhead to the caller (caller should know this function call will be expensive)