search

data-quest / EvasysPlugin

Plugin for Stud.IP to manage and provide evaluations with EvaSys by Electric Paper
GNU General Public License v2.0
1 stars 2 forks source link

adds separate socket-timeout config for soap calls; closes #28 #29

Closed crosscodr closed 3 years ago

crosscodr commented 3 years ago

Hi Rasmus, ich habe mal eine Konfigoption für einen eigenen socket-timeout im EvasysPlugin gebaut. Um deine Red-icon-Funktion nicht kaputt zu machen habe ich die soapCall-Methode etwas refactored. Bin für Anmerkungen und Verbesserungsvorschläge, und/oder das akzeptieren des Pull-Requests dankbar.

crosscodr commented 3 years ago

Hi @Krassmus, hast du hierzu noch Verbesserungsvorschläge? Mein commit ändert nichts an dem bisherigen Verhalten, bietet nur die Möglichkeit einen socket-timeout für die soap-calls zu EvaSys zu definieren. Dort habe ich den default von php übernommen (60 sekunden, default_socket_timeout).

Krassmus commented 3 years ago

Mir ist noch nicht so ganz klar, warum man das braucht. Aber es tut nicht weh.