search

clarat-org / clarat

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

Subscribe und Contact not working #563

Closed NilsVollmer closed 8 years ago

NilsVollmer commented 8 years ago

Gefunden bei der Funktionalitätsprüfung von #540 . Besteht auch aktuell auf production, hat also nichts mit der neuen URL Struktur zu tun.

Vermutlich seit der Dreifaltigkeit.. warum fällt uns das erst jetzt auf??

NilsVollmer commented 8 years ago

Okay es gab/gibt mehrere Probleme.. aktueller Stand ist:

  1. Bei jedem Button-Klick gab es eine ActionController::InvalidAuthenticityToken exception. Die kann man durch das selektive Deaktivieren des verify_authenticity_token für die create Methode verhindern (vgl. http://stackoverflow.com/questions/20875591/actioncontrollerinvalidauthenticitytoken-in-registrationscontrollercreate). Die Frage ist, ob das Sicherheitstechnisch relevant ist und nicht anders gelöst werden sollte.
  2. Nachdem die Exception erledigt ist, gibt es Probleme mit den Subscriptions: create sucht vergeblich nach der korrekten View-Datei und wenn man diesen fest auf create.js setzt, gibt es einen MissingMethod Error für subscription_path. Deshalb habe ich als Hotfix (analog zu Contact) einen redirect auf die root page mit success-flash eingebaut.. Wenn man das so lassen will, müssen dafür noch die Tests angepasst werden.
  3. Nachdem die Exception aus Punkt 1. behoben war, macht Contact manchmal Probleme und zwar weil die letzte URL als string gespeichert wird. Wenn die letzte Seite eine Search-Seite mit der langen Query URL ist, sprengt dies das 255 Zeichen limit von string (varchar) => Migration von string zu text mit limit von 1000 Zeichen.
NilsVollmer commented 8 years ago

Update: Hotfix deployed, Punkt 1 & 2 sollten dann zeitnah anders gelöst werden - Punkt 3 ist davon unabhängig und kann so bleiben.

NilsVollmer commented 8 years ago

Mailings: die Subscription-Mail kam nach ca. 30 Minuten an aber das Kontaktformular scheint keine Mails zu verschicken.. Die sollten an post@clarat.org gehen und da ist bis heute nichts angekommen, obwohl Julian und ich mehrere Anfragen gestartet haben.

Edit: Okay auch da gibt es mehrere Probleme, die durch das asynchrone oder verzögerte Verschicken entstehen. Deshalb landen die Mailings nämlich in sidekiq (claradmin). Dort fehlt dann die OfferMailer file aus clarat (zusammen mit den views zurück in _base verschoben) und der Gibbon API key war nicht gesetzt (gibbon.rb in initializers übernommen). Soweit zu den gelösten Problemen.

Dann sind in der sidekiq queue noch einige EmailPusherWorker Jobs (Subscription) mit invaliden IDs (zu hoch), von denen ich nicht weiß, wo sie herkommen.

NilsVollmer commented 8 years ago

Zusammen mit @KonstantinKo gelöst und als hotfix direkt auf master deployed.

Note to self: prüfen, ob die Mails auch ankommen.