• Alexander F.
    Alexander F.
    2015-06-06

    o.O z.b.?

    0
  • Lu Kaiser
    Lu Kaiser
    2015-06-06

    Danke!

    0
  • Faldrian 💫
    Faldrian 💫
    2015-06-06

    Halte durch! Fix einfach eine Sache nach der nächsten... oder einen zusammengehörigen Haufen nach dem nächsten. :)

    0
  • medprofiler@pod.geraspora.de
    medprofiler@pod.geraspora.de
    2015-06-06

    Quellcode Review: "Ist das Kunst oder kann das weg?" xD

    Ansonsten halte ich es mit Faldis Worten: Halte durch! ;)

    Deine Arbeit am Projekt ist echt gigantisch und ich sage mal nen fettes DANKE

    0
  • Benjamin Neff
    Benjamin Neff
    2015-06-06

    @Alexander: zB alles ...

    Irgendwie so jeden Bug den ich angucke und denke "joah, müsste nur ne Kleinigkeit im Controller sein. Das sollte in 10min gefixt sein ..." liegts schlussendlich irgendwo in der verkackten Federation ...

    Ausserdem findet man dann so Dinge im Code: "#this is done implicitly, somewhere else, but it doesnt work, says max. :'(" ... also es wird lieber alles doppelt federiert (ist vermutlich der Grund für die doppelten Fotos, aber an dem Problem bin ich grad nicht...), statt mal zu gucken, wo der Fehler wirklich ist ... aber vermutlich haben wir dem doppelt federieren und den Retries im Sidekiq zu verdanken, dass es überhaupt federiert ...

    Also irgendwie hab ich das Gefühl, die Federation wurde irgendwann mal geschrieben, es hat irgendwie funktioniert, Teilweise zumindest, und dann hat mans einfach gelassen ... ab und zu mal wieder irgendwo nen Pflaster drangeklebt und nen Hack ... damits irgend nen Problem verdeckt ...

    Ich hatte ja lange Angst die Federation anzupacken (vermutlich nicht nur ich, weil die Federation ist irgendwie alles so Code von 2011 ... das hat nie wieder jemand angefasst, höchstens mal von einem File ins andere verschoben), weil das ja irgendwie das Zentrale von Diaspora ist ... da kann man so viel kaputt machen ... Aber jedesmal wenn ich mir irgend nen Bug angucke, merke ich, dass das schon alles kaputt ist, da kann man nicht mehr viel kaputt machen ... das ist mehr ein Wunder ist, dass überhaupt was tut ...

    @admin password: Ich hab mittlerweile aufgehört zu fragen "Wieso wurde das eigentlich so gemacht?" oder "hat das irgend nen Sinn?" ... Weil darauf weiss meistens keiner mehr eine Antwort ... Und wenn man versucht anhand der history rauszufinden, wo das her kommt, und wieso das so gemacht wird, findet man so commit-messages wie "max, you're a princess" und ist kein bisschen schlauer ... ;)

    0
  • Alexander F.
    Alexander F.
    2015-06-06

    @Benjamin sind das commits von früher von den 4 Jungen und der Code auch? Vllt sollte man sich mal zusammen setzen und das komplette Federations Protokoll überarbeiten und neu ausarbeiten, bevor es irgendwann komplett kaputt ist und Diaspora lahm legt und ist ja auch nicht grade sehr attraktiv so zu versuchen andere Netzwerke mit anzubinden. Wo man dann hinterher sagt ja haben wir jetzt irgendwie so zu 80% geschafft aber wo die Fehler herkommen wissen wir nicht und bekommen wir auch nicht beseitigt.

    0
  • Alexander F.
    Alexander F.
    2015-06-06

    Achja und damit die Ansage nicht nur so klingt wie ja macht doch mal aber ich mach da nicht mit, ich würde auch gerne mithelfen wenn ich kann, ich kann sowas nur nicht gut anführen/leiten oder wie man es auch nennen möchte.

    0
  • Benjamin Neff
    Benjamin Neff
    2015-06-06

    ja, das meiste von dem code ist von vor 3-4 jahren ...

    Ich wollte auch mit Schnubbi mal alles überarbeiten, wollte aber vorher noch nen paar "Kleinigkeiten" fixen, aber irgendwie lande ich trotzdem jedesmal wieder in der Federation ...

    Das Protokoll an sich ist nicht das Problem, die Impl ist das Problem ... Und das es dann noch Friendica und RedMatrix gibt, die beide auch noch irgendwie das Protokoll implementieren, aber alle so nen bisschen anders ... machts dann auch nicht einfacher ;)

    0
  • Alexander F.
    Alexander F.
    2015-06-07

    hmm aso na dann Neuimplementieren und wir müssten uns wohl mal mit denen zusammen setzen oder irgendwie sowas...man um die uhrzeit fehlen mir ja die worte noch eher xD

    0
  • mʕ•ﻌ•ʔm jesuiSatire .. ᘛ⁐̤ᕐᐷ
    mʕ•ﻌ•ʔm jesuiSatire .. ᘛ⁐̤ᕐᐷ
    2015-06-07

    Wenn ich das richtig gelesen habe hatte Mike (friendica/redmatrix) den code damals für friendia selbst durchgearbeitet, und es jetzt für redmatrix wahrscheinlich noch mal getan. Der einzige von dem er feedback bekam damals war seiner Aussage nach Ilya. Mike will damit bestimmt auch nichts mehr zu tun haben, aber er begreift wahrscheinlich viel von dem Thema ich würde mal versuchen ihn anzupingen. Er erwähnte sogar das er alte Bekannte die rails coder sind befragt hatte. Ich mein die Leute müßen doch schon fast in Rente sein, so einen Prof dem die NSA auch auf'n Sack geht muß man doch irgendwie an Land ziehen können ..

    0
  • Matthias Ämmm
    Matthias Ämmm
    2015-06-07

    @Benjamin Hälst du es denn prinzipiell für möglihc die Federation auszubauen und durch was fertiges wie z.b. PumpIO zu ersetzen? Wir sind ja nicht die einzigen mit dem Problem: W3C Social WG

    0
  • ravenbird@nerdpol.ch
    ravenbird@nerdpol.ch
    2015-06-07

    Guten Morgen Zusammen! Leider habe ich Deinen Beitrag gerade erst gesehen Benjamin. Ich muss schmunzeln da das was Du als Entwickler beschreibst sich mit dem deckt was ich als normaler Anwender oftmals denke, wenn auch auf einen wesentlich weniger technischen Niveau.

    Die Frage die sich stellt ist demnach was mehr Sinn macht bzw. was mehr Aufwand verursacht. Die bestehende Federation zu fixen oder das Ganze neu zu konzipieren.

    Ich habe darüber hinaus nach wie vor die Meinung das der leider viel zu früh gestorbene Ilya Zhitomirskiy der eigentliche Vater und Entwickler von Diaspora war. Das nach seinem Tod damals nur noch wenig bis nichts sinnvolles und dafür viel Mist kam spricht klar dafür. Sprich der Community wurde damals Ende 2012 keine stabile Basis sondern eine Baustelle bei der es vorne und hinten hängt übergeben.

    Natürlich entspricht das nur meiner Meinung und Sichtweise. :-)

    0
  • (: aNNa :) blume
    (: aNNa :) blume
    2015-06-07

    Als oist die Föderation sowas wie BER in der Diaspora ? ;-)

    0
  • ravenbird@nerdpol.ch
    ravenbird@nerdpol.ch
    2015-06-07

    Scheint so, nur nicht so teuer...

    0
  • Faldrian 💫
    Faldrian 💫
    2015-06-07

    @Tuxxi

    Gibt es denn - oder hast du eine - Möglichkeit um die ganzen Federations-Dinge Ende-Zu-Ende zu testen, so dass man da überhaupt vernünftige Testes mit 2 Pods vollautomatisiert machen kann? Das wäre uns in dem Sektor ja irgendwie weiterhelfen, wenns verlässliche Testabdeckung gibt.

    0
  • herzmeister der welten
    herzmeister der welten
    2015-06-07

    manchmal ist doch grüne wiese das beste...

    0
  • Benjamin Neff
    Benjamin Neff
    2015-06-07

    @Matthias: Das Protokoll an sich, ist nicht das Problem, es ist nur halt zum Teil fehlerhaft implementiert. Da bringts auch nichts was anderes zu benutzen, solange das auch nicht richtig gemacht wird ;) Ausserdem muss man ja immernoch mit alten Pods kommunizieren können, also kann man da nicht einfach plötzlich was komplett anderes nehmen. Man muss einfach das bestehende Protokoll stabiliseren.

    Ziel ist es aber auch die Federation als eigenes Modul zu bauen, und nicht mehr fix in diaspora verdrahtet. Das machts dann 1. einfacher zu testen, 2. kann man das dann auch sonstwo verwenden und 3. könnte man das auch einfacher austauschen, bzw um vielleicht noch nen anderes Protokoll erweitern, oder was auch immer, aber kA ob das überhaupt Sinn macht.

    @Ravenbird: Naja, komplett neu konzipieren kann man halt eben nur schwer, da man immer noch mit alten Pods kompatibel bleiben muss, mindestens für ne gewisse Zeit. Das Protokoll an sich ist aber wie gesagt gar nicht wirklich das Problem, es gibt nur jede Menge an Spezialfälle die nicht abgedeckt sind, oder fehler die nicht richtig gehandelt werden ;) Zumal der Code zum Teil nen ziemliches Chaos ist ;)

    @Faldrian: Die einzelnen Endpunkte sind Unit-Getestet. Es gibt überreste von einer Integrations-Test-Umgebung, allerdings scheint die (soviel ich das sehen kann) nie richtig funktioniert zu haben, bzw war nie ganz fertig. Wenn man die Federation als gem hat, kann man das ganze aber auch einfacher testen :) Im moment teste ich zwischen nina und olivia ;) Also versuch erst die Probleme zu reproduzieren, debuggen und dann fixen, und dann nochmal testen, obs jetzt geht :) Aber das ersetzt natürlich keine automatischen Tests.

    0
  • Matthias Ämmm
    Matthias Ämmm
    2015-06-08

    Danke für deine Erläuterung! Gibt es denn sonst etwas, was Leute ohne Zeit und ohne RoR-Kenntnisse machen können?

    0
  • ravenbird@nerdpol.ch
    ravenbird@nerdpol.ch
    2015-06-08

    Auch von mir Danke für die Erklärung und den Einblick. Da ist offenbar noch viel zu tun.

    0
  • Michael 🇺🇦
    Michael 🇺🇦
    2015-06-08

    Ilya war die treibende Kraft. Mike hatte - soweit ich weiß - damals mit seiner Hilfe die Implementation für Friendica erstellt. Der Diaspora-Protokollteil in Friendica ist eigentlich recht übersichtlich, deswegen wundere ich mich, dass kein schneller Rewrite möglich zu sein scheint. Dann muss der Code ja anscheinend wirklich derbe im Kern verzahnt sein, dass man ihn nicht isoliert testen kann.

    0
  • Benjamin Neff
    Benjamin Neff
    2015-06-08

    @Matthias: Ohne Zeit wird schwierig :D aber ohne RoR-Kenntnisse kannst du ganz viel JS oder CSS dinge tun :)

    @Alexander: Weil du nach nem Beispiel gefragt hast: Als ich diesen Post geschrieben hab, wollte ich diesen Bug fixen: https://github.com/diaspora/diaspora/issues/4755 ... Bei der Problembeschreibung denkt man nicht an sowas wie Federation. Das klingt eher nach ner Kleinigkeit irgendwo ... Als ich dann aber versucht hab das zu Reproduzieren, hats erstmal immer funktioniert. Erst als ich die Federation (also nen 2. Testserver) ins Spiel brachte, konnte ich den Fehler reproduzieren ... und beim anschauen des Problems, sind mir dann direkt noch andere Probleme aufgefallen ... Rausgekommen ist dann folgender Pull-Request: https://github.com/diaspora/diaspora/pull/6082 der jetzt auch im 0.5.1.0 Release sein wird. Aber wie du sehen kannst, wars alles andere als ne Kleinigkeit ;)

    0
  • ravenbird@nerdpol.ch
    ravenbird@nerdpol.ch
    2015-06-09

    Also soweit ich das verstehe Benjamin ist das Protokoll an sich nicht das Problem, sondern die Spezialfälle und die Implementierung des Protokolls?

    0
  • Benjamin Neff
    Benjamin Neff
    2015-06-09

    japp, wenn alles so funktionieren würde, sie im Protokoll gedacht, wär schon einiges besser ;)

    0
  • ravenbird@nerdpol.ch
    ravenbird@nerdpol.ch
    2015-06-09

    Na denn Glück auf!

    0
  • mʕ•ﻌ•ʔm jesuiSatire .. ᘛ⁐̤ᕐᐷ
    mʕ•ﻌ•ʔm jesuiSatire .. ᘛ⁐̤ᕐᐷ
    2015-06-09

    "wenn alles so funktionieren würde, sie im Protokoll gedacht, wär schon einiges besser"

    Hört sich ja fast wie die Gesetzgebung an ..

    0
  • herzmeister der welten
    herzmeister der welten
    2015-06-09

    drum wurde das Wort "Protokoll" nicht von ungefähr in die IT importiert

    0