search

clarat-org / clarat

An Open Source Charity Directory
http://www.clarat.org/
MIT License
8 stars 10 forks source link

Bug: Orga-Update (auch create?) mit InlineCreation schmeißt Fehler #1422

Open NilsVollmer opened 6 years ago

NilsVollmer commented 6 years ago

Berichtet bei:

NilsVollmer commented 6 years ago

Update: Bug ist weder lokal noch auf staging zu reproduzieren aber auf production tritt er definitiv auf: ich habe eine Orga zum testen von Astrid genommen und kann dort keinen neuen Standort inline hinzufügen.

Leider hilft die Fehlermeldung in den Logs nicht wirklich und lokal tritt der Bug ja nicht auf.. Das der Bug aber nur auf production, sagt ja auch schonmal etwas aus.. vllt. hat es eher was mit Gmaps & Geokoordinaten etc. zu tun.. irgendetwas, was auf production anders als auf staging ist.

NilsVollmer commented 6 years ago

Update: ich habe jetzt einen potentiellen Fix aufgespielt, der allerdings eher ins Blaue geraten ist: in der Database hat das Location label ein limit von 255 und vllt. wird irgendwie zwischenzeitlich ein längeres Label erzeugt und dann direkt wieder überschrieben.. aber wie gesagt: ich bin mir nicht sicher, ob der Bug wirklich behoben ist weil ich immer noch nicht nachvollziehen konnte, wann er auftritt.

Da es nur auf production der Fall war, tippe ich auf eine irgendwie geartete race condition, die nur auf dem schnellen Production-Server auftritt.

Naja die Researcher sind informiert, sollen wie gewohnt arbeiten und sich melden falls das doch wieder nicht funktioniert.

NilsVollmer commented 6 years ago

Update: der Bug ist durch das limit(255) leider doch nicht gefixed.

NilsVollmer commented 6 years ago

Update: im Location Model ware noch einige komische Dinge, die potentiell bei RaceConditions Probleme machen könnten. Die habe ich jetzt rausgenommen und in den entsprechenden Operations umgesetzt. Vielleicht fixed das den Bug. Ist deployed und das Issue wieder pending..

NilsVollmer commented 6 years ago

Update: leider scheint der Bug nach wie vor zu bestehen :/

@KonstantinKo falls du dich dem hier in meiner Abwesenheit annehmen willst ;)

NilsVollmer commented 6 years ago

Erst einmal wieder pending, bis der Bug wieder auftritt. Dann müssen wir vllt. mal erweitertes Logging für diese Fall aktivieren, damit wir aus dem Production-only-bug schlauer werden.

NilsVollmer commented 6 years ago

Update: erweitertes Logging auf Production deployed.