Roba1993 / Happy-Hour

Das DHBW Happy-Hour Projekt vom Kurs WWI2012F
MIT License
4 stars 0 forks source link

Logging Entscheidung #131

Closed Neram closed 9 years ago

Neram commented 9 years ago

Wir müssen uns einigen, wie wir einheitlich loggen wollen. Ich sehe dafür drei Alternativen: 1) Wir erstellen einen extra Filter

2) Wir fügen in jede REST Funktion einen einheitlichen Logger ein

Bitte immer ABHAKEN, wenn ihr in eure REST Schnitstellen die Logger eingefügt habt, sowohl den bei der ANkunft der Anfrage als auch bei der Rückgabe, damit ich den Überblick behalte:

Neram commented 9 years ago

Da von euch bisher keine Rückmeldung kam, habe ich mich so eben mit Joans darauf geeinigt, dass wir nach jeder REST Funktion die aufgerufen wird als erstes ein log.debug erstellen, das wie folgt aussehen soll:

log.debug(request.requestMethod() + ":" + request.pathInfo());

Nach erfolgreicher Rückgabe soll ebefalls ein Log erstellt werden:

log.debug(response.success + response.description);

Neram commented 9 years ago

Bei einem seperaten Filter kann kein Response Objekt übergeben werden

Neram commented 9 years ago

Für die Rückgabe ist das response Objekt, das RESTresponse Objekt, dass ihr erstellt habt. Wenn ich es richtig sehe, müsste es also wie folgt aussehen, wenn euer RESTresponse Objekt z.B. "r" heißt:

log.debug( r.getStatus + r.description + r.data );

Neram commented 9 years ago

Bei Aufruf der REST Funktion haben wir nun einheitlich folgendes stehen: LOG.debug("HTTP-POST Anfrage eingetroffen: " + request.queryString());

Bitte ergänzt noch den Log-Eintrag für die Rückgabe!

stegmajo commented 9 years ago

Für die Rückgabe können wir nun auch die toString Funktion von RESTResponse nutzen: Also LOG.debug("RESTResponse verschickt: " + restResponse);

Neram commented 9 years ago

JA ist richtig, könnte man auch machen. Allerdings haben diejenigen, die es schon gemacht haben jetzt das andere verwendet und damit es einheitlich ist bitte ich es genauso zu machen.

stegmajo commented 9 years ago

Ok, erledigt

Neram commented 9 years ago

Der LOG-Eintrag wurde überall erstellt