dblib: fix SID parsing

CHANGELOG.md

@@ -1,5 +1,9 @@
 # Go Library
 
+## [v1.7.2 - 2023-05-16]
+### Fixed
+- dblib: fix SID parsing
+
 ## [v1.7.1 - 2023-05-10]
 ### New
 - pwlib: add vault method to get_password

dblib/tns.go

@@ -57,7 +57,7 @@ func CheckTNSadmin(tnsadmin string) (dn string, err error) {
 // BuildTnsEntry build map for entry
 func BuildTnsEntry(filename string, desc string, tnsAlias string) TNSEntry {
 	var service = ""
-	reService := regexp.MustCompile(`(?mi)SERVICE_NAME\s*=\s*([\w.]+)`)
+	reService := regexp.MustCompile(`(?mi)(?:SERVICE_NAME|SID)\s*=\s*([\w.]+)`)
 	s := reService.FindStringSubmatch(desc)
 	if len(s) > 1 {
 		service = s[1]
@@ -194,7 +194,7 @@ func tnsSanity(entries TNSEntries) (tnsEntries TNSEntries, deletes int) {
 			se++
 		}
 		if len(e.Service) == 0 {
-			log.Errorf("Entry %s has no Service set", k)
+			log.Errorf("Entry %s has no SERVICE_NAME or SID set", k)
 			se++
 		}
 		if len(e.Servers) == 0 {

dblib/tns_test.go

@@ -67,14 +67,18 @@ XE1 =(DESCRIPTION =
 	(ADDRESS_LIST = (ADDRESS=(PROTOCOL=TCP)(HOST=127.0.0.1)(PORT=1521)))
 	(CONNECT_DATA=(SERVER=DEDICATED)(SERVICE_NAME = XE1))
 )
+XE.SID =(DESCRIPTION =
+	(ADDRESS_LIST = (ADDRESS=(PROTOCOL=TCP)(HOST=127.0.0.1)(PORT=1521)))
+	(CONNECT_DATA=(SID = XESID))
+)
 XE.error = Error
 `
 
 const sqlnetora = `
 NAMES.DEFAULT_DOMAIN=local
 NAMES.DIRECTORY_PATH=(TNSNAMES,EZCONNECT,LDAP)
 `
-const entryCount = 5
+const entryCount = 6
 
 var tnsAdmin = "testdata"
 
@@ -142,6 +146,12 @@ func TestParseTns(t *testing.T) {
 				success: true,
 				service: "XE",
 			},
+			{
+				name:    "XE-SID",
+				alias:   "XE.SID",
+				success: true,
+				service: "XESID",
+			},
 			{
 				name:    "XE1-short-invalid",
 				alias:   "XE1",