- Queries
- All Stories
- Search
- Advanced Search
- Transactions
- Transaction Logs
Feed All Stories
All Stories
All Stories
Dec 5 2024
Dec 5 2024
aklapper requested review of D25848: Fix PhabricatorAuthInviteTestCase::testDuplicateInvite unit test.
pppery committed rP9c73d62c4466: Rewrite regex for project names to be not prone to catastrophic backtracking.
Rewrite regex for project names to be not prone to catastrophic backtracking
pppery updated the diff for D25838: Rewrite regex for project names to be not prone to catastrophic backtracking.
Rebase
I applied this patch locally on top of git master and output does not complain anymore about 'link-brackets.txt' (thus it's correct) but fails in link-edge-cases.txt now (thus it's likely not complete):
valerio.bozzolan updated the task description for T15967: Fix unit test PhutilPygmentizeParserTestCase.
valerio.bozzolan updated the diff for D25845: Repository Identity "Automatically Detected User": don't trust unverified emails.
run unit test
valerio.bozzolan changed the visibility for D25845: Repository Identity "Automatically Detected User": don't trust unverified emails.
valerio.bozzolan updated subscribers of T15965: Repository Identity "Automatically Detected User": it reads unverified emails, with spam concerns.
Adding @aklapper as subscriber in this security issue since I trust this user (unclear if this should be flagged as security thought, feel free to open)
valerio.bozzolan changed the edit policy for D25845: Repository Identity "Automatically Detected User": don't trust unverified emails.
valerio.bozzolan raised the priority of T15556: Improve Diffusion identity reassignment propagation from Low to Normal.
Dec 4 2024
Dec 4 2024
• l2dy added a comment to T15036: Phorge upstream mail should not use @upstream.phorge.dev addresses.
This seems to impact mail deliverability to @icloud.com addresses too.
Thanks a lot @valerio.bozzolan for addressing this so quickly!
valerio.bozzolan awarded T15036: Phorge upstream mail should not use @upstream.phorge.dev addresses a Cup of Joe token.
Please "remind" me about this ticket as often as possible, so I will fix it. I have everything I need to actually fix this, except the willpower.
valerio.bozzolan added inline comments to D25835: Strip surrounding whitespace from project and task titles.
valerio.bozzolan committed rP48fd3f1c40de: Delete Repository button: disable if not admin, but show popup to all.
Delete Repository button: disable if not admin, but show popup to all
valerio.bozzolan added a comment to T15036: Phorge upstream mail should not use @upstream.phorge.dev addresses.
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:
Dec 3 2024
Dec 3 2024
@mturdus: Thanks! LGTM
@valerio.bozzolan: Feel free to give this revised version another review :)
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).
Thanks. Confirming that this works as expected (after manually changing test_accents.mbox to make the arc unit output fail)
In D25844#22553, @valerio.bozzolan wrote:I hope you won't send me a lawyer
do you really think I could afford one
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.
MacFan4000 edited reviewers for D25839: Fix parsing of incoming mail with UTF-8 encoded headers, added: Blessed Committers; removed: • 3tayesh.
MacFan4000 changed the author of D25839: Fix parsing of incoming mail with UTF-8 encoded headers from • 3tayesh to aklapper.
@3tayesh Please don't do that again.
Policy changed the edit policy for D25839: Fix parsing of incoming mail with UTF-8 encoded headers.
Policy changed the visibility for D25839: Fix parsing of incoming mail with UTF-8 encoded headers.
git rebase origin/master
lol @taavi I've stolen some of your "ä" for the example file test_accents.mbox, I hope you won't send me a lawyer
• 3tayesh commandeered D25839: Fix parsing of incoming mail with UTF-8 encoded headers from aklapper.
Fix parsing of incoming mail with UTF-8 encoded headers
I tried to prepare a unit test but it's not working. But yes the patch itself seems to work thanks
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?
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.
Dec 2 2024
Dec 2 2024
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.
valerio.bozzolan updated the answer details for Q160: Is it possible to delete repositories as normal user? (Answer 199).
valerio.bozzolan added a project to T15964: "Delete Repository" button: it's a bit scaring (always active): UX.
aklapper closed T15962: Account registration: Handle too long Real Name more gracefully as Resolved by committing rPeb380f922c9a: Account registration: Restrict Real Name length.
Account registration: Restrict Real Name length
valerio.bozzolan added a project to T15742: When creating a Diffusion Repo, add helpful text to each field: UX.
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/
Thanks for the information. I can confirm that a normal user receives "You Shall Not Pass: rP", "You do not have permission to edit this object."
valerio.bozzolan awarded Q160: Is it possible to delete repositories as normal user? a Love token.
reet- edited projects for Q160: Is it possible to delete repositories as normal user?, added: Diffusion; removed Phorge.
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.
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"….
aklapper committed rP0bfdcaa005b1: Revert "Fix PHP 8.3 "Usage of ldap_connect with two arguments is deprecated"….
Revert "Fix PHP 8.3 "Usage of ldap_connect with two arguments is deprecated"…
Content licensed under Creative Commons Attribution-ShareAlike 4.0 (CC-BY-SA) unless otherwise noted; code licensed under Apache 2.0 or other open source licenses. · CC BY-SA 4.0 · Apache 2.0