From 57146fbbb81db0b832b67ad270c5a347e66a46e8 Mon Sep 17 00:00:00 2001
From: Harshavardhana <harsha@minio.io>
Date: Sun, 26 Jun 2016 03:03:52 -0700
Subject: [PATCH] main: minio --help should print regardless of root. (#1985)

Remove root check entirely.

Fixes #1964
---
 main.go           |  6 ------
 runtime-checks.go | 26 --------------------------
 2 files changed, 32 deletions(-)

diff --git a/main.go b/main.go
index 836895a10..ac934ce46 100644
--- a/main.go
+++ b/main.go
@@ -64,12 +64,6 @@ func init() {
 
 	// Set global trace flag.
 	globalTrace = os.Getenv("MINIO_TRACE") == "1"
-
-	// It is an unsafe practice to run network services as
-	// root. Containers are an exception.
-	if !isContainerized() && os.Geteuid() == 0 {
-		console.Fatalln("Please run ‘minio’ as a non-root user.")
-	}
 }
 
 func migrate() {
diff --git a/runtime-checks.go b/runtime-checks.go
index 804781c81..fc2d1d2b9 100644
--- a/runtime-checks.go
+++ b/runtime-checks.go
@@ -18,38 +18,12 @@ package main
 
 import (
 	"fmt"
-	"io/ioutil"
-	"os"
 	"runtime"
-	"strings"
 
 	"github.com/hashicorp/go-version"
 	"github.com/minio/mc/pkg/console"
 )
 
-// isContainerized returns true if we are inside a containerized environment.
-func isContainerized() bool {
-	// Docker containers contain ".dockerenv" at their root path.
-	if _, e := os.Stat("/.dockerenv"); e == nil {
-		return true
-	}
-
-	// Check if cgroup policies for init process contains docker string.
-	if cgroupData, e := ioutil.ReadFile("/proc/1/cgroup"); e == nil {
-		if strings.Contains(string(cgroupData), "/docker-") {
-			return true
-		}
-	}
-
-	// Check if env var explicitly set
-	if allow := os.Getenv("ALLOW_CONTAINER_ROOT"); allow == "1" || strings.ToLower(allow) == "true" {
-		return true
-	}
-
-	/* Add checks for non-docker containers here. */
-	return false
-}
-
 // check if minimum Go version is met.
 func checkGoVersion() {
 	// Current version.