Bug ReportBugs
ActivePublic

Watchers

  • This project does not have any watchers.

Details

Description

This task was filed via the "New Bug Report" form.

Recent Activity

Yesterday

OCram added a comment to T10252: Username autocompletion doesn't work with some non-US keyboard layouts.

It works great now with Windows + Chrome + German keyboard layout!

Wed, Aug 16, 6:41 AM · Restricted Project, Bug Report

Tue, Aug 15

epriestley moved T12689: Mail is still received after resigning from a revision from Backlog to Reviewer Rules on the Differential board.
Tue, Aug 15, 5:53 PM · Mail, Differential, Bug Report
epriestley moved T12920: Retracting requested changes can leave revision in "Needs Revision" state from Backlog to Reviewer Rules on the Differential board.
Tue, Aug 15, 5:53 PM · Restricted Project, Differential, Bug Report
epriestley closed T12548: Unable to initialize database with invalid credentials in configuration file as Invalid.

Closing this for lack of feedback, feel free to resurrect it if you get back to it.

Tue, Aug 15, 5:34 PM · Database, Bug Report
epriestley closed T12602: Assignees of tasks don't count as subscribers in maniphest search as Invalid.

This isn't a bug; they aren't subscribers, and aren't listed in "Subscribers" in the right-hand column or in "Subscribers" in "Edit Task".

Tue, Aug 15, 5:32 PM · Maniphest, Bug Report

Mon, Aug 14

epriestley added a comment to T12742: `arc land` shows only part of the error that causes the final squash to fail.

@aleb, if you like to continue discussion of this please use the appropriate channels, as @chad suggested above (at the time, they were Conpherence and Ponder, but the appropriate channel is now the community forum -- see: Discourse).

Mon, Aug 14, 4:04 PM · Bug Report
aleb added a comment to T12742: `arc land` shows only part of the error that causes the final squash to fail.

The error truncating behavior affects also Diffusion > R > Status, where only part of the stderr is shown, making it impossible to debug the problem without hacking Phabricator:

ab@phabricator-1-vm:/opt/bitnami$ diff apps/phabricator/libphutil/src/future/exec/CommandException.php.original apps/phabricator/libphutil/src/future/exec/CommandException.php
59c59
<     $limit = 1000;
---
>     $limit = 1000000000;
Mon, Aug 14, 11:09 AM · Bug Report

Fri, Aug 11

chad closed T12755: Aphlict doesn't work with the latest version of `ws` as Resolved by committing rP3b14c1fdd1cf: Update AphlictClientServer to support ws2 or ws3.
Fri, Aug 11, 10:17 PM · Aphlict, Bug Report
chad added a revision to T12755: Aphlict doesn't work with the latest version of `ws`: D18411: Update AphlictClientServer to support ws2 or ws3.
Fri, Aug 11, 10:03 PM · Aphlict, Bug Report

Thu, Aug 10

epriestley closed T12948: Remove the `bin/files purge` workflow as Resolved by committing rP8443366f32d3: Remove `bin/files purge` workflow.
Thu, Aug 10, 3:49 PM · Bug Report, Files
epriestley closed T12960: fancy date picker breaks browser if year < 1970 as Resolved by committing rPe89087fc5136: Fix a hang in fancy date picker for Ye Olde Time Years.
Thu, Aug 10, 3:48 PM · Bug Report
epriestley added a comment to T12871: Setting a restricted dashboard as the topmost item in the global menu can lock some users out.

T12956 has another situation where letting MenuItem generate 2+ items may be bad: we want to let a menu item steal the selection, but it's muddy to implement if each MenuItem can return several actual views.

Thu, Aug 10, 1:38 PM · Policy, Profile Menu Item, Dashboards, Bug Report
joshuaspence added a comment to T12948: Remove the `bin/files purge` workflow.

Thanks for the detailed response, I certainly didn't expect it.

Thu, Aug 10, 1:21 PM · Bug Report, Files
epriestley removed a watcher for Bug Report: bingogo009.
Thu, Aug 10, 1:17 PM
epriestley removed a watcher for Bug Report: saannjaay.
Thu, Aug 10, 1:17 PM
epriestley removed a watcher for Bug Report: Losy.
Thu, Aug 10, 1:17 PM
epriestley added a revision to T12948: Remove the `bin/files purge` workflow: D18384: Remove `bin/files purge` workflow.
Thu, Aug 10, 1:11 PM · Bug Report, Files
epriestley added a comment to T12948: Remove the `bin/files purge` workflow.

I believe the location of the OOM is a little misleading -- the real problem is loadFileData(). This returns the entire file as a string, and will thus:

Thu, Aug 10, 1:04 PM · Bug Report, Files
epriestley added a comment to T12959: Diffusion shows unexpected diff when an entire directory is removed.

The new rules on reports are still in flux a bit and extra-muddy for Community members, but I'd generally like to move to a "report + review" model for community reports, something like we currently have for code review. So the general idea is that your bug needs someone to review it and agree with it (e.g., reproduce a bug, or agree that a feature request is a brilliant new idea that makes sense and would be appropriate for the upstream and completely describes a reasonable, generally-experienced root problem) before it comes upstream.

Thu, Aug 10, 11:47 AM · Bug Report, Diffusion
epriestley added a comment to T12959: Diffusion shows unexpected diff when an entire directory is removed.

I think this is a real bug. Here's a more complete example of what's going wrong, why it's wrong, and how to reproduce it.

Thu, Aug 10, 11:09 AM · Bug Report, Diffusion
epriestley triaged T12960: fancy date picker breaks browser if year < 1970 as Normal priority.
Thu, Aug 10, 10:37 AM · Bug Report
epriestley added a revision to T12960: fancy date picker breaks browser if year < 1970: D18383: Fix a hang in fancy date picker for Ye Olde Time Years.
Thu, Aug 10, 10:36 AM · Bug Report
epriestley claimed T12960: fancy date picker breaks browser if year < 1970.
Thu, Aug 10, 10:36 AM · Bug Report
avivey created T12960: fancy date picker breaks browser if year < 1970.
Thu, Aug 10, 8:19 AM · Bug Report

Wed, Aug 9

chad added a comment to T12959: Diffusion shows unexpected diff when an entire directory is removed.

I would open to accepting this as a bug report if I felt it was deliberated designed to work the way you expect, but was not (like a regression). Or if information was presented that all other repository viewers do it this way. Otherwise I think it's more 'nitpick' or 'feature request'. A cursory look at GitHub, seems they show the full diff as well: https://github.com/phacility/phabricator/commit/5c1e4488dedafda08684b33a8a4786cf687d2811

Wed, Aug 9, 11:07 PM · Bug Report, Diffusion
joshuaspence added a comment to T12959: Diffusion shows unexpected diff when an entire directory is removed.
In T12959#230805, @chad wrote:

In general are you aware we no longer take bug reports or feature requests here?

Wed, Aug 9, 10:36 PM · Bug Report, Diffusion
chad added a comment to T12959: Diffusion shows unexpected diff when an entire directory is removed.

In general are you aware we no longer take bug reports or feature requests here?

Wed, Aug 9, 10:28 PM · Bug Report, Diffusion
joshuaspence renamed T12959: Diffusion shows unexpected diff when an entire directory is removed from Diffusion shows mysterious diff when an entire directory is removed to Diffusion shows unexpected diff when an entire directory is removed.
Wed, Aug 9, 10:23 PM · Bug Report, Diffusion
joshuaspence updated the task description for T12959: Diffusion shows unexpected diff when an entire directory is removed.
Wed, Aug 9, 10:23 PM · Bug Report, Diffusion
joshuaspence added a comment to T12959: Diffusion shows unexpected diff when an entire directory is removed.
In T12959#230799, @chad wrote:

What does "mysterious" mean.

Wed, Aug 9, 10:21 PM · Bug Report, Diffusion
chad added a comment to T12959: Diffusion shows unexpected diff when an entire directory is removed.

What does "mysterious" mean.

Wed, Aug 9, 10:20 PM · Bug Report, Diffusion
joshuaspence updated the task description for T12959: Diffusion shows unexpected diff when an entire directory is removed.
Wed, Aug 9, 10:13 PM · Bug Report, Diffusion
joshuaspence created T12959: Diffusion shows unexpected diff when an entire directory is removed.
Wed, Aug 9, 10:12 PM · Bug Report, Diffusion

Mon, Aug 7

bingogo009 added a watcher for Bug Report: bingogo009.
Mon, Aug 7, 1:09 AM

Sun, Aug 6

chad closed T12952: Reviewer not able to see diff as Resolved by committing rP8ca29a607a2b: Remove incorrect policy language on Diff reviewers.
Sun, Aug 6, 3:08 PM · Differential, Bug Report
chad merged task T12421: Can't verify email address if user doesn't have view policy for any space into T9150: Users without access to any spaces cannot use welcome link to set their password.
Sun, Aug 6, 3:40 AM · Bug Report
chad added a comment to T12575: All-day reccurring events reoccurr for two days instead of one.

I can confirm this.

Sun, Aug 6, 3:22 AM · Calendar, Bug Report
chad closed T11923: Unhandled Exception - Cant import columns for new workboard as Resolved.

Doesn't seem to reproduce any more - I have no issue importing from another milestone. Presumed fixed, but if not, please reopen with more detailed reproduction steps.

Sun, Aug 6, 3:20 AM · Projects, Bug Report
chad closed T12110: Persistent Chat UI broken-ish on clean install as Invalid.

Just launched a test instancepoo and everything seems fine.

Sun, Aug 6, 2:44 AM · Conpherence (v4), Bug Report
chad added a revision to T12952: Reviewer not able to see diff: D18349: Remove incorrect policy language on Diff reviewers.
Sun, Aug 6, 2:34 AM · Differential, Bug Report
joshuaspence added a comment to T12948: Remove the `bin/files purge` workflow.

I wonder if this OOM error can also be hit in other workflow though, given that it seems to occur in PhabricatorFileStorageEngine::getRawFileDataIterator.

Sun, Aug 6, 1:28 AM · Bug Report, Files
joshuaspence added a comment to T12951: File transform regeneration should not rely on passing a query string to a CDN'able resource.

Yep, agreed that the re-targeted proposal is a better solution... I had just assumed that this was a documentation oversight.

Sun, Aug 6, 12:15 AM · Bug Report, Files, Setup

Sat, Aug 5

chad claimed T12952: Reviewer not able to see diff.
Sat, Aug 5, 9:14 PM · Differential, Bug Report
chad added a comment to T12952: Reviewer not able to see diff.

The plan forward here is to correct the text. I will follow up with you on Discourse.

Sat, Aug 5, 9:00 PM · Differential, Bug Report
kugel- added a comment to T12952: Reviewer not able to see diff.

We require the "a revision's reviewers can always..." behavior. Please implement it instead of changing the text.

Sat, Aug 5, 8:54 PM · Differential, Bug Report
epriestley added a comment to T12952: Reviewer not able to see diff.

D7150 added this text ("a revision's reviewers can always...") and I said it "wasn't true but would be soon", but never made it true.

Sat, Aug 5, 7:07 PM · Differential, Bug Report

Fri, Aug 4

chad created T12952: Reviewer not able to see diff.
Fri, Aug 4, 10:25 PM · Differential, Bug Report
markusfluer added a revision to T12755: Aphlict doesn't work with the latest version of `ws`: D18338: Fixed issue: Due to API change of WebSockets event callback had to be changed.
Fri, Aug 4, 10:26 AM · Aphlict, Bug Report

Thu, Aug 3

chad added a commit to T10252: Username autocompletion doesn't work with some non-US keyboard layouts: rP020f3c729a0a: Fix username typeahead in Remarkup with German keyboard layout.
Thu, Aug 3, 5:46 PM · Restricted Project, Bug Report
chad added a comment to T10252: Username autocompletion doesn't work with some non-US keyboard layouts.

fwiw, I can't reproduce this on OSX chrome any more. going to go test windows in german.... wish me glück.

Thu, Aug 3, 4:42 PM · Restricted Project, Bug Report