btree: remove dependency on github.com/satori/go.uuid

[rsc]

The only call was uuid.NewV4().Bytes(), which really means "get me 16 random bytes".

uuid.NewV4 changed at some point to return an error in addition to the UUID, which made this code no longer compile.

There's no need to depend on a uuid package to get 16 random bytes. Do what the old package did, namely read from crypto/rand.Reader and expect it to succeed (or else panic), but without the dependency.

Fixes #187. Fixes #188.

[aviary2-wf]

Security Insights

No security relevant content was detected by automated scans.

Action Items

• Review PR for security impact; comment "security review required" if needed or unsure

• Verify aviary.yaml coverage of security relevant code

  Questions or Comments? Reach out on HipChat: InfoSec Forum.

[dustinhiatt-wf]

+1

[brianshannan-wf]

+1

[rsc]

Thanks!