Charcoal HQ

SmokeDetector

11:03

[ SmokeDetector | MS ] Bad keyword in body: Chinese Language Courses in connaught place by Sanjnapra on mathoverflow.net (@FrenzyLi)

tpu- by Glorfindel

[ metasmoke ] ci/circleci: js success on 285f2d5: Your tests passed on CircleCI!

[ metasmoke ] ci/circleci: rubocop success on 285f2d5: Your tests passed on CircleCI!

thesecretmaster

So far so good...

SmokeDetector

[ metasmoke ] ci/circleci: rails-tests success on 285f2d5: Your tests passed on CircleCI!

thesecretmaster

Yay, passing CI!

I'd try a deploy again, but now would prolly be a bad time for MS to explode.

Mithrandir

@iBug want a quick challenge?

SmokeDetector

[ SmokeDetector | MS ] Url in title, blacklisted website in body, blacklisted website in title, potentially bad ns for domain in body, potentially bad ns for domain in title: curlycrayy.com/vital-progenix/ by user48095 on rpg.SE

tpu- by suraj

Mithrandir

11:12

Add an option for rooms in rooms.yml to have Smokey reports always delete themselves after two minutes, whether or not it's a fp or deleted

John Dvorak

Why would a room want that?

Mithrandir

Apparently, the Restaurant does

SmokeDetector

[ SmokeDetector | MS ] Potentially bad keyword in answer: Mobile Media Converter stoped conversion by Victor on askubuntu.com

[ SmokeDetector | MS ] Url in title, bad keyword in body, bad keyword in title, bad keyword in username, blacklisted website in body, +4 more: goldencondor.org/vera-slim/ by Vera Slim on astronomy.SE

iBug

@Mithrandir ask RO to move 'em out, better?

SmokeDetector

tpu- by ByteCommander on Mobile Media Converter stoped conversion [MS]

iBug

11:21

You want a lot of (deleted) stick around?

sorta wanna [status-declined] on this

Mithrandir

I agree it's a bit silly, but...

Byte Commander

they could make smokey an RO and Smokey could get a feature to move old resolved reports to a trash room e.g. hourly or so.

angussidney

@ByteCommander urgh... messy. AFAIK, ChatExchange doesn't allow you to do this

Byte Commander

then we might have to add it :P

/shrig

/shrug

angussidney

@Mithrandir What advantage do they gain from having Smokey in there if the reports are just going to be deleted after 2mins, deleted or not?

Mithrandir

11:26

@angussidney If people are around, it gets noticed and deleted. If people aren't around, then it'll be deleted before anyone sees the message and the message is redundant

iBug

IMO periodically moving em out manually is a better solution

SmokeDetector

[ SmokeDetector | MS ] Bad keyword in body, bad keyword in title, blacklisted username, blacklisted website in body, blacklisted website in title, +7 more: www.visit4supplements.com/praltrix/ by jaimeldsan on security.SE

iBug

which is what SOCVR has been doing for a long time

SmokeDetector

tpu- by Shree

angussidney

@iBug that kinda defeats the 'before anyone sees the message' bit

Mithrandir

11:28

Just how long would it take to actually implement?

angussidney

We could consider adding the feature if the usecase is valid enough, and the two avaliable options (full on and deltionwatched) are justifyable insufficient

SmokeDetector

tpu- by tripleee on https://goldencondor.org/vera-slim/ [MS]

iBug

idk, might need to add another thread and queue and handle them

more complicated than an above-average optimization task

tripleee

!!/watch mypragati\.org

SmokeDetector

@tripleee Added mypragati\.org to watchlist

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

[ SmokeDetector ] SmokeDetector started at rev feac96c (SmokeDetector: Auto watch of mypragati\.org by tripleee --autopull) (running on teward/Solar Flare)

Restart: API quota is 17706.

iBug

11:34

processes with higher nice value gets less CPU right?

angussidney

@angussidney *deletionwatched *justifiably

iBug

With the "precompile-regex" branch merged in, my testing Smokey maintains a very stable CPU usage of around 15%

that's ... a bit higher than a t2.micro can maintain for long

I'll try again with that patch removed

SmokeDetector

[ SmokeDetector | MS ] Blacklisted website in body, body starts with title and ends in url, link at end of body, link following arrow in body, pattern-matching product name in body, +2 more: Perfect Keto Max>>>Perfect Keto scoop supplement by CarrieHamlin on drupal.SE

tpu- by tripleee

micsthepick

!!/scan puzzling.stackexchange.com/questions/54341/…

SmokeDetector

@micsthepick Post 1: This does not look like spam

micsthepick

11:39

!!/report puzzling.stackexchange.com/questions/54341/…

SmokeDetector

[ SmokeDetector | MS ] Manually reported answer: Raven's style IQ question by Benjamin on puzzling.SE (@Mithrandir)

micsthepick

I now have access to the VLQ queue in Puzzling

SmokeDetector

tpu- by suraj

micsthepick

no thanks to answering one of Randal'Thor's questions

!!/watch iqtestonline.net

SmokeDetector

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

micsthepick

11:43

@angussidney I think that now might be an appropriate time for me to ask again for code privileges, what do you think?

(I mean, if you are busy, then I can wait)

SmokeDetector

Merged SmokeDetector #2392.

angussidney

I'll take a look for you

SmokeDetector

Merged SmokeDetector #2392.

paper1111

@micsthepick friendly reminder, there is no more need to scan-then-report.

iBug

@micsthepick !!/report does a scan first, it's been there for almost 2 months

SmokeDetector

11:45

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

[ SmokeDetector ] SmokeDetector started at rev b5cbd6f (metasmoke: Merge pull request #2392 from Charcoal-SE/auto-blacklist-1535715728 --autopull) (running on teward/Solar Flare)

Restart: API quota is 17584.

iBug

!!/test perfect booster boost

SmokeDetector

> Pattern-matching product name in body, pattern-matching product name in title
----------
Title - Pattern-matching product name *perfect booster boost*
Body - Pattern-matching product name *perfect booster boost*

tripleee

@micsthepick try !!/amicodeprivileged now

iBug

@tripleee /ami/acode

SmokeDetector

[ SmokeDetector | MS ] Offensive answer detected, toxic answer detected: My Private key from myetherwallet.com was stolen, Can I add more steps to access my wallet? by wat on ethereum.SE

tripleee

11:48

@angussidney enough good PRs for me; clicked the box

SmokeDetector

tpu- by tripleee

iBug

@SmokeDetector n

angussidney

@micsthepick your track record looks good enough now, so ~~I've adjusted your privileges~~ tripleee beat me to it :) Before you start doing anything, I recommend re-reading the blacklisting guidelines. Don't abuse it, or standard pallet-of-bricks enforcement method applies, yada yada yada, you get the drill

Anyway, enjoy

iBug

!!/whois code_admin

SmokeDetector

@iBug I am aware of 32 code admins. Currently in this room: iBug, angussidney, tripleee, paper1111, micsthepick, Magisch, Mithrandir, Byte Commander, Henders, John Dvorak, thesecretmaster, Glorfindel, Makyen, Suraj Rao, Zoe, Ferrybig. Not currently in this room: Thomas Ward, doppelgreener, WELZ, K.Dᴀᴠɪs, CalvT, ProgramFOX, Ashish Ahuja, DavidPostill, Tetsuya Yamamoto, ArtOfCode, Paul Stenne, Andy, bwDraco, Undo, quartata, NobodyNada.

thesecretmaster

11:51

Wait when did I end up on that list?

angussidney

Don't forget that your blacklist entries go straight through now, so I suggest you double check first to make sure there aren't any mistakes before sending the command - although it isn't a huge problem if anything does make it through, everything can be fixed

thesecretmaster

Oh different list

Nevermind

iBug

@thesecretmaster Probably because Undo and Art trust you so much that they just gave you ~~the rubocop~~ the role.

SmokeDetector

[ SmokeDetector | MS ] Blacklisted website in body: Review & Rating not showing in google SERP by amritjames on stackoverflow.com

thesecretmaster

I forgot that the list comes from MS rather than one of Smokeys config files.

angussidney

11:54

Which list were you thinking about? The normal privileges list?

thesecretmaster

Isn't there a per-room list in rooms.yml or something like that?

iBug

@SmokeDetector k

ArtOfCode

@thesecretmaster yeah, that's Smokey privs

SmokeDetector

[ SmokeDetector | MS ] Blacklisted website in answer: SQLITE File is encrypted or is not a database by Goran on stackoverflow.com

[ SmokeDetector | MS ] Potentially bad ns for domain in answer: SQLITE File is encrypted or is not a database by Mark Crook on stackoverflow.com

[ SmokeDetector | MS ] Bad keyword in answer, bad keyword with email in answer, pattern-matching email in answer: How do I remove the "Comments" word out of "x Comments" in Disqus Tumblr themes? by rebecca maria on webapps.SE

[ SmokeDetector | MS ] Url-only title, blacklisted website in body, blacklisted website in title, link at end of body, pattern-matching product name in body, +4 more: supplementforuse.com/rapid-slim/ by user866287 on askubuntu.com

tpu- by Shree

fp- by tripleee on SQLITE File is encrypted or is not a database [MS]

tpu- by tripleee on SQLITE File is encrypted or is not a database [MS]

tpu- by tripleee on How do I remove the "Comments" word out of "x Comments" in Disqus Tumblr themes? [MS]

thesecretmaster

@ArtOfCode I'm looking for something in the prod logs, and it's > 1 day old. Is the 1k line limit [status-bydesign] or could I make a PR to show more?

(or do those logs get rotated out after a day?)

ArtOfCode

12:05

gonna say status-bydesign

displaying >1k lines is asking for trouble; grepping >10k lines likewise

micsthepick

!!/amicodeprivileged

SmokeDetector

@micsthepick ✓ You are a code-privileged user.

micsthepick

cool

thesecretmaster

@ArtOfCode Oh, but not a security/privacy thing?

ArtOfCode

however, feel free to add a "download prod log" thing, like we have for the query log

Magisch

12:06

hey @ArtOfCode

micsthepick

now back to this PPCG challenge that I suddenly discovered

ArtOfCode

@thesecretmaster nah

@Magisch o/

Magisch

long time no see

thesecretmaster

@Undo Are you OK with a "Download prod logs" button for devs?

I still don't know what to do about that mystery yarn error in deploy...

ArtOfCode

@thesecretmaster prod logs isn't cap logs

thesecretmaster

12:08

Yeah I know

Wait... what? @ArtOfCode Am I supposed to be an MS admin? Because I don't think so, but it looks like I am.

ArtOfCode

@thesecretmaster cc @Undo

Mithrandir

!!/whois admin

SmokeDetector

@Mithrandir I am aware of 6 admins. Currently in this room: angussidney, tripleee, thesecretmaster, ArtOfCode. Not currently in this room: Andy, Undo.

thesecretmaster

I'll just uncheck that for now.

ArtOfCode

can't say I added it, but as a developer it's kinda useful to have access to the stuff

thesecretmaster

12:12

Ok. I don't object to having it, but I also don't want to have it without being officially allowed to.

Mithrandir

+1 TSM for admin

thesecretmaster

Also, y'know, if I really needed admin I could still get it back.

But I don't think I do.

ArtOfCode

@thesecretmaster uh... could you?

SmokeDetector

Restart: API quota is 19309.

thesecretmaster

@ArtOfCode Impersonate an admin, give my self admin, go back to my account.

ArtOfCode

12:14

oh, true :P

forgot we had impersonation

SmokeDetector

Restart: API quota is 19307.

ArtOfCode

but that does also mean that if you do need to check something out as an admin, you can just impersonate one

thesecretmaster

Yep, but I wouldn't want to push any buttons from someone elses account.

SmokeDetector

Restart: API quota is 19300.

ArtOfCode

@thesecretmaster eh, use me or Undo and it'd be fine

thesecretmaster

12:17

K. I think I'm gonna try a deploy again and see if yarn magically fixed itself.

We're 10 commits behind master, which I don't love.

SmokeDetector

[ metasmoke-deploy ] deploy started by thesecretmaster

[ SmokeDetector | MS ] Bad keyword in body, bad keyword in title, blacklisted website in body, pattern-matching website in body, potentially bad ns for domain in body: KetoFit Trial Reviews by tuhefitoorm on stackoverflow.com

Glorfindel

sd k

thesecretmaster

Bah, foiled again!

@ArtOfCode Do you know how to add a step to cap deploy? Or does that require Undo?

ArtOfCode

the latter

thesecretmaster

I think it isn't installing tagging

And so the compile step fails because it's missing a dependency

I don't see a npm or yarn install anywhere in there.

@Undo Seems like cap deploy isn't installing my package. Could you please either do it manually or add it to the cap deploy script?

SmokeDetector

12:33

[ SmokeDetector | MS ] Manually reported answer: Add Custom registration fields in WooCommerce and phone field validation issue by Denish on stackoverflow.com

Stephen Kennedy

^ I reported this. User has multiple answers today all pointing to his blog.

SmokeDetector

naa- by Glorfindel

Glorfindel

yeah, probably best to mod flag one of their posts.

Stephen Kennedy

will do

iBug

@SmokeDetector k

SmokeDetector

12:36

[ SmokeDetector ] SmokeDetector started at rev 27c4b6f (iBug: Merge branch 'precompile-regex' --autopull) (running on teward/Solar Flare)

Restart: API quota is 17077.

iBug

Hey, does MS still send "deploy_updated" messages?

I found some code that I've never seen running (by seeing output).

Currently whenever a new pull instruction is received from MS, Smokey exits and nocrash.py pulls the updates.

github.com/Charcoal-SE/SmokeDetector/blob/master/…

SmokeDetector

[ SmokeDetector | MS ] Url in title, bad keyword in body, bad keyword in title, bad keyword in username, blacklisted website in body, +3 more: goldencondor.org/vera-slim/ by Vera Slim on ham.SE

iBug

@SmokeDetector k

Stephen Kennedy

@SmokeDetector Could/should we watch this domain?

iBug

!!/watch wordpressdeveloperonline\.com

SmokeDetector

12:51

@iBug Added wordpressdeveloperonline\.com to watchlist

Stephen Kennedy

ty

SmokeDetector

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

[ SmokeDetector ] SmokeDetector started at rev cd676ed (SmokeDetector: Auto watch of wordpressdeveloperonline\.com by iBug --autopull) (running on teward/Solar Flare)

Restart: API quota is 16915.

SmokeDetector

13:10

[ SmokeDetector | MS ] Blacklisted website in body: Have you know about the various gaming platform by Tyron P on superuser.com

tpu- by Glorfindel

[ SmokeDetector | MS ] Bad keyword in answer, bad keyword with email in answer, email in answer, pattern-matching email in answer: Can a party function without a Cleric or Paladin? by jayce on rpg.SE

tpu- by Glorfindel

Undo

@thesecretmaster no point in having dev permissions without admin permissions

well, I guess, but it's not like admin permissions let you screw anything up worse than dev permissions

@thesecretmaster Probably won't get anything from prod logs on a Cap issue; they're completely separate.

thesecretmaster

@Undo Was looking for logs for something else, not cap. I wanted the full prod logs because I needed further back than the last 1k lines.

Undo

Those logs are kinda scary; I don't trust that they're nicely censored. I prefer being a gate to mass-log stuff.

What's the yarn package that needs installed?

thesecretmaster

@Undo K, that's what I thought, then Art said it was OK.

@Undo tagging. But really a yarn/npm install should go in the cap deploy process.

Undo

Probably needs something like github.com/j-arnaiz/capistrano-yarn

thesecretmaster

13:23

That looks good, but I don't really know yarn or cap too well.

I can try and add it?

SmokeDetector

[ metasmoke-deploy ] deploy started by Undo

thesecretmaster

Or that

Undo

Trying adding it manually for now; should definitely be in Cap though

I like semicolons; they're the ultimate lazy person's punctuation

ArtOfCode

EC22 upgrading to 18.04.1... let's see how terribly this goes

thesecretmaster

@Undo Can we cache the bundle on travis for faster deploys?

Undo

13:28

Definitely could if you could figure out how

Don't even need smarts about it, just need to cache the whole workspace

thesecretmaster

docs.travis-ci.com/user/caching/#enabling-bundler-caching

ArtOfCode

> You have to download a total of 264 M. This download will take approximately 33 minutes with a 1 MBit DSL connection and about 10 hours with a 56k modem.

thesecretmaster

cache: bundler

ArtOfCode

Totally apart from those being entirely useless measurements at modern speeds, this is EC2:

Undo

well that's too easy

ArtOfCode

13:29

> 264 MB downloaded in 6s.

Undo

@ArtOfCode I love my 30MB/s transfer perf to S3

This is even better, somehow. Probably cached super close to the hardware

thesecretmaster

@Undo "Command "webpack" not found" from travis

ArtOfCode

well it's getting the updates from Amazon's mirror of the package repos, so those are almost certainly mirrored in every EC2 datacenter

Undo

well that seems worse

thesecretmaster

cap rollback?

Oh wait no

Also, why don't we have historical logs from travis?

ArtOfCode

13:32

because it's the same job every time, just restarted

thesecretmaster

Yes, but why do we do that?

Undo

yeah, it's technically rebuilds.

thesecretmaster

Logs are good

ArtOfCode

because the deploy script is its own repo, so without making another commit to that repo for every deploy we can't get a new job for it

Undo

Because it was the easiest thing to do. There was something crazy about starting new builds programmatically

thesecretmaster

13:34

@ArtOfCode Seems like travis pulls in MS and the other repo and runs a sh script from the other repo. Could easily also be parented to MS, no?

Undo

There's a bit of security there; the other repo is there so I can hold server secrets without exposing them to all MS contributors

Travis lets me put secrets in metasmoke-deploy that no one else can see. I can't do that without a repo I own

SmokeDetector

[ metasmoke-deploy ] deploy started by Undo

Undo

Okay, let's try this

Ran just sudo yarn install

Travis probably doesn't need a lot of those gems anyway, like mysql2.

thesecretmaster

Add cache: bundler?

Undo

Want to see it succeed first

SmokeDetector

13:39

[ SmokeDetector | MS ] Bad keyword in answer, bad keyword with email in answer: How to throw off suspicion for the big reveal in a way that it is not unfairly blindsiding them? by jayce weny on rpg.SE

tpu- by Glorfindel

Undo

there, that'll work

thesecretmaster

\o/

SmokeDetector

[ SmokeDetector | MS ] No whitespace in answer: oracleで1度問い合わせたSQLが2回目からは結果を返して来ません by user29940 on ja.stackoverflow.com

tpu- by Zoe on Transfer Word and pdf files between PC and iPhone [MS]

Glorfindel

@SmokeDetector f

SmokeDetector

tp by Zoe on "Put a knife to/at/on throat" - which one is preferred? [MS]

tp by Federico on Transfer Word and pdf files between PC and iPhone [MS]

thesecretmaster

13:44

Aaaand I think turbolinks doesn't play nice with my tagging lib :'(

SmokeDetector

fp by Federico on My Private key from myetherwallet.com was stolen, Can I add more steps to access my wallet? [MS]

tp by Zoe on How do I remove the "Comments" word out of "x Comments" in Disqus Tumblr themes? [MS]

tp by Zoe on Have you know about the various gaming platform [MS]

Undo

@thesecretmaster Turbolinks doesn't play nice with much :(

thesecretmaster

Any way to turn them off for a link?

uses google

SmokeDetector

[ SmokeDetector | MS ] Pattern-matching website in body, potentially bad keyword in body: What you do when you need essay paper due tomorrow? by Amy Grunewald on writing.SE

[ metasmoke ] ci/circleci: js success on b500684: Your tests passed on CircleCI!

[ metasmoke ] ci/circleci: rubocop success on b500684: Your tests passed on CircleCI!

thesecretmaster

I wonder how much time I've shaved off CI

Looks like 30 seconds to a minute

So...

Looks like CI failing was not all query_times_log

Tests just fail at random

I give up

Gonna check out some circle alternatives.

Undo

14:07

@thesecretmaster Example?

I saw tests failing for weird reasons on local a few days ago

thesecretmaster

The one running right now

circleci.com/gh/Charcoal-SE/metasmoke/2126

SmokeDetector

[ metasmoke ] ci/circleci: rails-tests failure on b500684: Your tests failed on CircleCI

thesecretmaster

Yeah, those ^^

Undo

Looks like a timeout

weird

thesecretmaster

I had a big fight with CI yesterday

Undo

14:08

I kinda like Circle's UI, but if it's that broken then yeah

thesecretmaster

It's related to how tests talk to mysql

Undo

Concurrency issue somewhere?

thesecretmaster

Maybe

But there's not logic to when it locks up

Were you here last night when I was fighting with it?

If not, have a look at this branch and the corresponding CI runs. There's no logic beyond "More DB calls = Earlier locking"

SmokeDetector

[ SmokeDetector | MS ] Potentially bad keyword in body, potentially bad keyword in title: iOS - how can we offer discount through promo code for in-app purchase? by Puneeth on stackoverflow.com

thesecretmaster

I also like Circle, but it's just broken

Undo

14:13

Wasn't here, but did you lock the tests to a certain order?

There's a test seed number or something

thesecretmaster

Nope. I was blaming MetricsUpdateJob for most of the evening

In the end I made tests skip it

And it worked

But now it's not working again

Undo

o.O

thesecretmaster

I also bet that if you re-run that job, it'll pass

SmokeDetector

[ SmokeDetector | MS ] Link at end of answer: Fee-only financial advice vs. asset management (which includes advice)? by Ross Paul on money.SE

[ metasmoke ] ci/circleci: js success on b500684: Your tests passed on CircleCI!

[ metasmoke ] ci/circleci: rubocop success on b500684: Your tests passed on CircleCI!

[ metasmoke ] ci/circleci: rubocop success on 27f37c1: Your tests passed on CircleCI!

thesecretmaster

@SmokeDetector Ignore this, this is me trying to speed up CI.

SmokeDetector

14:19

tpu- by ByteCommander

@thesecretmaster That message is not a report.

Undo

lol

Thomas Ward

lolololol

SmokeDetector

[ metasmoke ] ci/circleci: js success on 27f37c1: Your tests passed on CircleCI!

thesecretmaster

@SmokeDetector fp

SmokeDetector

@thesecretmaster That message is not a report.

fp- by ByteCommander on iOS - how can we offer discount through promo code for in-app purchase? [MS]

tpu- by ByteCommander on What you do when you need essay paper due tomorrow? [MS]

[ metasmoke ] ci/circleci: rails-tests success on b500684: Your tests passed on CircleCI!

Undo

14:21

yep, that one passed

seed was 5889 that passed, 33986 failed

SmokeDetector

[ metasmoke ] ci/circleci: rails-tests success on 27f37c1: Your tests passed on CircleCI!

[ metasmoke ] ci/circleci: rails-tests error on b500684: Your CircleCI tests were canceled

thesecretmaster

@Undo My bad, I broke CI

Gonna fix it in a sec

Undo

k

thesecretmaster

That commit message wasn't any good, but eh

Although SSHing in is pretty cool. Maybe we could use that to debug

SmokeDetector

[ metasmoke ] ci/circleci: js success on 4cdda9e: Your tests passed on CircleCI!

[ metasmoke ] ci/circleci: rubocop success on 4cdda9e: Your tests passed on CircleCI!

thesecretmaster

14:30

@Undo Working again

SmokeDetector

[ metasmoke ] ci/circleci: rails-tests success on 4cdda9e: Your tests passed on CircleCI!

SmokeDetector

14:45

[ metasmoke ] ci/circleci: js success on dd9c803: Your tests passed on CircleCI!

[ metasmoke ] ci/circleci: rubocop success on dd9c803: Your tests passed on CircleCI!

[ metasmoke ] ci/circleci: rails-tests failure on dd9c803: Your tests failed on CircleCI

[ metasmoke ] ci/circleci: js success on 765ff35: Your tests passed on CircleCI!

[ metasmoke ] ci/circleci: rails-tests failure on 765ff35: Your tests failed on CircleCI

[ metasmoke ] ci/circleci: rubocop success on 765ff35: Your tests passed on CircleCI!

thesecretmaster

Sorry for the CI spam again y'all

SmokeDetector

[ metasmoke ] ci/circleci: js success on 64f55a5: Your tests passed on CircleCI!

[ metasmoke ] ci/circleci: rubocop success on 64f55a5: Your tests passed on CircleCI!

[ metasmoke ] ci/circleci: rails-tests success on 64f55a5: Your tests passed on CircleCI!

[ SmokeDetector | MS ] Potentially bad keyword in answer: How can I get mouseover / mouseout to work with resizing? by Lu15 on stackoverflow.com

fp- by Glorfindel

[ metasmoke ] ci/circleci: js success on 99037e3: Your tests passed on CircleCI!

[ metasmoke ] ci/circleci: rubocop success on 99037e3: Your tests passed on CircleCI!

[ metasmoke ] ci/circleci: rails-tests success on 99037e3: Your tests passed on CircleCI!

[ metasmoke ] ci/circleci: js success on a81bae4: Your tests passed on CircleCI!

[ metasmoke ] ci/circleci: rubocop success on a81bae4: Your tests passed on CircleCI!

tp by CalvT on What you do when you need essay paper due tomorrow? [MS]

fp by CalvT on iOS - how can we offer discount through promo code for in-app purchase? [MS]

tp by CalvT on Fee-only financial advice vs. asset management (which includes advice)? [MS]

fp by CalvT

[ metasmoke ] ci/circleci: rails-tests success on a81bae4: Your tests passed on CircleCI!

[ metasmoke-deploy ] deploy started by thesecretmaster

[ metasmoke ] ci/circleci: js success on 8e443f7: Your tests passed on CircleCI!

[ metasmoke ] ci/circleci: rubocop success on 8e443f7: Your tests passed on CircleCI!

thesecretmaster

15:22

Any reason we don't unencode urls on this page?

SmokeDetector

[ metasmoke ] ci/circleci: rails-tests success on 8e443f7: Your tests passed on CircleCI!

iBug

@thesecretmaster No, let's get those strange things off

Byte Commander

15:43

Linus: Born to be rude

;-)

SmokeDetector

[ SmokeDetector | MS ] Potentially bad keyword in body: Akaunting Double entry, Updating / Deleting a bill item will not be deleted from Balance Sheet / Trial Balance? ✏️ by Brothersoft on stackoverflow.com

fp- by Glorfindel

[ SmokeDetector | MS ] Blacklisted website in body: Get Ounces of buds for half price all AAA buds by jugga on bitcoin.SE

tpu- by Glorfindel

J F

@thesecretmaster Did you ever get that figured out?

(I was on vacation, forgot to mention that here)

thesecretmaster

Yep, Art helped me out. Thank you though! Right now I'm finishing up adding that code to domain tagging.

Undo

@thesecretmaster Those are supposed to be domains, not URLs

thesecretmaster

Still, un-urlencoding could help?

J F

15:52

O.O tagging was still available on npm

thesecretmaster

Not anymore :P

Undo

@thesecretmaster Yeah, maybe. Valid domains shouldn't need it, though.

ArtOfCode

Not sure, actually

We might want those verbatim

SmokeDetector

@ɪʙᴜɢ: It seems this is your first time sending feedback to SmokeDetector. Make sure you've read the guidance on your privileges, the available commands, and what feedback to use in different situations.

ArtOfCode

heh

iBug

16:00

posted to the wrong place, plus wrong username

@Undo Some IDNA may have that

J F

@thesecretmaster Commentary on the tagging library: you generally don’t need to check in the node_modules directory — package.json and package-lock.json/yarn.lock (optional) are all you need to recreate the dependency structure.

The best practice for npm modules is to assign something to module.exports (or a key thereof since it starts out as an empty object) so you can do const foo = require('./bar') to retrieve the value of module.exports. (you can also do this with the export and export default keywords if you’re transpiling the code.)

thesecretmaster

@ArtOfCode For domain tag review... is the add button supposed to redirect you to the domain tag page?

J F

Also, I’m not sure what you’re using webpack.config.js for since you don’t have webpack as a dependency in your package.json or use it in any of the npm scripts.

You could probably get away without webpack by instructing users to manually import 'tagging/index.css' in their code, or bundle it some other way.

thesecretmaster

@JF I think I put it in because MS complained without it.

J F

Check out Prettier and Rollup too :)

@thesecretmaster very strange. Could that be because you have the import statement in main.js?

thesecretmaster

16:06

Maybe? Although the import is a relic of when I was trying to get the CSS bundled in. I never got that to work though.

iBug

@iBug Anyone answer this?

ArtOfCode

@thesecretmaster no?

thesecretmaster

@ArtOfCode Ah nevermind I did a stupid

So, url-decode or not, for spam domains?

SmokeDetector

16:31

fp by Nisse Engström on oracleで1度問い合わせたSQLが2回目からは結果を返して来ません [MS]

ArtOfCode

16:45

@thesecretmaster Decode for display, yes, but don't decode on creation or in the database.

thesecretmaster

I've got that and tagging changes in a PR that I'm about to open and ask for your review on.

Just because there's some JS weirdness that's a bit beyond me.

SmokeDetector

[ metasmoke ] ci/circleci: js failure on 5aa6c48: Your tests failed on CircleCI

[ metasmoke ] ci/circleci: rubocop success on 5aa6c48: Your tests passed on CircleCI!

thesecretmaster

@SmokeDetector Sorry, my machine doesn't have enough ram to run npm test

SmokeDetector

[ metasmoke ] ci/circleci: rails-tests success on 5aa6c48: Your tests passed on CircleCI!

ArtOfCode

@thesecretmaster wat

npm test is just the xo linter

shouldn't require much ram

thesecretmaster

16:54

FATAL ERROR: CALL_AND_RETRY_LAST Allocation failed - JavaScript heap out of memory

J F

got the full logs?

thesecretmaster

> xo


<--- Last few GCs --->

  127549 ms: Mark-sweep 1342.1 (1437.1) -> 1342.1 (1437.1) MB, 1023.0 / 0.0 ms [allocation failure] [scavenge might not succeed].
  128561 ms: Mark-sweep 1342.1 (1437.1) -> 1342.1 (1437.1) MB, 1011.8 / 0.0 ms [allocation failure] [scavenge might not succeed].
  129588 ms: Mark-sweep 1342.1 (1437.1) -> 1342.1 (1421.1) MB, 1026.6 / 0.0 ms [last resort gc].
  130614 ms: Mark-sweep 1342.1 (1421.1) -> 1342.0 (1421.1) MB, 1025.6 / 0.0 ms [last resort gc].


<--- JS stacktrace --->

SmokeDetector

[ SmokeDetector | MS ] Potentially bad keyword in username: how use multiple numeric format string for a simple string in c# by hahahahaaaa on stackoverflow.com

Thomas Ward

!!/watch Evil-?Droid

SmokeDetector

@ThomasWard Added Evil-?Droid to watchlist

Thomas Ward

16:59

found someone asking how to properly deploy android malware over on Ask Ubuntu, so time to watch it (my guess is they are going to do this on multiple sites0

J F

@thesecretmaster very strange. Do you have an unusually small amount of RAM?

thesecretmaster

4gb

J F

hm

not sure why it would fail then. Might be worth checking XO’s issues and opening a new one if there isn’t already one about this.

SmokeDetector

[ SmokeDetector ] SmokeDetector started at rev 6f032e1 (SmokeDetector: Auto watch of Evil-?Droid by Thomas Ward --autopull) (running on teward/Solar Flare)

Restart: API quota is 14091.

thesecretmaster

@JF It also shouldn't be using 1.5gb of ram though, which is what the logs seem to indicate.

J F

17:17

@iBug You can get the app, then copy-paste the file contents out of the app :)

SmokeDetector

17:32

[ SmokeDetector | MS ] Potentially bad keyword in body: Unexpected error when using feedparser.py ✏️ by WarpedReality on stackoverflow.com

fp by Nisse Engström on how use multiple numeric format string for a simple string in c# [MS]

fp by Nisse Engström

[ SmokeDetector | MS ] Potentially bad keyword in answer: Alternate Elemental Bases for Life ✏️ by Dan on worldbuilding.SE

[ SmokeDetector | MS ] Mostly punctuation marks in body, potentially bad keyword in title: help Evil-Droid :'( by Dj xXx on askubuntu.com

[ SmokeDetector | MS ] Potentially bad keyword in body: async/await return Promise { <pending> } ✏️ by Andrei Aleksandrov on stackoverflow.com

[ SmokeDetector | MS ] Username similar to website in answer: How exactly does sensory substitution work? by seeingwithsound on biology.SE

SmokeDetector

17:59

[ SmokeDetector | MS ] Potentially bad ns for domain in body: She lends no assistance in accomplishing by Joseph Stewart on english.SE

Transcript for