search

thewebhatesme / opm_server

Open Performance Monitor - Server
MIT License
13 stars 1 forks source link

Register Client #6

Open thewebhatesme opened 10 years ago

thewebhatesme commented 10 years ago

If a user want's to set up a client he must be able to request a client ID.

xyNNN commented 10 years ago

Its a simple create method which returns a identifier (client id) as a md5 hash or not? This ID is implemented in the config of the client to recognize the incoming packages.

thewebhatesme commented 10 years ago

Ich glaub ich wechsel in der Entwicklungszeit mal auf Deutsch.

Ich würde die Konfiguration des Clients gerne auf dem Server machen. Das heißt, dass Standort und Intervall beim Server hinterlegt sind und nicht im Client. Das war so die Idee hinter der ID.

xyNNN commented 10 years ago

Wäre natürlich sehr schön. Soll heißen. dass der Cronjob jede Minute ausgeführt werden sollte und das Intervall, wie oft die entsprechenden Tests durchlaufen werden, vom Server definiert wird? Soll also bei jedem Aufruf die Konfiguration vom Server geladen werden? Oder bei Änderung gepusht? Caching? ;)

thewebhatesme commented 10 years ago

Ich würde immer alles abrufen. Wenn die Last mal zu hoch wird, dann kann man sich da noch was überlegen. Ich glaube aber nicht, dass wir an der Stelle Performanceprobleme bekommen werden.

mikelohmann commented 10 years ago

Wir können hier doch einfach mit HTTP arbeiten. 304 Not modified, wenn sich die Config nicht geändert hat.

Am 13. März 2014 13:32 schrieb thewebhatesme notifications@github.com:

Ich würde immer alles abrufen. Wenn die Last mal zu hoch wird, dann kann man sich da noch was überlegen. Ich glaube aber nicht, dass wir an der Stelle Performanceprobleme bekommen werden.

— Reply to this email directly or view it on GitHubhttps://github.com/thewebhatesme/opm_server/issues/6#issuecomment-37527324 .

mikelohmann commented 10 years ago

Wir haben die Clientregistrierung in den Tests aufgeschrieben (OpmServerBundle/Features). Diese müssen implementiert werden und dann eben ein Controller angelegt werden, der die Registrierung machen kann. Ein einfacher CRD-Crontroller ist ausreichend, denke ich.

mikelohmann commented 10 years ago

Ich sehe das doch eher bei 1.0 in der Fassung: Client Registrieren udn ID manuell in Client eintragen. Was meint ihr?