Page MenuHomePhorge
Feed All Stories

Oct 21 2024

valerio.bozzolan added a comment to T15927: Fix Arcanist unit test ArcanistBundleTestCase->testNonlocalTrailingNewline() on non-English terminal.

Thanks, little cute Phorge kitten in profile picture <3

Oct 21 2024, 05:34 · User-valerio.bozzolan
valerio.bozzolan added inline comments to D25828: ManiphestReportController: Separate legacy and synthetic data handling.
Oct 21 2024, 05:30

Oct 20 2024

syncer added a comment to Q156: How to support project commercialy? (Answer 197).

Hire someone from the Community seems most optimal for many reasons

Oct 20 2024, 21:29
20after4 accepted D25828: ManiphestReportController: Separate legacy and synthetic data handling.

Nice, it's a good step towards cleaning up this mess!

Oct 20 2024, 21:08
20after4 added Q156: How to support project commercialy? (Answer 197).
Oct 20 2024, 21:01
20after4 accepted D25809: ArcanistBundleTestCase: fix support of non-English environments.
Oct 20 2024, 20:43
20after4 accepted D25829: Update mimemailparser from May 2011 version to 8.0.4.

It would be nice to have some unit tests for this but that seems like it might be quite a bit of work to implement.

Oct 20 2024, 20:37
aklapper added a comment to T15766: Cleanup a bit MimeMailParser.

@valerio.bozzolan Proposing to decline in favor of T15940: Update copy of external mimemailparser library / D25829: Update mimemailparser from May 2011 version to 8.0.4.
Let's not increase the diff to upstream (13 years difference) even more.

Oct 20 2024, 17:26 · Good Starter Task
aklapper requested review of D25829: Update mimemailparser from May 2011 version to 8.0.4.
Oct 20 2024, 17:22
aklapper added a revision to T15940: Update copy of external mimemailparser library: D25829: Update mimemailparser from May 2011 version to 8.0.4.
Oct 20 2024, 17:22
mainframe98 accepted D25809: ArcanistBundleTestCase: fix support of non-English environments.

Confirmed on KDE on Debian 12, everything set to Dutch: LC_ALL evaluates to nl_NL.UTF-8. Fails before applying this change, succeeds afterwards.

Oct 20 2024, 16:56
mainframe98 added a comment to T15927: Fix Arcanist unit test ArcanistBundleTestCase->testNonlocalTrailingNewline() on non-English terminal.

I can reproduce this (KDE on Debian 12, everything set to Dutch: LC_ALL evaluates to nl_NL.UTF-8). diff returns localized output, but doesn't when using LC_ALL=C.

Oct 20 2024, 16:55 · User-valerio.bozzolan
valerio.bozzolan added a comment to T15927: Fix Arcanist unit test ArcanistBundleTestCase->testNonlocalTrailingNewline() on non-English terminal.

Has anybody a terminal with non-English environment and that obtains a non-English output with this command?

Oct 20 2024, 12:25 · User-valerio.bozzolan
valerio.bozzolan added a comment to D25809: ArcanistBundleTestCase: fix support of non-English environments.

Thank you for trying. Let's continue the "how to reproduce" discussion in the bug report T15927

Oct 20 2024, 12:21
avivey added Q156: How to support project commercialy? (Answer 196).
Oct 20 2024, 10:00
avivey added a comment to Q153: php-fpm memory leak.

If it's the daemons that are having problems, you can probably see in the /daemons/ dashboard which tasks are causing problems, and narrow it down from there.

Oct 20 2024, 09:05

Oct 19 2024

aklapper updated the task description for T15940: Update copy of external mimemailparser library.
Oct 19 2024, 15:12
aklapper added a comment to D25828: ManiphestReportController: Separate legacy and synthetic data handling.

ARRRGH, I locally did this on master branch instead of a dedicated branch. Sorry, it's been a while!

Oct 19 2024, 11:00
aklapper added a comment to D25828: ManiphestReportController: Separate legacy and synthetic data handling.

For context, the ManiphestTransaction table in my installation has >10mio entries so some charts do not render but time out. One approach could be Materialized Views but I don't see how that's feasible given the "Project" tag parameter coming into play (and having thousands of project tags in my installation), so my vague followup idea for my installation is to completely disable the code which handles the legacy data only (as creating the modern data does not rely on querying the ManiphestTransaction table), and thus deliberately ignore ancient legacy data from before 2018 for the sake of performance. That's easier to achieve when the upstream spaghetti code doesn't constantly switch between handling legacy and modern/synthetic data.

Oct 19 2024, 10:49
aklapper requested review of D25828: ManiphestReportController: Separate legacy and synthetic data handling.
Oct 19 2024, 10:47
keithzg added a comment to D25723: PHPMailer: Merge class.phpmailer-lite.php into class.phpmailer.php.

For what it's worth I've applied this to my personal instance which tracks master, and I've foisted it on my work instance too---ostensibly tracking stable there, but ;)

Oct 19 2024, 04:07

Oct 18 2024

20after4 added a comment to Q153: php-fpm memory leak.
  1. Can you add a swap file to allow memory usage above 8gb? This will have poor performance but it may allow repo processing to continue.
  2. Can you tell if there is a repo which is still importing? Normal day to day repo tracking shouldn’t require tons of ram so the only situation that I can imagine needing this much ram is importing a large repository.
Oct 18 2024, 17:37
chris accepted D25816: Fix implicitly nullable parameter declarations for PHP 8.4.
Oct 18 2024, 12:52
chris accepted D25814: Fix implicitly nullable parameter declarations for PHP 8.4.
Oct 18 2024, 12:52
chris accepted D25813: Fix implicitly nullable parameter declarations for PHP 8.4.
Oct 18 2024, 12:50
aklapper edited the content of Next Up.
Oct 18 2024, 12:20
aklapper closed T15047: Officially raise minimum required PHP version to 7.2 as Resolved by committing rP23a49eb403c9: Bump PHP version requirement from 5.2.3 to 7.2.25.
Oct 18 2024, 12:20 · Phorge
aklapper closed T15047: Officially raise minimum required PHP version to 7.2, a subtask of T15935: PHP 8.4 fatal error / exceptions: Implicitly nullable parameter declarations deprecated, as Resolved.
Oct 18 2024, 12:20 · PHP 8 support
aklapper closed D25815: Bump PHP version requirement from 5.2.3 to 7.2.25.
Oct 18 2024, 12:20
aklapper committed rP23a49eb403c9: Bump PHP version requirement from 5.2.3 to 7.2.25.
Bump PHP version requirement from 5.2.3 to 7.2.25
Oct 18 2024, 12:20
s.quinn added a comment to Q153: php-fpm memory leak.

What is the normal memory usage for the update daemon during the repository update? It seems to be peaking around 8GB per process for our repositories. Maybe I need to add more RAM, i.e. 32 GB to satisfy this, but that's not easy on AWS.

Oct 18 2024, 11:19
s.quinn updated the answer details for Q153: php-fpm memory leak (Answer 195).
Oct 18 2024, 11:13
avivey added a comment to Q153: php-fpm memory leak.

Repository update is based on how recent the last commit is, but that's only for collecting the list of commits to analyze.
But that should be done in the Daemons, which should at least recover nicely and not break the web side of the application. I don't know if they should be running as php or php-fpm.

Oct 18 2024, 06:44

Oct 17 2024

s.quinn updated the answer details for Q153: php-fpm memory leak (Answer 195).
Oct 17 2024, 15:22
s.quinn added a comment to Q153: php-fpm memory leak.

OK, not the monitoring issue.

Oct 17 2024, 15:22
s.quinn updated the answer details for Q153: php-fpm memory leak (Answer 195).
Oct 17 2024, 15:02
s.quinn added Q153: php-fpm memory leak (Answer 195).
Oct 17 2024, 15:02
s.quinn reopened Q153: php-fpm memory leak.
Oct 17 2024, 14:56
nhg159083 added a watcher for Trusted Contributors: nhg159083.
Oct 17 2024, 13:25

Oct 16 2024

Iniquity added a comment to Update From Phabricator.

@Iniquity - Problem solved by removing that sentence lol - thanks. Now seems super-freshy

Thanks! :)

Oct 16 2024, 09:35
valerio.bozzolan added a comment to D25773: Fix submitting forms in a new tab using Ctrl+Return.

@BlankEclair Do you want a little help to land this?

Oct 16 2024, 09:25
valerio.bozzolan added a comment to Update From Phabricator.

@Iniquity - Problem solved by removing that sentence lol - thanks. Now seems super-freshy

Oct 16 2024, 09:08
valerio.bozzolan edited the content of Update From Phabricator.
Oct 16 2024, 09:08
valerio.bozzolan added a member for Trusted Contributors: bojan.
Oct 16 2024, 08:53
valerio.bozzolan edited the content of Update From Phabricator.
Oct 16 2024, 08:52
bojan added a comment to Update From Phabricator.

Arcanist is misspelled as "Archanist" in the 3rd step of "Updating" section.

Oct 16 2024, 08:41
valerio.bozzolan added Q157: Retrieving task comments using conduit (Answer 194).
Oct 16 2024, 07:36
sicenul asked Q157: Retrieving task comments using conduit.
Oct 16 2024, 07:19
sicenul attached a referenced file: F195188: image.png.
Oct 16 2024, 07:11

Oct 15 2024

avivey created Blog Post: 2024 Summer Edition.
Oct 15 2024, 16:48
avivey edited the content of Change Log.
Oct 15 2024, 16:42
avivey committed rPd3c0a7edc3fc: Promote 2024.35 to stable.
Promote 2024.35 to stable
Oct 15 2024, 16:42
avivey committed rARC05abd055019c: Promote 2024.35 to stable.
Promote 2024.35 to stable
Oct 15 2024, 16:37
valerio.bozzolan added a comment to T15949: Herald comment attributed to the wrong user in email notifications.

Uh! Out of curiosity, what does the notification look like in your web notifications?

Oct 15 2024, 06:52 · Herald, Maniphest
valerio.bozzolan added a member for Trusted Contributors: keithzg.
Oct 15 2024, 06:51
keithzg closed Q147: How does one get the value of a custom field in a custom field extension? as resolved.
Oct 15 2024, 04:15 · Maniphest
keithzg added a comment to Q147: How does one get the value of a custom field in a custom field extension? (Answer 183).

Okay, I got it! For future reference if anyone in a similar situation is stumbling across this and wants a fully working example to go off of, my more-than-a-little-cargo-culted code is currently:

Oct 15 2024, 04:15

Oct 14 2024

keithzg added a comment to Q147: How does one get the value of a custom field in a custom field extension? (Answer 183).

Many thanks, @20after4! That all largely did it; in addition to setting up the object a bit wrong (I knew it was a ManiphestTask instance, but was failing to get the actual object of it), I was also just using essentially my:custom-field rather than std:maniphest:my:custom-field.

Oct 14 2024, 23:07
Tgr updated the task description for T15949: Herald comment attributed to the wrong user in email notifications.
Oct 14 2024, 19:10 · Herald, Maniphest
Tgr created T15949: Herald comment attributed to the wrong user in email notifications.
Oct 14 2024, 19:09 · Herald, Maniphest
s.quinn closed Q153: php-fpm memory leak as invalid.
Oct 14 2024, 13:44
s.quinn added a comment to Q153: php-fpm memory leak.

Appears to be internal monitoring software causing this (not phorge), thanks for the replies.

Oct 14 2024, 13:43
valerio.bozzolan closed Q156: How to support project commercialy? as resolved.
Oct 14 2024, 13:25
valerio.bozzolan updated the answer details for Q156: How to support project commercialy? (Answer 193).
Oct 14 2024, 13:24
valerio.bozzolan added a member for Trusted Contributors: syncer.
Oct 14 2024, 13:13
valerio.bozzolan added Q156: How to support project commercialy? (Answer 193).
Oct 14 2024, 07:16

Oct 10 2024

syncer updated the question details for Q156: How to support project commercialy?.
Oct 10 2024, 23:09
syncer asked Q156: How to support project commercialy?.
Oct 10 2024, 22:48
sirocyl added a comment to T15630: Fix Fonts.

Noting a user complaint (I'm using a mildly hacked-up Phorge to host https://blog.styx-os.org/ - but the font stack is unchanged): https://fe.disroot.org/objects/5d376f6a-0acc-44fc-bd37-31de8630b647

Oct 10 2024, 09:21
avivey awarded T15948: Allow public API access when equivalent public web access exists a Like token.
Oct 10 2024, 06:54 · Conduit, Phorge

Oct 9 2024

20after4 added a comment to T15947: Add method to query if user is member of a list of projects.

Maybe express it as allowed by Policy xxx? (Related to T15277)

Oct 9 2024, 18:23 · Feature Requests
20after4 added a comment to T15947: Add method to query if user is member of a list of projects.

So essentially this:

Oct 9 2024, 18:17 · Feature Requests
20after4 added a comment to T15948: Allow public API access when equivalent public web access exists.

If I were starting today I would probably design the back-end APIs first, then make the web interface derive from those APIs, such that web requests and api requests are not really any different, at least with regard to enforcing access controls.

Oct 9 2024, 18:07 · Conduit, Phorge
valerio.bozzolan awarded T15948: Allow public API access when equivalent public web access exists a Mountain of Wealth token.
Oct 9 2024, 14:56 · Conduit, Phorge
fgaz created T15948: Allow public API access when equivalent public web access exists.
Oct 9 2024, 14:25 · Conduit, Phorge
avivey added a comment to T15947: Add method to query if user is member of a list of projects.

Maybe express it as allowed by Policy xxx? (Related to T15277)

Oct 9 2024, 10:29 · Feature Requests
aklapper created T15947: Add method to query if user is member of a list of projects.
Oct 9 2024, 09:46 · Feature Requests
valerio.bozzolan edited the content of 2024 week 35.
Oct 9 2024, 09:36
aklapper closed T15944: RuntimeException passing a bogus "service" URL parameter to Diffusion commit view as Resolved by committing rPb0bf1c689a9d: Avoid RuntimeException passing bogus "service" parameter to Diffusion commit….
Oct 9 2024, 09:19
aklapper closed D25826: Avoid RuntimeException passing bogus "service" parameter to Diffusion commit view.
Oct 9 2024, 09:19
aklapper committed rPb0bf1c689a9d: Avoid RuntimeException passing bogus "service" parameter to Diffusion commit….
Avoid RuntimeException passing bogus "service" parameter to Diffusion commit…
Oct 9 2024, 09:19
aklapper added a comment to T15944: RuntimeException passing a bogus "service" URL parameter to Diffusion commit view.

No clue :)

Oct 9 2024, 09:17
valerio.bozzolan added a comment to D25827: Allow zooming on pages.

P.S. maybe relevant - maybe not :D lol https://we.phorge.it/T15945#19568

Oct 9 2024, 08:26
20after4 accepted D25826: Avoid RuntimeException passing bogus "service" parameter to Diffusion commit view.
Oct 9 2024, 08:07
20after4 awarded D25826: Avoid RuntimeException passing bogus "service" parameter to Diffusion commit view a Yellow Medal token.
Oct 9 2024, 08:07
20after4 awarded D25827: Allow zooming on pages a Yellow Medal token.
Oct 9 2024, 08:06
20after4 accepted D25827: Allow zooming on pages.

This one seems worthy of a rubber stamp. And a gold star.

Oct 9 2024, 08:05
20after4 updated the answer details for Q155: Is there a way to add new tokens? (Answer 192).
Oct 9 2024, 08:03
20after4 added Q155: Is there a way to add new tokens? (Answer 192).
Oct 9 2024, 08:00
20after4 added a comment to Q153: php-fpm memory leak.

Are you importing some huge git repo or something? I haven't seen a lot of OOM with php-fpm on phorge/phabricator. It's generally not been an issue in the past so this is either a new bug or something specific to the environment. Then again, I've almost always ran Phorge with much more than 8GB of ram available.

Oct 9 2024, 07:50
20after4 added a comment to Q147: How does one get the value of a custom field in a custom field extension? (Answer 183).

For another example, here is a fairly straightforward use of that API which takes place entirely outside of a custom field subclass:

Oct 9 2024, 07:41
20after4 added a comment to Q147: How does one get the value of a custom field in a custom field extension? (Answer 183).

There is a pretty complicated and not really easy to follow subclass of ManiphestCustomField over here: https://phabricator.wikimedia.org/source/phab-extensions/browse/wmf%252Fstable/src/customfields/ReleaseDetailsCustomField.php

Oct 9 2024, 07:33
20after4 added a comment to Q147: How does one get the value of a custom field in a custom field extension? (Answer 183).

@keithzg what you might be missing is

$object = $this->getObject()
Oct 9 2024, 07:31
20after4 updated subscribers of T15501: Voluntary Usage Survey App.

There is a significant amount of Phabricator dark matter out there - companies/people using the software, it works well enough, not really easy to know they exist or anything about their usage. I'm sure at least some of them have moved to Phorge. Automattic/wordpress.com have moved to Phorge and I wasn't even aware that they were using Phabricator before that. This is despite the fact that I did a pretty extensive amount of research to identify every company using Phabricator back in ~2019 as part of my work for Wikimedia, with the goal of reaching out and trying to organize an informal Phabricator users group. We had the idea that the various corporate users probably had good reasons to be collaborating and at least talking to each other since most of them were not active in the upstream project. Anyway, that never really panned out, although it did trigger a flurry of interest and some ongoing discussions via email (maybe even one video meeting but I can't remember the details now.)

Oct 9 2024, 07:16 · Discussion Needed

Oct 8 2024

fgaz created T15946: Support downloading a compressed working copy (tar/zip) of a repository at a specific ref (commit, branch, or tag).
Oct 8 2024, 17:14 · Diffusion, Phorge
valerio.bozzolan added a comment to T15944: RuntimeException passing a bogus "service" URL parameter to Diffusion commit view.

Any guess about a supposed right value for service? lol Just for my curiosity

Oct 8 2024, 12:12

Oct 2 2024

valerio.bozzolan added a comment to T15945: Allow zooming Phorge pages on mobile.

Just talking about accessibility, maybe interesting, Wikipedia on mobile allows to scale, but setting a minimum and maximum scale:

Oct 2 2024, 14:47 · Accessibility
aklapper updated the summary of D25827: Allow zooming on pages.
Oct 2 2024, 12:45
aklapper requested review of D25827: Allow zooming on pages.
Oct 2 2024, 12:44
aklapper added a revision to T15945: Allow zooming Phorge pages on mobile: D25827: Allow zooming on pages.
Oct 2 2024, 12:44 · Accessibility