feat: some optimize

.github/workflows/benchmark.yml

@@ -0,0 +1,91 @@
+name: Benchmark
+on:
+  push:
+    branches:
+      - master
+    paths-ignore:
+      - "**/*.md"
+  pull_request:
+    paths-ignore:
+      - "**/*.md"
+permissions:
+  contents: write
+  pull-requests: write
+  deployments: write
+jobs:
+  benchmark:
+    runs-on: ubuntu-latest
+    steps:
+      - uses: actions/checkout@v4
+        with:
+          fetch-depth: 0 # to be able to retrieve the last commit
+      - uses: actions/setup-go@v5
+        with:
+          go-version: "stable"
+      - name: Run Benchmark
+        run: set -o pipefail; go test ./... -benchmem -run=^$ -bench . | tee output.txt
+      # NOTE: Benchmarks could change with different CPU types
+      - name: Get GitHub Runner System Information
+        uses: kenchan0130/[email protected]
+        id: system-info
+      - name: Get master branch SHA
+        id: get-master-branch-sha
+        run: |
+          SHA="$(git rev-parse origin/master)"
+          echo "sha=$SHA" >> $GITHUB_OUTPUT
+      - name: Get Benchmark Results from master branch
+        id: cache
+        uses: actions/cache/restore@v4
+        with:
+          path: ./cache
+          key: ${{ steps.get-master-branch-sha.outputs.sha }}-${{ runner.os }}-${{ steps.system-info.outputs.cpu-model }}-benchmark
+      # This will only run if we have Benchmark Results from master branch
+      - name: Compare PR Benchmark Results with master branch
+        uses: benchmark-action/[email protected]
+        if: steps.cache.outputs.cache-hit == 'true'
+        with:
+          tool: 'go'
+          output-file-path: output.txt
+          external-data-json-path: ./cache/benchmark-data.json
+          # Do not save the data (This allows comparing benchmarks)
+          save-data-file: false
+          fail-on-alert: true
+          # Comment on the PR if the branch is not a fork
+          comment-on-alert: ${{ github.event.pull_request.head.repo.fork == false }}
+          github-token: ${{ secrets.GITHUB_TOKEN }}
+          summary-always: true
+          alert-threshold: "150%"
+      - name: Store Benchmark Results for master branch
+        uses: benchmark-action/[email protected]
+        if: ${{ github.ref_name == 'master' }}
+        with:
+          tool: 'go'
+          output-file-path: output.txt
+          external-data-json-path: ./cache/benchmark-data.json
+          # Save the data to external file (cache)
+          save-data-file: true
+          fail-on-alert: false
+          github-token: ${{ secrets.GITHUB_TOKEN }}
+          summary-always: true
+          alert-threshold: "150%"
+      - name: Publish Benchmark Results to GitHub Pages
+        uses: benchmark-action/github-action-benchmark@v1
+        if: ${{ github.ref_name == 'master' }}
+        with:
+          tool: 'go'
+          output-file-path: output.txt
+          benchmark-data-dir-path: "benchmarks"
+          fail-on-alert: false
+          github-token: ${{ secrets.GITHUB_TOKEN }}
+          comment-on-alert: true
+          summary-always: true
+          # Save the data to external file (GitHub Pages)
+          save-data-file: true
+          alert-threshold: "150%"
+          auto-push: true
+      - name: Update Benchmark Results cache
+        uses: actions/cache/save@v4
+        if: ${{ github.ref_name == 'master' }}
+        with:
+          path: ./cache
+          key: ${{ steps.get-master-branch-sha.outputs.sha }}-${{ runner.os }}-${{ steps.system-info.outputs.cpu-model }}-benchmark

.github/workflows/govulncheck.yml

@@ -0,0 +1,25 @@
+name: Govulncheck
+on:
+  schedule:
+    - cron: "0 */6 * * *" # Run every 6 hours
+jobs:
+  govulncheck:
+    runs-on: ubuntu-latest
+    env:
+      GO111MODULE: on
+    steps:
+      - name: Fetch Repository
+        uses: actions/checkout@v4
+
+      - name: Install Go
+        uses: actions/setup-go@v5
+        with:
+          go-version: "stable"
+          check-latest: true
+          cache: false
+
+      - name: Install Govulncheck
+        run: go install golang.org/x/vuln/cmd/govulncheck@latest
+
+      - name: Run Govulncheck
+        run: govulncheck ./...

.github/workflows/mockery.yml

@@ -1,6 +1,9 @@
 name: Mockery
 on:
   pull_request:
+permissions:
+  contents: write
+  pull-requests: write
 jobs:
   mockery:
     runs-on: ubuntu-latest

contracts/database/orm/constants.go

@@ -1,12 +1,10 @@
 package orm
 
 const (
-	DriverMysql Driver = "mysql"
-	// DriverPostgresql DEPRECATED, use DriverPostgres instead.
-	DriverPostgresql Driver = "postgresql"
-	DriverPostgres   Driver = "postgres"
-	DriverSqlite     Driver = "sqlite"
-	DriverSqlserver  Driver = "sqlserver"
+	DriverMysql     Driver = "mysql"
+	DriverPostgres  Driver = "postgres"
+	DriverSqlite    Driver = "sqlite"
+	DriverSqlserver Driver = "sqlserver"
 )
 
 type Driver string

contracts/session/manager.go

@@ -6,5 +6,7 @@ type Manager interface {
 	// Driver retrieves the session driver by name.
 	Driver(name ...string) (Driver, error)
 	// Extend extends the session manager with a custom driver.
-	Extend(driver string, handler func() Driver) Manager
+	Extend(driver string, handler func() Driver) error
+	// ReleaseSession releases the session back to the pool.
+	ReleaseSession(session Session)
 }

crypt/aes_test.go

@@ -57,3 +57,35 @@ func (s *AesTestSuite) TestDecryptString() {
 	_, err = s.aes.DecryptString("eyJpdiI6IjEyMzQ1IiwidmFsdWUiOiIxMjM0NSJ9")
 	s.Error(err)
 }
+
+func Benchmark_EncryptString(b *testing.B) {
+	mockConfig := &configmock.Config{}
+	mockConfig.On("GetString", "app.key").Return("11111111111111111111111111111111").Once()
+	aes := NewAES(mockConfig, json.NewJson())
+	b.ResetTimer()
+
+	for i := 0; i < b.N; i++ {
+		_, err := aes.EncryptString("Goravel")
+		if err != nil {
+			b.Error(err)
+		}
+	}
+}
+
+func Benchmark_DecryptString(b *testing.B) {
+	mockConfig := &configmock.Config{}
+	mockConfig.On("GetString", "app.key").Return("11111111111111111111111111111111").Once()
+	aes := NewAES(mockConfig, json.NewJson())
+	payload, err := aes.EncryptString("Goravel")
+	if err != nil {
+		b.Error(err)
+	}
+	b.ResetTimer()
+
+	for i := 0; i < b.N; i++ {
+		_, err := aes.DecryptString(payload)
+		if err != nil {
+			b.Error(err)
+		}
+	}
+}

database/console/migrate.go

@@ -52,14 +52,14 @@
 		}
 
 		return migrate.NewWithDatabaseInstance(dir, "mysql", instance)
-	case orm.DriverPostgresql:
+	case orm.DriverPostgres:
 		dsn := db.NewDsnImpl(config, connection)
-		postgresqlDsn := dsn.Postgresql(writeConfigs[0])
-		if postgresqlDsn == "" {
+		postgresDsn := dsn.Postgres(writeConfigs[0])
+		if postgresDsn == "" {
 			return nil, nil
 		}
 
-		db, err := sql.Open("postgres", postgresqlDsn)
+		db, err := sql.Open("postgres", postgresDsn)
 		if err != nil {
 			return nil, err
 		}
@@ -114,6 +114,6 @@
 
 		return migrate.NewWithDatabaseInstance(dir, "sqlserver", instance)
 	default:
-		return nil, errors.New("database driver only support mysql, postgresql, sqlite and sqlserver")
+		return nil, errors.New("database driver only support mysql, postgres, sqlite and sqlserver")
 	}
 }

database/console/migrate_command_test.go

@@ -61,14 +61,14 @@ func TestMigrateCommand(t *testing.T) {
 			},
 		},
 		{
-			name: "postgresql",
+			name: "postgres",
 			setup: func() {
 				var err error
 				docker := gorm.NewPostgresDocker(docker.Postgres())
 				query, err = docker.New()
 				assert.Nil(t, err)
 				mockConfig = docker.MockConfig
-				createPostgresqlMigrations()
+				createPostgresMigrations()
 			},
 		},
 		{
@@ -120,7 +120,7 @@ INSERT INTO agents (name, created_at, updated_at) VALUES ('goravel', '2023-03-11
 `)
 }
 
-func createPostgresqlMigrations() {
+func createPostgresMigrations() {
 	_ = file.Create("database/migrations/20230311160527_create_agents_table.up.sql",
 		`CREATE TABLE agents (
   id SERIAL PRIMARY KEY NOT NULL,

database/console/migrate_creator.go

@@ -50,12 +50,12 @@
 
 	driver := receiver.config.GetString("database.connections." + receiver.config.GetString("database.default") + ".driver")
 	switch orm.Driver(driver) {
-	case orm.DriverPostgresql:
+	case orm.DriverPostgres:
 		if create {
-			return migration.PostgresqlStubs{}.CreateUp(), migration.PostgresqlStubs{}.CreateDown()
+			return migration.PostgresStubs{}.CreateUp(), migration.PostgresStubs{}.CreateDown()
 		}
 
-		return migration.PostgresqlStubs{}.UpdateUp(), migration.PostgresqlStubs{}.UpdateDown()
+		return migration.PostgresStubs{}.UpdateUp(), migration.PostgresStubs{}.UpdateDown()
 	case orm.DriverSqlite:
 		if create {
 			return migration.SqliteStubs{}.CreateUp(), migration.SqliteStubs{}.CreateDown()

database/console/migrate_fresh_command_test.go

@@ -54,14 +54,14 @@ func TestMigrateFreshCommand(t *testing.T) {
 			},
 		},
 		{
-			name: "postgresql",
+			name: "postgres",
 			setup: func() {
 				var err error
 				docker := gorm.NewPostgresDocker(docker.Postgres())
 				query, err = docker.New()
 				assert.Nil(t, err)
 				mockConfig = docker.MockConfig
-				createPostgresqlMigrations()
+				createPostgresMigrations()
 			},
 		},
 		{

database/console/migrate_refresh_command_test.go

@@ -54,14 +54,14 @@ func TestMigrateRefreshCommand(t *testing.T) {
 			},
 		},
 		{
-			name: "postgresql",
+			name: "postgres",
 			setup: func() {
 				var err error
 				docker := gorm.NewPostgresDocker(docker.Postgres())
 				query, err = docker.New()
 				assert.Nil(t, err)
 				mockConfig = docker.MockConfig
-				createPostgresqlMigrations()
+				createPostgresMigrations()
 			},
 		},
 		{

database/console/migrate_reset_command_test.go

@@ -44,14 +44,14 @@ func TestMigrateResetCommand(t *testing.T) {
 			},
 		},
 		{
-			name: "postgresql",
+			name: "postgres",
 			setup: func() {
 				var err error
 				docker := gorm.NewPostgresDocker(docker.Postgres())
 				query, err = docker.New()
 				assert.Nil(t, err)
 				mockConfig = docker.MockConfig
-				createPostgresqlMigrations()
+				createPostgresMigrations()
 			},
 		},
 		{

database/console/migrate_rollback_command_test.go

@@ -44,14 +44,14 @@ func TestMigrateRollbackCommand(t *testing.T) {
 			},
 		},
 		{
-			name: "postgresql",
+			name: "postgres",
 			setup: func() {
 				var err error
 				docker := gorm.NewPostgresDocker(docker.Postgres())
 				query, err = docker.New()
 				assert.Nil(t, err)
 				mockConfig = docker.MockConfig
-				createPostgresqlMigrations()
+				createPostgresMigrations()
 			},
 		},
 		{

database/console/migrate_status_command_test.go

@@ -44,14 +44,14 @@ func TestMigrateStatusCommand(t *testing.T) {
 			},
 		},
 		{
-			name: "postgresql",
+			name: "postgres",
 			setup: func() {
 				var err error
 				docker := gorm.NewPostgresDocker(docker.Postgres())
 				query, err = docker.New()
 				assert.Nil(t, err)
 				mockConfig = docker.MockConfig
-				createPostgresqlMigrations()
+				createPostgresMigrations()
 			},
 		},
 		{

database/db/dsn.go

@@ -9,7 +9,7 @@ import (
 
 type Dsn interface {
 	Mysql(config databasecontract.Config) string
-	Postgresql(config databasecontract.Config) string
+	Postgres(config databasecontract.Config) string
 	Sqlite(config databasecontract.Config) string
 	Sqlserver(config databasecontract.Config) string
 }
@@ -39,7 +39,7 @@ func (d *DsnImpl) Mysql(config databasecontract.Config) string {
 		config.Username, config.Password, host, config.Port, config.Database, charset, true, loc)
 }
 
-func (d *DsnImpl) Postgresql(config databasecontract.Config) string {
+func (d *DsnImpl) Postgres(config databasecontract.Config) string {
 	host := config.Host
 	if host == "" {
 		return ""

database/db/dsn_test.go

@@ -52,16 +52,16 @@ func (s *DsnTestSuite) TestMysql() {
 		testUsername, testPassword, testHost, testPort, testDatabase, charset, true, loc), dsn.Mysql(testConfig))
 }
 
-func (s *DsnTestSuite) TestPostgresql() {
-	connection := orm.DriverPostgresql.String()
+func (s *DsnTestSuite) TestPostgres() {
+	connection := orm.DriverPostgres.String()
 	dsn := NewDsnImpl(s.mockConfig, connection)
 	sslmode := "disable"
 	timezone := "UTC"
 	s.mockConfig.On("GetString", fmt.Sprintf("database.connections.%s.sslmode", connection)).Return(sslmode).Once()
 	s.mockConfig.On("GetString", fmt.Sprintf("database.connections.%s.timezone", connection)).Return(timezone).Once()
 
 	s.Equal(fmt.Sprintf("postgres://%s:%s@%s:%d/%s?sslmode=%s&timezone=%s",
-		testUsername, testPassword, testHost, testPort, testDatabase, sslmode, timezone), dsn.Postgresql(testConfig))
+		testUsername, testPassword, testHost, testPort, testDatabase, sslmode, timezone), dsn.Postgres(testConfig))
 }
 
 func (s *DsnTestSuite) TestSqlite() {