Page MenuHomePhorge
Feed All Stories

Mar 28 2025

valerio.bozzolan added a parent task for T16023: Regression in Config: console generates errors when some libraries are not managed by Git: T15243: The /config/ page should log git errors instead of silently ignore them.
Mar 28 2025, 16:02
valerio.bozzolan added a subtask for T15243: The /config/ page should log git errors instead of silently ignore them: T16023: Regression in Config: console generates errors when some libraries are not managed by Git.
Mar 28 2025, 16:02 · Config
valerio.bozzolan renamed T16023: Regression in Config: console generates errors when some libraries are not managed by Git from Config console generates errors when some libraries are not managed by Git to Regression in Config: console generates errors when some libraries are not managed by Git.
Mar 28 2025, 16:02
xtex added a comment to T16023: Regression in Config: console generates errors when some libraries are not managed by Git.

Thanks for the hint.

Mar 28 2025, 16:01
xtex closed T16023: Regression in Config: console generates errors when some libraries are not managed by Git as Resolved.
Mar 28 2025, 16:01
valerio.bozzolan assigned T16023: Regression in Config: console generates errors when some libraries are not managed by Git to xtex.

Well done \o/

Mar 28 2025, 16:00
xtex closed D25931: Config: Fix git errors when .git does not exist.
Mar 28 2025, 15:59
xtex committed rPa81f20e0f228: Config: Fix git errors when .git does not exist.
Config: Fix git errors when .git does not exist
Mar 28 2025, 15:59
valerio.bozzolan accepted D25931: Config: Fix git errors when .git does not exist.

sgtm

Mar 28 2025, 15:52
xtex updated the diff for D25931: Config: Fix git errors when .git does not exist.

Remove duplicated assignments to $stderr

Mar 28 2025, 15:50
valerio.bozzolan added inline comments to D25931: Config: Fix git errors when .git does not exist.
Mar 28 2025, 15:49
xtex updated the diff for D25931: Config: Fix git errors when .git does not exist.

Add link to T16023

Mar 28 2025, 15:49
xtex updated the diff for D25931: Config: Fix git errors when .git does not exist.

Fix lint message

Mar 28 2025, 15:48
xtex updated the diff for D25931: Config: Fix git errors when .git does not exist.

Update to detect stderr messages

Mar 28 2025, 15:47
valerio.bozzolan added a comment to T16023: Regression in Config: console generates errors when some libraries are not managed by Git.

As said in https://we.phorge.it/D25931#25023 I think this is a sub-task of T15243. And in this specific case, very probably we don't want to log anything. So your report could be considered a regression caused by T15243.

Mar 28 2025, 15:41
valerio.bozzolan added a comment to D25931: Config: Fix git errors when .git does not exist.

Thanks my friend. I 100% understand the problem and I'm happy you shared the stack trace in the task and I'm happy about this test plan that helped me to reproduce,

Mar 28 2025, 15:37
valerio.bozzolan updated the diff for D25931: Config: Fix git errors when .git does not exist.

arc diff

Mar 28 2025, 15:22
xtex requested review of D25931: Config: Fix git errors when .git does not exist.
Mar 28 2025, 14:55
xtex added a revision to T16023: Regression in Config: console generates errors when some libraries are not managed by Git: D25931: Config: Fix git errors when .git does not exist.
Mar 28 2025, 14:55
xtex created T16023: Regression in Config: console generates errors when some libraries are not managed by Git.
Mar 28 2025, 14:48
aklapper edited the content of Next Up.
Mar 28 2025, 10:23
aklapper closed D25921: Fix invalid CSS overflow value for .phui-oi-name.
Mar 28 2025, 10:23
aklapper committed rP06a62b347ac5: Fix invalid CSS overflow value for .phui-oi-name.
Fix invalid CSS overflow value for .phui-oi-name
Mar 28 2025, 10:23
aklapper updated the diff for D25921: Fix invalid CSS overflow value for .phui-oi-name.

git rebase master

Mar 28 2025, 10:21
valerio.bozzolan accepted D25921: Fix invalid CSS overflow value for .phui-oi-name.

technicallycorrect

Mar 28 2025, 10:18
valerio.bozzolan updated the test plan for D25921: Fix invalid CSS overflow value for .phui-oi-name.
Mar 28 2025, 10:18
valerio.bozzolan closed T16001: Celerity unit test not triggered: mitigate thanks to Herald, a subtask of T15209: The Celerity unit tests are not automatically run when touching whatever CSS or JS file, as Resolved.
Mar 28 2025, 10:01 · Bug Reports, Phorge Development Tools, Arcanist, User-valerio.bozzolan
valerio.bozzolan closed T16001: Celerity unit test not triggered: mitigate thanks to Herald as Resolved.

It seems H29 works lol. I've improved it a bit to do not trigger if the celerity map was already touched.

Mar 28 2025, 10:01 · User-valerio.bozzolan, phorge.it install
aklapper added a comment to D25930: Remove more obsolete IE9 filter CSS syntax.

No problem! I'll send you my crypto wallet address. Once the money has arrived you'll receive your clipboard content as an NFT.

Mar 28 2025, 10:01
valerio.bozzolan added a comment to D25930: Remove more obsolete IE9 filter CSS syntax.

Yep thanks, I need my clipboard of the 12 of August 2024 to get back the "D25772" to copy-paste it here and attract hackers asd

Mar 28 2025, 09:59
aklapper edited the content of Next Up.
Mar 28 2025, 09:58
aklapper closed D25930: Remove more obsolete IE9 filter CSS syntax.
Mar 28 2025, 09:57
aklapper committed rP64044ab8308c: Remove more obsolete IE9 filter CSS syntax.
Remove more obsolete IE9 filter CSS syntax
Mar 28 2025, 09:57
aklapper added a comment to D25930: Remove more obsolete IE9 filter CSS syntax.

My keylogger thanks you! Would you like to know anything else about your online patterns?

Mar 28 2025, 09:56
valerio.bozzolan added a comment to D25930: Remove more obsolete IE9 filter CSS syntax.

You clearly hacked my account lol

Mar 28 2025, 09:55
aklapper added a comment to D25930: Remove more obsolete IE9 filter CSS syntax.

30 seconds for a review?! I think you broke a record.

Mar 28 2025, 09:54
valerio.bozzolan accepted D25930: Remove more obsolete IE9 filter CSS syntax.

Macro wojak_depressed: WHY... WHY MICROSOFT

Mar 28 2025, 09:51
aklapper requested review of D25930: Remove more obsolete IE9 filter CSS syntax.
Mar 28 2025, 09:51
aklapper edited the content of Next Up.
Mar 28 2025, 06:07
aklapper closed D25928: Remove some pre-2013 vendor-prefixed -ms- CSS properties.
Mar 28 2025, 05:57
aklapper committed rP112174b900cb: Remove some pre-2013 vendor-prefixed -ms- CSS properties.
Remove some pre-2013 vendor-prefixed -ms- CSS properties
Mar 28 2025, 05:57
aklapper updated the summary of D25928: Remove some pre-2013 vendor-prefixed -ms- CSS properties.
Mar 28 2025, 05:57
aklapper updated the diff for D25928: Remove some pre-2013 vendor-prefixed -ms- CSS properties.

git rebase master

Mar 28 2025, 05:55
aklapper closed D25929: Remove obsolete IE9 filter CSS syntax.
Mar 28 2025, 05:50
aklapper committed rPe38f545743d8: Remove obsolete IE9 filter CSS syntax.
Remove obsolete IE9 filter CSS syntax
Mar 28 2025, 05:50

Mar 27 2025

valerio.bozzolan accepted D25929: Remove obsolete IE9 filter CSS syntax.

Macro wojak_depressed: WHY... WHY MICROSOFT

Mar 27 2025, 19:56
aklapper updated the summary of D25929: Remove obsolete IE9 filter CSS syntax.
Mar 27 2025, 19:17
aklapper requested review of D25929: Remove obsolete IE9 filter CSS syntax.
Mar 27 2025, 19:15
valerio.bozzolan accepted D25928: Remove some pre-2013 vendor-prefixed -ms- CSS properties.

chadyes

Mar 27 2025, 18:52
aklapper requested review of D25928: Remove some pre-2013 vendor-prefixed -ms- CSS properties.
Mar 27 2025, 18:11
Cigaryno added a project to T16010: Owners not triggering audit on No Owner Involvement: Good Starter Task.

Looks like a Good Starter Task.

Mar 27 2025, 12:49 · Good Starter Task, Owners, Bug Reports
aklapper closed D25927: Remove pre-2016 vendor-prefixed -webkit- CSS properties.
Mar 27 2025, 07:33
aklapper edited the content of Next Up.
Mar 27 2025, 07:33
aklapper committed rP047021649785: Remove pre-2016 vendor-prefixed -webkit- CSS properties.
Remove pre-2016 vendor-prefixed -webkit- CSS properties
Mar 27 2025, 07:33

Mar 26 2025

valerio.bozzolan accepted D25880: Replace some AphrontView::setUser() calls with AphrontView::setViewer().

makeitso

Mar 26 2025, 17:12
valerio.bozzolan accepted D25927: Remove pre-2016 vendor-prefixed -webkit- CSS properties.

technicallycorrect

Mar 26 2025, 17:08
aklapper requested review of D25927: Remove pre-2016 vendor-prefixed -webkit- CSS properties.
Mar 26 2025, 17:05
aklapper closed T16007: Discuss the policy for contributing to prototype applications as Resolved by committing rP29866413d1c7: Diviner: Contributing Code: Update section on Prototype Changes.
Mar 26 2025, 16:15 · Discussion Needed, User-Cigaryno
aklapper committed rP29866413d1c7: Diviner: Contributing Code: Update section on Prototype Changes.
Diviner: Contributing Code: Update section on Prototype Changes
Mar 26 2025, 16:15
aklapper closed D25909: Diviner: Contributing Code: Update section on Prototype Changes.
Mar 26 2025, 16:15
aklapper closed D25912: Apply "Can Bulk Edit Tasks" to "Move Tasks" workboard options.
Mar 26 2025, 13:23
aklapper committed rP97605e6aa95f: Apply "Can Bulk Edit Tasks" to "Move Tasks" workboard options.
Apply "Can Bulk Edit Tasks" to "Move Tasks" workboard options
Mar 26 2025, 13:23
aklapper added a comment to D25926: Make Diffusion repository management accessible to logged-out users.

robots.txt can have the solution for that (see below).
[...]
For search engines, the solution is to add this to robots.txt:

In theory yes if everyone behaved. In practice, robots.txt is ignored and LLM/AI crawlers are ruthless. (For example, GNOME GitLab admins recently installed Anubis to run background checks on your machine.)

Mar 26 2025, 13:02 · User-Cigaryno, Diffusion
valerio.bozzolan added a comment to D25912: Apply "Can Bulk Edit Tasks" to "Move Tasks" workboard options.

(I cannot double-accept so only just one accept asd)

Mar 26 2025, 12:48
Cigaryno updated the test plan for D25926: Make Diffusion repository management accessible to logged-out users.
Mar 26 2025, 12:44 · User-Cigaryno, Diffusion
Cigaryno added a comment to D25926: Make Diffusion repository management accessible to logged-out users.
  • more search engine rabbit holes (but maybe not that bad)

robots.txt can have the solution for that (see below).

Mar 26 2025, 12:41 · User-Cigaryno, Diffusion
valerio.bozzolan added a comment to D25926: Make Diffusion repository management accessible to logged-out users.

Uhm. Good points:

Mar 26 2025, 12:30 · User-Cigaryno, Diffusion
Cigaryno added a comment to D25926: Make Diffusion repository management accessible to logged-out users.

Why would a logged-out user (who does not want to or cannot create an account) want to know about Repository management log or Repository limits? I don't see how that's their business (or interest)?

Mar 26 2025, 12:29 · User-Cigaryno, Diffusion
Cigaryno added a comment to T15999: Make Phorge more accessible to logged-out users.

Some items in the task description make me a bit uncomfortable in my instance.

I don't think you need to be uncomfortable on your instance (phabricator.wikimedia.org)
For Herald, it looks to be restricted to trusted contributors to restrict who can create personal rules (they actually can vandalize tasks via personal rules with the action set to claim the task), that's not something to take care of at all on your instance.
Project members, maniphest reports, user tasks and badges are actually useful for logged-out users.
But everything that's Diffusion-related sounds pointless for your instance as every repo is a read-only mirror of the repos on a Gerrit instance.

Mar 26 2025, 12:26 · User-Cigaryno
aklapper added a comment to D25926: Make Diffusion repository management accessible to logged-out users.

Why would a logged-out user (who does not want to or cannot create an account) want to know about Repository management log or Repository limits? I don't see how that's their business (or interest)?

Mar 26 2025, 11:58 · User-Cigaryno, Diffusion
aklapper added a comment to T15999: Make Phorge more accessible to logged-out users.

Some items in the task description make me a bit uncomfortable in my instance. Why does everyone need to see Diffusion sync, pull, and push logs? Why Herald transcripts? Why repo management if you cannot manage? What are actual use cases which outweigh security implications?

Mar 26 2025, 11:57 · User-Cigaryno
Cigaryno updated the task description for T15140: switch from PHPExcel to PHPSpreadsheet for Maniphest task export.
Mar 26 2025, 11:26 · Maniphest
Cigaryno updated the task description for T15999: Make Phorge more accessible to logged-out users.
Mar 26 2025, 11:23 · User-Cigaryno
Cigaryno added a revision to T15999: Make Phorge more accessible to logged-out users: D25926: Make Diffusion repository management accessible to logged-out users.
Mar 26 2025, 11:16 · User-Cigaryno
Cigaryno updated the summary of D25926: Make Diffusion repository management accessible to logged-out users.
Mar 26 2025, 11:16 · User-Cigaryno, Diffusion
Cigaryno edited the content of Update From Phabricator.
Mar 26 2025, 11:14
Cigaryno added a comment to D25926: Make Diffusion repository management accessible to logged-out users.

There might be some security implications to this.
Why is this needed?

Mar 26 2025, 11:09 · User-Cigaryno, Diffusion
avivey added a comment to D25926: Make Diffusion repository management accessible to logged-out users.

There might be some security implications to this.
Why is this needed?

Mar 26 2025, 10:11 · User-Cigaryno, Diffusion
Cigaryno requested review of D25926: Make Diffusion repository management accessible to logged-out users.
Mar 26 2025, 09:20 · User-Cigaryno, Diffusion

Mar 25 2025

aklapper added inline comments to D25912: Apply "Can Bulk Edit Tasks" to "Move Tasks" workboard options.
Mar 25 2025, 16:56
Cigaryno updated the task description for T15490: Uninstalling/Installing an application should eventually involve MFA.
Mar 25 2025, 14:03 · User-valerio.bozzolan, Security
20after4 awarded D25913: Herald actions: Allow setting subtype on Maniphest tasks a Like token.
Mar 25 2025, 13:55
aklapper edited the content of Next Up.
Mar 25 2025, 13:28
aklapper closed D25920: Fix invalid CSS vertical-align value for .fancy-datepicker-core.
Mar 25 2025, 13:28
aklapper committed rPf62918b019fb: Fix invalid CSS vertical-align value for .fancy-datepicker-core.
Fix invalid CSS vertical-align value for .fancy-datepicker-core
Mar 25 2025, 13:28
aklapper updated the diff for D25920: Fix invalid CSS vertical-align value for .fancy-datepicker-core.

git rebase master

Mar 25 2025, 13:27
valerio.bozzolan accepted D25920: Fix invalid CSS vertical-align value for .fancy-datepicker-core.
Mar 25 2025, 13:24
aklapper edited the content of Next Up.
Mar 25 2025, 11:38
aklapper closed D25924: Remove invalid CSS border-shadow property for input.jx-tokenizer-input.
Mar 25 2025, 11:37
aklapper committed rPa8c2bce6fe98: Remove invalid CSS border-shadow property for input.jx-tokenizer-input.
Remove invalid CSS border-shadow property for input.jx-tokenizer-input
Mar 25 2025, 11:37
aklapper updated the diff for D25924: Remove invalid CSS border-shadow property for input.jx-tokenizer-input.

git rebase master

Mar 25 2025, 11:37
valerio.bozzolan accepted D25924: Remove invalid CSS border-shadow property for input.jx-tokenizer-input.

sgtm

Mar 25 2025, 11:18
aklapper edited the content of Next Up.
Mar 25 2025, 11:03
aklapper closed D25925: Fix invalid CSS border-left-color value for .phabricator-action-view-submenu .caret-right.
Mar 25 2025, 11:03
aklapper committed rP633d4037e5ac: Fix invalid CSS border-left-color value for .phabricator-action-view-submenu ..
Fix invalid CSS border-left-color value for .phabricator-action-view-submenu .
Mar 25 2025, 11:03
valerio.bozzolan accepted D25925: Fix invalid CSS border-left-color value for .phabricator-action-view-submenu .caret-right.

sgtm

Mar 25 2025, 10:56
aklapper requested review of D25925: Fix invalid CSS border-left-color value for .phabricator-action-view-submenu .caret-right.
Mar 25 2025, 10:07
aklapper requested review of D25924: Remove invalid CSS border-shadow property for input.jx-tokenizer-input.
Mar 25 2025, 09:56
aklapper edited the content of Next Up.
Mar 25 2025, 09:41
aklapper edited the content of Next Up.
Mar 25 2025, 09:35