Charcoal HQ

@quartata ╳ You are not a privileged user. Please see the privileges wiki page for information on what privileges are and what is expected of privileged users.

Andy

You should have two things printed. The results of the amiprivileged command and the not_privileged_warning

quartata

I meant I already printed out the output of the function invocation

Andy

What's that unicode character? It's different than the other two in that block

quartata

The failure isn't caused by not_privileged_warning being empty if that's what you were concerned about

@Andy it's U+2573

I know, it looks weird

Oct 13 at 15:18, by SmokeDetector

@EricC ╳ You are not a privileged user. Please see the privileges wiki page for information on what privileges are and what is expected of privileged users.

angussidney

1:58 AM

Sorry I broke the tests :/

quartata

no no it's not your fault

I mean

I should have checked that they work before I merged them

but more importantly if the test fails then something is wrong, that's not something to shy away from

there's about an 80% chance this is some odd Mock corner case

if so it's something we should know

angussidney

Maybe some of the commands are trying to find a value from the msg object that we haven't mocked

quartata

that's actually possible

we can track that

at least I know we can obtain calls

mock.mock_calls

Andy

What branch are you using? What version of Python? I just cloned your master and ran pytest. Everything passed...two warnings:

python3 -W default::Warning -m pytest test/home/andy/.virtualenvs/se/lib/python3.5/site-packages/_pytest/assertion/rewrite.py:7: PendingDeprecationWarning: the imp module is deprecated in favour of importlib; see the module's documentation for alternative uses
  import imp
======================================================= test session starts ========================================================
platform linux -- Python 3.5.2, pytest-3.2.3, py-1.4.34, pluggy-0.4.0
rootdir: /home/andy/my_repos/q_detector, inifile:

Thomas Ward

@Andy it fails in master branch with 3.5 on my end, same assertion error

quartata

2:08 AM

@Andy sure you're on master?

because I don't see test/test_chatcommands in there

Andy

I don't know what to tell you. This environment is less than ten minutes old. I created a virtualenvironment, cloned quartata's master branch, pip installed the requirements, initialized the submodules and ran the command above

Oh, wait there is a switch to deploy in the readme that I copy/pasted in.

un momento

Thomas Ward

oh you know what

Andy

E ImportError: No module named 'yaml' - That needs to be fixed. Someone put that in requirements.txt

Thomas Ward

@quartata it's failing because it's saying you are privileged

quartata

right

@Andy should be in there

> pyyaml

at the end of requirements.txt

@ThomasWard the assert is after we change the ID to 2

Thomas Ward

2:10 AM

@quartata so then your privilege checker is broken

Andy

ugh. stupid deploy branch. Why is that in the readme?

Thomas Ward

@quartata I know. 1 moment

quartata

@ThomasWard it's not though

I just tried it

Thomas Ward

show me your code you used to test.

because ^ that suggests otherwise

quartata

No code. I ran !!/amiprivileged in a room that I wasn't privileged in

Thomas Ward

2:12 AM

i'm not asking you to test in a real world i'm asking you to test in the theoretical.

quartata

yeah, running the test it complains

Thomas Ward

because ^ that suggests that the test is going to fubar all the time

quartata

but keep in mind the test isn't passing in a real User object

Thomas Ward

well

then we have to consider the test broken

because your changes broke something.

just saying.

you could always revert just the tests but...

quartata

Well considering I just merged in those brand-new tests I'd rather make them work

:P

here let me run it with pdb

Andy

2:15 AM

test/test_chatcommands.py .......✓ You are a privileged user. ╳ You are not a privileged user. Please see     [the privileges wiki page](https://charcoal-se.org/smokey/Privileges) for     information on what privileges are and what is expected of privileged users.

That is the test output

Thomas Ward

that's what i'm seeing, yes.

quartata

wait, where's that print coming from?

did I commit something bad

Andy

I added the line I mentioned above, to the test and reran it

print(chatcommands.amiprivileged(original_msg=msg), "\u2573 " + GlobalVars.not_privileged_warning)

quartata

oh ok

pdb confirms that the only people that are privileged are 1 and 167070 so it's nothing wrong in the room config

> ('stackexchange.com', 11540): {1, 16070}

stepping through the test rn

hmm

ok

Thomas Ward

and what i'm seeing is the debug terminal output halted and suspended by PyCharm when it fails the test.

confirmed, is_privileged is failing with that specific mock item

quartata

2:21 AM

I'm almost positive the problem is because Mock maintains its own id field.

experimenting

Thomas Ward

@quartata make it be uid or something then?

id should be considered a protected name by the way

quartata

Blame Manishearth

OK I don't think that's being problematic:

<Mock name='mock.room' id='4321837968'> 11540 stackexchange.com

(from print(room, room.id, room._client.host))

Although I did change it so that it sets the fields manually rather than using Mock's constructor

since that makes more Mocks

ohhhh

I got it

It's because of the user.is_moderator clause

since we didn't set that to false it's returning a new Mock object

which is truthy

OK, that test now passes but pytest exits with exit code 4 anyways

angussidney

Is that the same issue as before where it just exits halfway through the tests, or does it complete first then exit with 4?

quartata

exits halfway through

and I know why

angussidney

\o/

quartata

2:29 AM

exit code 4 is what we use for crashes if I'm not mistaken

we're probably making a call into something that's expecthook'd and it's failing

it's not pytest itself that's exiting

yeah, ok

it's probably breaking on one of the report calls that's supposed to succeed

then it would try to post a message

and that would probably go afoul

commenting that one out for the time being

or at least just the parts where it gets past the initial checks

angussidney

The issue with the current test format is that there isn't any way to test for actual chat output

quartata

woot, works

I just commented out the valid ones for the time being

the invalid ones passed

ok, one other failure

> assert chatcommands.allspam("http://stackexchange.com/users/me", original_msg=msg) == (
"The specified user has an abnormally high number of accounts. Please consider flagging for moderator "
"attention, otherwise use !!/report on the user's posts individually." # TODO: Update ID
)

fails

it instead returns "That doesn't look like a valid user URL"

angussidney

Woops, I forgot to replace my userID in there

5869449

quartata

cheeky jon skeet test there btw

ok interesting

so something else failed

a globalvars test

for a pretty cool reason

when you ran allspam it actually blacklisted two users

but the globalvars test checks to see if the blacklisted user array is empty

>       assert len(GlobalVars.blacklisted_users) == 0
E       assert 2 == 0

angussidney

Woops, should have deleted the blacklisted user pickle/array

quartata

2:41 AM

ok they pass now

although the blacklisted user one was skipped because I had a pickle

angussidney

\o/

quartata

we'll see if it runs on travis, just pushed

@angussidney You can see the asserts I commented out for now: github.com/Charcoal-SE/SmokeDetector/pull/1135/commits/…

angussidney

One of the allspam tests in there may break at some point, it's relying on a low-quality user with many posts but low rep. If they get upvoted at some point it may break, but I've provided instructions on how to fix the issue if it arises

quartata

travis taking its sweet time

angussidney

\o/ it works

quartata

2:48 AM

waiting on the codecov report

went up about 7% if I'm not mistaken

mission_accomplished.jpeg.exe

angussidney

7% from upstream or 7% from last commit?

quartata

went up 7 percentage points from last commit

angussidney

I find codecov's interface so confusing

Anyway, 7% is a pretty good improvement

quartata

upstream coverage is 64% so we're still down 11 percentage points for passing

angussidney

We should be able to increase that more if we add tests for the user whitelist as well, they're basically copy-paste from the blacklisted ones

quartata

2:53 AM

and I need to do chatcommunicate tests

angussidney

Thanks for sorting those tests out

Andy

3:16 AM

@quartata Why are these commented out? Why not fix the underlying problem?

quartata

@Andy fixing them will be very difficult. they're trying to actually submit reports

SmokeDetector

[ SmokeDetector | MS ] Username similar to website in answer: dpkg: warning: files list file for package 'x'missing; assuming package has no files currently installed in ubuntu 16.04 by Fitzeng on stackoverflow.com

quartata

Testing just the invalid cases is better than what we have currently

Now this isn't to say I won't try to make them work eventually, just not right now

Andy

Do they currently try to submit reports?

quartata

Hm?

The one's that aren't commented out error out before hitting the part where it's passed onto spamhandling (by design). It tests things like invalid URLs, more than 5 posts or in the case of allspam users with more than 100 rep

Andy

3:24 AM

Why do they pass currently, and with the code change they do not? What is the difference that makes them try to submit now that is currently being worked around?

quartata

These are all brand new tests

I'm not eliminating existing tests other than the old chatcommunicate tests which aren't compatible

Andy

ah

SmokeDetector

[ SmokeDetector | MS ] Repeating characters in title: unknown error -1=ffffffffffffffff Command /bin/sh failed with exit code 1 by voodoo on stackoverflow.com

Conflicting feedback on unknown error -1=ffffffffffffffff Command /bin/sh failed with exit code 1.

SmokeDetector

3:45 AM

[ SmokeDetector | MS ] Username similar to website in answer: Wordpress + Contact Form 7 + Contact Form 7 Datepicker -Custom validation no custom error message by TECH10MENT on stackoverflow.com

fp- by Makyen on dpkg: warning: files list file for package 'x'missing; assuming package has no files currently installed in ubuntu 16.04 [MS]

naa- by angussidney

SmokeDetector

4:24 AM

[ SmokeDetector | MS ] Bad keyword with a link in answer: What is the user agent string for facebook app from android by Nguyễn Đình Giang on stackoverflow.com

Jamal

sd k

angussidney

@SmokeDetector @AlexanderO'Mara what's up with the TP on this one?

Alexander O'Mara

4:44 AM

@angussidney Sorry, typo

Did the following f not override it?

SmokeDetector

[ SmokeDetector | MS ] URL-only title, bad keyword in body, bad keyword in title, link at end of body, pattern-matching website in body, +1 more: nutritionofhealth.co.uk/claire-hydrafirm-cream/ by jirewase on askubuntu.com

Jamal

sd k

angussidney

@AlexanderO'Mara strange.... it should have

DJMcMayhem

6:05 AM

I just got my first Marshall badge. :)

Jamal

Welcome to the club. :)

angussidney

Nice work!

Next thing you know they'll come flooding in :P

DJMcMayhem

Haha

SmokeDetector

[ SmokeDetector | MS ] Bad NS for domain in body, bad keyword in body, blacklisted website in body, link at end of body, pattern-matching website in body: : \\$1 This weight diminish persuading supplement by Fusent eas on drupal.SE

tpu- by angussidney

[ 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: It has no signs on slim plus garcinia body by Clave1991 on askubuntu.com

tpu- by angussidney

[ SmokeDetector | MS ] Few unique characters in answer, no whitespace in answer: A verb that gives a very close meaning to stubborn? by das on english.SE

tpu- by angussidney

SmokeDetector

6:53 AM

[ SmokeDetector | MS ] Bad NS for domain in body, bad keyword in body, blacklisted website in body, link at end of body, pattern-matching website in body, +1 more: It softens down your fat from within your body. by Whind19 on superuser.com

tpu- by angussidney

SmokeDetector

7:05 AM

[ SmokeDetector | MS ] Bad NS for domain in body, bad keyword in body, blacklisted website in body, link at end of body, pattern-matching website in body: HCA is the vivacious compound by Fohollea on workplace.SE

tpu- by angussidney

SmokeDetector

7:17 AM

Restart: API quota is 15112.

angussidney

!!/queuestatus

sd del-force

DavidPostill

!!/report superuser.com/a/1265599/337631

SmokeDetector

[ SmokeDetector | MS ] Manually reported answer: Reducing Video Play Rate by Neeraj Bhatt on superuser.com

[ SmokeDetector | MS ] Bad NS for domain in body, blacklisted website in body, link at end of body, pattern-matching website in body, potentially bad keyword in body, +1 more: that disregarding the way by Hear ring on drupal.SE

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

DavidPostill

@QPaysTaxes Yes. apowersoft 48 from 65 tp

SmokeDetector

[ SmokeDetector | MS ] Bad keyword in body, bad keyword in title, bad keyword in username, blacklisted website in body, link at end of body: Garcinia Fuel MumyBear by Body Slim Garcinia on askubuntu.com

DavidPostill

7:30 AM

It was posted 2 hours ago. For some reason smokey didn't pick it up.

Nisse Engström

@DavidPostill Member since 2 months with positive reputation on multiple sites...

SmokeDetector

[ SmokeDetector | MS ] Bad NS for domain in body, bad keyword in body, blacklisted website in body, link at end of body, pattern-matching website in body, +1 more: It is a method to stock up healthy by laialbert35 on askubuntu.com

Rob

!!/report stackoverflow.com/a/47119237/563532

Nisse Engström

@SmokeDetector tpu-

SmokeDetector

[ SmokeDetector | MS ] Manually reported answer: Turning off "Language Service Disabled" error message in VS2017 by Fahrettin Ozturk on stackoverflow.com

DavidPostill

7:36 AM

@NisseEngström Yeah, that's why I didn't nuke his account.

SmokeDetector

fp- by DavidPostill

DavidPostill

@SmokeDetector Who reported this?

Rob

@DavidPostill I did

Rants are flaggable, as far as I've seen, as rude/abusive

DavidPostill

@Rob Why? It's not spam. We don't report naa or rude/abusive manually to smokey. If smokey picks it up then by all means flag it.

Rob

If that's the case, I won't report the rude/abusive to smokey anymore. But I'm pretty sure there are regexes specifically to catch such things?

angussidney

7:40 AM

@DavidPostill we do report rude/abusive to smokey

Rob

For example: metasmoke.erwaysoftware.com/post/92075 - marked as 'offensive answer'

angussidney

Might be worth starting a GH conversation on that though, I know a few people have been a bit confused whether r/a is manually reportable

DavidPostill

Fair enough for rude/abusive. But I think that one is naa. How is it rude? Everybody complains about Microsoft :)

SmokeDetector

[ SmokeDetector | MS ] Bad NS for domain in body, bad keyword in body, blacklisted website in body, link at end of body, pattern-matching website in body, +2 more: The fixings in testo rev by Misfor34 on askubuntu.com

tpu- by DavidPostill

angussidney

@DavidPostill TBH I would also flag that one as n

SmokeDetector

7:45 AM

naa- by angussidney on Turning off "Language Service Disabled" error message in VS2017 [MS]

DavidPostill

@SmokeDetector n

Yeah, i changed my fp to naa

Rob

@DavidPostill I've been flagging them as r/a for a while now - haven't had any declined. As I see it, if it's a deliberately an attempt to derail a thread / rant - it's the same as typing gibberish.

Same as I mentioned here: meta.stackoverflow.com/questions/351303/…

DavidPostill

Dec 6 '16 at 22:23, by ArtOfCode

@bwDraco we're not looking for NAAs

Dec 6 '16 at 22:23, by ArtOfCode

any NAA's we happen to catch are incidental, but we're not trying to catch 'em

Rob

Yep - and I disagree that it's NAA - it's abuse in my view :)

DavidPostill

@Rob Fair enough.

angussidney

7:51 AM

I can see why you may call it R/A

But the reason why I said NAA was due to Hanlon's Razor - it's probably someone who thinks SE is a forum and expressing your frustration on such a change is allowed

SmokeDetector

[ SmokeDetector | MS ] Potentially bad keyword in body: Wikipedia API: Converting a Promise to a JSON object using fetch in JavaScript by phao5814 on stackoverflow.com

DavidPostill

@SmokeDetector why

SmokeDetector

@DavidPostill Body - Position 1258-1269: millionaire

fp- by DavidPostill

tpu by Zanna on Garcinia Fuel MumyBear [MS]

[ SmokeDetector | MS ] Blacklisted website in answer: Save Email as MSG file without using Outlook (COM object, etc.) or 3rd party software by Shane will on stackoverflow.com

tpu- by DavidPostill

fp by Zanna on How to create newspaper sections in Drupal 8? [MS]

[ SmokeDetector | MS ] Username similar to website in answer: How to run different website in a same domain with HTTP and HTTPS by Mahi Jat on stackoverflow.com

[ SmokeDetector | MS ] Blacklisted website in answer, blacklisted user: how to convert OST file PST file in java? by Shane will on stackoverflow.com

[ SmokeDetector | MS ] Bad keyword in body, bad keyword in title, blacklisted website in body, pattern-matching product name in body, pattern-matching website in body, +2 more: RLX Male Enhancement and omega (sunflower, walnut, Gra by user262542 on apple.SE

tpu- by angussidney

tpu- by Makyen on how to convert OST file PST file in java? [MS]

Zanna

8:12 AM

related to the above discussion

0

Q: Should rant posts be considered against the "Be Nice" policy ("Rude/Abusive" flaggable)?

This was brought up in another Meta thread about a disagreement over the use of Rude/Abusive flags on rants. In so much as the link for the Rude/Abusive flag points to the Be Nice policy. In my experience, rants are not respectful and ultimately should be considered violations of the Be Nice po...

SmokeDetector

[ SmokeDetector | MS ] Blacklisted website in answer, blacklisted user: Features Redemption use to work with OST files by Shane will on stackoverflow.com

tpu- by Makyen

angussidney

@Zanna there's a good answer on that one

Zanna

yeah I agree with Eliah Kagan

SmokeDetector

tpu- by DavidPostill on How to run different website in a same domain with HTTP and HTTPS [MS]

[ SmokeDetector | MS ] Bad NS for domain in body, blacklisted website in body, link at end of body, pattern-matching website in body, potentially bad keyword in body: In the long run I can work harder by Fountop on workplace.SE

tpu- by DavidPostill

SmokeDetector

8:44 AM

[ SmokeDetector | MS ] Bad NS for domain in body, bad keyword in body, blacklisted website in body, link at end of body, pattern-matching website in body, +3 more: This is a safe testo rev by Onner19 on superuser.com

[ SmokeDetector | MS ] Offensive answer detected: Counting occurrences of a word in a file by You dad on codereview.SE

tpu by Zanna

tpu by Zanna on This is a safe testo rev [MS]

SmokeDetector

9:16 AM

tpu- by rene

[ SmokeDetector | MS ] Bad keyword in body, bad keyword in title, bad keyword in username, blacklisted website in body, link at end of body: Garcinia Fuel MumyBear by Body Slim Garcinia on workplace.SE

tpu- by angussidney

[ SmokeDetector | MS ] Blacklisted website in answer: AdMob: Invoke Firebase method getInstance error by Shristy GC on stackoverflow.com

rene

!!/report meta.stackexchange.com/q/302881/158100

SmokeDetector

[ SmokeDetector | MS ] Manually reported question: REGARDER Thor: Ragnarok Streaming VF Film 2017 1080p by user374433 on meta.SE

[ SmokeDetector | MS ] Bad NS for domain in body, bad keyword in body, bad keyword in title, bad pattern in URL body, blacklisted website in body: Duoslim Forskolin - Directs digestion in the body by user170353 on webapps.SE

rene

9:32 AM

!!/watch watchonmovies\.com

SmokeDetector

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

rene

thanks dear

SmokeDetector

tpu- by rene

tpu- by rene on AdMob: Invoke Firebase method getInstance error [MS]

Nisse Engström

@SmokeDetector tpu-

SmokeDetector

Merged SmokeDetector #1194.

CI on 26fdb4c succeeded. Message contains 'autopull', pulling...

[ SmokeDetector ] SmokeDetector started at rev ad465a8 (metasmoke: Merge pull request #1194 from Charcoal-SE/auto-blacklist-1509874602.5813692) (running on Henders/EC2)

Restart: API quota is 14400.

Anonymous

9:36 AM

@SmokeDetector We all have those days

SmokeDetector

[ SmokeDetector | MS ] Username similar to website in answer: Gravitational time dilation on the surface of a planet by Knowledge Universe Online on physics.SE

tpu- by Mithrandir

Glorfindel

!!/watch knowledgeuniverseonline\.com

SmokeDetector

@Glorfindel Added knowledgeuniverseonline\.com to watchlist

Mithrandir

reminder

SmokeDetector

[ SmokeDetector | MS ] Bad NS for domain in body, bad NS for domain in title, bad keyword in body, bad keyword in title, blacklisted website in body, +6 more: www.7supplements.org/body-boost-garcinia/ by Insch1962 on askubuntu.com

tpu- by Glorfindel

CI on 9ff7f38 succeeded. Message contains 'autopull', pulling...

[ SmokeDetector ] SmokeDetector started at rev 9ff7f38 (SmokeDetector: Auto watch of knowledgeuniverseonline\.com by Glorfindel --autopull) (running on Henders/EC2)

Restart: API quota is 14341.

Mithrandir

9:47 AM

!!/watch farnian\.com

SmokeDetector

@Mithrandir Added farnian\.com to watchlist

Mithrandir

(spammed Scifi.SE, wasn't caught as far as I can tell)

SmokeDetector

CI on 01e2b8e succeeded. Message contains 'autopull', pulling...

[ SmokeDetector ] SmokeDetector started at rev 01e2b8e (SmokeDetector: Auto watch of farnian\.com by Mithrandir --autopull) (running on Henders/EC2)

Restart: API quota is 14328.

SmokeDetector

10:04 AM

[ SmokeDetector | MS ] Bad NS for domain in body, bad keyword in body, blacklisted website in body, link at end of body, pattern-matching website in body, +2 more: on the web. Starting right now, by Dinto haddly on drupal.SE

tpu- by Mithrandir

SmokeDetector

10:15 AM

[ SmokeDetector | MS ] Username similar to website in answer: How to implement responsive web design and its best practices by Animesh Shrivastav on stackoverflow.com

[ SmokeDetector | MS ] Username similar to website in answer: Responsive Web Design by Animesh Shrivastav on stackoverflow.com

[ SmokeDetector | MS ] Username similar to website in answer, blacklisted user: How to run different website in a same domain with HTTP and HTTPS by Mahi Jat on stackoverflow.com

fp- by Glorfindel on How to implement responsive web design and its best practices [MS]

[ SmokeDetector | MS ] Answer similar to existing answer on post: Тесты по JavaScript by Alex Prytkov on ru.stackoverflow.com

fp- by Glorfindel

[ SmokeDetector | MS ] Offensive answer detected: Why did Emilia give Subaru a fake name on the first time they met? by love on anime.SE

tpu- by Mithrandir

Glorfindel

@SmokeDetector How is this similar to the other answer? I don't get it.

@SmokeDetector f

SmokeDetector

fp- by Glorfindel on How to run different website in a same domain with HTTP and HTTPS [MS]

fp by Glorfindel on Best open source rest API Project [MS]

fp by Glorfindel on How do I make a website responsive? [MS]

fp by Glorfindel on Responsive Web Design: What and how? [MS]

fp by Glorfindel on Is reinstalling windows enough for deleting all data? [MS]

fp by Glorfindel on Can't post OpenGraph story: OAuthException, An unknown error has occurred, Test user [MS]

rene

10:32 AM

sorry for the joke comment @bummi but I had to make it ...

Processed Meat

2

Q: How to distinct ham from spam

On my way to familiarize myself with flagging I sometimes have to risk a decline to explore the limits. On that occasion, I stumbled over a border where I would like to ask for clarification. We use the spam flag, among other things, for excessive advertising (I know that strictly speaking, this ...

discussion flags spam self-promotion

rene

That is pretty much in context, thanks

SmokeDetector

[ SmokeDetector | MS ] Bad NS for domain in body, bad keyword in body, blacklisted website in body, link at end of body, pattern-matching website in body, +2 more: It is shown that this settling helps in the advance by Whant19 on superuser.com

tpu- by J F

[ SmokeDetector | MS ] Blacklisted website in body: Extracting mobile and working hours from website using regex by honeyboney on stackoverflow.com

fp- by Mithrandir

tpu- by DavidPostill on How to run different website in a same domain with HTTP and HTTPS [MS]

Conflicting feedback on How to run different website in a same domain with HTTP and HTTPS.

[ SmokeDetector | MS ] Blacklisted username: Google Assistant SDK to integrate it in an app by PUSSY on stackoverflow.com

naa- by Glorfindel

Glorfindel

10:48 AM

@SmokeDetector @DavidPostill a moderator looked at that post and deemed the disclosure was sufficient ...

@Makyen i.stack.imgur.com/uaREU.png — Bhargav Rao ♦ 2 hours ago

DavidPostill

@Glorfindel Huh. Well it's NAA then ...

SmokeDetector

[ SmokeDetector | MS ] Few unique characters in body, no whitespace in body, repeating characters in body: How to create a new value our of the number of rows times another value by Stefan Gaillard on stackoverflow.com

tp- by J F

@SmokeDetector v

sd - v

@SmokeDetector n

10:52 AM

@rene perfectly fine :), like it

rene

OK

Glorfindel

@DavidPostill yeah, that's probably better.

Transcript for