kaberett: A sleeping koalasheep (Avatar: the Last Airbender), with the dreamwidth logo above. (dreamkoalasheep)
kaberett ([personal profile] kaberett) wrote in [site community profile] dw_dev2015-08-30 02:35 pm
Entry tags:

Code tour, 13th August-30th August

All live in last night's code push! A lot of these relate to developers only, and the rest are fairly low-profile so you might not notice them, but nonetheless this is what's been being worked on. :-)

Issue 1354: Update links in automated emails to point to HTTPS (pull request)
Category: HTTPS Everywhere v1
Patch by: [github.com profile] kaberett
Description: Notification e-mails were set to (almost) always go out with links to entries, threads, comments and the like using the HTTP protocol, not the more secure HTTPS. Dreamwidth is in the process of moving over to using HTTPS everywhere; now when the switch is flipped notification e-mails will reflect this. Additionally and furthermore, for reasons that surpass understanding, the code that determined whether notification e-mails linked to https://kaberett.dreamwidth.org or http://kaberett.dreamwidth.org was in a file that otherwise deals with journal layouts, rather than anywhere at all more sensible. It's moved house to the module that handles stuff to do with user account objects instead, where people might expect ever to find it.

Issue 1522: antispam: make spam report sortable (pull request)
Category:
Patch by: [github.com profile] kareila
Description: Adds the ability to sort spam reports by the date the spam was left, as well as by the date that it was reported. In the case of accounts that were compromised (e.g. via a leaked e-mail token...) and then resecured, it could be the case that spam left was only noticed and reported after the account was resecured. But if the spam was left before the account was resecured it's not a problem and doesn't indicate that security's been compromised again... and it's nice to be able to check this automatically instead of having to go through every report manually. And we like things that make Antispam's job easier, oh yes we do.

Issue 1527: DW-hosted image permissions interact poorly with custom access filters (pull request)
Category:
Patch by: [github.com profile] kareila
Description: Dreamwidth provides (still pretty basic) image hosting, and one of the reasons it's still very firmly in beta is that, as it turns out, weird stuff was going on with access levels. Problem the first: if you tried to upload an image with security "public" while your default security setting was "access lock", uploads would hang forever. Problem the second: if you had an image with visibility set to "access only" and you included it in a post to a custom access filter, you were the only person who could see it. Neither of those things is true any more!

Issue 1543: RTE fails with 'Error: invalid user or site' when trying to parse hyphenated github.com users (pull request)
Category:
Patch by: [github.com profile] kareila
Description: Github usernames use hyphens not underscores. Dreamwidth, or at least the RTE, assumed that usernames used underscores only and never hyphens - and therefore popped up an error if you tried to post via the RTE including a link to, say, [github.com profile] me-and (if you look at the last code tour it will become rapidly apparent why this was noticed...). This was a pity, and the RTE no longer assumes it knows better than target sites about which characters are permitted in usernames!

Issue 1551: Embedded fonts from Google blocked for being non-secure (pull request)
Category: HTTPS Everywhere v1
Patch by: [github.com profile] zorkian
Description: Dreamwidth was pulling Google fonts from the http not the https version of the Google URL, resulting in warnings on https pages that some content was http (and therefore less secure). All that was required to make the warnings go away was switching to using the https URLs by default... so that's what's now happening.

Issue 1553: fix failing test in wtf.t (pull request)
Category:
Patch by: [github.com profile] kareila
Description: Dev side! Tests are designed to demonstrate that you've not broken anything new and exciting by writing the bit of code you've just written, so it's good if they don't fail by default. This test now exhibit the desired behaviour so it's easier to tell whether things are working as they ought.

Issue 1558: fixes for atom-post.t and post.t (pull request)
Category:
Patch by: [github.com profile] kareila
Description: And another couple of fixes to tests!

7 total issues resolved
Contributors: [github.com profile] kaberett, [github.com profile] kareila, [github.com profile] zorkian
azurelunatic: A glittery black pin badge with a blue holographic star in the middle. (Default)

[personal profile] azurelunatic 2015-08-30 06:52 pm (UTC)(link)
Yay code tour!

*waits for spam to look at sorting*
musyc: Silver flute resting diagonally across sheet music (Default)

[personal profile] musyc 2015-08-30 07:34 pm (UTC)(link)
Woo code-tour! And yay for what looks like a lot of backend stuff. XD
brainwane: My smiling face in front of a brick wall, May 2015. (Default)

Yay!

[personal profile] brainwane 2015-08-30 09:17 pm (UTC)(link)
Thanks especially for the Update links in automated emails to point to HTTPS fix! You know I love the HTTPS work.

What's RTE?
woggy: Dom (from the webcomic Megatokyo) talking on a phone (Dom)

Re: Yay!

[personal profile] woggy 2015-08-30 09:31 pm (UTC)(link)
Rich Text Editor

*makes warding gestures*
marahmarie: Sheep go to heaven, goats go to hell (Default)

Re: Yay!

[personal profile] marahmarie 2015-09-06 02:02 am (UTC)(link)
What are warding gestures?
azurelunatic: A glittery black pin badge with a blue holographic star in the middle. (Default)

Re: Yay!

[personal profile] azurelunatic 2015-09-06 06:07 am (UTC)(link)
It ranges from mere "get it away from me" miming to the more formal gestures usually only brought out for a vampire, demon, or broccoli.
marahmarie: Sheep go to heaven, goats go to hell (Default)

Re: Yay!

[personal profile] marahmarie 2015-09-07 02:35 am (UTC)(link)
Ah, like 'warding off'! Got it. :)
marahmarie: Sheep go to heaven, goats go to hell (Default)

Re: Yay! Oh, and...

[personal profile] marahmarie 2015-09-07 02:36 am (UTC)(link)
"Broccoli" lol

(Cauliflower, for me...)
azurelunatic: A glittery black pin badge with a blue holographic star in the middle. (Default)

Re: Yay! Oh, and...

[personal profile] azurelunatic 2015-09-07 03:05 am (UTC)(link)

Overcooked broccoli.

sgsabbage: (Default)

[personal profile] sgsabbage 2015-08-30 10:10 pm (UTC)(link)
Hurrah for the code tour!
karzilla: a green fist above the word SMASH! (Default)

[staff profile] karzilla 2015-08-31 01:09 pm (UTC)(link)
Yay code tour! Sorry you ended up having to do two this month.