-
Notifications
You must be signed in to change notification settings - Fork 9
[Dev] Funceble is too slow #95
Comments
@mitchellkrogza can you test the fix ? |
@funilrys 😢 it finished but in double the time it usually takes. It seems specifically .se .it and .com.br are slowing it down a lot. |
@mitchellkrogza saw it 👍 It's related to #94 I'm into it right now 😉 The only problem is for some obscure reason for those domains those line interpret the regex as single line https://github.com/funilrys/funceble/blob/dev/funceble#L1113-L1126 🤔 |
@funilrys let me know when you get your head around it. I'm online so will test immediately when you push a new fix. |
@mitchellkrogza let's test 👍 😸 |
Underway - https://travis-ci.org/mitchellkrogza/The-Big-List-of-Hacked-Malware-Web-Sites/builds/266243864 ..... lots of duplications occuring ??? |
Sorry but I need to laugh a second 😹 😹 😹 😹 😹 😹 😹 😹 😹 😹 😹 😹 😹 😹 😹 😹 😹 😹 😹 😹 😹 😹 😹 😹 😹 😹 😹 😹 😹 😹 😹 😹 😹 😹 😹 😹 😹 😹 😹 😹 😹 😹 😹 😹 😹 😹 😹 😹 😹 😹 😹 😹 😹 😹 😹 😹 😹 😹 😹 😹 😹 😹 😹 😹 😹 😹 😹 😹 😹 😹 😹 😹 😹 😹 😹 😹 😹 😹 😹 😹 😹 😹 😹 😹 😹 😹 😹 Let's fix it 👍 |
That was one hell of a good laugh 🤣 Cancelling build. |
Hell yeah 😹 |
Note: That issue occurs only if we call |
.md and .sk domains also giving that weird output |
With the last commit @mitchellkrogza ? |
Nope sorry, with the previous one .... my bad .... still running previous dev version on my ubuntu box and just saw those. Will wait for update. |
Why is that? |
A note for myself 😹 it's not happening when you use |
@mitchellkrogza Let's |
17514 that's the number of line I had to read to find where was the problem 😹 😹 😹 😹 😹 😹 😹 |
Okay, let me see the difference between https://travis-ci.org/mitchellkrogza/The-Big-List-of-Hacked-Malware-Web-Sites/builds/265988407 && https://travis-ci.org/mitchellkrogza/The-Big-List-of-Hacked-Malware-Web-Sites 😭 |
I feel your pain 😢 |
Note: [That] https://www.diffchecker.com/arpDbj4l don't explain everything 😭 |
Can't understand how the execution time has grown so much ??
|
The diffchecker tell us the difference between before (left) and now (right) so it's the refactoring that produces that issue 😭 |
Any way I can run this in full debug mode on by Ubuntu box? Maybe produce a debug output file to see what's causing the execution time to double? |
I already did it over a 100 domain list but the only way is to rewrite the way we read the whois output 😅 But if you want a huge and full debug .. |
Pulling latest dev branch now to my Ubuntu box and will run this for an hour and see. If I stop it will we still get the debug output needed? |
yes it's a real time input into executiondebug |
Running now but already I see the script pausing continuously with message |
And also like here:
|
What about |
Okay let's change the default timeout to 2 or 3 seconds |
Shall i wait for fix? And then run this again? Stopped it after just 5 minutes already 8mb debug file. |
stop ip :) My last commit only change the timeout from 30 to 3s |
Running now, no pausing yet. Let me run this on Hacked Malware Sites and see how it does on Travis |
Let's test this one 😭 This one there's no loop one line after another I only search the line that get the expiration date 😅 |
Okay, @mitchellkrogza let's resume my long last night and day :)
As you can see, we have a real problem which looks like it's out of the code ... |
To quote @mitchellkrogza from mitchellkrogza/The-Big-List-of-Hacked-Malware-Web-Sites@4b9dbd8:
The text was updated successfully, but these errors were encountered: