Charcoal HQ

thesecretmaster

12:00 AM

Deployed? Things seem fixed.

NobodyNada

thanks!

Undo

Yep

SmokeDetector

tpu- by Makyen on Domain specific language naming conventions [MS]

thesecretmaster

So, if anyone wants to try out graphql, I can answer questions/help with queries/change stuff. The routes are /api/graphql and /api/graphiql, both with nice little interfaces.

Undo

Shiny

Makyen

12:04 AM

@SmokeDetector Solicitations for participation in off-site surveys are considered spam. See: Are research survey questions expected to be handled with close votes?

NobodyNada

@Makyen My bad -- I read that too quickly

Undo

Yeah, this is awesome. Nicely done.

NobodyNada

I thought the link was to the DSL in question or something, rather than the survey

ArtOfCode

{
  post(id: 100000) {
    id
    body
    feedbacks {
      id
      user { username }
      feedback_type
    }
  }
}

SmokeDetector

tp- by NobodyNada on Domain specific language naming conventions [MS]

ArtOfCode

12:05 AM

Already half the information our userscripts need, and it's fast

Makyen

@NobodyNada np. We all do that from time to time.

ArtOfCode

@thesecretmaster Two things I can see we're going to need (I know you're still working on it, just making sure they've crossed your mind): autoflagging information (a flag_logs field on PostType), and the ability to fetch multiple objects at once (i.e. I want to get posts 120283 and 120282)

oh, and the ability to get posts by URL or UID

Undo

ah, can't pass link as a filter

Such nice syntax

SmokeDetector

[ SmokeDetector | MS ] URL-only title, blacklisted website in body, blacklisted website in title, link at end of body: mumybear.com/keto/ by kato on askubuntu.com

thesecretmaster

I'm working on some more stuff. Pagination is next, but they've got an oven (active) issue about that.

SmokeDetector

12:08 AM

tpu- by NobodyNada

ArtOfCode

@thesecretmaster do they have a stove issue as well?

thesecretmaster

I have a plan to get a good bit more done tonight.

Makyen

@thesecretmaster If it's a pizza oven, then we're good.

thesecretmaster

@ArtOfCode Yeah, about to request that they pull the gas out and stick it back in.

I hope it's a gas stove, not electric...

Undo

Gotta figure out how to architect a decent deploy solution

ArtOfCode

12:12 AM

@Undo how do you do it at the moment?

Undo

Locally, from laptop

ArtOfCode

@Undo aye, but what's the process?

Undo

Needs to hide secrets and be easily access-controllable

@ArtOfCode cap production deploy

That's it.

ArtOfCode

okay, and what does that need to work

thesecretmaster

@ArtOfCode { post1: post(id: 120283) { id title body } post2: post(id: 120292) { id title body } }

Undo

12:14 AM

@ArtOfCode The repo, the bundle installed, and credentials to hit the server

Hey, SmokeDetector hit 200 stars

ArtOfCode

@thesecretmaster possible to have a posts(ids: [1, 2, 3])?

thesecretmaster

Yep

Gimme some time

ArtOfCode

@Undo and finally, if I had a set of credentials would they let me do anything I shouldn't

Undo

ehhhhh, probably

ArtOfCode

likewise for posts(urls: []) and maybe posts(uids: [['stackoverflow', 12345678'], ['mathoverflow.net', 1234]])

Undo

12:16 AM

Actually, less that and more "If it's a CI job, everything's nice and auditable and I can't accuse you of doing nasty stuff"

ArtOfCode

Being accused of things isn't something I'm too worried about ;)

Undo

Generally good to have stuff out in the open.

J F

GraphiQL is still not working for me.

Undo

Idea is that there should be no way to get API tokens out of there without changing code in Git and running the deploy job. That's a pretty good defense against any meta unrest, too

ArtOfCode

@JF "not working"?

@Undo agree

I assume it's possible to create a CI job that specified people have access to

Undo

12:20 AM

Yep. Right now it's a non-team repo under me, so I can just add collaborators.

ArtOfCode

that works

Undo

Looking at travis-ci.org/Undo1/metasmoke-deploy, which does nothing but it has a name

J F

@ArtOfCode Rails stack trace gets retuned from the API route

Undo

permissions?

ArtOfCode

what trace?

thesecretmaster

12:23 AM

@ArtOfCode UID = SE ID or MS ID?

@Undo It's public

ArtOfCode

@thesecretmaster Combination of SE API site parameter and SE post ID (native_id on the Post model)

J F

{
  "message": "The server responded with invalid JSON, this is probably a server-side error",
  "response": "<!DOCTYPE html>\n<html lang=\"en\">\n<head>\n  <meta charset=\"utf-8\" />\n  <title>Action Controller: Exception caught</title>\n  <style>\n    body {\n      background-color: #FAFAFA;\n      color: #333;\n      margin: 0px;\n    }\n\n    body, p, ol, ul, td {\n      font-family: helvetica, verdana, arial, sans-serif;\n      font-size:   13px;\n      line-height: 18px;\n    }\n\n    pre {\n      font-size: 11px;\n      white-space: pre-wrap;\n    }\n\n    pre.box {\n      border: 1p…

(it’s quite long, @ArtOfCode ^)

thesecretmaster

Reload the page?

That means you POSTed to the wrong route, it was a bug before the most recent deploy

ArtOfCode

uh

Undo

Maybe a hard reload?

ArtOfCode

12:25 AM

no it doesn't

J F

@thesecretmaster comes from running that query

ArtOfCode

you're trying to get post 120283

which doesn't exist

J F

another query works fine.

ArtOfCode

that's what the ActiveRecord::RecordNotFound is

and "Couldn't find Post with 'id'=120283"

J F

ah, makes sense.

thesecretmaster

12:26 AM

^^ That. I should stick some handling around that

J F

GitHub’s API returns { data: { [theKey]: null }, errors: [{ message, type, path: [theKey], locations }] }

quartata

@Undo you still there?

ArtOfCode

attempting to run rake without having rake defined... probably not gonna work ;)

Undo

@quartata aye, for about thirty seconds

@ArtOfCode Travis apparently defaults to Ruby

quartata

actually, I probably don't need you

unless things break

Undo

12:34 AM

Gonna try to see if I can build the job up manually, since it needs to clone another repo and do everything from there

quartata

@ArtOfCode take away my code admin really quick?

afk

gone

!!/reboot

Goodbye, cruel world

[ SmokeDetector ] SmokeDetector started at rev 555828a (SmokeDetector: Auto unwatch of Guarani by doppelgreener --autopull) (running on ArtOfCode/EC2)

quartata

12:35 AM

!!/watch ravioli ravioli what's in the pocketoli

SmokeDetector

@quartata You don't have code privileges, but I've created PR#2156 for you.

Restart: API quota is 14099.

quartata

wtf

it didn't even get the code admins approved badge

PA isn't showing up as a status check

that can't be normal behavior

@ArtOfCode is there anything in the MS log? is this on our end?

I think this is legitimately a PA problem actually

because the PR doesn't even show up here pullapprove.com/Charcoal-SE/SmokeDetector/?state=all

ArtOfCode

We don't initiate PA, PA should pick up on new PRs by itself

since it didn't... yeah not us

quartata

of all the days

ArtOfCode

try again

quartata

12:39 AM

!!/watch please help how do I juic avocad

SmokeDetector

@quartata You don't have code privileges, but I've created PR#2157 for you.

quartata

nope

????

@ArtOfCode check for tracebacks in the logs just to be sure?

it would be from github_controller.rb of course

SmokeDetector

Merged SmokeDetector #2156.

quartata

oh wait here we go

uhhhh

what?

I already deleted that one

ok

whatever

SmokeDetector

Merged SmokeDetector #2157.

quartata

12:42 AM

there we go

I have no idea what happened with the first one

That was really weird

Definitely a PA thing tho

SmokeDetector

CI on 93901da succeeded. Message contains 'autopull', pulling...

quartata

@ArtOfCode all right I'm done, you can give code admin back

ArtOfCode

no tracebacks

SmokeDetector

[ SmokeDetector ] SmokeDetector started at rev 93901da (metasmoke: Merge pull request #2157 from Charcoal-SE/auto-blacklist-1525999193.0865896 --autopull) (running on ArtOfCode/EC2)

quartata

yeah it wasn't us, it's working fine

SmokeDetector

12:43 AM

Restart: API quota is 14063.

[ SmokeDetector ] SmokeDetector started at rev 9fb3f5a (quartata: rm test --autopull) (running on ArtOfCode/EC2)

Restart: API quota is 14052.

fp by quartata on Is this a Fallacy? You are X because you hold an opinion as X would [MS]

fp by quartata on Parsing JSON file in powershell with specific characters [MS]

fp by quartata on Geoserver ImageMosaic ignores existing datastore.properties (connect to PostGIS) and automatically generates index using shapefile [MS]

naa by quartata on How to download all di.fm & sky.fm playlists at once? [MS]

naa by quartata on liveReload plugin not refreshing page on save [MS]

quartata

@Undo you're right, System does have Smokey's API token

so we're already good to go with that

Basically just try it first for flags, then do the rest of the flag logic normally

@Undo when you get back can you destroy 1388 from the console

@ArtOfCode check keybase?

sd 2poof

sd poof

thesecretmaster

Bleh, graphql gets annoying fast

Well, graphql-ruby and graphql generally.

@quartata user?

yeah

... yeah, gone

12:57 AM

cool

thesecretmaster

1:09 AM

Bleh graphql seems to hate nested array args.

@ArtOfCode Seems like this'd have to be done with this syntax:

1 hour ago, by thesecretmaster

@ArtOfCode { post1: post(id: 120283) { id title body } post2: post(id: 120292) { id title body } }

ArtOfCode

no way to specify multiple arguments at once?

that syntax wouldn't be terrible, but it's not very DRY

thesecretmaster

I can't find one

I can do uids: [] site: [] or uids: ['site:id', 'site:id']

ArtOfCode

oh, is this just for uids?

so urls and ids as an array works?

thesecretmaster

yeah

Arrays are fine

ArtOfCode

okay, that's fine then

thesecretmaster

1:20 AM

nested arrays are not

ArtOfCode

probably the latter of your two suggestions

quartata

@ArtOfCode now that I'm redoing this part, can we revisit why we don't bother to wait for Metasmoke to create a post?

FWIW from personal experience metasmoke often creates the post before it makes it here, just because of chat ratelimiting and all the rooms we have to post to

ArtOfCode

originally, metasmoke dependency

experience is the same

quartata

that can be solved with an aggressive timeout

And in exchange it would cure all the ugly URL crap we do

I'll push a MS PR to make posts.json return the ID in plaintext for now, just in case we make that change

incidentally why is this an instance variable:

ArtOfCode

Don't know how long MS post create requests take, however

quartata

1:25 AM

github.com/Charcoal-SE/metasmoke/blob/master/app/controllers/…

@ArtOfCode we could do it via ActionCable, since that's guaranteed a worker

ArtOfCode

ew

quartata

..hm?

what's wrong with using ActionCable? I think it would be a good idea

we could replace sending feedback too

ArtOfCode

(a) receiving AC on the Rails end is a mess

(b) more importantly, ew

quartata

oh is it

figures

ArtOfCode

that sort of thing just should be an HTTP request

SmokeDetector

1:27 AM

[ SmokeDetector | MS ] Offensive answer detected: What was Steve Rogers's phone number? by user100355 on scifi.SE (@Mithrandir)

quartata

status pings as an HTTP request are still weird imo but eh

ArtOfCode

that's more weird, yes

but it technically is updating a resource, so eh

thesecretmaster

I just might start a fork of graphql so my life can make more sense.

SmokeDetector

[ SmokeDetector | MS ] Link at end of answer: Gett JSON from python API by Kelvin on stackoverflow.com

quartata

should we have a separate report for when a post is re-reported without any revisions?

If so, the easiest fix of all would be just to add revision ID to the UID

SmokeDetector

1:35 AM

tpu by WELZ on What was Steve Rogers's phone number? [MS]

quartata

and ditch URLs everywhere else

SmokeDetector

[ SmokeDetector | MS ] Potentially bad keyword in body: Parsing JSON file in powershell with specific characters by Adam Sanders on stackoverflow.com

[ SmokeDetector | MS ] Offensive answer detected, toxic answer detected: How to change color of an image in Illustrator by P.S.OvaHoes on graphicdesign.SE

tpu by quartata

thesecretmaster

Heading out for today, PRs tomorrow.

SmokeDetector

tpu- by Catija

[ SmokeDetector | MS ] Username similar to website in answer: Resolver problema com o Docker para Windows by Senio Caires on pt.stackoverflow.com

[ SmokeDetector | MS ] Bad keyword in answer, bad keyword with email in answer, email in answer, pattern-matching email in answer: Vaccinations and medicines for Peru and Ecuador by mark on travel.SE

quartata

1:51 AM

@SmokeDetector 0.9935

Undo

> fred from dubai

quartata

@SmokeDetector 0.92. so by themselves, it's just under. this means that people could make site-specific conditions, like on Hinduism, to catch trolls still

Which is perfect IMO

Undo

Skeptical, somehow

SmokeDetector

fp- by Makyen on Parsing JSON file in powershell with specific characters [MS]

tpu by quartata

Makyen

@SmokeDetector ignore-

SmokeDetector

1:55 AM

fp by quartata on Parsing JSON file in powershell with specific characters [MS]

tpu- by Catija

WELZ

!!/scan graphicdesign.stackexchange.com/a/109215/63979

SmokeDetector

[ SmokeDetector | MS ] Offensive answer detected, toxic answer detected, blacklisted user: How to change color of an image in Illustrator by P.S.OvaHoes on graphicdesign.SE

tpu- by Makyen

quartata

Blacklisted user makes very little difference. Bumps it up to a .93.

I think that's a good thing, although in this case it wouldn't be autoflagged anyways

WELZ

still alive -_-

Makyen

@WELZ Why, when it was reported just a few messages before that?

WELZ

1:57 AM

@Makyen I didn't see that, I just saw it so scanned it.

SmokeDetector

fp- by NobodyNada on Parsing JSON file in powershell with specific characters [MS]

Makyen

I guess he better thing is to have SD point the person at the initial report, assuming that it's an identical body (i.e. no changes, even during the grace period).

SmokeDetector

fp- by Makyen on Resolver problema com o Docker para Windows [MS]

Makyen

@SmokeDetector Google translate appears to indicate that there is disclosure.

SmokeDetector

@dorukayhan Your flag condition was disabled: Accuracy must be over 99.75%

ArtOfCode

2:05 AM

well that still works

cc @dorukayhan .

gah, superpings are terrible

NobodyNada

Man, it's been a long time since I've seen them around...

SmokeDetector

2:50 AM

fp- by Makyen on Gett JSON from python API [MS]

[ SmokeDetector | MS ] Toxic answer detected: Can one be killed to death? by not gabriel on english.SE

tpu- by Makyen

[ SmokeDetector | MS ] Potentially bad keyword in body: What is the best cure for sexomania? by Josh on health.SE

NobodyNada

@SmokeDetector fp- weird

SmokeDetector

fp- by Makyen

[ SmokeDetector | MS ] URL in title: The repository 'ppa.launchpad.net/tsvetko.tsvetkov/trusty-backports/ubuntu xenial Release' does not have a Release file by chyangba on elementaryos.SE

fp- by NobodyNada

[ SmokeDetector | MS ] Repeating characters in title: TSR fdddfdfdsmmmmmmmmmmmmmmmmmmmmm by Noor on stackoverflow.com

quartata

3:23 AM

@tripleee any chance you could take the ping out of the Halflife "not +1" notifications I get

the message itself I'd like to keep but I don't need every single one in my inbox anymore :p

tripleee

3:43 AM

@quartata you shoull be able to do it yourself, this is nowecontrolled by the @pulse unnotify command

or give me half an hour until I'm at my computer

SmokeDetector

4:11 AM

[ SmokeDetector | MS ] Bad keyword in body, bad keyword in title, blacklisted website in body, blacklisted website in title: skin4up.com/keto-6x/ by fonszeonz on apple.SE

tpu- by Tetsuya Yamamoto

Nisse Engström

@SmokeDetector tpu-

Tetsuya Yamamoto

@SmokeDetector Seems like cat in keyboard when typing title?

SmokeDetector

tpu- by Tetsuya Yamamoto on What was Steve Rogers's phone number? [MS]

tripleee

preparing to reboot Halflife, will follow up here when it's done

NobodyNada

@TetsuyaYamamoto They probably got a "your title is too short" and didn't care why there's a length limit

tripleee

4:26 AM

@quartata can you run the following command over in the Charcoal Test room

@pulse unnotify .

the dot at the end is significant

in other words, should be back up now, though still absolutely no idea why PulseMonitor goes into a headspin like that

SmokeDetector

[ SmokeDetector | MS ] URL in title, bad NS for domain in body, bad NS for domain in title, bad keyword in body, bad keyword in title, +4 more: boosthightestosterones.com/ketoslim-forskolin/ by gelnteoc on english.SE

tpu- by Tetsuya Yamamoto

tripleee

!!/watch 234\W*810\W*335\W*8599

SmokeDetector

@tripleee That pattern looks like it's already caught by Bad keyword in answer and Bad keyword in body; append -force if you really want to do that.

tripleee

"Bad keyword" how?

!!/blacklist-keyword keto\W?slim(?:\W?forskolin)?

SmokeDetector

@tripleee That pattern looks like it's already caught by Bad keyword in answer and Bad keyword in body; append -force if you really want to do that.

tripleee

4:36 AM

!!/blacklist-keyword-force keto\W?slim(?:\W?forskolin)?

SmokeDetector

@tripleee Blacklisted keto\W?slim(?:\W?forskolin)?

CI on bde5955 succeeded. Message contains 'autopull', pulling...

[ SmokeDetector ] SmokeDetector started at rev bde5955 (SmokeDetector: Auto blacklist of keto\W?slim(?:\W?forskolin)? by tripleee --autopull) (running on ArtOfCode/EC2)

Restart: API quota is 12824.

tripleee

!!/blacklist-website deathbymodernmedicine\.com

SmokeDetector

@tripleee Blacklisted deathbymodernmedicine\.com

[ SmokeDetector ] SmokeDetector started at rev 302f275 (SmokeDetector: Auto blacklist of deathbymodernmedicine\.com by tripleee --autopull) (running on ArtOfCode/EC2)

Restart: API quota is 12811.

Suraj Rao

\o/ Marshal in GraphicDesign.SE

A J

@SurajRao Congo.

user4412195

4:56 AM

!!/scan codereview.stackexchange.com/a/194166

SmokeDetector

[ SmokeDetector | MS ] Bad keyword in answer, blacklisted website in answer, potentially bad NS for domain in answer: C++ Student Class by Irene Hynes on codereview.SE

tpu- by tripleee

user4412195

!!/report bitcoin.stackexchange.com/a/75004 bitcoin.stackexchange.com/a/75003

SmokeDetector

[ SmokeDetector | MS ] Manually reported answer (batch report: post 1 out of 2): I have 23 words of my 24 word mnemonic (and passphrase) for my ledger nano-s, is it possible for me to brute-force the last word (not in order) by Laura spears on bitcoin.SE

user4412195

By two different instances of some Laura [Ss]pears

SmokeDetector

[ SmokeDetector | MS ] Manually reported answer (batch report: post 2 out of 2): How to find bitcoin core address by Laura Spears on bitcoin.SE

tpu- by tripleee on I have 23 words of my 24 word mnemonic (and passphrase) for my ledger nano-s, is it possible for me to brute-force the last word (not in order) [MS]

tpu- by tripleee

[ SmokeDetector | MS ] Pattern-matching website in body, pattern-matching website in title: error: Call to a member function getRows() on a non-object in G:\\PleskVhosts\\nriitibhopal.org\\nrigroupindia.com\\index.php on line 3 by N R I Bhopal on stackoverflow.com

fp- by suraj

fp feedback on autoflagged post: error: Call to a member function getRows() on a non-object in G:\PleskVhosts\nriitibhopal.org\nrigroupindia.com\index.php on line 3 MS (@Glorfindel @JakeSymons @Federico)

[ SmokeDetector | MS ] URL-only title, bad NS for domain in body, bad NS for domain in title, bad keyword in body, bad keyword in title, +5 more: quicksupplementfact.com/keto-6x/ by srps on askubuntu.com

tpu- by suraj

SmokeDetector

5:22 AM

[ SmokeDetector | MS ] Bad NS for domain in body, bad keyword in body, blacklisted website in body, pattern-matching website in body: I wrote this essay in less than a day by XieverDenson on graphicdesign.SE

[ SmokeDetector | MS ] Bad NS for domain in body, bad keyword in body, bad keyword in title, blacklisted website in body, link at end of body, +1 more: Keto 6X Reviews by Keve Kiosan on askubuntu.com

tpu- by Makyen

tpu- by Makyen on I wrote this essay in less than a day [MS]

Makyen

!!/watch undergroundsoymarket

SmokeDetector

@Makyen Added undergroundsoymarket to watchlist

[ SmokeDetector ] SmokeDetector started at rev ffd7519 (SmokeDetector: Auto watch of undergroundsoymarket by Makyen --autopull) (running on ArtOfCode/EC2)

Restart: API quota is 12584.

[ SmokeDetector | MS ] Bad keyword in body, blacklisted website in body, link following arrow in body, pattern-matching website in body: Keto PureFit Unadulterated Keto PureFit by anthonyodson on graphicdesign.SE

tpu- by tripleee

Makyen

!!/watch keto\W*purefit

SmokeDetector

@Makyen Added keto\W*purefit to watchlist

[ SmokeDetector | MS ] Bad keyword in body, bad keyword in title, blacklisted website in body, link at end of body, pattern-matching website in body, +3 more: What Is Crazy Bulk Anvarol Anavar? by lokiston on superuser.com (@Zoe)

[ SmokeDetector ] SmokeDetector started at rev 1186f81 (SmokeDetector: Auto watch of keto\Wpurefit by Makyen --autopull*) (running on ArtOfCode/EC2)

Restart: API quota is 12516.

tripleee

5:38 AM

!!/watch anvarol

SmokeDetector

@tripleee Added anvarol to watchlist

[ SmokeDetector ] SmokeDetector started at rev 97a1b26 (SmokeDetector: Auto watch of anvarol by tripleee --autopull) (running on ArtOfCode/EC2)

Restart: API quota is 12500.

tripleee

!!/blacklist-keyword anavar

SmokeDetector

@tripleee Blacklisted anavar

[ SmokeDetector ] SmokeDetector started at rev 73f6ce9 (SmokeDetector: Auto blacklist of anavar by tripleee --autopull) (running on ArtOfCode/EC2)

Restart: API quota is 12496.

[ SmokeDetector | MS ] Bad keyword in body, bad keyword in title: \\Http error 500 on hosting laravel project on alibaba cloud server by vvek olee on stackoverflow.com

fp- by Makyen

fp feedback on autoflagged post: \Http error 500 on hosting laravel project on alibaba cloud server MS (@Makyen @M.A.R.)

@M.A.R. Your flag conditions were manually disabled by @System: Total accuracy must be >= 99.75%.

@Makyen Your flag conditions were manually disabled by @System: Total accuracy must be >= 99.75%.

[ SmokeDetector | MS ] Potentially bad keyword in answer: PDF viewer with selectable text by Jason on askubuntu.com

[ SmokeDetector | MS ] Bad keyword in answer: What do "Goto Definition" and "Goto Reference" do in Sublime Text by user903909 on superuser.com (@Zoe)

tpu- by tripleee

tripleee

!!/unwatch qoppa\.com

SmokeDetector

@tripleee Removed qoppa\.com from watchlist

fp- by tripleee on PDF viewer with selectable text [MS]

CI on 4f85182 succeeded. Message contains 'autopull', pulling...

[ SmokeDetector ] SmokeDetector started at rev 4f85182 (SmokeDetector: Auto unwatch of qoppa\.com by tripleee --autopull) (running on ArtOfCode/EC2)

Restart: API quota is 12419.

[ SmokeDetector | MS ] Bad keyword in body, bad keyword in title, blacklisted website in body, link at end of body, pattern-matching website in body: Pro Diet Plus - Suppresses appetite by aannieha on askubuntu.com

tpu- by tripleee

Transcript for