SOBotics / Guttenberg

A bot, searching for plagiarism on Stack Overflow.
http://stackapps.com/q/7197/43403
GNU General Public License v3.0
23 stars 10 forks source link

Problems with checkuser command #134

Open FelixSFD opened 6 years ago

FelixSFD commented 6 years ago

https://chat.stackoverflow.com/transcript/message/40779108#40779108

Quota was depleted after 7 requests (according to the logs)

jdd-software commented 6 years ago

Do you have the actual log, the message is when google error out, not really checking if quota is expired.

FelixSFD commented 6 years ago

There was no error in the logs. It just finished after 7 queries

 INFO [SoBoticsCommandsList:39] 2018-01-11 22:09:49,826 - Mention: @gut checkuser 6242024
 INFO [SoBoticsCommandsList:91] 2018-01-11 22:09:49,827 - Message: @gut checkuser 6242024
 INFO [CheckUser:74] 2018-01-11 22:09:49,833 - Executing command on user id: 6242024
 INFO [Room:264] 2018-01-11 22:09:50,041 - Task added - sending message 'Check user: 6242024 - START' to room 111347.
 INFO [CheckUser:94] 2018-01-11 22:09:50,369 - Query: Please try with java transformation! Find the difference between the start and end date, Create the new records using rowgen in java transformation, exact term:Please try with java transformation! Find the difference between the start and end date, Create the new records using rowgen in java transformation
 INFO [CheckUser:94] 2018-01-11 22:09:51,818 - Query: I suggest you to use Informatica cloud, there you will find tons of connectors, exact term:I suggest you to use Informatica cloud, there you will find tons of connectors
 INFO [CheckUser:94] 2018-01-11 22:09:52,737 - Query: The timestamp part in the given string is invalid, exact term: The timestamp part in the given string is invalid
 INFO [CheckUser:94] 2018-01-11 22:09:53,721 - Query: While importing file file , just select All_type in document type, exact term:While importing file file , just select All_type in document type
 INFO [CheckUser:94] 2018-01-11 22:09:54,383 - Query: In session properties, select target flat file then click set file propeties, exact term: In session properties, select target flat file then click set file propeties
 INFO [CheckUser:94] 2018-01-11 22:09:55,019 - Query: Instead of ls use ls whateveryouwant | xargs -n 1 basename If you know the path, then ls /path/*, exact term:ls
 INFO [CheckUser:94] 2018-01-11 22:09:56,457 - Query: CTRL+ALT+BREAK can work and I think this is duplicate question,Please research before asking question, exact term:CTRL+ALT+BREAK
 INFO [Room:264] 2018-01-11 22:09:57,055 - Task added - sending message '     # Score Post                                   On-Site                                           Off-site                                          
    -----------------------------------------------------------------
     1  0.62 https://stackoverflow.com/a/46341224   https://stackoverflow.com/a/46343933                                                                
     2  0.00 https://stackoverflow.com/a/47807800   https://stackoverflow.com/tags/informatica/new                                                      
     3  0.63 https://stackoverflow.com/a/47807849   https://stackoverflow.com/a/47839458                                                                
     4  0.00 https://stackoverflow.com/a/47807916   https://stackoverflow.com/tags/informatica/new                                                      
     5  0.00 https://stackoverflow.com/a/47807976   https://stackoverflow.com/tags/informatica/new                                                      
     6  0.67 https://stackoverflow.com/a/47830474   https://stackoverflow.com/a/217006                https://superuser.com/questions/193599/why-does-this-not-work-ls-txt-xargs-cat-all-txt-all-files-into-single
     7  0.00 https://stackoverflow.com/a/48206905                                                                                                       ' to room 111347.
 INFO [SoBoticsCommandsList:91] 2018-01-11 22:10:11,964 - Message: :-(
Bhargav-Rao commented 6 years ago

I am sure that it is something related to multiple instances running the command or some mixup like that. Check this one, where Gut replies twice https://chat.stackoverflow.com/transcript/111347?m=41038041#41038041

Check user: 3215402 - START Check user: 3215402 - START

as opposed to this one https://chat.stackoverflow.com/transcript/111347?m=39969534#39969534

Check user: 4056907 - START

Bhargav-Rao commented 6 years ago

For starters, https://github.com/SOBotics/Guttenberg/blob/develop/src/main/java/org/sobotics/guttenberg/commandlists/SoBoticsCommandsList.java#L62 should be inside the else, which follows after that.

This would prevent issues like this https://chat.stackoverflow.com/transcript/message/41172958#41172958

Bhargav-Rao commented 6 years ago

I guess, this must be fixed, right? The issue was with the multiple instances, IIRC.