We should maybe rewrite a bit the proposed solution, since ideally it's possible to use upstream.phorge.dev but it should be at least reachable to pass some anti-spam checkers I guess. At the moment it isn't:

Wed, Dec 4, 06:49 · phorge.it install

Tue, Dec 3

aklapper updated the test plan for D25790: Show table of contents by default on wide screens.

Tue, Dec 3, 23:07

aklapper accepted D25790: Show table of contents by default on wide screens.

@mturdus: Thanks! LGTM

Tue, Dec 3, 23:06

aklapper added a comment to D25835: Strip surrounding whitespace from project and task titles.

@valerio.bozzolan: Feel free to give this revised version another review :)

Tue, Dec 3, 23:02

aklapper accepted D25838: Rewrite regex for project names to be not prone to catastrophic backtracking.

Big thanks for digging deep into that regex (on which I gave up).

Tue, Dec 3, 22:43

aklapper accepted D25844: Add first unit test for mimemailparser headers.

Thanks. Confirming that this works as expected (after manually changing test_accents.mbox to make the arc unit output fail)

Tue, Dec 3, 22:24

aklapper updated the test plan for D25844: Add first unit test for mimemailparser headers.

Tue, Dec 3, 21:20

taavi added a comment to D25844: Add first unit test for mimemailparser headers.

In D25844#22553, @valerio.bozzolan wrote:

I hope you won't send me a lawyer

do you really think I could afford one

Tue, Dec 3, 17:18

MacFan4000 edited reviewers for D25839: Fix parsing of incoming mail with UTF-8 encoded headers, added: O1: Blessed Committers, taavi, valerio.bozzolan; removed: Blessed Committers.

Tue, Dec 3, 17:14

MacFan4000 edited reviewers for D25839: Fix parsing of incoming mail with UTF-8 encoded headers, added: Blessed Committers; removed: • 3tayesh.

Tue, Dec 3, 17:13

MacFan4000 changed the author of D25839: Fix parsing of incoming mail with UTF-8 encoded headers from • 3tayesh to aklapper.

Tue, Dec 3, 17:12

avivey changed the visibility for D25043: Upload compressed logo, update HTML.

Tue, Dec 3, 16:22

avivey changed the visibility for D25041: Home Page.

Tue, Dec 3, 16:21

avivey added a comment to D25839: Fix parsing of incoming mail with UTF-8 encoded headers.

@3tayesh Please don't do that again.

Tue, Dec 3, 16:17

avivey changed the visibility for D25839: Fix parsing of incoming mail with UTF-8 encoded headers.

Tue, Dec 3, 16:17

Policy changed the edit policy for D25839: Fix parsing of incoming mail with UTF-8 encoded headers.

Tue, Dec 3, 16:16

Policy changed the visibility for D25839: Fix parsing of incoming mail with UTF-8 encoded headers.

Tue, Dec 3, 16:16

valerio.bozzolan updated the test plan for D25844: Add first unit test for mimemailparser headers.

Tue, Dec 3, 15:53

valerio.bozzolan updated the diff for D25844: Add first unit test for mimemailparser headers.

git rebase origin/master

Tue, Dec 3, 15:52

valerio.bozzolan added a comment to D25844: Add first unit test for mimemailparser headers.

lol @taavi I've stolen some of your "ä" for the example file test_accents.mbox, I hope you won't send me a lawyer

Tue, Dec 3, 15:48

valerio.bozzolan updated the summary of D25844: Add first unit test for mimemailparser headers.

Tue, Dec 3, 15:47

valerio.bozzolan requested review of D25844: Add first unit test for mimemailparser headers.

Tue, Dec 3, 15:44

valerio.bozzolan added a revision to T15960: Incoming mail parsing fails if specific headers have UTF-8 encoded text in them: D25844: Add first unit test for mimemailparser headers.

Tue, Dec 3, 15:44 · Bug Reports

• 3tayesh commandeered D25839: Fix parsing of incoming mail with UTF-8 encoded headers from aklapper.

Tue, Dec 3, 15:09

aklapper closed T15960: Incoming mail parsing fails if specific headers have UTF-8 encoded text in them as Resolved by committing rPc589529155d6: Fix parsing of incoming mail with UTF-8 encoded headers.

Tue, Dec 3, 12:39 · Bug Reports

aklapper closed D25839: Fix parsing of incoming mail with UTF-8 encoded headers.

Tue, Dec 3, 12:39

aklapper committed rPc589529155d6: Fix parsing of incoming mail with UTF-8 encoded headers.

Fix parsing of incoming mail with UTF-8 encoded headers

Tue, Dec 3, 12:39

valerio.bozzolan accepted D25839: Fix parsing of incoming mail with UTF-8 encoded headers.

I tried to prepare a unit test but it's not working. But yes it seems to work thanks

Tue, Dec 3, 12:01

valerio.bozzolan added a comment to D25839: Fix parsing of incoming mail with UTF-8 encoded headers.

(Sorry I cannot test now) Does the test plan work with just this?

Tue, Dec 3, 11:24

valerio.bozzolan added a comment to D25843: Delete Repository button: disable if not admin, but show popup to all.

Nice question avivey. We could always disable this but maybe "newcomer admins" may not try to click it, so with more confusion from the other (Dark) Side.

Tue, Dec 3, 05:52

Mon, Dec 2

avivey accepted D25843: Delete Repository button: disable if not admin, but show popup to all.

Since the button doesn't do anything in any case except say why it doesn't do anything, it might always be disabled? Do we have any other similar buttons?
I'm never sure if users discover that disabled buttons are actually clickable (and try to explain why they're disabled). It's a UI feature I really like, but I don't think I've seen it anywhere else.

Mon, Dec 2, 21:24

valerio.bozzolan requested review of D25843: Delete Repository button: disable if not admin, but show popup to all.

Mon, Dec 2, 20:19

valerio.bozzolan added a revision to T15964: "Delete Repository" button: it's a bit scaring (always active): D25843: Delete Repository button: disable if not admin, but show popup to all.

Mon, Dec 2, 20:19 · UX, Diffusion

valerio.bozzolan added a project to T15964: "Delete Repository" button: it's a bit scaring (always active): UX.

Mon, Dec 2, 20:15 · UX, Diffusion

valerio.bozzolan created T15964: "Delete Repository" button: it's a bit scaring (always active).

Mon, Dec 2, 20:13 · UX, Diffusion

aklapper closed D25841: Account registration: Restrict Real Name length.

Mon, Dec 2, 19:36

aklapper closed T15962: Account registration: Handle too long Real Name more gracefully as Resolved by committing rPeb380f922c9a: Account registration: Restrict Real Name length.

Mon, Dec 2, 19:36

aklapper committed rPeb380f922c9a: Account registration: Restrict Real Name length.

Account registration: Restrict Real Name length

Mon, Dec 2, 19:36

valerio.bozzolan added a project to T15742: When creating a Diffusion Repo, add helpful text to each field: UX.

Mon, Dec 2, 17:37 · UX, Diffusion

• l2dy added a comment to T15062: Introducing Phixator 2.

Is the extension still maintained? While there is no extension store yet, it's possible to host repositories here. https://we.phorge.it/w/docs/extensions/phactory/

Mon, Dec 2, 13:58 · Phactory: Community Projects

aklapper reopened T15724: PHP 8.3: Usage of ldap_connect with two arguments is deprecated, a subtask of T15064: Make Phorge compatible with PHP 8.1/8.2/8.3/8.4, as Open.

Mon, Dec 2, 09:28 · PHP 8 support

aklapper reopened T15724: PHP 8.3: Usage of ldap_connect with two arguments is deprecated as Open by committing rP0bfdcaa005b1: Revert "Fix PHP 8.3 "Usage of ldap_connect with two arguments is deprecated"….

Mon, Dec 2, 09:28 · Deprecation Notice, PHP 8 support

aklapper closed T15963: LDAP over TLS is broken as Resolved by committing rP0bfdcaa005b1: Revert "Fix PHP 8.3 "Usage of ldap_connect with two arguments is deprecated"….

Mon, Dec 2, 09:28 · Bug Reports, Auth

aklapper closed D25842: Revert "Fix PHP 8.3 "Usage of ldap_connect with two arguments is deprecated" exception".

Mon, Dec 2, 09:28

aklapper added a reverting change for D25792: Fix PHP 8.3 "Usage of ldap_connect with two arguments is deprecated" exception: rP0bfdcaa005b1: Revert "Fix PHP 8.3 "Usage of ldap_connect with two arguments is deprecated"….

Mon, Dec 2, 09:28

aklapper committed rP0bfdcaa005b1: Revert "Fix PHP 8.3 "Usage of ldap_connect with two arguments is deprecated"….