Change the default path for CRUD helpers to be lower case (#1276)

If the structure contains a capital letter the default path will contain also the capital letter. Since path are case sensitive only on some servers it creates confusion. To avoid it, the paths in the url's are converted to lower case.
gofr-dev · Dec 19, 2024 · 7e90187 · 7e90187
1 parent d6306d6
commit 7e90187
Show file tree

Hide file tree

Showing 3 changed files with 26 additions and 22 deletions.
diff --git a/pkg/gofr/crud_handlers_test.go b/pkg/gofr/crud_handlers_test.go
@@ -76,7 +76,7 @@ func Test_scanEntity(t *testing.T) {
 				entityType: reflect.TypeOf(userTestEntity{}),
 				primaryKey: "id",
 				tableName:  "user_test_entity",
-				restPath:   "userTestEntity",
+				restPath:   "usertestentity",
 				constraints: map[string]gofrSql.FieldConstraints{"id": {AutoIncrement: true, NotNull: false},
 					"name": {AutoIncrement: false, NotNull: true},
 				},
@@ -180,10 +180,16 @@ func Test_getRestPath(t *testing.T) {
 			want:       "custom_path",
 		},
 		{
-			name:       "Test without RestPathOverrider interface",
+			name:       "Test without RestPathOverrider interface - with lower case transformation",
 			object:     &struct{}{},
 			structName: "TestStruct",
-			want:       "TestStruct",
+			want:       "teststruct",
+		},
+		{
+			name:       "Test without RestPathOverrider interface",
+			object:     &struct{}{},
+			structName: "test_struct",
+			want:       "test_struct",
 		},
 	}
 	for _, tt := range tests {

diff --git a/pkg/gofr/crud_helpers.go b/pkg/gofr/crud_helpers.go
@@ -21,7 +21,7 @@ func getRestPath(object any, structName string) string {
 		return v.RestPath()
 	}
 
-	return structName
+	return strings.ToLower(structName)
 }
 
 func hasAutoIncrementID(constraints map[string]sql.FieldConstraints) bool {

diff --git a/pkg/gofr/datasource/README.md b/pkg/gofr/datasource/README.md
@@ -69,21 +69,19 @@ Therefore, GoFr utilizes a pluggable approach for new datasources by separating
 
 ## Supported Datasources
 
-| Datasource      | Health-Check | Logs | Metrics | Traces | As Driver |
-|-----------------|--------------|------|---------|--------|-----------|
-| MySQL           | ✅            | ✅    | ✅       | ✅      |           |
-| Redis           | ✅            | ✅    | ✅       | ✅      |           |
-| PostgreSQL      | ✅            | ✅    | ✅       | ✅      |           |
-| MongoDB         | ✅            | ✅    | ✅       | ✅      | ✅         |
-| SQLite          | ✅            | ✅    | ✅       | ✅      |           |
-| BadgerDB        | ✅            | ✅    | ✅       | ✅      | ✅         |
-| Cassandra       | ✅            | ✅    | ✅       | ✅      | ✅         |
-| ClickHouse      |              | ✅    | ✅       | ✅      | ✅         |
-| FTP             |              | ✅    |         |        | ✅         |
-| SFTP            |              | ✅    |         |        | ✅         |
-| Solr            |              | ✅    | ✅       | ✅      | ✅         |
-| Dgraph          | ✅            | ✅    | ✅       | ✅      |           |
-| Azure Event Hub |              | ✅    | ✅       |        | ✅         |
-| OpenTSDB        | ✅            | ✅    |         | ✅      | ✅         |
-
-
+| Datasource       | Health-Check | Logs | Metrics | Traces | As Driver |
+|------------------|:------------:|:----:|:-------:|:------:|:---------:|
+| MySQL            | ✅          | ✅    | ✅    | ✅      |           |
+| REDIS            | ✅          | ✅    | ✅    | ✅      |           |
+| PostgreSQL       | ✅          | ✅    | ✅    | ✅      |           |
+| MongoDB          | ✅          | ✅    | ✅    |  ✅     | ✅         |
+| SQLite           | ✅          | ✅    | ✅    | ✅      |           |
+| BadgerDB         | ✅          | ✅    | ✅    |    ✅   | ✅         |
+| Cassandra        | ✅          | ✅    | ✅    |  ✅     | ✅         |
+| ClickHouse       |             | ✅    | ✅    |   ✅    | ✅         |
+| FTP              |             | ✅    |       |          | ✅         |
+| SFTP             |             | ✅    |       |          | ✅         |
+| Solr             |             | ✅    | ✅    |   ✅    |   ✅      |
+| DGraph           | ✅          | ✅    | ✅    |   ✅    |           |
+| Azure Event Hubs |             | ✅    | ✅    |          |✅       |
+| OpenTSDB         | ✅         | ✅     |       |   ✅    |   ✅     |