Skip to content
This repository was archived by the owner on Jan 8, 2019. It is now read-only.

Remove unused "admin users" feature #1245

Merged
merged 1 commit into from
Jul 31, 2018
Merged

Remove unused "admin users" feature #1245

merged 1 commit into from
Jul 31, 2018

Conversation

http-418
Copy link
Contributor

We don't use multiple users on any of our bootstraps, anywhere.

This is a security hole rather than a feature, and it broke one
of my hardware clusters. That's a good reminder to delete it.

If we make a conscious decision to pursue multi-user bootstraps,
that needs to be thought out more fully than just tossing some user
resources into bcpc::bootstrap

We don't use multiple users on any of our bootstraps, anywhere.

This is a security hole rather than a feature, and it broke one
of my hardware clusters.  That's a good reminder to delete it.
Copy link
Member

@vt0r vt0r left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I've been meaning to remove this. Thanks!

@vt0r vt0r merged commit de17e37 into bloomberg:master Jul 31, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants