Page MenuHomePhorge

l2dy (Zero King)
UserEmail Not Verified

Projects

Today

  • Clear sailing ahead.

Tomorrow

  • Clear sailing ahead.

Thursday

  • Clear sailing ahead.

User Details

User Since
Oct 4 2023, 01:53 (8 w, 6 d)
Availability
Available

Recent Activity

Yesterday

l2dy added a comment to D25478: Use predictable filenames when downloading raw diffs from a revision.

id, vs and /new/ in URL all affect content of the generated file.

Mon, Dec 4, 16:09

Sun, Dec 3

l2dy added a comment to D25478: Use predictable filenames when downloading raw diffs from a revision.
In D25478#13957, @speck wrote:

What do you think about keeping the current naming scheme but whitelisting a handful of query params to use instead of using them all? The timestamp adds noise and it's probably nice to see the ID of the diff in the filename, e.g. D25478.id1541.diff? Maybe just allow the the id parameter and regex that it's value is just a string of numbers.

Sun, Dec 3, 11:04

Sun, Nov 26

l2dy updated the summary of D25478: Use predictable filenames when downloading raw diffs from a revision.
Sun, Nov 26, 15:38
l2dy requested review of D25478: Use predictable filenames when downloading raw diffs from a revision.
Sun, Nov 26, 15:35

Sat, Nov 25

l2dy added a comment to T15675: Visually indicate new user accounts.

Should this be configurable and behind a feature flag, as some Phorge installations may not want this (plus may slow down performance a little bit)?

Sat, Nov 25, 13:34 · Feature Requests

Mon, Nov 20

valerio.bozzolan awarded Q86: Policy lifecycle of security tasks a Love token.
Mon, Nov 20, 22:15 · Discussion Needed

Mon, Nov 13

l2dy closed D25467: Align logo image and text in site header.
Mon, Nov 13, 09:56
l2dy committed rPaa8af1d79e8b: Align logo image and text in site header (authored by l2dy).
Align logo image and text in site header
Mon, Nov 13, 09:56
l2dy added a comment to D25467: Align logo image and text in site header.

Uh! Nice!

Interestingly this CSS change only has effects on Chromium 119 and it does not change anything in my Mozilla Firefox 118. Can you give a quick look?

Mon, Nov 13, 09:46
l2dy added a comment to Q79: Is arc.land.onto.default configuration deprecated? (Answer 107).

This is related to https://secure.phabricator.com/T13546. "classic" Arcanist workflows were being upgraded to Toolsets, but the migration was not complete. See comments starting with TOOLSETS: , e.g. in ArcanistConfigurationSource.php.

Mon, Nov 13, 01:18
l2dy closed D25470: Initialize static variable to make linter happy.
Mon, Nov 13, 00:52
l2dy committed rP361fcd0cac36: Initialize static variable to make linter happy (authored by l2dy).
Initialize static variable to make linter happy
Mon, Nov 13, 00:52
l2dy closed D25469: Fix reentrancy guard in setDebugTimeLimit().
Mon, Nov 13, 00:51
l2dy committed rP486fc95b29fd: Fix reentrancy guard in setDebugTimeLimit() (authored by l2dy).
Fix reentrancy guard in setDebugTimeLimit()
Mon, Nov 13, 00:51
l2dy closed D25468: Fix typo in DiffusionLowLevelGitRefQuery.
Mon, Nov 13, 00:50
l2dy committed rP4535e8753c8b: Fix typo in DiffusionLowLevelGitRefQuery (authored by l2dy).
Fix typo in DiffusionLowLevelGitRefQuery
Mon, Nov 13, 00:50

Sun, Nov 12

l2dy requested review of D25470: Initialize static variable to make linter happy.
Sun, Nov 12, 18:21
l2dy requested review of D25469: Fix reentrancy guard in setDebugTimeLimit().
Sun, Nov 12, 18:12
l2dy requested review of D25468: Fix typo in DiffusionLowLevelGitRefQuery.
Sun, Nov 12, 17:49
l2dy asked Q86: Policy lifecycle of security tasks.
Sun, Nov 12, 16:17 · Discussion Needed
l2dy closed D25464: Enforce viewable MIME types config on PDF documents.
Sun, Nov 12, 16:13
l2dy committed rP16d9cc12af45: Enforce viewable MIME types config on PDF documents (authored by l2dy).
Enforce viewable MIME types config on PDF documents
Sun, Nov 12, 16:13
l2dy added inline comments to D25464: Enforce viewable MIME types config on PDF documents.
Sun, Nov 12, 16:00
l2dy updated the diff for D25464: Enforce viewable MIME types config on PDF documents.

Phrasing

Sun, Nov 12, 15:57
l2dy added inline comments to D25464: Enforce viewable MIME types config on PDF documents.
Sun, Nov 12, 15:51
l2dy updated the question details for Q83: Pull security fixes from Mozilla's fork.
Sun, Nov 12, 15:42 · Security, Phorge
l2dy added a comment to D25464: Enforce viewable MIME types config on PDF documents.
In D25464#13372, @speck wrote:

This will require documentation of some sort, specifically for the upgrade notes to indicate that if someone relies on rendering PDFs currently then after upgrading they would need to update that configuration.

I think making a task to document the issue and linking in the release notes would be a good approach and what upstream would typically have done.

Sun, Nov 12, 15:40
l2dy closed Q81: Documentation for "Restarting Phabricator" is missing as resolved.
Sun, Nov 12, 10:58 · Phorge
l2dy added Q81: Documentation for "Restarting Phabricator" is missing (Answer 110).
Sun, Nov 12, 10:58
l2dy closed Q85: How does amending author information work with `arc land`? as resolved.
Sun, Nov 12, 10:57
l2dy requested review of D25467: Align logo image and text in site header.
Sun, Nov 12, 09:41
l2dy added a revision to T15666: Align logo image and text in site header: D25467: Align logo image and text in site header.
Sun, Nov 12, 09:41 · Phorge
l2dy updated the task description for T15666: Align logo image and text in site header.
Sun, Nov 12, 09:30 · Phorge
l2dy updated the task description for T15666: Align logo image and text in site header.
Sun, Nov 12, 09:21 · Phorge
l2dy created T15666: Align logo image and text in site header.
Sun, Nov 12, 09:20 · Phorge
l2dy updated the summary of D25464: Enforce viewable MIME types config on PDF documents.
Sun, Nov 12, 07:08
l2dy updated the summary of D25464: Enforce viewable MIME types config on PDF documents.
Sun, Nov 12, 06:13

Sat, Nov 11

l2dy added a comment to D25464: Enforce viewable MIME types config on PDF documents.
In D25464#13328, @speck wrote:

The Referenced Files section of this diff looks like someone is looking for a vulnerability. Any idea what’s happening here?

Sat, Nov 11, 23:18
l2dy added a comment to Q85: How does amending author information work with `arc land`? (Answer 109).

arc diff throws an error and arc land ignores it. Instead of getting it to work with Differential, I'm amending the commit manually with git commit --amend -s after arc land --hold.

Sat, Nov 11, 23:16
l2dy closed D25462: Fix view policy inheritance on image transforms.
Sat, Nov 11, 23:10
l2dy committed rP005fea5a14f4: Fix view policy inheritance on image transforms (authored by dkl).
Fix view policy inheritance on image transforms
Sat, Nov 11, 23:10
l2dy updated the diff for D25462: Fix view policy inheritance on image transforms.

Update commit message

Sat, Nov 11, 23:07
l2dy updated the diff for D25464: Enforce viewable MIME types config on PDF documents.

Update config description

Sat, Nov 11, 10:45
l2dy updated the question details for Q83: Pull security fixes from Mozilla's fork.
Sat, Nov 11, 08:41 · Security, Phorge
l2dy added a comment to Q83: Pull security fixes from Mozilla's fork (Answer 108).

Mozilla's fixes were quick but not in good quality for direct submission to Phorge. I have adapted or rewritten two of the changes into D25462 and D25464.

Sat, Nov 11, 08:25
l2dy updated the question details for Q83: Pull security fixes from Mozilla's fork.
Sat, Nov 11, 08:13 · Security, Phorge
l2dy requested review of D25464: Enforce viewable MIME types config on PDF documents.
Sat, Nov 11, 08:13
l2dy added a comment to D25462: Fix view policy inheritance on image transforms.

Before landing this change, I would like to know the right way to credit both the original author and my modifications with Arcanist (Q85).

Sat, Nov 11, 02:43
l2dy closed D25463: Correct default database prefix in documentation.
Sat, Nov 11, 02:33
l2dy committed rPc97a50472c5b: Correct default database prefix in documentation (authored by l2dy).
Correct default database prefix in documentation
Sat, Nov 11, 02:33
l2dy updated the diff for D25463: Correct default database prefix in documentation.

Reword

Sat, Nov 11, 02:26
l2dy requested review of D25463: Correct default database prefix in documentation.
Sat, Nov 11, 02:24
l2dy added inline comments to D25462: Fix view policy inheritance on image transforms.
Sat, Nov 11, 00:34
l2dy added inline comments to D25462: Fix view policy inheritance on image transforms.
Sat, Nov 11, 00:27

Fri, Nov 10

l2dy asked Q85: How does amending author information work with `arc land`?.
Fri, Nov 10, 18:01
l2dy changed the visibility for Q83: Pull security fixes from Mozilla's fork.
Fri, Nov 10, 16:40 · Security, Phorge
l2dy updated the question details for Q83: Pull security fixes from Mozilla's fork.
Fri, Nov 10, 16:08 · Security, Phorge
l2dy added a comment to Q83: Pull security fixes from Mozilla's fork.

@valerio.bozzolan If you didn't get an answer, try asking in #conduit. I didn't realize that # needs to be quoted in Remarkup. 😢

Fri, Nov 10, 16:05 · Security, Phorge
l2dy added a comment to D25462: Fix view policy inheritance on image transforms.

@valerio.bozzolan Upstream removed code that enforces $always_visible when $file->getIsProfileImage() is true in PhabricatorFileQuery.php, but I'm afraid that this change may break other things, so I did not apply it here, but on the other hand kept the description of "Image will be Public".

Fri, Nov 10, 15:58
l2dy updated the summary of D25462: Fix view policy inheritance on image transforms.
Fri, Nov 10, 15:57
l2dy updated the diff for D25462: Fix view policy inheritance on image transforms.

Mention upstream commit

Fri, Nov 10, 15:56
l2dy added a comment to Q83: Pull security fixes from Mozilla's fork.

It would be great if Mozilla's team could join forces with Phorge. Would you (the core team) contact them in Conduit on chat.mozilla.org and mozilla.slack.com?

Fri, Nov 10, 15:39 · Security, Phorge
l2dy updated the diff for D25462: Fix view policy inheritance on image transforms.

Fix lint warnings

Fri, Nov 10, 15:13
l2dy requested review of D25462: Fix view policy inheritance on image transforms.
Fri, Nov 10, 15:10
l2dy updated the question details for Q83: Pull security fixes from Mozilla's fork.
Fri, Nov 10, 13:39 · Security, Phorge
l2dy added a comment to T15045: Support SSL/TLS for MariaDB connections.

I have reviewed it and made some comments. On a remotely related topic, TLS handshakes are expensive and persistent connections can reduce latency and server load by reusing TLS connections, so maybe we should make it configurable outside of cluster.databases as well.

Fri, Nov 10, 13:07 · Security

Thu, Nov 9

l2dy closed D25459: Fix doc link to Restarting Phorge.
Thu, Nov 9, 11:34
l2dy committed rPce5e0f3e333e: Fix doc link to Restarting Phorge (authored by l2dy).
Fix doc link to Restarting Phorge
Thu, Nov 9, 11:34

Wed, Nov 8

l2dy requested review of D25459: Fix doc link to Restarting Phorge.
Wed, Nov 8, 14:15

Mon, Nov 6

l2dy added a project to Q83: Pull security fixes from Mozilla's fork: Phorge.
Mon, Nov 6, 13:35 · Security, Phorge
l2dy updated the question details for Q83: Pull security fixes from Mozilla's fork.
Mon, Nov 6, 13:20 · Security, Phorge
l2dy asked Q83: Pull security fixes from Mozilla's fork.
Mon, Nov 6, 13:20 · Security, Phorge

Nov 5 2023

l2dy asked Q82: Consider replacing MyISAM with InnoDB?.
Nov 5 2023, 16:58 · Phorge
avivey awarded Q81: Documentation for "Restarting Phabricator" is missing a Grey Medal token.
Nov 5 2023, 07:42 · Phorge
l2dy asked Q81: Documentation for "Restarting Phabricator" is missing.
Nov 5 2023, 04:00 · Phorge
l2dy asked Q80: Bug: passing null to strlen in readLocalArcConfig.
Nov 5 2023, 01:38 · Arcanist

Oct 31 2023

l2dy asked Q79: Is arc.land.onto.default configuration deprecated?.
Oct 31 2023, 11:36 · Arcanist