Page MenuHomePhabricator
Feed Advanced Search

Feb 13 2024

epriestley closed T13701: Perform a large data export from Phacility as Resolved.

(Of course, it'll probably just work the first time now...)

Feb 13 2024, 8:04 PM · Files, Phacility

Feb 12 2024

epriestley added a comment to T13701: Perform a large data export from Phacility.

The export process is already robust at a coarse level: the dump is retained on disk and the process can be retried at the "upload the whole file again" level, then picked up with bin/host export using the --database or --database-file flags (probably with --keep-file).

Feb 12 2024, 11:29 PM · Files, Phacility
epriestley added a comment to T13701: Perform a large data export from Phacility.

The (anonymized) error the process encountered occurred while transferring the dump to central storage was:

Feb 12 2024, 11:19 PM · Files, Phacility
epriestley triaged T13701: Perform a large data export from Phacility as Low priority.
Feb 12 2024, 11:10 PM · Files, Phacility

Oct 26 2022

epriestley closed T13683: Security Guidance: References to Files in Remarkup as Resolved.

There are some remaining non-security bugs with this that I'll follow up on in T13682. I believe the security side of this is now resolved.

Oct 26 2022, 8:03 PM · Guides, Files, Security
epriestley added a comment to T13683: Security Guidance: References to Files in Remarkup .

The details of this attack will be disclosed at a later date, once installs have had some sort of plausible chance to upgrade.

Oct 26 2022, 8:01 PM · Guides, Files, Security
epriestley closed T13685: "RemarkupField" returns "RemarkupValue" when editing defaults, which fails to survive serialization as Resolved.

I filed T13687 as a followup for preventing this particular sort of error (where a Phobject is incorrectly serialized directly).

Oct 26 2022, 7:56 PM · EditEngine, Files

Jul 8 2022

epriestley added a comment to T13685: "RemarkupField" returns "RemarkupValue" when editing defaults, which fails to survive serialization.

That's very likely the same problem, and I think it should be fixed by updating to the current stable (rPf2a7db1 or newer).

Jul 8 2022, 3:57 AM · EditEngine, Files
Krinkle added a comment to T13685: "RemarkupField" returns "RemarkupValue" when editing defaults, which fails to survive serialization.

I believe we may be hitting either the problem one of the above commit fixes, or suffering from a similar caused as side-effect from it.

Jul 8 2022, 3:35 AM · EditEngine, Files

Jun 14 2022

epriestley added a revision to T13685: "RemarkupField" returns "RemarkupValue" when editing defaults, which fails to survive serialization: Restricted Differential Revision.
Jun 14 2022, 4:32 PM · EditEngine, Files
epriestley added a comment to T13685: "RemarkupField" returns "RemarkupValue" when editing defaults, which fails to survive serialization.

...ideally this sort of thing should fail loudly at serialization time...

Jun 14 2022, 4:29 PM · EditEngine, Files
epriestley triaged T13685: "RemarkupField" returns "RemarkupValue" when editing defaults, which fails to survive serialization as Normal priority.
Jun 14 2022, 1:09 PM · EditEngine, Files

May 27 2022

epriestley triaged T13683: Security Guidance: References to Files in Remarkup as Normal priority.
May 27 2022, 6:13 PM · Guides, Files, Security

Mar 15 2021

epriestley moved T13105: Plans: Rich presentation and diff rendering pipelines for various file types from Backlog to Next on the Differential board.
Mar 15 2021, 5:11 PM · Plans, Files, Differential, Diffusion

Mar 11 2021

epriestley added a comment to T12919: Healthcheck endpoint for alternate file domain.

Resource (and Phame and Phurl) domains now respond to /status/.

Mar 11 2021, 10:06 PM · Files
epriestley closed T12919: Healthcheck endpoint for alternate file domain as Resolved by committing Restricted Diffusion Commit.
Mar 11 2021, 10:05 PM · Files
epriestley added a revision to T12919: Healthcheck endpoint for alternate file domain: Restricted Differential Revision.
Mar 11 2021, 9:42 PM · Files
epriestley triaged T12919: Healthcheck endpoint for alternate file domain as Wishlist priority.
Mar 11 2021, 8:55 PM · Files
epriestley added a parent task for T12919: Healthcheck endpoint for alternate file domain: T13636: Some "well-known" resources can't be served from ResourceSite.
Mar 11 2021, 8:55 PM · Files

Mar 10 2021

epriestley closed T13629: Provide alt text for images in more contexts as Resolved.

This promoted without exploding.

Mar 10 2021, 6:21 PM · Files

Mar 5 2021

epriestley added a comment to T13629: Provide alt text for images in more contexts.

These contexts are covered:

Mar 5 2021, 12:53 AM · Files
epriestley added a revision to T13629: Provide alt text for images in more contexts: Restricted Differential Revision.
Mar 5 2021, 12:48 AM · Files
epriestley triaged T13629: Provide alt text for images in more contexts as Low priority.
Mar 5 2021, 12:21 AM · Files

Apr 27 2020

epriestley added a revision to T13522: Compute an internal content hash of large files using hash-of-hashes: Restricted Differential Revision.
Apr 27 2020, 3:29 PM · Differential, Files
epriestley triaged T13522: Compute an internal content hash of large files using hash-of-hashes as Wishlist priority.
Apr 27 2020, 2:48 PM · Differential, Files

Jan 13 2020

epriestley merged T13470: Unable to upload (crashes) large files (eg. 20mb) via file application upload section: https://secure.phabricator.com/file/upload/ into T13256: Update storage engines in Files to a modern multi-service configuration.
Jan 13 2020, 7:26 PM · Files

Sep 8 2019

epriestley triaged T13407: If an S3 request returns with HTTP 404, error handling behavior is poor as Wishlist priority.
Sep 8 2019, 8:22 PM · Files

Jul 31 2019

epriestley updated the task description for T13352: Very large objects may not be distributable via CDN (CloudFront has 20GB object limit).
Jul 31 2019, 12:14 AM · Files, Infrastructure

Jul 30 2019

epriestley triaged T13352: Very large objects may not be distributable via CDN (CloudFront has 20GB object limit) as Low priority.
Jul 30 2019, 11:42 PM · Files, Infrastructure

Jun 24 2019

epriestley closed T13326: "bin/files integrity" misparses file selection arguments as Resolved by committing Restricted Diffusion Commit.
Jun 24 2019, 11:02 PM · Files
epriestley added a revision to T13326: "bin/files integrity" misparses file selection arguments: Restricted Differential Revision.
Jun 24 2019, 10:13 PM · Files

Jun 22 2019

epriestley added a comment to T13306: Provide a mechanism for importing local disk file backups into the Phacility cluster.

See one followup in T13326. The "import from disk" part seems to have worked properly in production.

Jun 22 2019, 4:51 PM · Phacility, Files
epriestley created T13326: "bin/files integrity" misparses file selection arguments.
Jun 22 2019, 4:50 PM · Files

Jun 17 2019

epriestley closed T13313: "Download Raw Diff" may fail in a misleading way after failing to engage the chunk engine if the raw diff is larger than 4MB as Resolved by committing Restricted Diffusion Commit.
Jun 17 2019, 8:31 PM · Differential, Files
epriestley added a revision to T13313: "Download Raw Diff" may fail in a misleading way after failing to engage the chunk engine if the raw diff is larger than 4MB: Restricted Differential Revision.
Jun 17 2019, 4:51 PM · Differential, Files
epriestley created T13313: "Download Raw Diff" may fail in a misleading way after failing to engage the chunk engine if the raw diff is larger than 4MB.
Jun 17 2019, 2:40 PM · Differential, Files

Jun 5 2019

epriestley added a revision to T13306: Provide a mechanism for importing local disk file backups into the Phacility cluster: Restricted Differential Revision.
Jun 5 2019, 9:36 PM · Phacility, Files
epriestley claimed T13306: Provide a mechanism for importing local disk file backups into the Phacility cluster.
Jun 5 2019, 5:59 PM · Phacility, Files
epriestley created T13306: Provide a mechanism for importing local disk file backups into the Phacility cluster.
Jun 5 2019, 5:59 PM · Phacility, Files

May 24 2019

thuffir added a comment to T4752: Daemons may not have permission to remove files from local disk storage engine.

setfacl -Rdm u:phab-phd:rw /var/local/phabricator/filestore
setfacl -Rm u:phab-phd:rw /var/local/phabricator/filestore

May 24 2019, 9:21 AM · Files

Feb 25 2019

epriestley added a parent task for T4752: Daemons may not have permission to remove files from local disk storage engine: T13256: Update storage engines in Files to a modern multi-service configuration.
Feb 25 2019, 12:55 PM · Files
epriestley added a subtask for T13256: Update storage engines in Files to a modern multi-service configuration: T4752: Daemons may not have permission to remove files from local disk storage engine.
Feb 25 2019, 12:55 PM · Files

Feb 23 2019

epriestley closed T9169: Downloading Attachment Extremely Slow as Wontfix.

We no longer offer support for this kind of problem (that technically has reproduction steps, but is sufficiently involved to reproduce that no one has time to follow them, e.g. build a new server from scratch with assorted specific software versions).

Feb 23 2019, 3:37 PM · Bug Report (Needs Information), Files
epriestley triaged T13256: Update storage engines in Files to a modern multi-service configuration as Low priority.
Feb 23 2019, 3:26 PM · Files

Jun 28 2018

20after4 added a comment to T13103: Document favicon customization and validate the "ui.favicons" configuration option.

Can't wait for "connect 2" to come out on playstation six.

Jun 28 2018, 11:53 AM · Files, Aphront, Infrastructure

Jun 1 2018

epriestley closed T13148: css: ginormous images go over preview panel in Files as Resolved by committing Restricted Diffusion Commit.
Jun 1 2018, 9:53 PM · Files
epriestley added a revision to T13105: Plans: Rich presentation and diff rendering pipelines for various file types: Restricted Differential Revision.
Jun 1 2018, 8:14 PM · Plans, Files, Differential, Diffusion
epriestley added a revision to T13148: css: ginormous images go over preview panel in Files: Restricted Differential Revision.
Jun 1 2018, 8:14 PM · Files
epriestley claimed T13148: css: ginormous images go over preview panel in Files.
Jun 1 2018, 8:09 PM · Files

May 4 2018

epriestley closed T13135: Chrome does not play some videos with type "video/quicktime", but does play them with type "video/mp4" as Resolved by committing Restricted Diffusion Commit.
May 4 2018, 4:28 PM · Files
epriestley added a revision to T13135: Chrome does not play some videos with type "video/quicktime", but does play them with type "video/mp4": Restricted Differential Revision.
May 4 2018, 4:25 PM · Files
epriestley added a comment to T13135: Chrome does not play some videos with type "video/quicktime", but does play them with type "video/mp4".

I like that reasoning a lot better than mine, and simply omitting type appears to produce the correct behavior in every browser, at least for this file. I'll try that instead, and we can revisit this after we write a video transcoder in PHP and can offer files in multiple formats.

May 4 2018, 3:14 PM · Files
asherkin added a comment to T13135: Chrome does not play some videos with type "video/quicktime", but does play them with type "video/mp4".

My knowledge is centred around broadcast video / IPTV (UDP multicast) rather than HTML5 video, but that seems fairly reasonable. I don't think anything should try and download the file twice unless it is a complete clownshoes implementation that probably has multiple other serious bugs wasting bandwidth. I think the worst case with no type that might be hit here is that the browser could decide to download the entire file on load to figure out the format and duration, rather than starting with byte-range requests, but this is easy to test and unlikely (I'd expect any sane one to always request chunks).

May 4 2018, 3:11 PM · Files
epriestley added a comment to T13135: Chrome does not play some videos with type "video/quicktime", but does play them with type "video/mp4".

I can imagine that two <source /> tags might, in some bizarre world, cause browsers to download the file twice when you click "Play", if they're super confused about how to process videos, don't notice that the URLs are the same, and don't hit any caching. But that's a pretty minor bad effect, and I didn't immediately see any kind of bad behavior locally.

May 4 2018, 2:50 PM · Files
epriestley added a comment to T13135: Chrome does not play some videos with type "video/quicktime", but does play them with type "video/mp4".

Thanks! I wouldn't mind parsing that but I'm hesitant to ask installs to install it -- but it's helpful in understanding that I'm not completely crazy here and that there's at least some basis for "video/mp4" being a quasi-legitimate way to label the video file.

May 4 2018, 2:47 PM · Files
asherkin added a comment to T13135: Chrome does not play some videos with type "video/quicktime", but does play them with type "video/mp4".

mediainfo is the tool for this, but is really overkill for programmatic use.

May 4 2018, 1:29 PM · Files

May 3 2018

epriestley added a comment to T13135: Chrome does not play some videos with type "video/quicktime", but does play them with type "video/mp4".

One easy approach we could take is just:

May 3 2018, 3:45 PM · Files
epriestley triaged T13135: Chrome does not play some videos with type "video/quicktime", but does play them with type "video/mp4" as Normal priority.
May 3 2018, 3:31 PM · Files

May 1 2018

epriestley closed T13132: `arc download` broken for non-CDN Phabricator as Resolved by committing Restricted Diffusion Commit.
May 1 2018, 5:08 PM · Conduit, Files
epriestley added projects to T13132: `arc download` broken for non-CDN Phabricator : Files, Conduit.
May 1 2018, 4:06 PM · Conduit, Files

Apr 28 2018

epriestley added a revision to T13105: Plans: Rich presentation and diff rendering pipelines for various file types: Restricted Differential Revision.
Apr 28 2018, 1:55 PM · Plans, Files, Differential, Diffusion
epriestley added a comment to T13105: Plans: Rich presentation and diff rendering pipelines for various file types.

From PHI604, for completeness, on the newer behavior of "Hide Blame":

Apr 28 2018, 1:52 PM · Plans, Files, Differential, Diffusion

Apr 27 2018

epriestley added a comment to T13103: Document favicon customization and validate the "ui.favicons" configuration option.

validation for the configuration option

Apr 27 2018, 7:00 PM · Files, Aphront, Infrastructure

Apr 25 2018

epriestley added a revision to T13103: Document favicon customization and validate the "ui.favicons" configuration option: Restricted Differential Revision.
Apr 25 2018, 1:45 PM · Files, Aphront, Infrastructure

Apr 17 2018

epriestley added a subtask for T13105: Plans: Rich presentation and diff rendering pipelines for various file types: T13126: Plans: Diffusion cleanup for document engine.
Apr 17 2018, 10:02 PM · Plans, Files, Differential, Diffusion
epriestley added a revision to T13105: Plans: Rich presentation and diff rendering pipelines for various file types: Restricted Differential Revision.
Apr 17 2018, 1:46 PM · Plans, Files, Differential, Diffusion
epriestley added a revision to T13105: Plans: Rich presentation and diff rendering pipelines for various file types: Restricted Differential Revision.
Apr 17 2018, 12:54 PM · Plans, Files, Differential, Diffusion

Apr 16 2018

epriestley added a comment to T13105: Plans: Rich presentation and diff rendering pipelines for various file types.

See T13125 for a more detailed breakout of coverage plans.

Apr 16 2018, 12:50 PM · Plans, Files, Differential, Diffusion

Apr 15 2018

epriestley added a comment to T13105: Plans: Rich presentation and diff rendering pipelines for various file types.

This particular page here is fataling:

Apr 15 2018, 5:06 PM · Plans, Files, Differential, Diffusion

Apr 11 2018

epriestley added a revision to T13105: Plans: Rich presentation and diff rendering pipelines for various file types: Restricted Differential Revision.
Apr 11 2018, 9:47 PM · Plans, Files, Differential, Diffusion
epriestley added a revision to T13105: Plans: Rich presentation and diff rendering pipelines for various file types: Restricted Differential Revision.
Apr 11 2018, 9:30 PM · Plans, Files, Differential, Diffusion
epriestley added a revision to T13105: Plans: Rich presentation and diff rendering pipelines for various file types: Restricted Differential Revision.
Apr 11 2018, 9:15 PM · Plans, Files, Differential, Diffusion
epriestley added a revision to T13105: Plans: Rich presentation and diff rendering pipelines for various file types: Restricted Differential Revision.
Apr 11 2018, 8:45 PM · Plans, Files, Differential, Diffusion
epriestley added a comment to T13105: Plans: Rich presentation and diff rendering pipelines for various file types.
  • Source view rendering in Paste has a missing background color.
Apr 11 2018, 7:06 PM · Plans, Files, Differential, Diffusion

Apr 9 2018

epriestley added a revision to T13105: Plans: Rich presentation and diff rendering pipelines for various file types: Restricted Differential Revision.
Apr 9 2018, 1:09 PM · Plans, Files, Differential, Diffusion
epriestley added a revision to T13105: Plans: Rich presentation and diff rendering pipelines for various file types: Restricted Differential Revision.
Apr 9 2018, 12:40 PM · Plans, Files, Differential, Diffusion
epriestley added a revision to T13105: Plans: Rich presentation and diff rendering pipelines for various file types: Restricted Differential Revision.
Apr 9 2018, 12:26 PM · Plans, Files, Differential, Diffusion
epriestley added a revision to T13105: Plans: Rich presentation and diff rendering pipelines for various file types: Restricted Differential Revision.
Apr 9 2018, 12:20 PM · Plans, Files, Differential, Diffusion
epriestley added a revision to T13105: Plans: Rich presentation and diff rendering pipelines for various file types: Restricted Differential Revision.
Apr 9 2018, 12:16 PM · Plans, Files, Differential, Diffusion
epriestley added a comment to T13105: Plans: Rich presentation and diff rendering pipelines for various file types.

Still needs work:

Apr 9 2018, 12:15 PM · Plans, Files, Differential, Diffusion

Apr 8 2018

epriestley added a revision to T13105: Plans: Rich presentation and diff rendering pipelines for various file types: Restricted Differential Revision.
Apr 8 2018, 7:53 PM · Plans, Files, Differential, Diffusion
epriestley added a revision to T13105: Plans: Rich presentation and diff rendering pipelines for various file types: Restricted Differential Revision.
Apr 8 2018, 4:34 PM · Plans, Files, Differential, Diffusion
epriestley added a revision to T13105: Plans: Rich presentation and diff rendering pipelines for various file types: Restricted Differential Revision.
Apr 8 2018, 3:13 PM · Plans, Files, Differential, Diffusion
epriestley added a revision to T13105: Plans: Rich presentation and diff rendering pipelines for various file types: Restricted Differential Revision.
Apr 8 2018, 1:33 PM · Plans, Files, Differential, Diffusion
epriestley added a comment to T13105: Plans: Rich presentation and diff rendering pipelines for various file types.
  • The secret $1-2,7-9 syntax for highlighting multiple noncontiguous blocks of lines in Diffusion is no longer supported.
Apr 8 2018, 1:31 PM · Plans, Files, Differential, Diffusion

Apr 5 2018

epriestley added a revision to T13105: Plans: Rich presentation and diff rendering pipelines for various file types: Restricted Differential Revision.
Apr 5 2018, 6:40 PM · Plans, Files, Differential, Diffusion
epriestley added a comment to T13105: Plans: Rich presentation and diff rendering pipelines for various file types.
  • If a file is non-UTF8, we should try the repository's "Encoding" as a default (or maybe just remove this feature).
Apr 5 2018, 5:50 PM · Plans, Files, Differential, Diffusion
epriestley added a comment to T13105: Plans: Rich presentation and diff rendering pipelines for various file types.

Junk I'm about to break in Diffusion (but plan to restore):

Apr 5 2018, 5:38 PM · Plans, Files, Differential, Diffusion
epriestley added a revision to T13105: Plans: Rich presentation and diff rendering pipelines for various file types: Restricted Differential Revision.
Apr 5 2018, 4:17 PM · Plans, Files, Differential, Diffusion
epriestley added a revision to T13105: Plans: Rich presentation and diff rendering pipelines for various file types: Restricted Differential Revision.
Apr 5 2018, 2:33 PM · Plans, Files, Differential, Diffusion

Mar 30 2018

epriestley added a revision to T13105: Plans: Rich presentation and diff rendering pipelines for various file types: Restricted Differential Revision.
Mar 30 2018, 6:15 PM · Plans, Files, Differential, Diffusion
epriestley added a revision to T13105: Plans: Rich presentation and diff rendering pipelines for various file types: Restricted Differential Revision.
Mar 30 2018, 4:48 PM · Plans, Files, Differential, Diffusion

Mar 28 2018

WTreutterer added a comment to T13105: Plans: Rich presentation and diff rendering pipelines for various file types.

Changeset rPfb4ce851c4ae: Add a PDF document "rendering" engine allows to view PDFs from the Files application. However, {Fxxx, ...} references to a PDF file in Phriction still only allow to download the file or open a lightbox with only an icon (again with a download link).

Mar 28 2018, 8:08 AM · Plans, Files, Differential, Diffusion

Mar 26 2018

epriestley added a revision to T13105: Plans: Rich presentation and diff rendering pipelines for various file types: Restricted Differential Revision.
Mar 26 2018, 2:23 PM · Plans, Files, Differential, Diffusion
epriestley added a revision to T13105: Plans: Rich presentation and diff rendering pipelines for various file types: Restricted Differential Revision.
Mar 26 2018, 2:20 PM · Plans, Files, Differential, Diffusion
epriestley added a revision to T13105: Plans: Rich presentation and diff rendering pipelines for various file types: Restricted Differential Revision.
Mar 26 2018, 2:16 PM · Plans, Files, Differential, Diffusion

Mar 23 2018

epriestley added a revision to T13105: Plans: Rich presentation and diff rendering pipelines for various file types: Restricted Differential Revision.
Mar 23 2018, 8:49 PM · Plans, Files, Differential, Diffusion
epriestley added a revision to T13105: Plans: Rich presentation and diff rendering pipelines for various file types: Restricted Differential Revision.
Mar 23 2018, 2:41 PM · Plans, Files, Differential, Diffusion
epriestley added a revision to T13105: Plans: Rich presentation and diff rendering pipelines for various file types: Restricted Differential Revision.
Mar 23 2018, 2:06 PM · Plans, Files, Differential, Diffusion
epriestley added a revision to T13105: Plans: Rich presentation and diff rendering pipelines for various file types: Restricted Differential Revision.
Mar 23 2018, 12:10 PM · Plans, Files, Differential, Diffusion