00:00:27 <Eksell> Unstable branch on CRAWL.XTAHUA.COM updated to: 0.23-a0-458-g93a6ea9 (34)
00:26:34 <gammafunk> floraline: I see that of dev accounts, the following are registered if you want to make them dgl admins: advil bh ebering elliptic gammafunk Lasty MarvinPA
00:27:19 <gammafunk> I'm sure a few more active devs will be registering soon
00:28:14 <gammafunk> and there's one last bit of official server setup you could consider: setting up an IRC bot to announce rebuilds and possibly milestones; for the former it needs to join ##crawl-dev and for the latter you'd need have it use a registered nick and join ##crawl
00:28:18 <gammafunk> ??henzell
00:28:19 <Sequell> henzell[1/1]: Linley Henzell is the original creator of Crawl. "Henzell" is also the {CAO} announcement bot. Based on {Sizzell}, which is based on {Gretell}. Uses ! as its command prefix. Note: non-announce functions now in {Sequell}.
00:28:27 <gammafunk> ??sizzell
00:28:28 <Sequell> sizzell[1/2]: Like Gretell, except that |amethyst is typing everything himself. Announces games and upgrades on crawl.s-z.org ({CSZO}). Uses "%" for its commands ("%dump" etc), but does not do nick mapping.
00:28:31 <gammafunk> ??sizzell[2
00:28:32 <Sequell> sizzell[2/2]: Source at https://github.com/crawl/sizzell
00:28:40 <gammafunk> I believe that's the bot most dgl based servers use
00:29:15 <gammafunk> hrm, does that support sasl
00:30:22 <gammafunk> never mind, I guess it doesn't
00:31:12 <gammafunk> until we give sizzell sasl support, guess it can only announce rebuilds, which is not so important
00:40:36 <alexjurkiewicz> floraline: thanks
00:47:17 <floraline> gammafunk: those users are now dgl admin
00:47:40 <gammafunk> floraline: thanks
01:27:35 <Gretell> Unstable branch on crawl.develz.org updated to: 0.23-a0-458-g93a6ea9 (34)
01:58:21 <Gretell> Windows builds of master branch on crawl.develz.org updated to: 0.23-a0-458-g93a6ea9
02:58:02 <Gretell> Monster database of master branch on crawl.develz.org updated to: 0.23-a0-458-g93a6ea9
05:52:24 <Henzell> acidosis1 (L9 GnSk) ERROR in 'tileweb.cc' at line 219: Socket write error: Resource temporarily unavailable (D:7)
06:41:15 <Henzell> Devilfish (L15 HuWn) Crash caused by signal #15: Terminated (Swamp:2)
07:32:40 <ebering> !tell gammafunk thanks for the eyes on the ely patch
07:32:40 <Sequell> ebering: OK, I'll let gammafunk know.
07:32:50 <ebering> Yermak: thanks for the catch
07:36:37 <ebering> !tell floraline thanks for hosting the experimental!
07:36:38 <Sequell> ebering: OK, I'll let floraline know.
09:57:54 <Cheibriados> 03ebering02 07https://github.com/crawl/crawl/pull/909 * 0.23-a0-459-g8fbb052: fixup! Make ghost demons immune to pacification 10(2 hours ago, 2 files, 3+ 2-) 13https://github.com/crawl/crawl/commit/8fbb052cfdcc
09:57:54 <Cheibriados> 03ebering02 07https://github.com/crawl/crawl/pull/909 * 0.23-a0-460-gea0a806: fixup! Simplify Heal Other formula 10(2 hours ago, 1 file, 0+ 4-) 13https://github.com/crawl/crawl/commit/ea0a806e10ae
09:57:54 <Cheibriados> 03ebering02 07https://github.com/crawl/crawl/pull/909 * 0.23-a0-461-g30c0990: fixup! Display pacification chance in Heal Other targeter 10(2 hours ago, 1 file, 10+ 2-) 13https://github.com/crawl/crawl/commit/30c099051ba0
09:57:54 <Cheibriados> 03ebering02 07https://github.com/crawl/crawl/pull/909 * 0.23-a0-462-g512f561: fixup! fixup! Make ghost demons immune to pacification 10(85 minutes ago, 1 file, 1+ 1-) 13https://github.com/crawl/crawl/commit/512f561b9c66
09:57:54 <Cheibriados> 03ebering02 07https://github.com/crawl/crawl/pull/909 * 0.23-a0-463-g7f85caa: fixup! fixup! Simplify Heal Other formula 10(83 minutes ago, 1 file, 1+ 2-) 13https://github.com/crawl/crawl/commit/7f85caa964ad
11:16:20 <Cheibriados> 03stenella02 {ebering} 07* 0.23-a0-459-ge787970: Rest the Ancestor 10(2 weeks ago, 11 files, 62+ 9-) 13https://github.com/crawl/crawl/commit/e787970c0630
11:20:14 <ebering> Yermak: ^ you might like this change
11:28:24 <Yermak> ebering, yep, we discissed it with switch__
11:59:03 <Henzell> Nino (L4 HuWn) Crash caused by signal #15: Terminated (D:2)
12:10:41 <Jorgrell> Unstable branch on crawl.jorgrun.rocks updated to: 0.23-a0-459-ge787970c06 (34)
12:27:02 <ebering> advil: did you take down lua.dcss.xyz on purpose?
12:27:16 <advil> ebering it should be moved to lua.dcss.io
12:27:19 <advil> er
12:27:31 <advil> http://doc.dcss.io/
12:27:38 <advil> I still haven't updated it recently
12:27:59 <advil> the .xyz domain name is flagged as spam by basically everything
12:28:06 <advil> it's blocked in entirety on my campus even
12:29:17 <ebering> oh
12:29:21 <ebering> I see
12:29:30 <Henzell> Unstable branch on crawl.akrasiac.org updated to: 0.23-a0-459-ge787970 (34)
12:29:36 <advil> so I'm moving what little I had there to the io domain name
12:29:38 <ebering> it would be cool if you could make it auto update
12:29:41 <advil> eventually I need to move the private list too
12:30:50 <advil> yeah, probably a good idea...I don't have too much cpu on that server so I just need to make sure it's not too painful to do
12:30:53 <ebering> I'm writing a trunk updates blog post, is it stable enough to link to from the cdo homepage?
12:30:56 <advil> yes
12:34:51 <advil> !tell gammafunk yeah, there's still a trickle of constriction crashes going on, haven't kept good track of them but it's the usual stuff, summons and penetration javelins or whatever
12:34:52 <Sequell> advil: OK, I'll let gammafunk know.
12:35:30 <advil> (I'm slightly on a crawl break right now if that hasn't been apparent)
12:38:16 <ebering> no worries, just don't want to make a link that'll die in a day
12:38:24 <ebering> or slam you with unexpected traffic
12:41:29 <advil> traffic to static stuff should be fine, I have unlimited bandwidth or something
12:41:45 <advil> or "unlimited" bandwidth, maybe
15:40:21 <gammafunk> advil: I added some checks for aliveness to constriction damage that should handle those cases, but I wasn't able to replicate the one crash I saw. If find any recreatable constriction-related crash, let me know
15:40:30 <gammafunk> !gitgrep 1 constriction
15:40:30 <Sequell> %git HEAD^{/constriction}
15:40:30 <Cheibriados> 07gammafunk02 * 0.23-a0-457-g6f6daef: Add more checks for invalid constrictions 10(20 hours ago, 1 file, 7+ 5-) 13https://github.com/crawl/crawl/commit/6f6daefbf7df
15:41:24 <gammafunk> ebering: Yeah, I was planning on writing one after I got new labs merged
15:41:47 <gammafunk> the trunk updates post, that is
15:42:18 <gammafunk> it's ok to post one before I guess, but if we do make one, we need the changelog updated
15:42:49 <gammafunk> I usually just do something like git log --oneline 0.23-a0..HEAD
15:44:03 <gammafunk> and go through the list. We don't add absolutely every change, only things that have significant gameplay impact or that are important new features on the UI/configuration side of things
16:38:08 <ebering> gammafunk: oh, couldn't remember if PF was updating the changelog in concert when he was doing them more frequently
16:38:18 <ebering> anyhow, changelog update pending
16:43:12 <ebering> %git HEAD
16:43:12 <Cheibriados> 07stenella02 {ebering} * 0.23-a0-459-ge787970: Rest the Ancestor 10(2 weeks ago, 11 files, 62+ 9-) 13https://github.com/crawl/crawl/commit/e787970c0630
16:43:41 <gammafunk> ebering: yeah, it's good to have it updated at the same time you do the post regardless
16:43:54 <gammafunk> especially since to make the post you have to effectively look through the same commits
16:44:06 <gammafunk> but also just so people checking the changelog have something current
16:44:21 <gammafunk> it's also fine to just update the changelog without making a post
16:44:27 <gammafunk> we're just usually somewhat lazy about that
16:45:50 <ebering> yea, I'd like to be a bit less lazy about it
16:45:55 <ebering> we'll see if I stick to it though
16:45:57 <ebering> no promises
16:47:23 <gammafunk> it doesn't need to be as frequently as a weekly post; even monthly or every other month would be fine, and the same person doesn't have to do it every time either
16:47:41 <ebering> for sure
16:49:09 <gammafunk> |amethyst: would adding CKO to the CAO scoring pages necessitate anything big (like a rebuild of the database), and assuming not, would you have time to add CKO to scoring in the near term?
16:49:42 <gammafunk> I was going to make an announcement post about CKO soon since it's otherwise set up
16:50:32 <Cheibriados> 03ebering02 07* 0.23-a0-460-g9be7e05: changelog through 0.23-a0-459-ge787970 10(4 minutes ago, 1 file, 50+ 0-) 13https://github.com/crawl/crawl/commit/9be7e05e39b5
16:50:32 <Cheibriados> 03ebering02 07* 0.23-a0-461-gff60f68: checkwhite INSTALL.txt 10(3 minutes ago, 1 file, 25+ 25-) 13https://github.com/crawl/crawl/commit/ff60f6859ba1
17:00:51 <gammafunk> I have a few changelog tweaks I'll push
17:00:53 <gammafunk> I probably wouldn't add the wanderer equipment announcement to the the changelog, since that's a pretty minor UI thing
17:00:54 <ebering> gammafunk: I have some vague ideas about newlab layouts, what sort of dimensions are you thinking
17:01:19 <Cheibriados> 03ebering02 07https://github.com/crawl/crawl/pull/909 * 0.23-a0-462-g4994b30: Change pacification to depend only on monster type 10(28 hours ago, 1 file, 6+ 6-) 13https://github.com/crawl/crawl/commit/4994b305688e
17:01:19 <Cheibriados> 03ebering02 07https://github.com/crawl/crawl/pull/909 * 0.23-a0-463-g4abe239: Simplify Heal Other formula 10(24 hours ago, 3 files, 31+ 31-) 13https://github.com/crawl/crawl/commit/4abe2390e611
17:01:19 <Cheibriados> 03ebering02 07https://github.com/crawl/crawl/pull/909 * 0.23-a0-464-ga614697: Display pacification chance in Heal Other targeter 10(23 hours ago, 1 file, 38+ 11-) 13https://github.com/crawl/crawl/commit/a614697e851e
17:01:19 <Cheibriados> 03ebering02 07https://github.com/crawl/crawl/pull/909 * 0.23-a0-465-g4e30461: Make player ghosts and pan lords immune to pacification 10(22 hours ago, 2 files, 15+ 5-) 13https://github.com/crawl/crawl/commit/4e30461676fc
17:01:19 <Cheibriados> New branch created: pull/910 (2 commits) 13https://github.com/crawl/crawl/pull/910
17:01:20 <Cheibriados> 03floraline02 07https://github.com/crawl/crawl/pull/910 * 0.23-a0-460-g45c1e20: Add webtiles form to change user email 10(2 hours ago, 4 files, 90+ 0-) 13https://github.com/crawl/crawl/commit/45c1e2016c58
17:01:20 <Cheibriados> 03floraline02 07https://github.com/crawl/crawl/pull/910 * 0.23-a0-461-g07a5185: Make webtiles email change form more user friendly 10(15 minutes ago, 5 files, 53+ 11-) 13https://github.com/crawl/crawl/commit/07a5185bc6cc
17:01:22 <gammafunk> have you seen the initial one I'm using to develop the portal?
17:01:22 <ebering> gammafunk: pretty minor but I got asked about it 20 times during csdc week 1
17:02:15 <gammafunk> ebering: yeah, I'm not taking it out, but it's the sort of thing I've seen removed in the past from changelogs (there's not exactly consensus on what's changelog worthy anyhow)
17:02:16 <ebering> gammafunk: yes but I lost the link and it's not on the wiki page for it
17:02:36 <gammafunk> https://cdn.discordapp.com/attachments/205316046230388737/514663213833388061/unknown.png
17:02:44 <gammafunk> bottom part is a bit cut off
17:02:57 <gammafunk> it's not so much dimensions as it is the approach of the layout
17:03:23 <gammafunk> the basic idea I have in mind is to have a set of reusable (within the same map) arenas and a bit set of monster encounter sets
17:03:30 <gammafunk> divided into two tiers (currently)
17:03:36 <gammafunk> easier/harder
17:03:50 <ebering> ok, my vague ideas aren't too of base
17:03:58 <ebering> I'll try to send you some sketches in a bit
17:04:00 <gammafunk> and those subvaults you see are going to be arenas presenting a choice
17:04:14 <gammafunk> there is one big issue to consider with teleport that I wanted to ask MPA about as well
17:04:31 <gammafunk> but I'll probably make a fuller wiki writeup after I have my basic map 100% working and reasonable
17:04:45 <gammafunk> I have some changelog tweaks I'll push in next 15 mins
17:05:12 <ebering> sweet
17:10:43 <Jorgrell> Unstable branch on crawl.jorgrun.rocks updated to: 0.23-a0-461-gff60f6859b (34)
17:12:30 <gammafunk> ebering: shockingly you left out your most impactful change so far, monster equipment identification!
17:13:09 <ebering> oh oops
17:13:13 <ebering> that one made the blog post
17:13:24 <ebering> which I edited into the change log
17:14:48 <gammafunk> ok I can add then
17:15:28 <gammafunk> wow
17:15:54 <gammafunk> "Wand charges have been reduced, according to the following *schedule* of average charges per wand"
17:16:05 <gammafunk> schedule...this must be canadian english
17:16:12 <geekosaur> more or less, yes
17:16:23 <gammafunk> first they force me to write armour and jewellery all the time and now this
17:17:51 <gammafunk> I randomly write "armour" or "colour" instead of armor or color all the time now, and it's crawl's fault
17:35:27 <switch__> same
17:35:38 <switch__> i don't even remember how to spell jewellery correctly
17:46:54 <minmay> at least you can pronounce it
17:48:29 <alexjurkiewicz> gammafunk: that's a nice syntax for DES,did you write it?
17:49:36 <gammafunk> alexjurkiewicz: oh, you mean the vim highlighter? That's in the repo; originally made by greensnark with some recent modifications from emtedronai and myself
17:49:42 <gammafunk> and I have another local tweak I've not pushed
17:50:28 <gammafunk> dat/vim/syntax/levdes.vim I think
17:51:14 <gammafunk> not really sure why that file resides in those subdirectories when it's the only vim file in the repo, seems like it might belong more in docs/develop/levels
17:51:47 <gammafunk> it's not perfect, unfortunately, but it does make des maps easier to read in vim
17:59:19 <gammafunk> floraline: So if I wanted to just change my password via webtiles when I already know it, do I have to use the "Forgot Password" link to do that?
18:02:13 <floraline> yes that would work
18:02:13 <Sequell> floraline: You have 3 messages. Use !messages to read them.
18:02:25 <floraline> i'm planning on adding a password change dialog anyway
18:02:42 <floraline> !messages
18:02:43 <Sequell> (1/3) minmay said (1d 20h 36m 12s ago): how do you feel about giving me the "minmay" account on cko? whoever registered it and suicided a game on it (presumably to break my streak) is not me
18:02:52 <floraline> !messages
18:02:52 <Sequell> (1/2) bh said (1d 20h 33m 22s ago): can you register me 'bh'? the server won't let me
18:02:54 <floraline> !messages
18:02:55 <Sequell> (1/1) ebering said (10h 26m 16s ago): thanks for hosting the experimental!
18:05:03 <floraline> the forgot password feature needs to be tweaked to support cases where multiple users have the same email
18:05:25 <floraline> i'm thinkin the email will just contain multiple reset links for each username and you can just choose which ones to click on
18:05:33 <ebering> floraline: that's not good
18:05:45 <floraline> what's not good?
18:05:52 <ebering> oh, I guess the attack I'm imagining doesn't work
18:06:09 <ebering> where user a intentionally makes their e-mail address one they know belongs to user b
18:06:13 <floraline> yeah as far as i can tell all you can do right now is prevent someone from receiving their reset email
18:06:20 <floraline> which is not that big of a deal
18:10:36 <floraline> i think we also need to start sending confirmation emails
18:10:42 <Cheibriados> 03ebering02 07* 0.23-a0-462-g7c42f86: Nerf the acid effect of minor destruction 10(3 hours ago, 1 file, 1+ 1-) 13https://github.com/crawl/crawl/commit/7c42f86923ab
18:13:08 -!- MarvinPA_ is now known as MarvinPA
18:16:13 <gammafunk> floraline: confirmation emails before the registration is completed? that sounds like a nice idea
18:20:54 <alexjurkiewicz> ??cko
18:20:55 <Sequell> cko[1/1]: Crawl server owned by floraline hosted in NY, USA. Webtiles at https://crawl.kelbi.org:8081/#lobby or console via ssh to crawl.kelbi.org with port 22 and username 'terminal' and using CAO keys (found at https://crawl.kelbi.org/crawl/).
18:21:41 <Lantell> Unstable branch on underhound.eu updated to: 0.23-a0-461-gff60f6859b (34)
18:35:19 <switch__> it would allow you to auth via email ownership, which sort of solves the theft problem
18:35:34 <switch__> of course, it doesn't do anything retroactively
18:42:51 <floraline> switch__: it wouldn't solve username theft, you could still sign up with your own email or with no email
18:43:32 <floraline> if we want to continue to allow sign ups with no email, which i think we ought to, then all we would really get by having confirmation emails is avoiding typos
18:43:34 <switch__> well, it gives you a contact point
18:43:50 <switch__> rather than nothing at all
18:44:05 <switch__> if you have to resolve a dispute, for example
18:44:41 <switch__> (which i think is possible, if you have the time of account creation, though this can be problematic)
18:45:00 <switch__> but it's not something you can even attempt if you have to way to talk to the account owner
18:45:04 <switch__> no*
18:46:13 <geekosaur> might cut down on the griefer accounts too
18:46:33 <switch__> i don't think you need to allow signups without an email
18:46:39 <switch__> basically nobody else on the internet does it
18:47:31 <switch__> it's something i personally enjoy when i come across it, but being able to solve problems like account theft are more important than my 15 seconds of convenience, since my account names don't get stolen
18:55:21 <ebering> this is yak shaving
18:55:29 <ebering> the solution to account theft is single sign on
19:08:17 <switch__> you can step into SSO much easier if you have an email account to base ownership off of
19:10:40 <Jorgrell> Unstable branch on crawl.jorgrun.rocks updated to: 0.23-a0-462-g7c42f86923 (34)
19:14:37 <ebering> %git 8c0b41491
19:14:37 <Cheibriados> 07gammafunk02 * 0.23-a0-420-g8c0b414: Reveal equipment for monsters behind glass 10(12 days ago, 1 file, 3+ 0-) 13https://github.com/crawl/crawl/commit/8c0b41491556
19:14:59 <ebering> !bug 11724
19:15:00 <Sequell> https://crawl.develz.org/mantis/view.php?id=11724
19:15:01 <ebering> also fixed that
19:18:58 <switch__> oh, by the way
19:19:01 <switch__> thanks for fixing up my PR
19:19:05 <switch__> <3
19:28:46 <Cheibriados> 03gammafunk02 07* 0.23-a0-463-gabef753: Some changelog updates (now through 0.23-a0-462-g7c42f869) 10(24 minutes ago, 1 file, 37+ 18-) 13https://github.com/crawl/crawl/commit/abef753f2456
19:35:40 <gammafunk> ebering: thanks, I saw that bug but wasn't sure if it was the same issue
19:37:11 <ebering> indeed it was
20:00:31 <Cheibriados> 03ebering02 07* 0.23-a0-464-g7bc77c8: Relocate some on-sight logic to the correct place 10(48 minutes ago, 3 files, 4+ 5-) 13https://github.com/crawl/crawl/commit/7bc77c830af5
20:00:31 <Cheibriados> 03ebering02 07* 0.23-a0-465-g8f93ed4: Prevent invalid_monster messages when converting 10(5 minutes ago, 1 file, 7+ 0-) 13https://github.com/crawl/crawl/commit/8f93ed435fec
20:03:12 <Cheibriados> 03ebering02 07* 0.23-a0-466-gb59e870: Change pacification to depend only on monster type 10(31 hours ago, 1 file, 6+ 6-) 13https://github.com/crawl/crawl/commit/b59e870becd6
20:03:12 <Cheibriados> 03ebering02 07* 0.23-a0-467-gcf71bd9: Simplify Heal Other formula 10(27 hours ago, 3 files, 31+ 31-) 13https://github.com/crawl/crawl/commit/cf71bd98f681
20:03:12 <Cheibriados> 03ebering02 07* 0.23-a0-468-g28a1794: Display pacification chance in Heal Other targeter 10(26 hours ago, 1 file, 38+ 11-) 13https://github.com/crawl/crawl/commit/28a179404a73
20:03:12 <Cheibriados> 03ebering02 07* 0.23-a0-469-gb5b494a: Make player ghosts and pan lords immune to pacification 10(25 hours ago, 2 files, 15+ 5-) 13https://github.com/crawl/crawl/commit/b5b494a93fe6
20:03:40 <ebering> gotta get that change log out of date asap
20:10:42 <Jorgrell> Unstable branch on crawl.jorgrun.rocks updated to: 0.23-a0-463-gabef753f24 (34)
20:11:53 <ebering> %git e122d05
20:11:53 <Cheibriados> 07gammafunk02 * 0.23-a0-344-ge122d05: Remove an unused variable 10(5 weeks ago, 1 file, 1+ 2-) 13https://github.com/crawl/crawl/commit/e122d056520d
20:12:31 <gammafunk> ebering: that view_monster_equipement() call was left there specifically because without it, the wrong equipement message is printed for autoexplore
20:12:39 <ebering> oh
20:12:44 <gammafunk> the message happens before the call to seen_monster()
20:12:56 <gammafunk> which could possibly be refactored but that's why I had to leave it there
20:13:00 <ebering> I see
20:14:39 <ebering> oh, it's in the commit message
20:14:59 <gammafunk> yeah, definitely something that should be refactored to avoid an extra loop over monsters if possible
20:15:24 <gammafunk> but the simple solution for now is to leave that call in, otherwise you get messages about "It is wielding a runed longsword" etc
20:15:42 <gammafunk> and then the item is properly ided after that
20:16:40 <ebering> yea I'm putting it back in with an XXX comment explaining why it's there
20:17:21 <gammafunk> oh yeah, I should have added said comment in the first place
20:18:35 <ebering> do_conversions can move though, since if the attitude change happens that gets its own message
20:20:55 <Cheibriados> 03ebering02 07* 0.23-a0-470-g6f7cfd9: Unrelocate some on-sight logic (gammafunk) 10(3 minutes ago, 1 file, 5+ 0-) 13https://github.com/crawl/crawl/commit/6f7cfd97396d
20:48:22 <switch__> https://github.com/crawl/crawl/blob/07a5185bc6cce5624617398a03242a237b0687bd/crawl-ref/source/webserver/util.py#L137 is it just me or is this a strange regex
20:49:05 <amalloy> that looks like a pretty normal "anything that could plausibly be an email address" regex
20:49:16 <ebering> well it's definitely not rfc-nnnn compliant checking of email address validity
20:49:27 <amalloy> right, it doesn't support quoting
20:50:13 <switch__> why are there so many @ signs. have i forgotten regex?
20:50:29 <ebering> switch__: [^@] matches all characters except @
20:51:09 <switch__> oic
20:51:30 <Naruni> Hi
20:51:46 <switch__> hello
20:51:57 <Naruni> Has the conversation ever come up about a cloud being dissipated instantly when a door is closed on it?
20:52:13 <switch__> not that i've seen, why?
20:52:21 <Naruni> I just think it's kind of odd
20:52:58 <Naruni> I can cast a very high powered conjure flame (so it will have a long duration) then shut the door and poof it's gone
20:52:58 <ebering> it's in the "odd edge case but the alternative is worse" file
20:53:06 <switch__> have you ever closed the door on a cloud of flame in real life?
20:53:27 <Naruni> Not flame, but lead-acid gas, yes.
20:53:51 <switch__> well then i'm sure you understand that once the door shuts, the hazardous gas is gone forever
20:54:11 <switch__> i'm pretty sure it was a programming choice to avoid a difficult to manage case
20:55:08 <switch__> that regex really should be changed to sanitize properly
20:55:19 <switch__> for fear of SQLi
20:55:21 <Naruni> ebering, I could write a function to have the door randomly place clouds on opposite sides of the door of the same type, with lower power
20:55:52 <switch__> i'm sure that patch would be welcome, but the cloud code is fairly complicated
20:55:56 <Naruni> to simulate the dispersal of the cloud.  I think it would be pretty cool feature
20:56:19 <Naruni> switch__, I've worked with the cloud code before I don't think it's too cosmic
20:56:36 <Naruni> Just wanted to get a feel of what the concensus is before I put work on it
20:56:41 <gammafunk> Naruni: I don't think that's a good idea, seems very spoilery
20:57:18 <gammafunk> and likely to induce players into some pretty weird antics with manipulating doors and clouds
20:57:20 <Naruni> gammafunk, I tried thinking about that, but then I remembered a patch I put in years ago about clouds dissipating 4x faster when OOS
20:58:19 <floraline> switch__: the webtiles email regex is extremely hands-off, the best next step to properly validate emails would be to send a confirmation mail, that's the only way we would know for sure that they entered a correct and valid address
20:58:19 <Naruni> if closing the door generates clouds 'created by' the actor who closed it, the cloud OOS would pretty much instantly be gone, hence the much lower power when re-creating the cloud
20:58:20 <floraline> and python should be helping us prevent sql injection with its parameter substitution
20:58:22 <switch__> floraline, i'm more concerned about security than i am about invalid emails
20:58:27 <amalloy> switch__: if the database code were vulnerable to sql injection (i don't know whether it is or not), that regex would be the wrong place to fix it
20:58:58 <switch__> i didn't see sanitizing anywhere else, but i didn't look very hard
20:59:14 <floraline> it's my understanding that the parameter substitution is enough to prevent sql injection
20:59:16 <amalloy> there shoudn't be any sanitizing
20:59:16 <gammafunk> Naruni: It's not a queston of how quickly the cloud dissipates but one of why we would want this more complicated behaviour in the first place
20:59:21 <ebering> Naruni: clouds now disappear immediately out of LOS
20:59:37 <amalloy> like floraline said, if you just use your DB client properly then silly text tricks don't matter
20:59:37 <switch__> then disregard my concern, floraline, i'm unfamiliar
20:59:46 <floraline> https://github.com/crawl/crawl/blob/master/crawl-ref/source/webserver/userdb.py#L181
20:59:47 <Naruni> ebering, ah, thanks for letting me know
21:00:13 <Naruni> gammafunk, I mentioned earlier the example of my conjure flame.  It just seems quirky
21:00:28 <Naruni> If you guys aren't concerned about it, I can learn to live with it :)
21:00:34 <gammafunk> using doors to move clouds seems very quirky to me
21:00:58 <amalloy> switch__: see https://github.com/crawl/crawl/blob/a4718cdc7d3ae8ad2e96ae79249df51e9345b10f/crawl-ref/source/webserver/userdb.py#L181
21:00:59 <Naruni> hah well different strokes for different folks
21:01:09 <amalloy> it's inserted correctly, using a pameterized query
21:01:14 <switch__> gotcha
21:08:26 <gammafunk> MarvinPA: Regarding our conversation of sac love the other day, for phial of floods and sack of spiders, how do you feel about simply not creating summons for those two and just having them apply the flood beam and net, respectively.
21:08:57 <gammafunk> And then for the other pure summoning evokers we can actually disallow them. Likewise we could forbid tukima's dance.
21:09:00 <minmay> sack of nets
21:09:21 <gammafunk> yeah it's a bit weird; obviously we could just forbid their use at all
21:10:46 <Jorgrell> Unstable branch on crawl.jorgrun.rocks updated to: 0.23-a0-470-g6f7cfd9739 (34)
21:10:53 <minmay> then you could rename sac love to sac sack
21:11:35 <minmay> does sack of spiders still leave behind permanent webs?
21:11:55 <gammafunk> don't think so
21:12:00 <gammafunk> since it only targets monsters with them
21:12:09 <gammafunk> and the webs gets destroyed when they break free
21:13:31 <minmay> good, if it did you'd want to empty it before sacrificing so that you had the web traps to lure monsters back to
21:17:41 <minmay> I guess that happens with digging and random effects with sac artifice too
21:29:52 <Cheibriados> Level/staircase/vault generation bug in depths 13https://crawl.develz.org/mantis/view.php?id=11772 by Frosteey
21:34:21 <travis-ci> The build passed. (master - b5b494a #10629 : Edgar A. Bering IV): https://travis-ci.org/crawl/crawl/builds/459554781
21:34:47 <ebering> thanks travis
21:42:39 <gammafunk> said you passed but didn't give you a grade
21:43:11 <ebering> it's crawl, the grade is always AAA
21:43:18 <gammafunk> dang
21:52:57 <ebering> do we have an rnd fund???? https://www.amazon.com/Mazes-Labyrinths-Their-History-Development/dp/048622614X/
21:53:09 <ebering> (actually, that book is probably out of copyright)
21:56:06 <ebering> indeed
21:56:08 <ebering> https://www.biodiversitylibrary.org//bibliography/16552#/summary
22:18:45 <alexjurkiewicz> lol
22:27:16 <travis-ci> The build passed. (master - 6f7cfd9 #10630 : Edgar A. Bering IV): https://travis-ci.org/crawl/crawl/builds/459557763