Remove Atlas config option

gofiber · Dec 1, 2020 · e5d9b06 · e5d9b06
1 parent 1d00ff6
commit e5d9b06
Show file tree

Hide file tree

Showing 3 changed files with 2 additions and 25 deletions.
diff --git a/mongodb/README.md b/mongodb/README.md
@@ -41,7 +41,6 @@ store := mongodb.New()
 
 // Initialize custom config
 store := mongodb.New(mongodb.Config{
-	Atlas:      false,
 	Host:       "127.0.0.1",
 	Port:       27017,
 	Database:   "fiber",
@@ -67,11 +66,6 @@ type Config struct {
 	// Optional. Default is ""
 	Connection string
 
-	// Whether the DB is hosted on MongoDB Atlas
-	//
-	// Optional. Default is false
-	Atlas bool
-
 	// Host name where the DB is hosted
 	//
 	// Optional. Default is "127.0.0.1"
@@ -113,7 +107,6 @@ type Config struct {
 ```go
 var ConfigDefault = Config{
 	Connection: "",
-	Atlas:      false,
 	Host:       "127.0.0.1",
 	Port:       27017,
 	Database:   "fiber",

diff --git a/mongodb/config.go b/mongodb/config.go
@@ -7,11 +7,6 @@ type Config struct {
 	// Optional. Default is ""
 	Connection string
 
-	// Whether the DB is hosted on MongoDB Atlas
-	//
-	// Optional. Default is false
-	Atlas bool
-
 	// Host name where the DB is hosted
 	//
 	// Optional. Default is "127.0.0.1"
@@ -51,7 +46,6 @@ type Config struct {
 // ConfigDefault is the default config
 var ConfigDefault = Config{
 	Connection: "",
-	Atlas:      false,
 	Host:       "127.0.0.1",
 	Port:       27017,
 	Database:   "fiber",

diff --git a/mongodb/mongodb.go b/mongodb/mongodb.go
@@ -39,12 +39,7 @@ func New(config ...Config) *Storage {
 	if cfg.Connection != "" {
 		dsn = cfg.Connection
 	} else {
-		dsn = "mongodb"
-		if cfg.Atlas {
-			dsn += "+srv://"
-		} else {
-			dsn += "://"
-		}
+		dsn = "mongodb://"
 		if cfg.Username != "" {
 			dsn += url.QueryEscape(cfg.Username)
 		}
@@ -54,12 +49,7 @@ func New(config ...Config) *Storage {
 		if cfg.Username != "" || cfg.Password != "" {
 			dsn += "@"
 		}
-		// Cannot specify port when using MongoDB Atlas
-		if cfg.Atlas {
-			dsn += url.QueryEscape(cfg.Host)
-		} else {
-			dsn += fmt.Sprintf("%s:%d", url.QueryEscape(cfg.Host), cfg.Port)
-		}
+		dsn += fmt.Sprintf("%s:%d", url.QueryEscape(cfg.Host), cfg.Port)
 	}
 
 	// Set mongo options