Fix(connection/raw query) handle promise when db connection lost

[stevealexandre]

I'm currently developping a gta fivem resource where it could use oxmysql optionnally so I need to check if the database is available or not even after the oxmysql resource is started.
Currently oxmysql do not check if the database is still available after the start while for any reason, the connection can be lost.
Following my tests, I got a promise error not managed which cannot let us handle this case :

After my analysis of your code, I found a quick fix for that which result of this with the test code after:

Here my sample code for testing db connection :

local mysqlHost = GetConvar("mysql_connection_string", "")
local mysqlReady = false
if mysqlHost ~= "" then
    Citizen.CreateThread(function()
        ---Verify if MySQL is ready to send to database

        while GetResourceState("oxmysql") ~= "started" or not exports.oxmysql.isReady() do
            Citizen.Wait(30000)
        end
        -- test callback mode
       exports.oxmysql.query(nil, 'SELECT VERSION() as version', {}, function(result, error)
            if error then
                print("MySQL is not ready !")
                mysqlReady = false
            else
                print("MySQL is ready !")
                mysqlReady = true
            end
        end)

        -- test async mode
        local result = exports.oxmysql.query_async(nil, 'SELECT VERSION() as version', {});
        if result then
            print("MySQL async is ready !")
            mysqlReady = true
        else
            print("MySQL async is not ready !")
            mysqlReady = false
        end
    end)
end

I don't have a full vision of impact which could happens for other resources so feel free to suggest/review my fix.
Maybe a method to test the real current status of database connection could be great and not only trusting local state.