Page MenuHomePhabricator
Feed Advanced Search

Mar 24 2016

svmagnum created T10660: Php 7 installation.
Mar 24 2016, 5:43 AM · Bug Report
chad closed T10658: Mail Delivery fails as Invalid.

Steps to reproduce means that we can follow what you've done and see the same error locally. Unfortunately there isn't anything here we can use to reproduce the issue. Please feel free to reopen if you can come up with the complete steps to reproduce the issue. Specifically, the steps should be repeatable and observable. For example "Every time I do X, Y, and Z, this error shows up in my log."

Mar 24 2016, 12:49 AM · Mail, Bug Report
Luke081515.2 added a comment to T10658: Mail Delivery fails.

Sry, the relevant steps are:

  • metamta.mail-adapter is PhabricatorMailImplementationPHPMailerLiteAdapter

and I got the same settings since more than a half year, (I don't know, if additional e-mail configs are required, if so, please ask) but since a short time (I don't know the exact date, but it is less than one week) there were no emails send, and the daemon throws this error (at the task description). I hope this helps.

Mar 24 2016, 12:09 AM · Mail, Bug Report

Mar 23 2016

chad added a comment to T10658: Mail Delivery fails.

We require reproduction steps for bug reports, please. There isn't enough information here for us to make a guess as to the root problem.

Mar 23 2016, 11:22 PM · Mail, Bug Report
kymera.travis added a comment to T10657: Descriptions not escaped for Amazon Web Services Settings.

Thanks for sharing but I don't think that'll be necessary. The only place this will likely be an issue is on the config screen (and there its mostly moot).

Mar 23 2016, 11:19 PM · Bug Report
Luke081515.2 created T10658: Mail Delivery fails.
Mar 23 2016, 11:14 PM · Mail, Bug Report
chad added a comment to T10657: Descriptions not escaped for Amazon Web Services Settings.

Sorry, if that wasn't clear. "We" means just this install, not Phabricator by default.

Mar 23 2016, 10:28 PM · Bug Report
chad added a comment to T10657: Descriptions not escaped for Amazon Web Services Settings.

It's not the default value, I'm sharing what ours is set to in case you want to use it.

Mar 23 2016, 10:25 PM · Bug Report
kymera.travis added a comment to T10657: Descriptions not escaped for Amazon Web Services Settings.

This doesn't appear to be the default value on our install, unless you just meant its what been set here?

Mar 23 2016, 10:24 PM · Bug Report
chad added a comment to T10657: Descriptions not escaped for Amazon Web Services Settings.

Also related is T3278. There is no current means of having different flavors of Remarkup, it's all or nothing. So a "no object" Remarkup would similar to a "object only" Remarkup, and so on. For now installs just should either use a regex like above or the admins should tough it out until T3278.

Mar 23 2016, 10:18 PM · Bug Report
chad closed T10657: Descriptions not escaped for Amazon Web Services Settings as Invalid.

We set remarkup.ignored-object-names to /^(?:(Q|V)\d|S3)$/ under Config → Core

Mar 23 2016, 10:11 PM · Bug Report
kymera.travis updated the task description for T10657: Descriptions not escaped for Amazon Web Services Settings.
Mar 23 2016, 10:07 PM · Bug Report
kymera.travis renamed T10657: Descriptions not escaped for Amazon Web Services Settings from Descriptions not escaped for Amazon to Descriptions not escaped for Amazon Web Services Settings.
Mar 23 2016, 10:04 PM · Bug Report
kymera.travis created T10657: Descriptions not escaped for Amazon Web Services Settings.
Mar 23 2016, 10:03 PM · Bug Report
richardvanvelzen added a comment to T10635: Loading differential revision slow when lots of unit test messages exist.

In the extreme case where you have, say, a million tests, I'd expect there is probably little value in reporting each test into Harbormaster as a "pass", and your harness might want to summarize all passes into "999,998 additional tests passed" and submit two failures and one aggregate-pass.

Mar 23 2016, 6:16 PM · Restricted Project, Haskell.org, Harbormaster, Performance
epriestley added a comment to T10635: Loading differential revision slow when lots of unit test messages exist.

T9704 also discusses slowness on inserting the tests. That may be unnecessarily slow right now, but I don't expect clients to submit unlimited numbers of tests in one API call. Instead, submit tests in chunks (say, of 1K tests per page or whatever) by calling harbormaster.sendmessage repeatedly with a work status.

Mar 23 2016, 5:58 PM · Restricted Project, Haskell.org, Harbormaster, Performance
epriestley updated subscribers of T10635: Loading differential revision slow when lots of unit test messages exist.
Mar 23 2016, 5:49 PM · Restricted Project, Haskell.org, Harbormaster, Performance
epriestley added a comment to T10635: Loading differential revision slow when lots of unit test messages exist.

We don't currently have a sortable column on the unit message table, since pass, fail, etc., aren't naturally sortable by any key MySQL can construct.

Mar 23 2016, 5:34 PM · Restricted Project, Haskell.org, Harbormaster, Performance
epriestley triaged T10645: Accepting a closed diff causes diff to be re-opened as Low priority.
Mar 23 2016, 5:19 PM · Differential, Bug Report
epriestley added a comment to T10645: Accepting a closed diff causes diff to be re-opened.

This is probably not a true race in the strict technical sense, I suspect you can reproduce it by doing one of these at walking speed:

Mar 23 2016, 5:18 PM · Differential, Bug Report
amckinley added a comment to T10651: Attempting to create a credential of type API Token always results in an exception.

Works now; thanks!

Mar 23 2016, 4:55 PM · Passphrase, Bug Report
epriestley closed T10651: Attempting to create a credential of type API Token always results in an exception as Resolved by committing Restricted Diffusion Commit.
Mar 23 2016, 1:53 AM · Passphrase, Bug Report
epriestley added a comment to T10651: Attempting to create a credential of type API Token always results in an exception.

Whoops, thanks. This got rebased across 200 years of changes and I didn't re-test creating a credential. D15512 should fix it.

Mar 23 2016, 1:52 AM · Passphrase, Bug Report
epriestley added a revision to T10651: Attempting to create a credential of type API Token always results in an exception: Restricted Differential Revision.
Mar 23 2016, 1:52 AM · Passphrase, Bug Report

Mar 22 2016

avivey added a project to T10651: Attempting to create a credential of type API Token always results in an exception: Passphrase.
Mar 22 2016, 11:47 PM · Passphrase, Bug Report
amckinley created T10651: Attempting to create a credential of type API Token always results in an exception.
Mar 22 2016, 11:41 PM · Passphrase, Bug Report
chad added a project to T10612: Editing a panel causes it to duplicate in dashboard: Dashboards.
Mar 22 2016, 11:04 PM · Dashboards (v2), Bug Report
Taskle added a comment to T10642: Default search boolean logic is mildly inconvenient and not configurable in Amazon RDS.

@epriestley I understand you have the setup check - my point is that for RDS users we skip this check when we see we can't do anything about it. And then months later it's forgotten about and we still draw that conclusion.

Mar 22 2016, 9:49 PM · Search, Bug Report
epriestley added a comment to T10642: Default search boolean logic is mildly inconvenient and not configurable in Amazon RDS.

just convinced that Phab search didn't work and feeling powerless over it

Mar 22 2016, 9:35 PM · Search, Bug Report
Taskle added a comment to T10642: Default search boolean logic is mildly inconvenient and not configurable in Amazon RDS.

Amazon RDS is incredibly popular so although you haven't seen this request from other installs, my guess is that many are like we have been for the past 1.5 years - just convinced that Phab search didn't work and feeling powerless over it...I understand the squeaky wheel tends to get the oil, but it's worth maybe considering what % of installs use RDS since all of them will have this problem.

Mar 22 2016, 9:12 PM · Search, Bug Report
epriestley added a comment to T10648: Task project list ellipsis displays unecessary.

This should be fixed in HEAD of master. Let us know if you're still seeing issues after updating. Thanks for the report!

Mar 22 2016, 4:29 PM · Bug Report
epriestley closed T10648: Task project list ellipsis displays unecessary as Resolved by committing Restricted Diffusion Commit.
Mar 22 2016, 4:29 PM · Bug Report
epriestley added a revision to T10648: Task project list ellipsis displays unecessary: Restricted Differential Revision.
Mar 22 2016, 4:21 PM · Bug Report
epriestley added a comment to T10646: Actions are not populated when creating herald rule (Chrome v49 issue).

I think this should be fixed in HEAD, let us know if you're still seeing it after updating. Thanks for the report!

Mar 22 2016, 4:14 PM · Herald, Bug Report
epriestley closed T10646: Actions are not populated when creating herald rule (Chrome v49 issue) as Resolved by committing Restricted Diffusion Commit.
Mar 22 2016, 4:13 PM · Herald, Bug Report
epriestley added a revision to T10646: Actions are not populated when creating herald rule (Chrome v49 issue): Restricted Differential Revision.
Mar 22 2016, 4:11 PM · Herald, Bug Report
ofbeaton created T10648: Task project list ellipsis displays unecessary.
Mar 22 2016, 3:54 PM · Bug Report
Pawka updated the task description for T10646: Actions are not populated when creating herald rule (Chrome v49 issue).
Mar 22 2016, 3:24 PM · Herald, Bug Report
Pawka added a comment to T10646: Actions are not populated when creating herald rule (Chrome v49 issue).

@chad False alarm, I've checked wrong link. Problem existed on Maniphest global rules but it seems it is fixed already on upstream.

Mar 22 2016, 3:24 PM · Herald, Bug Report
chad added a comment to T10646: Actions are not populated when creating herald rule (Chrome v49 issue).

I'm still able to select stuff fine and use the form, did you try the dropdown? (I do see the error in the console).

Mar 22 2016, 2:14 PM · Herald, Bug Report
Pawka created T10646: Actions are not populated when creating herald rule (Chrome v49 issue).
Mar 22 2016, 1:55 PM · Herald, Bug Report
krisbuist added a project to T10645: Accepting a closed diff causes diff to be re-opened: Differential.
Mar 22 2016, 11:10 AM · Differential, Bug Report
krisbuist updated the task description for T10645: Accepting a closed diff causes diff to be re-opened.
Mar 22 2016, 11:09 AM · Differential, Bug Report
krisbuist created T10645: Accepting a closed diff causes diff to be re-opened.
Mar 22 2016, 11:05 AM · Differential, Bug Report
chad added a comment to T10642: Default search boolean logic is mildly inconvenient and not configurable in Amazon RDS.

Elasticsearch may also be a solution. Also, looks like we might be able to detect RDS installs by querying variables in MySQL.

Mar 22 2016, 1:31 AM · Search, Bug Report
epriestley added a comment to T10638: A Task can end up on a Milestone workboard column without being in that Milestone.

Yeah -- it's not very clear on its own, but this comment is specifically referring to that issue:

Mar 22 2016, 1:26 AM · Projects (v3), Bug Report
epriestley added a project to T10642: Default search boolean logic is mildly inconvenient and not configurable in Amazon RDS: Search.
Mar 22 2016, 1:23 AM · Search, Bug Report
epriestley added a comment to T10642: Default search boolean logic is mildly inconvenient and not configurable in Amazon RDS.

We can resolve this by parsing the query and rewriting the query we submit to the engine as "+x +y" if ft_boolean_syntax has not been adjusted.

Mar 22 2016, 1:14 AM · Search, Bug Report
eserrano added a comment to T10638: A Task can end up on a Milestone workboard column without being in that Milestone.

Hmm... maybe I'm not explaining my expectation well, but the "issue" I'm referring to is the fact that after steps (1)-(3) I now have a task that is in the "Subproject A" column in my "Project X" workboard but is not *actually* in Subproject A anymore.

Mar 22 2016, 1:13 AM · Projects (v3), Bug Report
epriestley renamed T10642: Default search boolean logic is mildly inconvenient and not configurable in Amazon RDS from Search experience is not useable on Amazon RDS to Default search boolean logic is mildly inconvenient and not configurable in Amazon RDS.
Mar 22 2016, 1:11 AM · Search, Bug Report
Taskle created T10642: Default search boolean logic is mildly inconvenient and not configurable in Amazon RDS.
Mar 22 2016, 1:00 AM · Search, Bug Report
epriestley added a comment to T10638: A Task can end up on a Milestone workboard column without being in that Milestone.

T10486 is related; T6027 and T5214 are somewhat related.

Mar 22 2016, 12:10 AM · Projects (v3), Bug Report

Mar 21 2016

eserrano added a comment to T10638: A Task can end up on a Milestone workboard column without being in that Milestone.

Thanks for looking @chad

Mar 21 2016, 11:47 PM · Projects (v3), Bug Report
chad added a comment to T10638: A Task can end up on a Milestone workboard column without being in that Milestone.

Does the task go away from the workboard when you refresh? I think that's the missing step here, that you're editing the task on the workboard.

Mar 21 2016, 11:09 PM · Projects (v3), Bug Report
chad added a comment to T10638: A Task can end up on a Milestone workboard column without being in that Milestone.

I followed steps listed here but didn't get your same outcome. Can you clarify the steps to reproduce, the expected result, and the version of Phabricator you're using.

Mar 21 2016, 10:55 PM · Projects (v3), Bug Report
eserrano created T10638: A Task can end up on a Milestone workboard column without being in that Milestone.
Mar 21 2016, 10:27 PM · Projects (v3), Bug Report
Shredder121 added a comment to T10637: Markup of new Details card is missing a <table />.

I see you already found an existing issue, sorry for the noise.

We don't keep this install at HEAD at all times

I understand, I didn't think this was the case anyway. I just figured I'd let this issue be known so it will at least be fixed.

Mar 21 2016, 9:58 PM · Bug Report
chad added a comment to T10637: Markup of new Details card is missing a <table />.

No problem. We don't keep this install at HEAD at all times, for what it's worth.

Mar 21 2016, 9:55 PM · Bug Report
chad updated subscribers of T10626: Spacing issue in Diffusion.
Mar 21 2016, 9:54 PM · Bug Report, Diffusion
chad merged task T10637: Markup of new Details card is missing a <table /> into T10626: Spacing issue in Diffusion.
Mar 21 2016, 9:54 PM · Bug Report
Shredder121 added a comment to T10637: Markup of new Details card is missing a <table />.

We personally are not on any version with the new layout, so I can't say it fixes it.
I just drive-by reported a bug I saw on this instance.

Mar 21 2016, 9:53 PM · Bug Report
chad added a comment to T10637: Markup of new Details card is missing a <table />.

Version information is required for bug reports, as is updating before you file a bug report.

Mar 21 2016, 9:50 PM · Bug Report
Shredder121 created T10637: Markup of new Details card is missing a <table />.
Mar 21 2016, 9:43 PM · Bug Report
chad added a comment to T10636: Custom Field.

(Nearly every text area in Phabricator can take files via drag and drop.)

Mar 21 2016, 9:26 PM · Bug Report
chad closed T10636: Custom Field as Invalid.

This doesn't appear to be a Bug Report

Mar 21 2016, 9:25 PM · Bug Report
• camilorojas added a comment to T10635: Loading differential revision slow when lots of unit test messages exist.
Mar 21 2016, 9:07 PM · Restricted Project, Haskell.org, Harbormaster, Performance
• camilorojas created T10636: Custom Field.
Mar 21 2016, 9:01 PM · Bug Report
ablekh added a comment to T10632: Error displaying a Differential revision.

Thank you both for clarifications. Will follow your advice.

Mar 21 2016, 6:59 PM · Bug Report
epriestley changed the status of T10234: When using a PhabricatorRemarkupCustomInlineRule with caching disabled, custom rules may end up with incorrect engines from Invalid to Resolved by committing Restricted Diffusion Commit.
Mar 21 2016, 6:24 PM · Remarkup, Bug Report
epriestley added a revision to T10234: When using a PhabricatorRemarkupCustomInlineRule with caching disabled, custom rules may end up with incorrect engines: Restricted Differential Revision.
Mar 21 2016, 6:21 PM · Remarkup, Bug Report
eadler added a comment to T10234: When using a PhabricatorRemarkupCustomInlineRule with caching disabled, custom rules may end up with incorrect engines.

for anyone else that runs into this in the future: http://www.gossamer-threads.com/lists/wiki/mediawiki-cvs/626638

Mar 21 2016, 6:12 PM · Remarkup, Bug Report
richardvanvelzen created T10635: Loading differential revision slow when lots of unit test messages exist.
Mar 21 2016, 6:07 PM · Restricted Project, Haskell.org, Harbormaster, Performance
epriestley moved T10633: Render timezones explicitly in email from Backlog to Unprototype (v1) on the Calendar board.
Mar 21 2016, 4:41 PM · Calendar, Mail, Countdown, Bug Report
epriestley added a project to T10633: Render timezones explicitly in email: Calendar.

I think this isn't tremendously important for Countdown but should definitely be sorted out before we unbeta Calendar.

Mar 21 2016, 4:41 PM · Calendar, Mail, Countdown, Bug Report
eadler updated the task description for T10633: Render timezones explicitly in email.
Mar 21 2016, 4:16 PM · Calendar, Mail, Countdown, Bug Report
epriestley added a comment to T10633: Render timezones explicitly in email.

It might not be bad to always show timezones in email explicitly to deal with cases related to T3025 better (e.g., you fly to the east coast for a conference, get prompted to reconcile your timezone after T3025, and it may not be obvious that this also affects mail you receive until you return and re-reconicle back to PST).

Mar 21 2016, 4:15 PM · Calendar, Mail, Countdown, Bug Report
epriestley triaged T10633: Render timezones explicitly in email as Wishlist priority.

With metamta.one-mail-per-recipient on, we can adjust this to the viewer's preferences.

Mar 21 2016, 4:13 PM · Calendar, Mail, Countdown, Bug Report
eadler updated the task description for T10633: Render timezones explicitly in email.
Mar 21 2016, 4:10 PM · Calendar, Mail, Countdown, Bug Report
eadler updated the task description for T10633: Render timezones explicitly in email.
Mar 21 2016, 4:09 PM · Calendar, Mail, Countdown, Bug Report
eadler created T10633: Render timezones explicitly in email.
Mar 21 2016, 4:09 PM · Calendar, Mail, Countdown, Bug Report
chad closed T10632: Error displaying a Differential revision as Invalid.

Unfortunately we can't take bug reports missing required information. Specifically:

Mar 21 2016, 1:52 PM · Bug Report
richardvanvelzen added a comment to T10632: Error displaying a Differential revision.

Visit /config/module/versions on your install. If it does not exist, you're not running HEAD.

Mar 21 2016, 9:08 AM · Bug Report
ablekh added a comment to T10632: Error displaying a Differential revision.

@richardvanvelzen Thank you for your comment. However, I don't have /config/module/versions/ or any Phabricator-related versions file or directory, for that matter, on my system. If you meant /conf/, then this directory contains only the files __init_conf__.php, keys and local. Please clarify.

Mar 21 2016, 8:36 AM · Bug Report
richardvanvelzen added a comment to T10632: Error displaying a Differential revision.
  • You can obtain the versions at /config/module/versions/ on your install
  • The max nesting level is set to PHP_INT_MAX (see libphutil/scripts/__init_script__.php:44), so you should never see this error.
Mar 21 2016, 8:07 AM · Bug Report
ablekh updated the task description for T10632: Error displaying a Differential revision.
Mar 21 2016, 7:02 AM · Bug Report
ablekh created T10632: Error displaying a Differential revision.
Mar 21 2016, 7:02 AM · Bug Report

Mar 19 2016

chad closed T10626: Spacing issue in Diffusion as Resolved by committing Restricted Diffusion Commit.
Mar 19 2016, 3:34 PM · Bug Report, Diffusion
chad added a revision to T10626: Spacing issue in Diffusion: Restricted Differential Revision.
Mar 19 2016, 3:16 PM · Bug Report, Diffusion
joshuaspence created T10626: Spacing issue in Diffusion.
Mar 19 2016, 12:16 AM · Bug Report, Diffusion

Mar 18 2016

Krenair added a comment to T10625: Links like "((prototype))" are not expanded properly in email.

Yep, that worked. Thanks.

Mar 18 2016, 11:30 PM · Phurl, Bug Report
epriestley added a comment to T10625: Links like "((prototype))" are not expanded properly in email.

That one worked OK, yell if you're still seeing an issue?

Mar 18 2016, 11:02 PM · Phurl, Bug Report
epriestley added a comment to T10625: Links like "((prototype))" are not expanded properly in email.

Test: User Guide: Prototype Applications

Mar 18 2016, 11:02 PM · Phurl, Bug Report
epriestley closed T10625: Links like "((prototype))" are not expanded properly in email as Resolved by committing Restricted Diffusion Commit.
Mar 18 2016, 11:00 PM · Phurl, Bug Report
epriestley added a revision to T10625: Links like "((prototype))" are not expanded properly in email: Restricted Differential Revision.
Mar 18 2016, 10:59 PM · Phurl, Bug Report
epriestley added a comment to T10625: Links like "((prototype))" are not expanded properly in email.

Oh -- I misunderstood. The issue isn't that we're getting invalid markup or emitting markup in a text context, but that the link is relative.

Mar 18 2016, 10:51 PM · Phurl, Bug Report
chad edited projects for T10625: Links like "((prototype))" are not expanded properly in email, added: Phurl; removed Diviner.
Mar 18 2016, 10:49 PM · Phurl, Bug Report
epriestley added a comment to T10625: Links like "((prototype))" are not expanded properly in email.

This is probably very easy to fix, let me spend 3 seconds on it.

Mar 18 2016, 10:49 PM · Phurl, Bug Report
epriestley added a comment to T10625: Links like "((prototype))" are not expanded properly in email.

Yes, Phurl provides URL shortening. U10 is the root object providing ((prototype)).

Mar 18 2016, 10:48 PM · Phurl, Bug Report
chad added a comment to T10625: Links like "((prototype))" are not expanded properly in email.

Yeaaah, I would assume this worked for free with Remarkup, but I guess not. Yeah it's Phurl,

Mar 18 2016, 10:48 PM · Phurl, Bug Report
Krenair added a comment to T10625: Links like "((prototype))" are not expanded properly in email.

So this actually in Phurl (not clear from the link) and not valid itself because Phurl is a prototype?

Mar 18 2016, 10:43 PM · Phurl, Bug Report