[Gelöst] Discovery über Ping

[foxriver76]

Hi Leute,

ich hätte mal eine Frage bezüglich der Discovery Funktion mittels Ping. In meinem Fall suche ich ein Gerät, dass ich über die REST-API eindeutig identifizieren könnte. UPnP nicht vorhanden.

Wie ist das korrekte Vorgehen? So wie ich das aktuell verstehe:

sende PING an Broadcast 
Für jedes antwortende Device: 
	Spreche die API an und prüfe ob ich eine korrekte Antwort bekomme

Gibt es da ein sinnvolleres Vorgehen?

beste Grüße

fox

[foxriver76]

Push, mit Wink an Apollon 8-)

[apollon77]

Sehr viel aber nicht alles.

Discovery selbst macht Ping und alle Discovery checks mit Typ Ping werden dann aufgerufen.

Schau in Discovery mal diese durch. Da gibts welche die am Ende dann einen http request an eine bestimmte url der ip machen und nach Antwort auswerten ob’s gefunden wurde.

Gefunden?

Gesendet vom Handy …

[foxriver76]

Vielen Dank, habe gute Beispiele gefunden, bzw habt ihr ja schon einen Großteil der Arbeit erledigt.

Nur habe ich eben versucht den Discovery Adapter zu erweitern, habe entsprechendes File nach /lib/adapters geschoben, Upload und Neustart v Discovery gemacht. Nur im debug log sehe ich, dass er mein File gar nicht berücksichtigt. Habe auch logging ins File eingebaut was nicht ausgespuckt wird. Hast du eine Idee woran es liegen könnte?

! ````
const tools = require(__dirname + '/../tools.js');
! function addSonnen(ip, device, data, options, callback) {
var instance = tools.findInstance(options, 'sonnen', (obj) => {
return (obj.native.ip === ip || obj.native.ip === device._name);
});
! if (!instance) {
instance = {
_id: tools.getNextInstanceID('sonnen', options),
common: {
name: 'sonnen',
title: 'sonnen (' + ip + (device._name ? (' - ' + device._name) : '') + ')'
},
native: native,
comment: {
add: [ip]
}
};
options.newInstances.push(instance);
callback(null, true, ip);
} else {
callback(null, false, ip);
} // endElse
} // endAddSonnen
! function detect(ip, device, options, callback) {
adapter.log.debug('Sonnen search');
tools.httpGet('http://' + ip + ':8080/api/v1/status', (err, data) => {
if (err) {
if (callback) {
return callback(null, false, ip);
} // endIf
} else {
let testData;
try {
testData = JSON.parse(data);
} catch (e) {
testData = null;
}
if (testData && testData.hasOwnProperty('GridFeedIn_W')) {
var native = {
ip: ip
}
addSonnen(ip, device, options, native, callback);
} else if(callback) {
return callback(null, false, ip);
} // endElse

      } // endElse

! });
} // endDetect
! exports.detect = detect;
exports.type = ['ip'];
exports.timeout = 1500;

[apollon77]

Hast Du auch das richtige Verzeichnis erwischt wo das file (lib/adapter) drin liegt?

[foxriver76]

@apollon77:

Hast Du auch das richtige Verzeichnis erwischt wo das file (lib/adapter) drin liegt? `

Ich gehe davon aus:

ls -lah /opt/iobroker/node_modules/iobroker.discovery/lib/adapters/ | grep son
-rwxrwxrwx 1 pi pi 1,4K Aug  6 21:34 epson_stylus_px830.js
-rwxrwxrwx 1 pi pi 1,7K Aug  6 22:16 sonnen.js
-rwxrwxrwx 1 pi pi 6,7K Aug  6 21:34 sonos.js

Er sollte ja einfach über jeden Adapter drüber iterieren, im debug zeigt er jeden an außer meinen hinzugefügten. Müsste eigentlich irgend ein ganz stupider Fehler sein.

[Bluefox]

Du hattest ein paar Fehler drin. Schaue ob es läuft: https://github.com/ioBroker/ioBroker.di … /sonnen.js

[foxriver76]

@Bluefox:

Du hattest ein paar Fehler drin. Schaue ob es läuft: https://github.com/ioBroker/ioBroker.di … /sonnen.js `

Nach wie vor, geht er bei mir gar nicht in den Code rein. Im Debug Log stehen die aufgerufenen Suchmethoden je Adapter, jedoch nicht für meinen:

Ausschnitt aus dem Log, gesamter Lauf ist zu umfangreich:

2018-08-07 23:27:31.825 - debug: discovery.0 Test 192.168.178.26 maxcube
2018-08-07 23:27:31.826 - debug: discovery.0 Test 192.168.178.26 megad
2018-08-07 23:27:31.828 - debug: discovery.0 Test 192.168.178.26 miele
2018-08-07 23:27:31.829 - debug: discovery.0 Test 192.168.178.26 mihome
2018-08-07 23:27:31.829 - debug: discovery.0 Test 192.168.178.26 musiccast
2018-08-07 23:27:31.830 - debug: discovery.0 Test 192.168.178.26 nut
2018-08-07 23:27:31.831 - debug: discovery.0 Test 192.168.178.26 openhab
2018-08-07 23:27:31.835 - debug: discovery.0 Test 192.168.178.26 ping
2018-08-07 23:27:31.836 - debug: discovery.0 Test 192.168.178.26 ping DETECTED!
2018-08-07 23:27:31.837 - debug: discovery.0 Test 192.168.178.26 samsung
2018-08-07 23:27:31.838 - debug: discovery.0 Test 192.168.178.26 sonos
2018-08-07 23:27:31.842 - debug: discovery.0 Test 192.168.178.26 sql
2018-08-07 23:27:31.844 - debug: discovery.0 Test 192.168.178.26 squeezebox

beste Grüße

fox

[unltdnetworx]

Das gleiche Problem hatte ich auch. Das liegt daran, dass es eine Function gibt, die überprüft ob deine Erweiterung bereits im Repo besteht und ansonsten ignoriert. Bin gerade unterwegs, darum kann ich nicht genau schauen wie diese heißt. Aber man kann zwei Zeilen davon auskommentieren und schon läuft es auch mit deinem Code.

Näheres heute Abend, falls gewünscht.

[foxriver76]

@unltdnetworx:

Das gleiche Problem hatte ich auch. Das liegt daran, dass es eine Function gibt, die überprüft ob deine Erweiterung bereits im Repo besteht und ansonsten ignoriert. Bin gerade unterwegs, darum kann ich nicht genau schauen wie diese heißt. Aber man kann zwei Zeilen davon auskommentieren und schon läuft es auch mit deinem Code.

Näheres heute Abend, falls gewünscht. `

An so was dachte ich auch schon, hatte dann nur ehrlich gesagt keine Lust mehr zu debuggen.

Entsprechend wäre es super cool, wenn du mir Namen + Location der Funktion sagen könntest.

[unltdnetworx]

In der main.js des Discovery gibt es die Function:

function isValidAdapter(adapterName, type, dependencies) {
    if (!adapters.hasOwnProperty(adapterName)) return false;
    var adapter = adapters[adapterName];
    if (typeof adapter.type === 'string' && adapter.type !== type) return false;
    if (typeof adapter.type === 'object' && adapter.type.indexOf(type) === -1) return false;
    return ((!!adapter.dependencies) === dependencies);
}

Hier meine ich die erste if-Zeile auskommentiert zu haben. Falls es so nicht klappt, melde ich mich heute Abend.

Edit: Ich schaue doch besser nochmal nach. Stimmt wohl so eher nicht.

[foxriver76]

@unltdnetworx:

In der main.js des Discovery gibt es die Function:

function isValidAdapter(adapterName, type, dependencies) {
    if (!adapters.hasOwnProperty(adapterName)) return false;
    var adapter = adapters[adapterName];
    if (typeof adapter.type === 'string' && adapter.type !== type) return false;
    if (typeof adapter.type === 'object' && adapter.type.indexOf(type) === -1) return false;
    return ((!!adapter.dependencies) === dependencies);
}

Hier meine ich die erste if-Zeile auskommentiert zu haben. Falls es so nicht klappt, melde ich mich heute Abend.

Edit: Ich schaue doch besser nochmal nach. Stimmt wohl so eher nicht. `

Das wars leider nicht. Trotzdem schon mal vielen Dank.

[unltdnetworx]

Jetzt hab ichs glaub ich:

! ```
`function enumAdapters(repository) {
var dir = fs.readdirSync(__dirname + '/lib/adapters');

for (var f = 0; f < dir.length; f++) {
    var parts = dir[f].split('.');
    if (parts[parts.length - 1] === 'js') {
        parts.pop();

        var moduleName = __dirname + '/lib/adapters/' + dir[f];
        var aName      = parts.join('.');

        if (adapters && adapters[aname] && adapters[aname].reloadModule) {
            var module = require.resolve(moduleName);
            delete require.cache[module];
            delete adapters[aname];
        }

! // check if this adapter available in repository
if (!adapters[aname] && (!repository || repository.indexOf(aName) !== -1)) {
adapters[aname] = require(moduleName);
}
}
}
}! In der letzte if-Zeile muss[code]&& (!repository || repository.indexOf(aName) !== -1)[/code]` entfernt werden. Ich hab mir die Zeile kopiert und ausgeklammert im Entwicklungssystem.[/aname][/aname][/aname][/aname][/aname]

[foxriver76]

@unltdnetworx:

Jetzt hab ichs glaub ich:

! ```
`function enumAdapters(repository) {
var dir = fs.readdirSync(__dirname + '/lib/adapters');

for (var f = 0; f < dir.length; f++) {
    var parts = dir[f].split('.');
    if (parts[parts.length - 1] === 'js') {
        parts.pop();
        
        var moduleName = __dirname + '/lib/adapters/' + dir[f];
        var aName      = parts.join('.');
        
        if (adapters && adapters[aname] && adapters[aname].reloadModule) {
            var module = require.resolve(moduleName);
            delete require.cache[module];
            delete adapters[aname];
        }

! // check if this adapter available in repository
if (!adapters[aname] && (!repository || repository.indexOf(aName) !== -1)) {
adapters[aname] = require(moduleName);
}
}
}
}! In der letzte if-Zeile muss[code]&& (!repository || repository.indexOf(aName) !== -1)[/code]entfernt werden. Ich hab mir die Zeile kopiert und ausgeklammert im Entwicklungssystem. ! Wow, da ist ja sogar nen Comment im Code --> dann bin ich selbst schuld :D ! Ist auf jeden Fall der Ursprung des Problems, vielen Dank.[/aname][/aname][/aname][/aname][/aname] ```