Page MenuHomePhabricator
Feed All Stories

Jun 9 2016

avivey added a project to T11125: Ability to change LDAP password from within Phabricator: LDAP.
Jun 9 2016, 9:44 PM · LDAP, Feature Request
sinayion created T11125: Ability to change LDAP password from within Phabricator.
Jun 9 2016, 9:35 PM · LDAP, Feature Request
avivey added a comment to T11121: Git tags are not merged during arc land.

I do discourage it, but I was using italics to highlight the difference, which is pretty delicate.

Jun 9 2016, 9:33 PM · Arcanist, Bug Report
eadler added a comment to T11121: Git tags are not merged during arc land.

Your usage of italic makes me think that perhaps tagging practices like
this are to be discouraged. Is this the case?

Jun 9 2016, 9:09 PM · Arcanist, Bug Report
julioromano added a comment to T11121: Git tags are not merged during arc land.

Your usage of italic makes me think that perhaps tagging practices like
this are to be discouraged. Is this the case?

Jun 9 2016, 9:06 PM · Arcanist, Bug Report
avivey added a comment to T11121: Git tags are not merged during arc land.

If that herald is blocking your push, it means you're pushing new commits, not only new tags.

Jun 9 2016, 9:01 PM · Arcanist, Bug Report
julioromano added a comment to T11121: Git tags are not merged during arc land.

That's what we've tried to do so far, but the git push tag command is being
blocked by our Herald rule because there is no approved differential
revision associated with that push.

Jun 9 2016, 8:59 PM · Arcanist, Bug Report
eadler added a comment to T11121: Git tags are not merged during arc land.

T10329 is likely also related if you really want to do what you've been doing so far.

Jun 9 2016, 8:48 PM · Arcanist, Bug Report
avivey added a comment to T11121: Git tags are not merged during arc land.

In that case, you should create the tag when you release, and only release from code that was already pushed, not from code that is "about to be pushed".

Jun 9 2016, 8:42 PM · Arcanist, Bug Report
julioromano added a comment to T11121: Git tags are not merged during arc land.

We'd like to use them to tag releases.

Jun 9 2016, 8:39 PM · Arcanist, Bug Report
avivey added a comment to T11121: Git tags are not merged during arc land.

This should probably be a Ponder question (See Support Resources).

Jun 9 2016, 8:37 PM · Arcanist, Bug Report
epriestley moved T8510: Typeahead project proposals in Maniphest advanced search do not always include exact matches from The Queue to Paused on the Prioritized board.

D16094 should improve things, let me know if you identify specific problems after deploying that. It's deployed on this server now, although we have fewer similarly-named projects so it's harder to see the effects.

Jun 9 2016, 8:28 PM · Typeahead, Prioritized, Wikimedia, Infrastructure, Search
epriestley added a comment to T11120: HTML mails malformed.

Config → All Settings → phpmailer.smtp-encoding

Jun 9 2016, 8:20 PM · Mail, Bug Report
kugel- added a comment to T11120: HTML mails malformed.

How can I enable base64 encoding within phabricator?

Jun 9 2016, 8:19 PM · Mail, Bug Report
epriestley added a revision to T11120: HTML mails malformed: Restricted Differential Revision.
Jun 9 2016, 8:19 PM · Mail, Bug Report
epriestley added a comment to T11120: HTML mails malformed.

Let's just make base64 the default and see what that breaks. ¯\_(ツ)_/¯

Jun 9 2016, 8:17 PM · Mail, Bug Report
epriestley added a revision to T8510: Typeahead project proposals in Maniphest advanced search do not always include exact matches: Unknown Object (Differential Revision).
Jun 9 2016, 8:16 PM · Typeahead, Prioritized, Wikimedia, Infrastructure, Search
eadler added a comment to T11120: HTML mails malformed.

More specifically, I believe phabricator is violating the RFC when it sends overly long lines and that postfix (or whatever is munging the output) is taking a rather silly action as a result.
base64 encoding the output should work around the problem.

Jun 9 2016, 8:11 PM · Mail, Bug Report
eadler added a comment to T11120: HTML mails malformed.

From RFC 2822 There are two limits that this standard places on the number of characters in a line. Each line of characters MUST be no more than 998 characters, and SHOULD be no more than 78 characters, excluding the CRLF.
I believe the behavior of splitting the lines using a ! is implementation specific.

Jun 9 2016, 8:08 PM · Mail, Bug Report
avivey added a comment to T11120: HTML mails malformed.

If your email goes through postfix, it breaks lines after 1000 bytes, rain or shine.
The solution I got from IT was "Set phabricator to encode the mail using base64".

Jun 9 2016, 8:04 PM · Mail, Bug Report
kugel- added a comment to T11120: HTML mails malformed.

The client was thunderbird and the webclient (based Open-Xchange). I received it on my personal mail account, outside the company.

Jun 9 2016, 7:57 PM · Mail, Bug Report
chad added a comment to T11120: HTML mails malformed.

Received: from giant.haxx

Jun 9 2016, 7:55 PM · Mail, Bug Report
epriestley added a comment to T8510: Typeahead project proposals in Maniphest advanced search do not always include exact matches.

It's expected that the ordering in "Browse" is alphabetical/raw, and I don't plan to change that.

Jun 9 2016, 7:50 PM · Typeahead, Prioritized, Wikimedia, Infrastructure, Search
epriestley added a comment to T11120: HTML mails malformed.

Interesting. The same mail was delivered to me intact:

Jun 9 2016, 7:46 PM · Mail, Bug Report
20after4 added a comment to T8510: Typeahead project proposals in Maniphest advanced search do not always include exact matches.

Here's what happens when adding wikidata project tag using the comment form typeahead:

Jun 9 2016, 7:46 PM · Typeahead, Prioritized, Wikimedia, Infrastructure, Search
kugel- added a comment to T11120: HTML mails malformed.

Here's the mail that I received for your comment. You can see that the M stream is broken with "!" in two places. This seems to match the behavior I observed initially, although there was also an extraneous blank.

Jun 9 2016, 7:33 PM · Mail, Bug Report
epriestley moved T11034: Provide more information about projects, etc. in tokenizer browse dialog to make distinguishing between similar projects easier from Backlog to The Queue on the Prioritized board.
Jun 9 2016, 7:29 PM · Restricted Project, Design, Typeahead, Prioritized, Wikimedia
epriestley moved T8510: Typeahead project proposals in Maniphest advanced search do not always include exact matches from Preflight to The Queue on the Prioritized board.
Jun 9 2016, 7:29 PM · Typeahead, Prioritized, Wikimedia, Infrastructure, Search
epriestley added a comment to T7643: Improve prose diffs (was: description changes don't generate usable diffs).
  • We currently use white-space: pre-wrap to handle linebreaks in prose diffs, but at least one client (Airmail) doesn't handle that properly. We could use explicit <br /> to improve behavior.
Jun 9 2016, 7:25 PM · Prioritized, Transactions
epriestley moved T10227: Provide an extension point for HTTP behaviors (proxies, SSL trust) from Preflight to Paused on the Prioritized board.

I think this should work now. I tested it locally with HTTP and SOCKS proxies via Charles and those both seemed to behave properly.

Jun 9 2016, 7:20 PM · Prioritized, Restricted Project, Feature Request
epriestley added a revision to T10227: Provide an extension point for HTTP behaviors (proxies, SSL trust): Unknown Object (Differential Revision).
Jun 9 2016, 6:50 PM · Prioritized, Restricted Project, Feature Request
epriestley added a revision to T10227: Provide an extension point for HTTP behaviors (proxies, SSL trust): Unknown Object (Differential Revision).
Jun 9 2016, 6:47 PM · Prioritized, Restricted Project, Feature Request
epriestley added a revision to T10227: Provide an extension point for HTTP behaviors (proxies, SSL trust): Unknown Object (Differential Revision).
Jun 9 2016, 6:45 PM · Prioritized, Restricted Project, Feature Request
storrgie added a comment to T9640: Make Phabricator compatible with PHP7.

Macro whatcouldgowrong:

Jun 9 2016, 6:32 PM · Prioritized, Restricted Project, Infrastructure
svmagnum added a comment to T9640: Make Phabricator compatible with PHP7.

I have commented out internals of entire function that contains apc_sma_info() request and everything seems to be working fine :)

Jun 9 2016, 6:17 PM · Prioritized, Restricted Project, Infrastructure
epriestley closed T11123: Implement differential.revision.search as Resolved.

Sounds good. I'll call this done for now, let us know if you run into issues.

Jun 9 2016, 5:39 PM · Conduit, Differential, Feature Request
noliveira87 added a comment to T11124: Repository commits stuck on "importing commits".

Thank you. We will upgrade this weekend and report back.

Jun 9 2016, 5:36 PM · Support Request, Subversion, Diffusion
chad added a comment to T11124: Repository commits stuck on "importing commits".

Your versions above are over a year old. Sorry for not catching that sooner.

Jun 9 2016, 5:36 PM · Support Request, Subversion, Diffusion
chad closed T11124: Repository commits stuck on "importing commits" as Invalid.

We don't support old versions of Phabricator. Per documentation we require all bugs and support requests be performed with a current version.

Jun 9 2016, 5:32 PM · Support Request, Subversion, Diffusion
noliveira87 added a comment to T11124: Repository commits stuck on "importing commits".

I've read through this article and I am unable to determine what the next step should be. I've already used the repository tool to reparse the bad commit but new commits all go into the same state. One thing to note, my version of the repository tool does not have an --importing flag.

Jun 9 2016, 5:18 PM · Support Request, Subversion, Diffusion
chad added a comment to T11116: Ponder email for newly created question includes content that suggests the question was also edited.

parrot

Jun 9 2016, 5:04 PM · Ponder, Bug Report
chad added a comment to T11116: Ponder email for newly created question includes content that suggests the question was also edited.

https://nylas.com/

Jun 9 2016, 5:03 PM · Ponder, Bug Report
chad added a comment to T11116: Ponder email for newly created question includes content that suggests the question was also edited.

All the cool companies use Phabricator.

Jun 9 2016, 4:59 PM · Ponder, Bug Report
chad added a comment to T11116: Ponder email for newly created question includes content that suggests the question was also edited.

I should switch to Nylas since they use Phabricator.

Jun 9 2016, 4:59 PM · Ponder, Bug Report
timhirsh added a comment to T11123: Implement differential.revision.search.

I was able to filter by status using the queryKey, which is much more valuable than displaying the status. So omitting status from the response is ok by me considering the level of effort required to refactor the numeric constants. Thanks again, much appreciated!

Jun 9 2016, 4:55 PM · Conduit, Differential, Feature Request
chad added a comment to T11116: Ponder email for newly created question includes content that suggests the question was also edited.

Airmail on Mac, it has a few quirks with our HTML emails, but I haven't tried to debug it yet. I'm sure we're doing something weird since all other HTML seems fine in it.

Jun 9 2016, 4:51 PM · Ponder, Bug Report
epriestley added a comment to T11116: Ponder email for newly created question includes content that suggests the question was also edited.

I think we're relying on white-space: pre-wrap; right now. We could explicitly do \n -> <br /> conversion which might fix newlines, at least.

Jun 9 2016, 4:51 PM · Ponder, Bug Report
cspeckmim added a comment to T11116: Ponder email for newly created question includes content that suggests the question was also edited.

Which client do you use? I imagine the html relies on <br> or something, but would \n vs. \r\n be involved at all?

Jun 9 2016, 4:49 PM · Ponder, Bug Report
eadler added a project to T11121: Git tags are not merged during arc land: Arcanist.
Jun 9 2016, 4:48 PM · Arcanist, Bug Report
epriestley added a comment to T11116: Ponder email for newly created question includes content that suggests the question was also edited.

maybe by 2050 email will work most of the time for most people

Jun 9 2016, 4:48 PM · Ponder, Bug Report
chad added a comment to T11116: Ponder email for newly created question includes content that suggests the question was also edited.

I use a crappy email client

Jun 9 2016, 4:48 PM · Ponder, Bug Report
epriestley added a comment to T11116: Ponder email for newly created question includes content that suggests the question was also edited.

The actual diff is further down and rendered correctly for me, at least -- that's just how we currently render "added one line to the middle of a 500 line document" right now:

Jun 9 2016, 4:48 PM · Ponder, Bug Report
epriestley added a comment to T11116: Ponder email for newly created question includes content that suggests the question was also edited.

No, that's this stuff from T7643#179240:

Jun 9 2016, 4:47 PM · Ponder, Bug Report
chad added a comment to T11116: Ponder email for newly created question includes content that suggests the question was also edited.

I got a super long email from Phriction today, is that also an EditEngine freebie?

Jun 9 2016, 4:45 PM · Ponder, Bug Report
epriestley added a comment to T9789: Make it easier to write custom transaction types.

I'm leaning toward exploring this in the near term. We have four applications which may convert soon:

Jun 9 2016, 4:40 PM · Modernization, Transactions
epriestley added a comment to T7643: Improve prose diffs (was: description changes don't generate usable diffs).

Oh, one more thing:

Jun 9 2016, 4:34 PM · Prioritized, Transactions
epriestley added a comment to T11123: Implement differential.revision.search.

I return the other fields, but don't currently return "status" because I'd have to return an internal numeric constant. So we may need to fix that more thoroughly before you can move forward.

Jun 9 2016, 4:25 PM · Conduit, Differential, Feature Request
timhirsh added a comment to T11123: Implement differential.revision.search.

Awesome, thanks for the quick turnaround. This looks good to me. Currently I'm using revision ID, title, status, and authorPHID in the response for display purposes. I'll try your recommendation for 'open' revisions using the queryKey to handle the status.

Jun 9 2016, 4:20 PM · Conduit, Differential, Feature Request
epriestley added a comment to T11123: Implement differential.revision.search.

Actually, this is slightly more involved because you want to select open revisions.

Jun 9 2016, 4:07 PM · Conduit, Differential, Feature Request
epriestley added a revision to T11123: Implement differential.revision.search: Unknown Object (Differential Revision).
Jun 9 2016, 4:02 PM · Conduit, Differential, Feature Request
chad added a comment to T11124: Repository commits stuck on "importing commits".

https://secure.phabricator.com/book/phabricator/article/repository_imports/

Jun 9 2016, 4:00 PM · Support Request, Subversion, Diffusion
chad removed projects from T11124: Repository commits stuck on "importing commits": Repositories, Phabricator, Audit.
Jun 9 2016, 3:59 PM · Support Request, Subversion, Diffusion
chad added a comment to T11124: Repository commits stuck on "importing commits".

https://secure.phabricator.com/book/phabricator/article/support/

Jun 9 2016, 3:58 PM · Support Request, Subversion, Diffusion
epriestley added a comment to T11123: Implement differential.revision.search.

You essentially only need the revision IDs in the response, right?

Jun 9 2016, 3:55 PM · Conduit, Differential, Feature Request
noliveira87 added a comment to T11124: Repository commits stuck on "importing commits".

I created the task, then changed the tags manually (selecting only the tags I felt necessary - also removing the BUG tag). Seems I may have found a bug with the bug tracker. ;)

Jun 9 2016, 3:54 PM · Support Request, Subversion, Diffusion
epriestley added a comment to T11124: Repository commits stuck on "importing commits".

That is, if you go to the Diffusion workboard, we give you a generic create form instead of a choice between the various normal create forms, currently.

Jun 9 2016, 3:54 PM · Support Request, Subversion, Diffusion
epriestley added a comment to T11124: Repository commits stuck on "importing commits".

You can file stuff without those tags from workboards, and maybe "Support Request" just got caught in the crossfire of typing every possible tag, so that's a legitimate-albeit-unusual pathway here. But I wouldn't expect the short-lived "New Support Request" form to be accessible any longer.

Jun 9 2016, 3:53 PM · Support Request, Subversion, Diffusion
chad added a comment to T11124: Repository commits stuck on "importing commits".

This tracker is for bug reports and feature requests, and we're surprised you were able to file something without either of those tags.

Jun 9 2016, 3:51 PM · Support Request, Subversion, Diffusion
epriestley renamed T11123: Implement differential.revision.search from Search for revisions by project via conduit to Implement differential.revision.search.
Jun 9 2016, 3:51 PM · Conduit, Differential, Feature Request
epriestley added a comment to T11124: Repository commits stuck on "importing commits".

How did you file this task?

Jun 9 2016, 3:50 PM · Support Request, Subversion, Diffusion
noliveira87 created T11124: Repository commits stuck on "importing commits".
Jun 9 2016, 3:47 PM · Support Request, Subversion, Diffusion
timhirsh created T11123: Implement differential.revision.search.
Jun 9 2016, 3:47 PM · Conduit, Differential, Feature Request
epriestley closed T10886: Unpublished, unauthored inline comments in Pholio are incorrectly counted on the image badge as Resolved by committing Restricted Diffusion Commit.
Jun 9 2016, 3:43 PM · Pholio, Bug Report
epriestley added a comment to T6523: Allow creating/adding images to mocks with a vanilla upload control.

After D16088:

Jun 9 2016, 3:42 PM · Prioritized, Pholio, Wikimedia
epriestley moved T6523: Allow creating/adding images to mocks with a vanilla upload control from Backlog to Paused on the Prioritized board.
Jun 9 2016, 3:41 PM · Prioritized, Pholio, Wikimedia
epriestley added a revision to T6523: Allow creating/adding images to mocks with a vanilla upload control: Unknown Object (Differential Revision).
Jun 9 2016, 3:40 PM · Prioritized, Pholio, Wikimedia
chad added a project to T11120: HTML mails malformed: Mail.
Jun 9 2016, 3:29 PM · Mail, Bug Report
chad added a comment to T11122: Stables windows are bugged and you can see just terrain and players.

(assume you mean to post this on a different Phabricator)

Jun 9 2016, 3:24 PM · Windows, Bug Report
chad closed T11122: Stables windows are bugged and you can see just terrain and players as Wontfix.

nyancat

Jun 9 2016, 3:23 PM · Windows, Bug Report
epriestley added a revision to T10886: Unpublished, unauthored inline comments in Pholio are incorrectly counted on the image badge: Unknown Object (Differential Revision).
Jun 9 2016, 3:19 PM · Pholio, Bug Report
• Argaf created T11122: Stables windows are bugged and you can see just terrain and players.
Jun 9 2016, 3:14 PM · Windows, Bug Report
epriestley closed T5761: Add an update comment to tasks that are linked to Pholio mocks when they are updated as Wontfix.

We haven't seen other requests for this and it's consistent with the similar behavior in Differential.

Jun 9 2016, 3:12 PM · Pholio
epriestley added a comment to T6523: Allow creating/adding images to mocks with a vanilla upload control.

This should also fix the mobile workflow stragglers from T2644.

Jun 9 2016, 3:11 PM · Prioritized, Pholio, Wikimedia
epriestley merged T2644: Make Pholio work better on mobile into T6523: Allow creating/adding images to mocks with a vanilla upload control.
Jun 9 2016, 3:10 PM · Prioritized, Pholio, Wikimedia
epriestley merged task T2644: Make Pholio work better on mobile into T6523: Allow creating/adding images to mocks with a vanilla upload control.
Jun 9 2016, 3:10 PM · Pholio
epriestley merged T8712: Unpublished pholio inlines count towards the total number of inlines into T10886: Unpublished, unauthored inline comments in Pholio are incorrectly counted on the image badge.
Jun 9 2016, 3:10 PM · Pholio, Bug Report
epriestley merged task T8712: Unpublished pholio inlines count towards the total number of inlines into T10886: Unpublished, unauthored inline comments in Pholio are incorrectly counted on the image badge.
Jun 9 2016, 3:10 PM · Pholio
epriestley moved T8712: Unpublished pholio inlines count towards the total number of inlines from Backlog to v2 on the Pholio board.
Jun 9 2016, 3:10 PM · Pholio
epriestley moved T6523: Allow creating/adding images to mocks with a vanilla upload control from Backlog to v2 on the Pholio board.
Jun 9 2016, 3:09 PM · Prioritized, Pholio, Wikimedia
epriestley moved T10886: Unpublished, unauthored inline comments in Pholio are incorrectly counted on the image badge from Backlog to v2 on the Pholio board.
Jun 9 2016, 3:09 PM · Pholio, Bug Report
• luca.looz92 added a comment to T9640: Make Phabricator compatible with PHP7.

I have just tried ubuntu 16.04 LTS with php 7, apcu and apcu-bc and i can confirm that with apcu enabled there are issues with php random crashing with "zend_mm_heap corrupted" likewise T10987

Jun 9 2016, 3:05 PM · Prioritized, Restricted Project, Infrastructure
epriestley added a project to T7924: Unprototype Calendar (v1): Prioritized.
Jun 9 2016, 2:58 PM · Restricted Project, Prioritized, Calendar
julioromano created T11121: Git tags are not merged during arc land.
Jun 9 2016, 2:58 PM · Arcanist, Bug Report
epriestley added a project to T10747: Import and export ICS from Calendar: Prioritized.
Jun 9 2016, 2:57 PM · Restricted Project, Prioritized, Calendar
epriestley claimed T2334: Implement Google Calendar v3 API into Calendar.
Jun 9 2016, 2:57 PM · Restricted Project, Prioritized, Calendar
epriestley moved T10747: Import and export ICS from Calendar from Backlog to Unprototype (v1) on the Calendar board.
Jun 9 2016, 2:57 PM · Restricted Project, Prioritized, Calendar
epriestley moved T2334: Implement Google Calendar v3 API into Calendar from Future Work (v2+) to Unprototype (v1) on the Calendar board.
Jun 9 2016, 2:57 PM · Restricted Project, Prioritized, Calendar
epriestley added a comment to T10969: Incorrect Phriction link in email.

Anchor tags got cleaned up in D16083.

Jun 9 2016, 2:56 PM · Phriction, Bug Report
epriestley renamed T11073: Allow the user to specify a date in a different timezone using date controls from Allow the user to specify a timezone when creating a countdown to Allow the user to specify a date in a different timezone using date controls.
Jun 9 2016, 2:53 PM · KDE, Calendar, Feature Request
epriestley closed T11090: Marking comments as done in Differential isn't always saved correctly as Invalid.

This report doesn't include reproduction instructions and doesn't clearly demonstrate a problem: all of the screenshots are explicable through user error.

Jun 9 2016, 2:52 PM · Restricted Project, Differential, Bug Report