Charcoal HQ

@NobodyNada except that the tooltip doesn't explain what he actually texted.

And I refuse to set my keyboard to Dvorak just to figure it out.

@Glorfindel I did; it says "okay google send a text"

K, thx

[ SmokeDetector | MS ] Blacklisted username: How to execute a while loop for 5 minutes c# by Mother Fucker on stackoverflow.com (@dorukayhan)

5:15 PM

questions's crap, but not spam

fp- by Glorfindel

@NobodyNada flagged for mod attention (to change the username)

The question ended with "P.S. How to change username?" but someone edited it out

yay for power ^.^

5:33 PM

Literature has entered private beta.

cool

@ArtOfCode @Undo preliminary diff/pull-req for the two different Flake8 tests here: github.com/Charcoal-SE/SmokeDetector/pull/467

marked as WIP because I want to make sure things don't fail first before it's ready for review

... downside: we have to tell Flake which config file to use. Upside: we have two separate Flake8 runs with two separate configuration files to dictate what we ignore and what directory is explicitly being tested.

I have a grand total of zero idea about how the whole flake8/CI process works, so I can't help on this

OK

looks sane to me

Magisch

@ArtOfCode So would you say your knowledge of it is .... flaky?

5:48 PM

@Undo and it looks like it ran with those CI changes without issue on both Travis and Circle, so yeah

... aaactually, @ThomasWard why not keep tox.ini, then special-case for tox_tests.ini?

@Undo We could. Force of habit for me is have multiple

Because it's kinda nice to be able to run flake8 in the console locally

but ok one second

@Undo symlink of tox.ini -> tox_code.ini could also work, but i'll rename

Probably better to rename here.

5:50 PM

@Magisch sigh

@Undo yep. pushed, give it a minute to pass/work

@Undo I hope you don't mind that I put ./test/ checks above code checks in the CI process

if the test code fails we should probably not be going on anyways :P

Doesn't matter

cool

i didn't realize initially we had two separate CI configurations o.O

one for Travis, one for Circle

then I pulled my head out of my butt :P

@ThomasWard Yeah, that's a historical thing. We race them because usually one's faster than the other.

i'm happy having both though :)

so we're all good

(sometimes Travis dies for no reason)

LOL the travis build hasn't even started yet xD

@Undo if it looks sane, then OK the merge and execute it. Looks good to me either way though :)

5:57 PM

There you go

I'm just glad everything is working as it should heh

with all these revisions I'm surprised something hasn't fireballed into instant death xD

[ SmokeDetector ] SmokeDetector started at rev 5510b55 (Undo1: Merge pull request #467 from Charcoal-SE/flake8-ci-tests --autopull) (running on Undo's EC2)

Restart: API quota is 3879.

6:09 PM

[ SmokeDetector | MS ] Link at end of answer: _beginthreadex() returns valid handle and crashes when executing thread using Codeblocks/Mingw with GNU GCC Compiler by XXXBold on stackoverflow.com (@dorukayhan)

[ SmokeDetector | MS ] Bad keyword in body, bad keyword in title, repeated URL at end of long post: vitanoria moet kunnen worden by user84396 on graphicdesign.stackexchange.com

@SmokeDetector k

!!/blacklist-website skinscam\.com

@Glorfindel Blacklisted skinscam\.com

i'm just glad all the blacklist stuff is working again

Just the name of that website ...

that was an evil refactor, trying to get it to work with a single pattern heh

@Glorfindel burn it with fire.

Jan Dvorak

6:18 PM

What does the text say?

It's Dutch, give me a minute to translate.

New record for autoflags: 90 posts flagged.

@ArtOfCode nice

Actually, it is Dutch but I have no f@@@ing clue what they're trying to sell.

[ SmokeDetector | MS ] Bad keyword in body, bad keyword in title, repeated URL at end of long post: Vitanoria und Profollix für schönes Haar und glattere Haut? by user220602 on apple.stackexchange.com

6:20 PM

The most interesting about that spam is that it mixes some German words into a Dutch text.

@Glorfindel -756 spam points

@SmokeDetector k

@Undo do we need the two automatic branches I'm seeing on GitHub?

no

can I burn them?

6:27 PM

aye

After they get merged or declined, there's no use for them to stay around

done

[ SmokeDetector | MS ] Pattern-matching website in body: review my logo&website - need feedback by Katherine J. Page on stackoverflow.com (@dorukayhan)

[ SmokeDetector | MS ] Bad keyword in body, link at end of body: Ondanks herstel van de laatste by user84396 on askubuntu.com (@ThomasWard)

@SmokeDetector k

Why is my blacklist entry not working yet?

@SmokeDetector k

!!/rev

@ThomasWard 5510b55 (Undo1: Merge pull request #467 from Charcoal-SE/flake8-ci-tests --autopull)

6:30 PM

!!/pull

[ SmokeDetector ] SmokeDetector started at rev da3419c (SmokeDetector: Auto blacklist of skinscam\.com by Glorfindel --autopull) (running on Undo's EC2)

CI broken. Might be due to line length refactor.

because somehow it missed autopull

@Undo eesh, that's not good

Restart: API quota is 3669.

@ThomasWard nah, it's fine. I wrote it, then broke it, so I can probably fix it

6:31 PM

indeed.

[ SmokeDetector | MS ] Bad keyword with email in answer: How to Trace Ancestors to Switzerland by Michelle Carpenter Homan on genealogy.stackexchange.com

f?

naa- by teward

I'd say naa

because it's not answering the question being asked, really

it doesn't answer "how" to track

but meh

(the email alone is bad enough)

@SmokeDetector why

@ThomasWard Post - Keyword wife with email michellehoman@yahoo.com

6:34 PM

lol

naa or fp

but the email capture is a valid detection of the email in the answer :P

hi folks

@SmokeDetector That message is not a report.

lol

@Undo need me to make a github issue about autopull not working, or no?

I've got it

cool

6:37 PM

[ SmokeDetector ] SmokeDetector started at rev 46a9d4c (Undo1: Actually send autopull message --autopull) (running on Undo's EC2)

Restart: API quota is 3627.

Well it should work

!!/blacklist-website this\.is\.a\.test\.website\.com

!!/alive?

chat con

[ SmokeDetector ] SmokeDetector started at rev 46a9d4c (Undo1: Actually send autopull message --autopull) (running on Undo's EC2)

Restart: API quota is 3616.

!!/blacklist-website this\.is\.a\.test\.website\.com

@Undo Blacklisted this\.is\.a\.test\.website\.com

CI on 5f2a245 succeeded. Message contains 'autopull', pulling...

[ SmokeDetector ] SmokeDetector started at rev 5f2a245 (SmokeDetector: Auto blacklist of this\.is\.a\.test\.website\.com by Undo --autopull) (running on Undo's EC2)

Restart: API quota is 3612.

for future reference, this is the websocket payload:

{"commit_status"=>{"status"=>"success", "ci_url"=>"https://travis-ci.org/Charcoal-SE/SmokeDetector/builds/193142666", "commit_sha"=>"5f2a2455bc88f498bee5106f14a0b60fc60c9ca9", "commit_message"=>"Auto blacklist of this\\.is\\.a\\.test\\.website\\.com by Undo --autopull"}} (via streamed from smokedetector_messages)

6:42 PM

CI on 39e6458 succeeded. Message contains 'autopull', pulling...

[ SmokeDetector ] SmokeDetector started at rev 39e6458 (Undo1: Revert "Auto blacklist of this\.is\.a\.test\.website\.com by Undo --autopull") (running on Undo's EC2)

Restart: API quota is 3603.

So the prognosis is "I don't think it's broken"

interesting

@Undo could it be a glitch in CI, maybe a brief network interrupt so it missed the payload?

unlikely, given two servers in the same Amazon AZ

hmm

well, it's working, but didn't in that one case shrugs

geez, keybase takes an eternity to update

!!/alive?

6:51 PM

@Undo Of course

!!/coffee

@ThomasWard brews coffee for @ThomasWard

dumps the coffee all over @SmokeDetector

oops my bad

7:15 PM

CI on e5954a4 succeeded.

CI on 84cfe26 succeeded.

!!/pull

[ SmokeDetector ] SmokeDetector started at rev 84cfe26 (Thomas Ward: IDE Suppressons in test code.) (running on Undo's EC2)

@Undo I ran project wide inspects via PyCharm, and it threw some annoying things, so I've gone and poked some of them.

Restart: API quota is 3353.

Great

You don't need to ping me on these kinds of things, by the way :)

7:18 PM

the rest is just me putting in IDE warning suppressions in the test code so it stops bitching

@Undo yes, I know, but I expect some things to break ^ namely tests even if it's suppressions

given enough time i'll have a billion inspection suppressions in here XD

8:02 PM

@Undo do you mind if I put the fact I'm listed as a developer in the Charcoal/SmokeDetector project on my resume? Trying to keep it updated with all the skills and such I have and contributions I've made :P

Sure

great, I'll add it now. (I have a lot of my volunteer / nonpaid activity there heh)

done. (I also added this to LinkedIn because I try and keep that and my resume synced up)

Mithrandir

8:30 PM

@Undo On that list---> it says that you people get pinged a lot ;)

This would be true

Even truer now. Self-fulfilling prophecy, apparently.

@Undo what do you mean?

:)

9:00 PM

[ SmokeDetector | MS ] Blacklisted user: Tkinter No attribute "Destroy" by CatsInSpace on stackoverflow.com (@dorukayhan)

fp- by Glorfindel

sd remove - why

1. [:34864523] <processed without return value>
2. [:34864518] <skipped>
3. [:34864513]
Blacklisted user - blacklisted for http://stackoverflow.com/a/41574604 (https://m.erwaysoftware.com/posts/by-url?url=//stackoverflow.com/a/41574604) by http:metasmoke

@ThomasWard Suppose you could host Smokey this weekend? I'll be out on a camping thing.

have fun!

[ SmokeDetector | MS ] Bad keyword in body, bad keyword in title: Does this patent not encompass the sale and use of the vast majority of cognition supplements on the market? by Ilde on patents.stackexchange.com

fp- by NobodyNada

9:03 PM

Nevermind, that's premature

@SmokeDetector autoflagged

@Glorfindel That post was not automatically flagged by metasmoke.

That surprises me. It has a weight of 190, which is above the threshold some people (including me) have set.

What's your reason count?

1

9:07 PM

@Undo I sure can if needed

whether premature or not :)

The post is also not mentioned in the unflagged list.

Is that because it isn't new, but posted July last year?

The flag logs say it attempted to flag but failed

do you have an account on the patent site?

Yes, I have.

BTW, what happened to the flag logs link?

@NobodyNada @Undo ate it in his sleep.

:P

9:09 PM

sent @Undo

oh whoops

oh well, you have a public key for me now

@Undo when do you want me to take it over, Friday?

or now?

@SmokeDetector because it had more than one revision

ProTip: hover over "fail" in the flag logs to see the error message

@ArtOfCode OK, I didn't know about that 'rule'.

@Glorfindel Protects us from spam-flagging vandalism

@ThomasWard I'm experimenting with getting my Pi running, leaving in the afternoon Friday US time.

Then would be fine

9:13 PM

Makes sense, but why does Smokey attempt to flag it in the first place?

@Undo OK. I'll be happy to take it over as long as you'd like, hell if you and Art toss me SSH pub keys I'll even grant you sudo access on Aurora

so you can mess with Smokey as needed.

@Glorfindel Because three are matching conditions.

@ThomasWard heh, that's more trusting than I am

Although I guess it helps to have your security ducks in a row anyway

@Undo well, I do have the big bad "Nuke it" button

and the knowledge to restrict sudo down to a specific set of commands :p

@Undo I'm gonna bet that system is firewalled, LAN-isolated, and not running anything else important ;)

@ArtOfCode Aurora?

9:14 PM

aye

offsite VPS, firewalled, not running anything else important, and DDoS protected IPs

ESXi2 is LAN-isolated, on my network here at my apartment though

LAN-isolated, offsite VPS, much the same effect

but that's the backup instance :P

@ArtOfCode indeed.

@ArtOfCode also helps to have that huge "BURN IT" button called "Reinstall VPS"

:P

@Undo 'more than one revision' seems like something Smokey can detect up-front, instead of attempting to flag.

Doesn't look like a Failure to me.

@Glorfindel Not really. It'd be another API call

Easiest thing would be to just change metasmoke to not generate a failing flag log on that.

9:17 PM

could use "Ignored" instead

(as a suggestion for such an alternative)

@Undo fine with me.

Oh, by the way: this situation reminds me that I do want to increase my threshold on Ask Patents. However, I don't see the Sites anymore, so I can neither change them nor create a new condition.

I'm gonna try MS#70... this may go terribly wrong.

@Glorfindel refresh the page, known bug. I think @angus is working on it.

@ArtOfCode thx

I'd love it if he (or somebody else) can make a meaningful sort order of the dropdown.

@Glorfindel What would you propose?

@Glorfindel currently ordered by site creation date

9:20 PM

(not sure what it is now)

I mean, it takes longer to find Ask Patents in that list of sites than it takes to find a good PHP question on Stack Overflow.

4

I'd rather like it to be alphabetical, with SOFU at the top

@Undo seconded.

@ArtOfCode Good luck, but if it's horribly slow I'll probably shoot it down. Just a fair warning ;)

@Undo Oh, I would too. I'd take any improvement over the current slowness of MySQL REGEXP, though.

That said, on second thoughts I'm gonna leave this for now... I'm not sure coding a complex thing while exhausted is a good idea.

9:27 PM

no it's not, @ArtOfCode

also, Internet is working wonderfully today

@ArtOfCode *five minutes later...*

@ArtOfCode What about your email delay? :P

@Undo still negligible, thanks to my now-several-months-old server move ;)

what'd you want the pub key for anyway, @Undo?

@ArtOfCode Considering giving someone other than me SSH access to metasmoke's server. It's probably getting a dedicated server sometime in the near future anyway, and I'm leaving for a while and autoflagging is a thing now.

Main issue is your protection; having access to those tokens could open one up to nastiness. Current userbase is probably trusting.

Also, access to PII. Limited to emails, but still

9:32 PM

@Undo even if I wasn't biased I'd tend to agree. Might wanna do a User.all.pluck(:email) and send a BCC notification around about it, though, even if they are trusting.

[ SmokeDetector | MS ] Pattern-matching website in answer: What is first electronic rights? by Ken Forslund on writers.stackexchange.com

and even if it's not an RFC just a "this is happening"; people should know who has access to their PII.

fp- by NobodyNada

@ArtOfCode Brings up another question: Do we need to store emails?

Ideally, it'd be nice to have this all backed by SE's OAuth, then I don't have to deal with any login stuff.

@Undo go with yes. They're a useful pseudo-primary key, and it means we have a method of contacting people if we really need to

9:34 PM

@ArtOfCode That's true

as in, "your API write token may have leaked, sorry", which you wouldn't really want to tell someone publicly but they do need to know

any reason we have for using those emails is edge-casey but important

connection now peaking at 82Mbps down

throws packets at @ArtOfCode

@ThomasWard ooh, another delivery! Wonder what's in this one.

probably old socks.

Life Pro Tip #22837a: Order useful-but-inconsequential items from people who take a while to deliver. Then, when it arrives two weeks later, you get a nice surprise and a useful new thing.

9:38 PM

aka "buy stuff on Amazon and live in the UK"

@Undo Nah, Amazon operate in the UK too ya know :P

True

that's about the quickest way to get things delivered

'specially with six free months of prime because students get everything free

yes, we the students are spoiled :p

which reminds me, I should still do that coursework

reads sheet - knows this already - considers coursework done

Anonymous

9:46 PM

@ArtOfCode Story of my college years :P

I mean, inheritance, really? Can I start year 2 now, please?

Anonymous

The best part for me was learning Agile in my software engineering course, a year after learning it on-the-job

Magisch

@ArtOfCode You gotta remember

the average college student doesn't program in their free time and doesnt develop complicated heuristics and designs guis

Yeah, and they're who this course is designed for. Which makes sense, and is fine, but can I skip ahead?

Hmm. My flags used today just went down from 13 to 12 ...

9:52 PM

@DavidPostill Time has this funny habit of marching on, yes

"today" == "24 hours preceding this precise moment"

@ArtOfCode Fair enough :)

"today" != "2017-01-18T00:00:00.000000Z..2017-01-18T23:59:59.999999Z"

I was expecting UTC days ...

@ArtOfCode Maybe it should be

@Undo know how to implement that?

9:53 PM

@ArtOfCode It is here :)

@ArtOfCode where('DATE(created_at) = ?', Date.today) seems to work okay

@Undo Fair enough

It'd be a lot more complicated if the server weren't already on UTC

@Undo if you're considering giving me MS server access, probably a smart move to move everything else off it first. Which you've probably thought of already, but doesn't harm saying again.

@Undo do Smokey and MS both run on the same server right now?

10:08 PM

@ThomasWard No

Try to keep those separate.

cool

Testing rpi, hold on...

throws a wrench into @Undo's RPi

i should repurpose my RPi heh

[ SmokeDetector ] SmokeDetector started in reverted mode at rev 84cfe26 (Thomas Ward: IDE Suppressons in test code.) (running on Undo's Raspberry Pi)

what died

10:17 PM

!!/master

um that is master

!!/gitstatus

[ SmokeDetector ] SmokeDetector started at rev 84cfe26 (Thomas Ward: IDE Suppressons in test code.) (running on Undo's Raspberry Pi)

@ThomasWard yeah, but it was on the actual master branch, not deploy, triggering the reverted mode warning

Restart: API quota is 9996.

@Undo ah, right.

@Undo I could write that into the startup process, to make sure we always git checkout deploy at runtime

10:18 PM

No, there are valid times when you want to run it at a certain commit

mmkay

Manual rollbacks, for one, if things get really bad

true, though we're usually fast enough to revert our own broken commits heh

or someone else is able to fix them :P

one reason development on some things goes faster :)

!!/blacklist-website this\.is\.a\.test\.to\.ignore

git error, hold on

whoopsies :P

@Undo mind sharing what the error was?

so I can add a capture on it, if one doesn't exist.

(i.e. the actual git module error that was thrown)

10:21 PM

@ThomasWard Not capturable, I just don't have keys set up right

ah, okay.

It looks like it would have recovered, if I hadn't killed it

cool

[ SmokeDetector ] SmokeDetector started in reverted mode at rev 84cfe26 (Thomas Ward: IDE Suppressons in test code.) (running on Undo's Raspberry Pi)

!!/master

10:26 PM

[ SmokeDetector ] SmokeDetector started at rev 84cfe26 (Thomas Ward: IDE Suppressons in test code.) (running on Undo's Raspberry Pi)

Restart: API quota is 9988.

!!/blacklist-website just\.a\.test

@Undo You don't have code privileges, but I've created a pull request for you.

o.O

wat

@Undo did you revoke your own privs briefly?

Disabled for testing

there's a checkbox in metasmoke

ooh cool

10:29 PM

So it's quite a bit slower, but still operational.

!!/test whatever

> Would not be caught for title, body, and username.

indeed

I'd probably chalk that up to network issues and maybe another thing, not sure what

(the 'issue' being "it's not AWS")

or Aurora, with its nice gig uplink

Metasmoke going offline temporarily

10:33 PM

just kidding :P

[ SmokeDetector ] SmokeDetector started at rev 84cfe26 (Thomas Ward: IDE Suppressons in test code.) (running on Undo's EC2)

^ also, keyboards are hard

?

whoa, new EC2 EIP /8 block. Just got a 34.0.0.0/8

o.O

10:44 PM

@Undo huh, I thought Amazon IPs were 54.0.0.0/8 mostly

54 and 52 are the ones I've seen

guess when we switched to the pi

~~only slightly faster than breaking even, actually~~. Although that's misleading because threading

@Undo Hm? Surely breaking even would be 1.0633 posts per second.

@SmokeDetector f

@ArtOfCode yeah, I can't math

638 posts in 600 seconds...

10:48 PM

still, a good order of magnitude slower than EC2

@Undo any chance we could benchmark the EC2 vs. Aurora?

@ThomasWard Sure, we'll get that anyway when we switch over on Friday

cool

i've just updated Aurora's code base to latest, it's been sitting for a few weeks without updates.

unless you've got some CPU benchmarks to run, in which case I have a fresh t2.nano right here

Not sure if you're already aware of this (you probably are), but FYI, I marked the last report as a false positive with FDSC and Smokey didn't notice.

10:49 PM

@NobodyNada yeah, I did too. Websockets.

My fault, I restarted metasmoke and it dropped off

!!/reboot

to fix that

ah. Do you guys send a heartbeat or something?

[ SmokeDetector ] SmokeDetector started at rev 84cfe26 (Thomas Ward: IDE Suppressons in test code.) (running on Undo's Raspberry Pi)

Restart: API quota is 9846.