Page MenuHomePhabricator
Feed Advanced Search

Jun 27 2016

thoughtpolice renamed T11219: Exception on 'View All' at Phame top-level (on upstream) from Error on 'View All' with Phame (on upstream) to Exception on 'View All' at Phame top-level (on upstream).
Jun 27 2016, 3:58 PM · Phame, Bug Report
thoughtpolice created T11219: Exception on 'View All' at Phame top-level (on upstream).
Jun 27 2016, 3:54 PM · Phame, Bug Report
epriestley closed T10982: Harbormaster builds fail to allocate resource as Invalid.

This isn't an actionable bug report because it doesn't include reproduction instructions. Reports MUST include reproduction instructions; see Contributing Bug Reports and Providing Reproduction Steps.

Jun 27 2016, 3:32 PM · Harbormaster, Bug Report
ilendir added a comment to T10982: Harbormaster builds fail to allocate resource.

Nevertheless, I'm still not sure how to debug this error message / my configuration.
Is there a way to test/run the Drydock Host Blueprint and get more information than the command failed?

Jun 27 2016, 3:10 PM · Harbormaster, Bug Report
epriestley added a revision to T10982: Harbormaster builds fail to allocate resource: Restricted Differential Revision.
Jun 27 2016, 2:55 PM · Harbormaster, Bug Report
ilendir added a comment to T10982: Harbormaster builds fail to allocate resource.

I would say, definitively. 'xxxxx' looks more like a placeholder for me, but that might be a personal preference.

Jun 27 2016, 2:48 PM · Harbormaster, Bug Report
epriestley added a comment to T10982: Harbormaster builds fail to allocate resource.

If we masked with ***** instead of xxxxx, do you think that would have been more intuitive?

Jun 27 2016, 1:51 PM · Harbormaster, Bug Report
kugel- added a comment to T10608: Arc diff failing to generate diff with svn 1.9.3.

I am also affected. Is there any known workaround?

Jun 27 2016, 1:06 PM · Arcanist, Bug Report
ilendir added a comment to T10982: Harbormaster builds fail to allocate resource.

So, the information is just masked but really present in the real command call? The ssh man-page states what the options before the xxxxxs mean, I'm just not sure if the call is right when the information is masked. E.g. the clone call...does phabricator call with git@phabricator... or with the username provided for the ssh connection. anyway. Is there a log where the real command is written? Why is it even masked here?

Jun 27 2016, 12:44 PM · Harbormaster, Bug Report
asherkin added a comment to T10982: Harbormaster builds fail to allocate resource.

the -l xxxxx is your configured SSH user, the -i xxxxx is the path to your configured SSH private key (it's written out to a temporary file).

Jun 27 2016, 11:29 AM · Harbormaster, Bug Report

Jun 25 2016

epriestley closed T11212: One of our audits is displaying comments from another audit as Invalid.

Bug reports MUST include reproduction steps that allow us to reproduce the issue locally. Without this information, we can not move forward.

Jun 25 2016, 2:15 PM · Quicksand, Audit, Bug Report
epriestley closed T11214: bin/phd status lists long dead daemons as Invalid.

Bug reports MUST include reproduction steps that allow us to reproduce the issue locally. Without this information, we can not move forward.

Jun 25 2016, 2:15 PM · Daemons, Bug Report
joshuaspence added a project to T11178: Some linters fail without a pathway forward if the PHP "xml" extension is not installed: Lint.
Jun 25 2016, 11:21 AM · Lint, Arcanist, Bug Report
joshuaspence updated the task description for T11178: Some linters fail without a pathway forward if the PHP "xml" extension is not installed.
Jun 25 2016, 11:19 AM · Lint, Arcanist, Bug Report
joshuaspence renamed T11205: Hard/Impossible to disable SSH built-in URIs from Hard/Impossible to disable SSH built-in uris to Hard/Impossible to disable SSH built-in URIs.
Jun 25 2016, 10:51 AM · Diffusion (v3), Bug Report
joshuaspence added a project to T11169: User without "can edit" permission, can change task via comment action: Policy.
Jun 25 2016, 10:50 AM · Policy, Bug Report
joshuaspence added a project to T11207: User can change properties on the task, even when not having edit policy rights: Policy.
Jun 25 2016, 10:49 AM · Restricted Project, Policy, Bug Report
joshuaspence added a project to T11212: One of our audits is displaying comments from another audit: Audit.
Jun 25 2016, 6:01 AM · Quicksand, Audit, Bug Report
joshuaspence updated the task description for T11214: bin/phd status lists long dead daemons.
Jun 25 2016, 5:46 AM · Daemons, Bug Report
joshuaspence added a project to T11214: bin/phd status lists long dead daemons: Daemons.
Jun 25 2016, 5:45 AM · Daemons, Bug Report

Jun 24 2016

anton.vladimirov created T11214: bin/phd status lists long dead daemons.
Jun 24 2016, 11:06 PM · Daemons, Bug Report
epriestley merged task T10610: ExecFuture does not kill child processes on timeout into T4551: consider allowing SIGTERM for ExecFuture timeout.
Jun 24 2016, 9:38 PM · libphutil, Bug Report
chad merged task T11213: Inline comments show up in reverse order in the summary into T8739: Inline comments not sorted.
Jun 24 2016, 8:47 PM · Bug Report
anton.vladimirov created T11213: Inline comments show up in reverse order in the summary.
Jun 24 2016, 8:45 PM · Bug Report
avivey added a comment to T11209: JIRA links with really long names overflow in Revision Details view.

happycat

Jun 24 2016, 8:41 PM · Remarkup, Doorkeeper, Restricted Project, Bug Report, Restricted Project, Differential
chad closed T11209: JIRA links with really long names overflow in Revision Details view as Resolved by committing Restricted Diffusion Commit.
Jun 24 2016, 8:41 PM · Remarkup, Doorkeeper, Restricted Project, Bug Report, Restricted Project, Differential
anton.vladimirov added a comment to T11212: One of our audits is displaying comments from another audit.

Some more info I dug up:

  • There are 2 commits that appear to have flipped comments, i.e. the comments that appear on audit1 actually apply to audit2 and vice versa.
  • These commits are sequential and were pushed simultaneously (i.e. the author made multiple commits on master locally before git pushing them all)
  • In both cases only comments made by the author appear out of place.
Jun 24 2016, 8:27 PM · Quicksand, Audit, Bug Report
anton.vladimirov created T11212: One of our audits is displaying comments from another audit.
Jun 24 2016, 8:09 PM · Quicksand, Audit, Bug Report
chad added a revision to T11209: JIRA links with really long names overflow in Revision Details view: Restricted Differential Revision.
Jun 24 2016, 7:10 PM · Remarkup, Doorkeeper, Restricted Project, Bug Report, Restricted Project, Differential
avivey added a comment to T11209: JIRA links with really long names overflow in Revision Details view.

I think it starts out as a URL, and then replaced on the client-side with the name after some AJAX. Most of the time I don't have time to see the "before" text.

Jun 24 2016, 7:05 PM · Remarkup, Doorkeeper, Restricted Project, Bug Report, Restricted Project, Differential
chad added a comment to T11209: JIRA links with really long names overflow in Revision Details view.

I don't understand why it's a tag. Does the JIRA link look like a URL or an actual name?

Jun 24 2016, 7:03 PM · Remarkup, Doorkeeper, Restricted Project, Bug Report, Restricted Project, Differential
avivey added a comment to T11209: JIRA links with really long names overflow in Revision Details view.

css is it's own world of magick.

Jun 24 2016, 7:02 PM · Remarkup, Doorkeeper, Restricted Project, Bug Report, Restricted Project, Differential
chad added a comment to T11209: JIRA links with really long names overflow in Revision Details view.

I'm faking my way through it.

Jun 24 2016, 7:02 PM · Remarkup, Doorkeeper, Restricted Project, Bug Report, Restricted Project, Differential
avivey added a comment to T11209: JIRA links with really long names overflow in Revision Details view.

Here's Chrome's Style page for the element:

pasted_file (990×464 px, 101 KB)

Jun 24 2016, 7:00 PM · Remarkup, Doorkeeper, Restricted Project, Bug Report, Restricted Project, Differential
chad added a comment to T11209: JIRA links with really long names overflow in Revision Details view.

I'm just surprised to hear something is still overflowing after D16081.

Jun 24 2016, 6:53 PM · Remarkup, Doorkeeper, Restricted Project, Bug Report, Restricted Project, Differential
avivey added a comment to T11209: JIRA links with really long names overflow in Revision Details view.

DoorkeeperTagView is also used for DifferentialAsanaRepresentationField, which is probably easier to set up (?).

Jun 24 2016, 6:51 PM · Remarkup, Doorkeeper, Restricted Project, Bug Report, Restricted Project, Differential
avivey added a comment to T11209: JIRA links with really long names overflow in Revision Details view.

The value is rendered here: https://secure.phabricator.com/diffusion/P/browse/master/src/applications/differential/customfield/DifferentialJIRAIssuesField.php;70463e8a16d4cbe0c83b23b9328c4a359072c506$53

Jun 24 2016, 6:48 PM · Remarkup, Doorkeeper, Restricted Project, Bug Report, Restricted Project, Differential
chad added a comment to T11209: JIRA links with really long names overflow in Revision Details view.

I'd need to see it to fix it. These are tags?

Jun 24 2016, 6:42 PM · Remarkup, Doorkeeper, Restricted Project, Bug Report, Restricted Project, Differential
avivey renamed T11209: JIRA links with really long names overflow in Revision Details view from JIRA links with really long names in Revision Details view overflow to JIRA links with really long names overflow in Revision Details view.
Jun 24 2016, 6:25 PM · Remarkup, Doorkeeper, Restricted Project, Bug Report, Restricted Project, Differential
eadler added a project to T11209: JIRA links with really long names overflow in Revision Details view: Remarkup.
Jun 24 2016, 6:21 PM · Remarkup, Doorkeeper, Restricted Project, Bug Report, Restricted Project, Differential
eadler added projects to T11209: JIRA links with really long names overflow in Revision Details view: Restricted Project, Doorkeeper.
Jun 24 2016, 6:20 PM · Remarkup, Doorkeeper, Restricted Project, Bug Report, Restricted Project, Differential
avivey created T11209: JIRA links with really long names overflow in Revision Details view.
Jun 24 2016, 6:20 PM · Remarkup, Doorkeeper, Restricted Project, Bug Report, Restricted Project, Differential
eadler added a project to T10323: "Action With No Effect" when marking inline comments as done: Restricted Project.
Jun 24 2016, 6:19 PM · Restricted Project, Restricted Project, Bug Report, Differential
kibs added a comment to T11207: User can change properties on the task, even when not having edit policy rights.

Alright, thanks for clearing it up, and sorry for calling it a bug :) I am new to phabricator, trying to see if I can configure it for my needs where permissions have a big role. Maybe I should explain what I am trying to do, so my request maybe makes more sense. You are of course welcome to close this task as wontfix, I just, as a new user, found it confusing that the edit policy wasn't strictly enforced around the application, view policy seems more strict though, which is nice :)

Jun 24 2016, 6:10 PM · Restricted Project, Policy, Bug Report
ilendir added a comment to T10982: Harbormaster builds fail to allocate resource.

I have a somewhat similar problem when I try to start a working copy job:

Jun 24 2016, 12:19 PM · Harbormaster, Bug Report
epriestley added a comment to T11207: User can change properties on the task, even when not having edit policy rights.

There are many situations where you can make an edit in one way but not in another. For example, you can't edit the subscribers for a task via "Edit Task" on the detail page, workboards, or task list, or via the API, or via mail commands but you can edit them via "Subscribe", "Unsubscribe", with Herald, by commenting with @username, or via the action dropdown.

Jun 24 2016, 12:06 PM · Restricted Project, Policy, Bug Report
kibs added a comment to T11207: User can change properties on the task, even when not having edit policy rights.

Thanks for the quick reply.

Jun 24 2016, 12:02 PM · Restricted Project, Policy, Bug Report
epriestley added a comment to T11207: User can change properties on the task, even when not having edit policy rights.

This is expected. You do not need "edit" permission to comment on a task, subscribe to it, award it tokens, or take other actions available in the action dropdown.

Jun 24 2016, 11:59 AM · Restricted Project, Policy, Bug Report
kibs created T11207: User can change properties on the task, even when not having edit policy rights.
Jun 24 2016, 8:22 AM · Restricted Project, Policy, Bug Report
chad closed T11206: No categorization of bugs as Invalid.

Assume you are testing. Please use http://phacility.com if you want to try Phabricator at no cost. This is a working installation.

Jun 24 2016, 6:36 AM · Bug Report
• alan.raj created T11206: No categorization of bugs.
Jun 24 2016, 6:31 AM · Bug Report
avivey created T11205: Hard/Impossible to disable SSH built-in URIs.
Jun 24 2016, 1:57 AM · Diffusion (v3), Bug Report

Jun 23 2016

chad added a comment to T11204: Sometimes maniphest status is not correctly updated after maniphest.update conduit api call returns success response.

This issue is not always reproducible but happens very frequently:

Jun 23 2016, 9:47 PM · Bug Report (Needs Information)
chad added a comment to T11204: Sometimes maniphest status is not correctly updated after maniphest.update conduit api call returns success response.

Updating Phabricator is required before submitting a bug report.

Jun 23 2016, 9:35 PM · Bug Report (Needs Information)
zhiyuchen updated the task description for T11204: Sometimes maniphest status is not correctly updated after maniphest.update conduit api call returns success response.
Jun 23 2016, 9:32 PM · Bug Report (Needs Information)
zhiyuchen updated the task description for T11204: Sometimes maniphest status is not correctly updated after maniphest.update conduit api call returns success response.
Jun 23 2016, 9:30 PM · Bug Report (Needs Information)
zhiyuchen created T11204: Sometimes maniphest status is not correctly updated after maniphest.update conduit api call returns success response.
Jun 23 2016, 9:26 PM · Bug Report (Needs Information)
epriestley closed T11203: Observing svn+ssh on non-standard ssh port as Resolved by committing Restricted Diffusion Commit.
Jun 23 2016, 2:40 PM · Bug Report
epriestley added a revision to T11203: Observing svn+ssh on non-standard ssh port: Restricted Differential Revision.
Jun 23 2016, 2:35 PM · Bug Report
enckse added a comment to T11203: Observing svn+ssh on non-standard ssh port.

I've applied the patch and updated by URI accordingly (removed the ssh config as well) - seems to work

Jun 23 2016, 2:13 PM · Bug Report
epriestley added a comment to T11203: Observing svn+ssh on non-standard ssh port.

Yes, put the port in the URI like a normal URI (host.com:1234).

Jun 23 2016, 2:02 PM · Bug Report
enckse added a comment to T11203: Observing svn+ssh on non-standard ssh port.

I'm willing to give it a go, just to confirm before I do that what my URI would be in the UI?

svn+ssh://svn.host.name:port/srv/repo/url/path/
Jun 23 2016, 2:01 PM · Bug Report
epriestley closed T11201: Exception when destroying an object with edges that no longer have corresponding edge definitions as Resolved by committing Restricted Diffusion Commit.
Jun 23 2016, 1:45 PM · Bug Report
epriestley added a comment to T11203: Observing svn+ssh on non-standard ssh port.

If you have time, can you let me know if this patch fixes things?

Jun 23 2016, 1:42 PM · Bug Report
enckse created T11203: Observing svn+ssh on non-standard ssh port.
Jun 23 2016, 1:06 PM · Bug Report
epriestley renamed T11201: Exception when destroying an object with edges that no longer have corresponding edge definitions from Exception when destroy diffusion repository to Exception when destroying an object with edges that no longer have corresponding edge definitions.
Jun 23 2016, 12:53 PM · Bug Report
epriestley merged task T10525: "Ghost" user created in external accounts table under certain circumstances into T5953: Build a modular user account batch importer.
Jun 23 2016, 12:49 PM · LDAP, Auth, Bug Report
epriestley added a comment to T10525: "Ghost" user created in external accounts table under certain circumstances.

I'm just going to merge this into T5953. The LDAP importer is a contributed patch from yesteryear which we wouldn't bring upstream today, and unlikely to see improvements outside of a more formal upstream effort in T5953.

Jun 23 2016, 12:49 PM · LDAP, Auth, Bug Report
epriestley closed T10489: Unexpected Remarkup behavior for links as Wontfix.

git blame suggests that this hasn't ever matched on word boundaries (it didn't in August 2012, when it was first written -- see rPHU45f7bd6a).

Jun 23 2016, 12:46 PM · Bug Report, Remarkup
epriestley closed T10433: Repo in Diffusion has no padding after Branches box if footer is not configured as Resolved.

I can't reproduce this on the linked install or locally either; presuming we fixed it at some point.

Jun 23 2016, 12:38 PM · Diffusion, Bug Report
epriestley merged task T10380: Editing subscribers list should allow selection of exact or closest matched username into T8510: Typeahead project proposals in Maniphest advanced search do not always include exact matches.
Jun 23 2016, 12:35 PM · Wikimedia, Maniphest, Bug Report
epriestley closed T10376: Error handling for mail send-test needs improvement as Wontfix.

We haven't seen other users encounter difficulty with this, and the error message is relatively specific already.

Jun 23 2016, 12:34 PM · Mail, Bug Report
epriestley closed T10326: Property Lists use "Projects", not "Tags" as Resolved.

This is sort of resolved since Diffusion moved to PHUITwoColumnView and project tags are now listed in the sidebar, labeled "Tags", while VCS tags are listed in the main list, also labeled "Tags". Labeling them both "tags" isn't great, but context provides some help and this doesn't immediately appear to be particularly confusing.

Jun 23 2016, 12:31 PM · Projects, Diffusion, Bug Report
epriestley added a comment to T11201: Exception when destroying an object with edges that no longer have corresponding edge definitions.

When we destroy an object, we load all its edges and destroy them too.

Jun 23 2016, 12:18 PM · Bug Report
epriestley claimed T11201: Exception when destroying an object with edges that no longer have corresponding edge definitions.
Jun 23 2016, 12:17 PM · Bug Report
epriestley added a revision to T11201: Exception when destroying an object with edges that no longer have corresponding edge definitions: Restricted Differential Revision.
Jun 23 2016, 12:16 PM · Bug Report
netroby added a comment to T11201: Exception when destroying an object with edges that no longer have corresponding edge definitions.

The repository created last year.

Jun 23 2016, 5:59 AM · Bug Report
netroby added a comment to T11201: Exception when destroying an object with edges that no longer have corresponding edge definitions.

I am running Phabricator up to date (I upgrade it today, using git pull --rebase)

Jun 23 2016, 5:58 AM · Bug Report
chad merged T11202: 502 while access /config/issue into T10987: /config/issue/ causes coredump in PHP-FPM.
Jun 23 2016, 3:40 AM · Bug Report
chad merged task T11202: 502 while access /config/issue into T10987: /config/issue/ causes coredump in PHP-FPM.
Jun 23 2016, 3:40 AM · Bug Report
netroby updated the task description for T11202: 502 while access /config/issue.
Jun 23 2016, 3:29 AM · Bug Report
netroby updated the task description for T11202: 502 while access /config/issue.
Jun 23 2016, 3:28 AM · Bug Report
netroby created T11202: 502 while access /config/issue.
Jun 23 2016, 3:27 AM · Bug Report
chad updated subscribers of T11201: Exception when destroying an object with edges that no longer have corresponding edge definitions.

Edge Constant 39 was TYPE_OBJECT_USES_CREDENTIAL, which was removed a few months back.

Jun 23 2016, 2:35 AM · Bug Report
chad added a comment to T11201: Exception when destroying an object with edges that no longer have corresponding edge definitions.

What are the steps to reproduce the error, specifically? I can't trigger this exception.

Jun 23 2016, 1:59 AM · Bug Report
netroby created T11201: Exception when destroying an object with edges that no longer have corresponding edge definitions.
Jun 23 2016, 1:52 AM · Bug Report

Jun 22 2016

chad closed T11200: Cannot access installation Phabricator script (install_ubuntu.sh) from the website. as Resolved.

Sorry for the disruption.

Jun 22 2016, 9:00 PM · Bug Report
avivey added a comment to T11200: Cannot access installation Phabricator script (install_ubuntu.sh) from the website..

Should probably be fixed once D16170 is deployed...

Jun 22 2016, 8:58 PM · Bug Report
ograchikov created T11200: Cannot access installation Phabricator script (install_ubuntu.sh) from the website..
Jun 22 2016, 8:49 PM · Bug Report
chad closed T11164: Badges: Unwanted entry at the history as Resolved by committing Restricted Diffusion Commit.
Jun 22 2016, 8:26 PM · Badges, Bug Report
chad added a revision to T11164: Badges: Unwanted entry at the history: Restricted Differential Revision.
Jun 22 2016, 8:00 PM · Badges, Bug Report
epriestley closed T11198: browser timezone popup shows for disabled accounts as Resolved by committing Restricted Diffusion Commit.
Jun 22 2016, 6:54 PM · Bug Report
epriestley added a comment to T11198: browser timezone popup shows for disabled accounts.

Thanks. D16167 should hide this and the similar HTTP/HTTPS check for unapproved, disabled, unverified, etc., users, who can't act on them.

Jun 22 2016, 6:47 PM · Bug Report
epriestley added a revision to T11198: browser timezone popup shows for disabled accounts: Restricted Differential Revision.
Jun 22 2016, 6:45 PM · Bug Report
scode added a comment to T11199: User visible error message on X-Forwarded-For header.

Thanks! That is exactly what we had, and I was not aware of it. And yes, we have this on purpose because of there being a load balancing (trusted) proxy in front of Phabricator. For whatever reason, I assumed Phabricator had it's own X-Forwarded-For handling. Mainly my mistake here was jumping to the assumption that Phabricator was looking at X-Forwarded-For based on the value - even though the error message never indicated that it was.

Jun 22 2016, 6:42 PM · Bug Report
epriestley added a comment to T11199: User visible error message on X-Forwarded-For header.

Since I typed up most of this anyway:

Jun 22 2016, 6:37 PM · Bug Report
scode added a comment to T11199: User visible error message on X-Forwarded-For header.

Closed as "invalid". I apologize for the noise. This was an undocumented Twitter specific patch.

Jun 22 2016, 6:25 PM · Bug Report
scode closed T11199: User visible error message on X-Forwarded-For header as Invalid.
Jun 22 2016, 6:24 PM · Bug Report
scode created T11199: User visible error message on X-Forwarded-For header.
Jun 22 2016, 6:18 PM · Bug Report
eadler added a comment to T11198: browser timezone popup shows for disabled accounts.

this is a rather minor nit, but figured I should report it regardless.

Jun 22 2016, 6:14 PM · Bug Report