status-im
/
consul
mirror of https://github.com/status-im/consul.git
2
0
Fork 0
Code Issues Projects Releases Wiki Activity

Reduce the size of the UUID Lookup Length restriction from `8` to `2`. 

I'm torn on this.  It's useful from a UX perspective for an operator to
be able to type in something that's short.  At the same time, by
enforcing an `8` character length, we reduced the probability of a user
depending on the behavior and having it suddenly stop working in the
future when a duplicate prefix is injected into the environment.
This commit is contained in:
Sean Chittenden 2017-02-02 12:12:18 -08:00
parent 01588efc2a
commit fe49c0a0ab
No known key found for this signature in database
GPG Key ID: 4EBC9DC16C2E5E16
1 changed files with 1 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
consul/state/catalog.go
View File

@ -13,7 +13,7 @@ const (
// minUUIDLookupLen is used as a minimum length of a node name required before
// we test to see if the name is actually a UUID and perform an ID-based node
// lookup.
minUUIDLookupLen = 8
minUUIDLookupLen = 2
)
// Nodes is used to pull the full list of nodes for use during snapshots.