-
Notifications
You must be signed in to change notification settings - Fork 9
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
main.js was obfuscated with regular commit #42
Comments
@stoffel7 Can you revert this please? |
Why? |
Is the source available somewhere in the repository as an open-source file? If not, please add the source so we can look at it, fix bugs or use the code to understand this adapter better. What was the reason to obfuscate the file in the first place? |
Hallo, um was geht es, die Sourcen stammen von Sebastian Schulz und mir |
Ja, aber es ist doch ein Open-Source Projekt, oder nicht? Wenn ihr gegen einen Baum fahrt, dann wäre es doch toll, dass jeder das geschriebene weiterverwenden kann. Ich denke, ich muss hier nicht die Vorteile von Open-Source nennen auf GitHub. Aber den Code zu "verschleiern" ist entgegen der Open-Source Idee. Vielleicht war es ja ein Versehen? Deswegen habe ich nach dem Grund gefragt. Anders gefragt: Hat es einen Grund, dass niemand mehr den Code lesen und weiterentwickeln kann oder soll? |
Der Ursprungsadapter für das USB Gateway stammt von Sebastian, ich habe in der main.js den Anteil für den eNet-Server beigetragen, mußte aber ein GHV einhalten, daher die Verschlüsselung, aber ich denke das ist ( aus versch. Gründen) nicht mehr notwendig. Kannst du den Adapter updaten, wenn du die aktuellen Sourcen hättest? Oder es muss mir jemdan helfen beim update. |
Den letzten Satz verstehe ich nicht so ganz. Die Funktionsweise ist durch die Verschleierung nicht eingeschränkt. Nutzen kann ich den Adapter. Aber es gibt mit dem aktuellen Stand keine Möglichkeit der Weiterentwicklung. Wenn es nicht mehr nötig ist, dann schieb doch bitte die unverschlüsselten Sourcen hoch. :) |
Wie oben schon geschrieben, der Adapter stammt von Sebastian, ich hab ihm nur die main.js erweitert, |
Danke. Open-source ohne Open-Source ist irgendwie... Sinnbefreit und am Ende hilft das keinem. |
Hi. Wenn es nur darum geht, zu verstehen, wie man den eNet-Server anspricht, könnt ihr euch das auch in meinem Homekit-Adapter abschauen. Ist allerdings in Python geschrieben: https://github.com/sweh/enet_homekit |
Nein, es geht darum, dass ein Teil eines Open-Source Adapters nicht Open-Source ist. Wenn man dies hier weiterentwickeln möchte, dann sollte der Code nicht obfuscated sein. Niemand kann das hier debuggen oder selbst auf der Basis einen Fork erstellen. Es wurde sich auch für die Open-Source MIT Lizenz entschieden: https://github.com/SebastianSchultz/ioBroker.enet/blob/master/LICENSE Dieselbe Lizenz, wie der quelloffene https://github.com/ioBroker/ioBroker auch, der ebenfalls ein Open-Source Projekt ist. Ich verstehe auch nach wie vor die Beweggründe nicht, den Code unleserlich zu gestalten. Es hat doch nur Nachteile. Was ist, wenn das Backup zu Hause mal weg ist oder die Person, die das hier geschrieben hat, nicht mehr unter uns weilt? Dann ist das Projekt nicht mehr benutzbar. Das ist schade. Und darum geht es hier in diesem Issue. Ihr könnt das hier auch schließen und sagen: Nein, wir wollen nicht, dass ihr den Code debuggen oder weiterentwickeln könnt. |
As I opened the ticket, I also wanted to comment on this again: What bothered me, was that the obfuscation was done as part of a regular commit merged with multiple other features and bug fixes. The commit message said nothing about an obfuscation. Further development in the current state is not possible anymore. |
Ich habe jetzt meinen Anteil am Sourcecode im Original hochgeladen, wie schon mehrmals erwähnt, der Adapter stammt von Sebastian. Die Beweggründe für "Verschlüsselung" habe ich oben schon erwähnt. |
Hi,
the main.js was was obfuscated and rendered unusable and non-modifiable with cb3a617
Is there a specific reason why this happened? If not, please upload a unobfuscated version...
Thanks,
Marius
The text was updated successfully, but these errors were encountered: