MySQL: Spalten mit NOT NULL werden nicht immer gesetzt

Scout24 / restapi-php-sdk

SDK für PHP-Entwickler um die Arbeit mit der API von ImmobilienScout24 zu vereinfachen. Über die API erhält man Ergebnislisten, Objektdaten, uvm. von ImmobilienScout und kann diese Daten inhherhalb seines Webservice integrieren. Die Dokumentation zum SDK findet man unter https://github.com/Immocaster/php-sdk/wiki.

64 stars 46 forks source link

MySQL: Spalten mit NOT NULL werden nicht immer gesetzt #25

Closed mindhaq closed 9 years ago

mindhaq commented 9 years ago

Die automatisch generierte Tabelle ist nicht kompatibel mit den abgesetzten INSERT statements.

ic_expire und ic_username werden beide als NOT NULL definiert.

Bei saveRequestToken() wird ic_username nicht gesetzt.

Bei saveApplicationToken() wird ic_expire nicht gesetzt.

Beide INSERTs schlagen daher fehl; im ersten Fall wird dann trotzdem zum oauth weiter geleitet, der Fehler wird "verschluckt".

fmielke commented 9 years ago

Ja, ic_username sollte null sein dürfen, denn dieser erhält null, wenn die Authentifizierung nicht komplett durchgeht. ic_desc wird dann mit "REQUEST" gefüllt.

ic_expire ist bei erfolgreicher Authentifizierung grundsätzlich '0000-00-00 00:00:00.000000', weil Access Token und Access Token Secret immer unendlich lang gültig sind. Wenn ein "REQUEST" Eintrag erfolgt, ist dieser ein Stunde gültig (obwohl die Dauer auf 24h erhöht wurde).

Für beides überlegen wir uns eine Lösung. Danke dir!

fmielke commented 9 years ago

Fixed in f3e2913d56050ac42fc7f2ea934f466021f9c913.

ic_username nun nullable und ic_expire wird auch bei saveApplcationToken gesetzt.

Vielen Dank, @mindhaq !!!