Oracle DB monitor -- steps in #2796 does not seem to work

[Sriram-Kumar1]

📑 I have found these related issues/pull requests

#2493

🏷️ Feature Request Type

New monitor

🔖 Feature description

Should be able to monitor connection to an Oracle DB

✔️ Solution

Install oracle db node package
add option for monitoring oracle db accepting host name , port, service name, user name and password

❓ Alternatives

No response

📝 Additional Context

made changes mentioned in #2796 but the oracle db as an option is not showing in the drop down. I am new to this framework any help would be appreciated.

[Sriram-Kumar1]

running uptime-kuma Version: 1.23.13

1. installed oracledb node package
2. Changes done in following files

src\pages\EditMonitor.vue

Lines 77-80


Oracledb

Lines 388-393

                       <!--Oracle -->
                      <template v-if="monitor.type === 'Oracledb'">
                            <div class="my-3">
                        <input id="sqlConnectionString" v-model="monitor.databaseConnectionString" type="text" class="form-control" placeholder='"user":"<USER>","password":"<PASSWORD>","connectionString":"(DESCRIPTION =(ADDRESS_LIST=(ADDRESS     =(PROTOCOL=TCP)(HOST=<HOSTADDRESS>)(PORT=<PORT>)))(CONNECT_DATA=(SERVER=DEDICATED)(SERVICE_NAME=<SERVICENAME>)))"'>
                           </div>
                     </template>

3. edited file server\model\monitor.js to handle OracleDb monitor:

Line 8-9

const { tcping, ping, dnsResolve, checkCertificate, checkStatusCode, getTotalClientInRoom, setting, mssqlQuery, postgresQuery, mysqlQuery, oracledbQuery,mqttAsync, setSetting, httpNtlm, radius, grpcQuery,
redisPingAsync, mongodbPing, kafkaProducerAsync, getOidcTokenClientCredentials, rootCertificatesFingerprints, axiosAbortSignal

line 861-868

  } else if (this.type === "oracledb") {
            let startTime = dayjs().valueOf();

             await oracledbQuery(this.databaseConnectionString, this.databaseQuery);
              bean.msg = "";
              bean.status = UP;
              bean.ping = dayjs().valueOf() - startTime;        
  }

4. cd to project dir and run npm rebuild, then start again uptime-kuma.

5. After the changes are done, I dont see the oracle db as an option in the add monitor page.

[CommanderStorm]

Regarding oracle
Monitoring oracle databases won't be supported as this would mean having to drop arm support (which is a non-starter)
This monitor will thus not be merged. Consider asking oracle for arm support.

Regarding your post. Could you specify what you mean by the following? (npm run setup for expample will delete any local changes by design)

cd to project dir and run npm rebuild, then start again uptime-kuma.

given how brittle such a patch would be (you will need to maintain your fork), I suggest to use the push monitor instead with a script of your choosing.

[Sriram-Kumar1]

@CommanderStorm thank you. I could not quiet understand arm support. (ARM as in processor)?.

I observe that oracle site is showing that their thick client is supported in ARM processor

https://node-oracledb.readthedocs.io/en/latest/user_guide/installation.html#aarch64

not sure if I am missing something obvious.

coming to the steps, I repeated the steps mentioned in #2796

after I made the changes in the above mentioned files I did a npm rebuild and restarted uptime-kuma from pm2

I will look up push monitor and see how to go about it.

[CommanderStorm]

Things must have changed since we last looked at this

the node-oracledb driver is a pure JavaScript module that directly connects to Oracle Database 12.1 or later. This ‘Thin’ mode does not need Oracle Client libraries. The database can be on the same machine as Node.js, or it can be remote.

We either overlooked thin mode or (more likely) that is a newer development..
=> Should be possible to do that way

When this was last looked into, oracle required binary blobs and those were neither bundled with the package, nor build for arm.

after I made the changes in the above mentioned files

Could you provide that as a pull request? Reviewing the text above for what is missing is pretty hard
Among others, the above code is missing oracledbQuery and the Monitor-Registration (see the "new monitoring type" part of our contribution guide