Skip to content

Commit 54cd1c7

Browse files
authored
[1.15] - Cherry pick oracle fixes (#3718)
Signed-off-by: yaron2 <[email protected]>
1 parent 5df338e commit 54cd1c7

File tree

3 files changed

+18
-9
lines changed

3 files changed

+18
-9
lines changed

go.mod

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -107,7 +107,7 @@ require (
107107
github.com/redis/go-redis/v9 v9.6.3
108108
github.com/riferrei/srclient v0.6.0
109109
github.com/sendgrid/sendgrid-go v3.13.0+incompatible
110-
github.com/sijms/go-ora/v2 v2.7.18
110+
github.com/sijms/go-ora/v2 v2.8.22
111111
github.com/spf13/cast v1.6.0
112112
github.com/spiffe/go-spiffe/v2 v2.1.7
113113
github.com/stealthrocket/wasi-go v0.8.1-0.20230912180546-8efbab50fb58
@@ -224,7 +224,7 @@ require (
224224
github.com/eapache/go-xerial-snappy v0.0.0-20230731223053-c322873962e3 // indirect
225225
github.com/eapache/queue v1.1.0 // indirect
226226
github.com/emicklei/go-restful/v3 v3.10.1 // indirect
227-
github.com/emirpasic/gods v1.12.0 // indirect
227+
github.com/emirpasic/gods v1.18.1 // indirect
228228
github.com/fatih/color v1.17.0 // indirect
229229
github.com/felixge/httpsnoop v1.0.4 // indirect
230230
github.com/fsnotify/fsnotify v1.7.0 // indirect

go.sum

Lines changed: 4 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -559,8 +559,9 @@ github.com/edsrzf/mmap-go v1.0.0/go.mod h1:YO35OhQPt3KJa3ryjFM5Bs14WD66h8eGKpfaB
559559
github.com/emicklei/go-restful/v3 v3.8.0/go.mod h1:6n3XBCmQQb25CM2LCACGz8ukIrRry+4bhvbpWn3mrbc=
560560
github.com/emicklei/go-restful/v3 v3.10.1 h1:rc42Y5YTp7Am7CS630D7JmhRjq4UlEUuEKfrDac4bSQ=
561561
github.com/emicklei/go-restful/v3 v3.10.1/go.mod h1:6n3XBCmQQb25CM2LCACGz8ukIrRry+4bhvbpWn3mrbc=
562-
github.com/emirpasic/gods v1.12.0 h1:QAUIPSaCu4G+POclxeqb3F+WPpdKqFGlw36+yOzGlrg=
563562
github.com/emirpasic/gods v1.12.0/go.mod h1:YfzfFFoVP/catgzJb4IKIqXjX78Ha8FMSDh3ymbK86o=
563+
github.com/emirpasic/gods v1.18.1 h1:FXtiHYKDGKCW2KzwZKx0iC0PQmdlorYgdFG9jPXJ1Bc=
564+
github.com/emirpasic/gods v1.18.1/go.mod h1:8tpGGwCnJ5H4r6BWwaV6OrWmMoPhUl5jm/FMNAnJvWQ=
564565
github.com/envoyproxy/go-control-plane v0.6.9/go.mod h1:SBwIajubJHhxtWwsL9s8ss4safvEdbitLhGGK48rN6g=
565566
github.com/envoyproxy/go-control-plane v0.9.0/go.mod h1:YTl/9mNaCwkRvm6d1a2C3ymFceY/DCBVvsKhRF0iEA4=
566567
github.com/envoyproxy/go-control-plane v0.9.1-0.20191026205805-5f8ba28d4473/go.mod h1:YTl/9mNaCwkRvm6d1a2C3ymFceY/DCBVvsKhRF0iEA4=
@@ -1514,8 +1515,8 @@ github.com/shoenig/go-m1cpu v0.1.6/go.mod h1:1JJMcUBvfNwpq05QDQVAnx3gUHr9IYF7GNg
15141515
github.com/shoenig/test v0.6.4 h1:kVTaSd7WLz5WZ2IaoM0RSzRsUD+m8wRR+5qvntpn4LU=
15151516
github.com/shoenig/test v0.6.4/go.mod h1:byHiCGXqrVaflBLAMq/srcZIHynQPQgeyvkvXnjqq0k=
15161517
github.com/shurcooL/sanitized_anchor_name v1.0.0/go.mod h1:1NzhyTcUVG4SuEtjjoZeVRXNmyL/1OwPU0+IJeTBvfc=
1517-
github.com/sijms/go-ora/v2 v2.7.18 h1:xl9CUeBlFi261AOKekiiFnfcp3ojHFEedLxIzsj909E=
1518-
github.com/sijms/go-ora/v2 v2.7.18/go.mod h1:EHxlY6x7y9HAsdfumurRfTd+v8NrEOTR3Xl4FWlH6xk=
1518+
github.com/sijms/go-ora/v2 v2.8.22 h1:3ABgRzVKxS439cEgSLjFKutIwOyhnyi4oOSBywEdOlU=
1519+
github.com/sijms/go-ora/v2 v2.8.22/go.mod h1:QgFInVi3ZWyqAiJwzBQA+nbKYKH77tdp1PYoCqhR2dU=
15191520
github.com/sirupsen/logrus v1.0.6/go.mod h1:pMByvHTf9Beacp5x1UXfOR9xyW/9antXMhjMPG0dEzc=
15201521
github.com/sirupsen/logrus v1.2.0/go.mod h1:LxeOpSwHxABJmUn/MG1IvRgCAasNZTLOkJPxbbu5VWo=
15211522
github.com/sirupsen/logrus v1.4.0/go.mod h1:LxeOpSwHxABJmUn/MG1IvRgCAasNZTLOkJPxbbu5VWo=

state/oracledatabase/oracledatabaseaccess.go

Lines changed: 12 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -348,8 +348,16 @@ func (o *oracleDatabaseAccess) ensureStateTable(stateTableName string) error {
348348
}
349349

350350
func tableExists(db *sql.DB, tableName string) (bool, error) {
351-
var tblCount int32
352-
err := db.QueryRow("SELECT count(table_name) tbl_count FROM user_tables WHERE table_name = upper(:tablename)", tableName).Scan(&tblCount)
353-
exists := tblCount > 0
354-
return exists, err
351+
//nolint:gosec
352+
query := fmt.Sprintf("SELECT 1 FROM %s WHERE ROWNUM = 1", tableName)
353+
354+
var dummy int
355+
err := db.QueryRow(query).Scan(&dummy)
356+
if err != nil {
357+
if errors.Is(err, sql.ErrNoRows) {
358+
return true, nil // Table exists but is empty
359+
}
360+
return false, nil // Likely a table does not exist error
361+
}
362+
return true, nil
355363
}

0 commit comments

Comments
 (0)