Set safe context as default.

Allow use of unsafe context.

Author: andreiavrammsd

context.go

@@ -362,15 +362,15 @@ func (c *context) Cookies() []*http.Cookie {
 }
 
 func (c *context) Get(key string) interface{} {
-	if c.echo.SafeContext {
+	if !c.echo.UnsafeContext {
 		c.lock.RLock()
 		defer c.lock.RUnlock()
 	}
 	return c.store[key]
 }
 
 func (c *context) Set(key string, val interface{}) {
-	if c.echo.SafeContext {
+	if !c.echo.UnsafeContext {
 		c.lock.Lock()
 		defer c.lock.Unlock()
 	}
@@ -607,4 +607,3 @@ func (c *context) Reset(r *http.Request, w http.ResponseWriter) {
 	// NOTE: Don't reset because it has to have length c.echo.maxParam at all times
 	// c.pvalues = nil
 }
-

context_test.go

@@ -528,21 +528,54 @@ func TestContextStore(t *testing.T) {
 	c := &context{
 		echo: e,
 	}
+
 	c.Set("name", "Jon Snow")
 	testify.Equal(t, "Jon Snow", c.Get("name"))
 }
 
-func TestContextSafeStore(t *testing.T) {
+func BenchmarkContext_Store(b *testing.B) {
+	e := &Echo{}
+
+	c := &context{
+		echo: e,
+	}
+
+	for n := 0; n < b.N; n++ {
+		c.Set("name", "Jon Snow")
+		if c.Get("name") != "Jon Snow" {
+			b.Fail()
+		}
+	}
+}
+
+func TestContextUnsafeStore(t *testing.T) {
 	e := &Echo{}
-	e.SafeContext = true
+	e.UnsafeContext = true
 
 	c := &context{
 		echo: e,
 	}
+
 	c.Set("name", "Jon Safe")
 	testify.Equal(t, "Jon Safe", c.Get("name"))
 }
 
+func BenchmarkContext_UnsafeStore(b *testing.B) {
+	e := &Echo{}
+	e.UnsafeContext = true
+
+	c := &context{
+		echo: e,
+	}
+
+	for n := 0; n < b.N; n++ {
+		c.Set("name", "Jon Snow")
+		if c.Get("name") != "Jon Snow" {
+			b.Fail()
+		}
+	}
+}
+
 func TestContextHandler(t *testing.T) {
 	e := New()
 	r := e.Router()

echo.go

@@ -84,7 +84,7 @@ type (
 		Validator        Validator
 		Renderer         Renderer
 		Logger           Logger
-		SafeContext      bool
+		UnsafeContext    bool
 	}
 
 	// Route contains a handler and information for matching against requests.