Ga terug   Computer Idee Forum > Software > Software Overige

Antwoord
 
Discussietools Weergave
Oud 5 September 2018, 19:19   #11
anoniem14733
Guest
 
Berichten: n/a
Standaard

Ohja, origineel heb ik in de api het script weer aan de praat gekregen (voordat ik begon met posten) door:
Code:
$array_headers = split("\r\n",$this->result_header);
te veranderen naar:
Code:
$array_headers = explode("\r\n",$this->result_header);
Toch staat er nu nog 1 split statement in, moet dat ook naar explode of moet dat gewoon blijven staan, het gaat om deze:
Code:
return split("\n",$this->fetch_body());
Vooralsnog zie ik er geen foutmelding voor, de eerdere die verandert is naar explode gaf een php fatal error.
  Met citaat antwoorden
Oud 5 September 2018, 19:36   #12
anoniem14945
Guest
 
Berichten: n/a
Standaard

EDIT: dit is een reactie op #9 /EDIT

'k zou zelf toch voor die aanpassing in php.ini gaan
Er is op zich niks mis met het niet initializeren van variabelen binnen PHP. PHP is zo ontworpen om dat zelf te doen als de programmeur het niet doet. Daarom levert het ook slechts een notice op en geen error of warning. Er is dus ook niks mis om deze notices in productie uit te schakelen met
Code:
error_reporting=E_ALL & ~E_NOTICE & ~E_STRICT & ~E_DEPRECATED

Citaat:
$_SESSION['cod'] = substr(md5(rand(000, 999) * md5(rand(000, 999))), 1, 5);
Wat hier gebeurt (ik heb het even bold gemaakt) is dat twee waarden worden vermenigvuldigd. De eerste is een willekeurig getal tussen 0 en 999. De tweede de MD5-hash van een ander willekeurig getal tussen 0 en 999. Nu is een MD5-hash geen getal, maar een string met letters en cijfers. En dat vermenigvuldigt nogal lastig en dat probeert PHP je te vertellen.

Het gaat hier om een willekeurige code die gegenereerd wordt, waarbij het er niet echt toe lijkt te doen hoe deze gegenereerd wordt. Ik zou er dit van maken:
PHP-code:
$_SESSION['cod'] = substr(md5(rand(000999) * rand(000999)), 15); 
Dus eerst de twee getallen vermenigvuldigen en dan pas hashen.
Ik zou ook die 1 veranderen in een ander getal tussen 0 en 26, nu dat iedereen weet dat het een 1 is. Je moet de uitkomst van deze functie toch zien als een soort van password. De willekeurigheid daarvan is op zich al twijfelachtig, maar helemaal als men ook nog eens precies weet welk deel van de MD5-hash gekozen wordt. Door die 1 te wijzigen is het in ieder geval nog een factor 27 willekeuriger.
  Met citaat antwoorden
Oud 5 September 2018, 19:41   #13
anoniem14945
Guest
 
Berichten: n/a
Standaard

Citaat:
Oorspronkelijk geplaatst door BlackTiger Bekijk bericht
PHP Fatal error: Uncaught Error: Call to undefined function emtpy()
typefoutje

Citaat:
Toch staat er nu nog 1 split statement in, moet dat ook naar explode of moet dat gewoon blijven staan, het gaat om deze:
Ja, die ook graag even wijzigen. split() bestaat niet meer en explode() doet in dit geval hetzelfde.
  Met citaat antwoorden
Oud 5 September 2018, 21:44   #14
anoniem14733
Guest
 
Berichten: n/a
Standaard

Citaat:
typefoutje
Ah, dat maakt me nieuwsgierig naar de juiste code.

Citaat:
De eerste is een willekeurig getal tussen 0 en 999. De tweede de MD5-hash
Klopt, er wordt hiermee een random md5 wachtwoord gegenereerd als het goed is.
Gebeurt dat met jouw aangepaste code ook nog steeds? Want dat is wel belangrijk.

Als dat het geval is ga ik die gelijk proberen. De split wijzig ik dan sowieso nog even.
  Met citaat antwoorden
Oud 6 September 2018, 17:32   #15
anoniem14945
Guest
 
Berichten: n/a
Standaard

Citaat:
Oorspronkelijk geplaatst door BlackTiger Bekijk bericht
Ah, dat maakt me nieuwsgierig naar de juiste code.
Wijzig emtpy in empty. p en t omgedraaid

Citaat:
Oorspronkelijk geplaatst door BlackTiger Bekijk bericht
Klopt, er wordt hiermee een random md5 wachtwoord gegenereerd als het goed is.
Gebeurt dat met jouw aangepaste code ook nog steeds? Want dat is wel belangrijk.
Ja, de uitkomst is nog steeds MD5.
  Met citaat antwoorden
Oud 6 September 2018, 17:59   #16
anoniem14733
Guest
 
Berichten: n/a
Standaard

Oh dju... typo... en ik zag 't niet, door de bomen het bos niet meer gezien ondertussen. LoL.
Heel hartelijk dank (Obelix71 ook trouwens) voor het meedenken.

Inmiddels zijn er 0 notices. V.w.b. die location heb ik niets veranderd, misschien gaat dat nu goed omdat er geklikt wordt vanaf een andere pagina of iets anders, geen idee.

Dan de wijziging van die MD5 heeft tot gevolg dat ook die notice weg is.

En de wijziging van die $Betaling dingens zoals jij aangaf liet ook die notice verdwijnen.

Oftewel, 2 wijzigingen en alles werkt weer perfectos.

Hartstikke bedankt!!
  Met citaat antwoorden
Oud 6 September 2018, 21:53   #17
anoniem14945
Guest
 
Berichten: n/a
Standaard

Fijn om te lezen dat het gelukt is om de notices weg te werken én dat het ook nog werkt! Graag gedaan
  Met citaat antwoorden
Antwoord

Discussietools
Weergave

Regels voor berichten
You may not post new threads
You may not post replies
You may not post attachments
You may not edit your posts

BB code is Aan
Smileys zijn Aan
[IMG]-code is Aan
HTML-code is Uit

Forumnavigatie


Alle tijden zijn GMT +1. Het is nu 04:04.


Powered by vBulletin Version 3.8.6
Copyright ©2000 - 2019, Jelsoft Enterprises Ltd.