Enhance GetBridgeDBInstance: Add Error Handling for LevelDB Initialization

[vinayak0035]

Overview

Currently, the GetBridgeDBInstance function does not handle scenarios where bridgeDB may be nil. If the database fails to open for any reason, returning a nil object can lead to potential runtime panics or errors when the returned database instance is used. Adding error handling will help prevent these issues and improve the robustness of the code.

Changes Made

• Updated the GetBridgeDBInstance function signature to return an error alongside the database instance.
• Added error handling within the sync.Once block to capture and return any errors encountered while opening the LevelDB database.

Code Changes

Before:

func GetBridgeDBInstance(filePath string) *leveldb.DB {
    bridgeDBOnce.Do(func() {
        bridgeDB, _ = leveldb.OpenFile(filePath, nil)
    })
    return bridgeDB
}

After:

func GetBridgeDBInstance(filePath string) (*leveldb.DB, error) {
    var err error
    bridgeDBOnce.Do(func() {
        bridgeDB, err = leveldb.OpenFile(filePath, nil)
    })
    if err != nil {
        return nil, err
    }
    return bridgeDB, nil
}

Benefits

• Enhanced reliability of the application.
• Easier debugging by providing clear error messages.
• Improved user experience by avoiding crashes due to nil references.