Page MenuHomePhabricator
Feed All Stories

Jul 3 2016

nickhutchinson added a watcher for Windows: nickhutchinson.
Jul 3 2016, 12:03 PM
epriestley added a comment to T10855: Allow external systems to edit the content of dashboard panels so they can publish information.

We probably need to do almost all of this work before adding new panel types anyway, since building a new panel type on top of CustomFields would make moving everything from CustomFields to EditEngine harder.

Jul 3 2016, 9:11 AM · Dashboards, Restricted Project, Conduit, Feature Request
epriestley added a comment to T9360: Unbeta Phame.
In T9360#183602, @chad wrote:

Is [HTTPS] simple? Not sure where to start? Nothing is jumping out at me here.

Jul 3 2016, 9:06 AM · Phame
epriestley edited subtasks for T9897: Unbeta Feedback on Phame, added: T6299: Remarkup links to objects in Phame break if a CNAME is set; removed: T4936: Phame is not accessible on a custom domain without `policy.allow-public`.
Jul 3 2016, 9:00 AM · Phame
epriestley removed a parent task for T4936: Phame is not accessible on a custom domain without `policy.allow-public`: T9897: Unbeta Feedback on Phame.
Jul 3 2016, 9:00 AM · Phame
epriestley added a parent task for T6299: Remarkup links to objects in Phame break if a CNAME is set: T9897: Unbeta Feedback on Phame.
Jul 3 2016, 9:00 AM · Haskell.org, Remarkup, Phame
epriestley removed a subtask for T9897: Unbeta Feedback on Phame: T6299: Remarkup links to objects in Phame break if a CNAME is set.
Jul 3 2016, 9:00 AM · Phame
epriestley removed a parent task for T6299: Remarkup links to objects in Phame break if a CNAME is set: T9897: Unbeta Feedback on Phame.
Jul 3 2016, 9:00 AM · Haskell.org, Remarkup, Phame
Danny_B added a comment to T11254: Provide a way for quick revert of all activities of the most recent task editor..

Suggestion from one chat I had:

I mean even having just a "Revert" button for each change would be easier
Even if you had 50 to click, it's better than manually undoing :)

Jul 3 2016, 8:53 AM · Abuse, Maniphest, Wikimedia, Feature Request
epriestley merged T11262: On slow connections, it's possible to submit the Create Task form on workboards multiple times into T11145: Dialog submit buttons don't disable on click.
Jul 3 2016, 8:37 AM · Restricted Project, PHUI, Bug Report
epriestley merged task T11262: On slow connections, it's possible to submit the Create Task form on workboards multiple times into T11145: Dialog submit buttons don't disable on click.
Jul 3 2016, 8:37 AM · Maniphest, Bug Report
eadler added a comment to T10855: Allow external systems to edit the content of dashboard panels so they can publish information.

FWIW we'd be perfectly fine with the following:

  • allow editing phriction documents via Conduit
  • create a panel which shows the contents of a specific phriction page
Jul 3 2016, 5:51 AM · Dashboards, Restricted Project, Conduit, Feature Request
eadler added a project to T11254: Provide a way for quick revert of all activities of the most recent task editor.: Abuse.
Jul 3 2016, 5:49 AM · Abuse, Maniphest, Wikimedia, Feature Request
hach-que added projects to T11262: On slow connections, it's possible to submit the Create Task form on workboards multiple times: Bug Report, Maniphest.
Jul 3 2016, 5:20 AM · Maniphest, Bug Report
hach-que created T11262: On slow connections, it's possible to submit the Create Task form on workboards multiple times.
Jul 3 2016, 5:20 AM · Maniphest, Bug Report

Jul 2 2016

chad added a comment to T9360: Unbeta Phame.

Oh, and we select the correct protocol for custom domains now, but don't enforce it when serving content. That is, these both work, and we always link to "https", but "http" should redirect you directly to "https" instead of serving content:

Jul 2 2016, 8:13 PM · Phame
chad renamed T11261: Simplify Dashboard Panel Layouts from Simply Dashboard Panel Layouts to Simplify Dashboard Panel Layouts.
Jul 2 2016, 7:47 PM · Dashboards
epriestley added a comment to T10855: Allow external systems to edit the content of dashboard panels so they can publish information.

The actual panel transactions that we need to update for this aren't too bad. The board transactions are a little more involved, but we can skip them.

Jul 2 2016, 7:36 PM · Dashboards, Restricted Project, Conduit, Feature Request
epriestley moved T5267: Localize Phabricator from Backlog to The Queue on the Prioritized board.
Jul 2 2016, 7:24 PM · Prioritized, Localization
epriestley moved T10855: Allow external systems to edit the content of dashboard panels so they can publish information from Preflight to The Queue on the Prioritized board.
Jul 2 2016, 7:24 PM · Dashboards, Restricted Project, Conduit, Feature Request
epriestley removed a project from T11051: "Explain Why" isn't properly explaining why revisions were closed: Prioritized.

This could still be improved, but the immediate confusion we encountered appears fixed.

Jul 2 2016, 7:18 PM · Restricted Project, Differential, Bug Report
epriestley removed a project from T7916: Can't paste images from clipboard in Firefox / Safari: Prioritized.
Jul 2 2016, 7:17 PM · Restricted Project, Firefox, Safari, PHUI
epriestley moved T3875: Allow arc to perform cascading rebases, maybe during or after `arc land` from Backlog to Rugged Terrain on the Prioritized board.
Jul 2 2016, 7:15 PM · Prioritized, Restricted Project, Arcanist
epriestley moved T11085: Prevent users from generating an ambiguous commit message by editing revisions with the web UI from Backlog to The Queue on the Prioritized board.
Jul 2 2016, 7:12 PM · Prioritized, Restricted Project, Arcanist, Bug Report
epriestley added a comment to T11085: Prevent users from generating an ambiguous commit message by editing revisions with the web UI.

We can fix the highly-tractable web UI part, at least.

Jul 2 2016, 7:12 PM · Prioritized, Restricted Project, Arcanist, Bug Report
epriestley renamed T11085: Prevent users from generating an ambiguous commit message by editing revisions with the web UI from arcanist incorrectly parses fields inside fields to Prevent users from generating an ambiguous commit message by editing revisions with the web UI.
Jul 2 2016, 7:12 PM · Prioritized, Restricted Project, Arcanist, Bug Report
epriestley moved T11035: Only evaluate newly added mentions/subscribers when applying transactions to remarkup blocks from The Queue to Paused on the Prioritized board.
Jul 2 2016, 7:11 PM · Prioritized, Wikimedia, Remarkup
epriestley removed a project from T9789: Make it easier to write custom transaction types: Prioritized.

(Just bookkeeping; there's no external priority on this.)

Jul 2 2016, 7:10 PM · Modernization, Transactions
epriestley moved T11114: Move Differential to EditEngine from Backlog to Rugged Terrain on the Prioritized board.
Jul 2 2016, 7:09 PM · Prioritized, Restricted Project, Infrastructure, EditEngine, Differential
epriestley moved T2543: Add a formal "Draft" / "Not Yet Ready for Review" state to Differential from Preflight to Rugged Terrain on the Prioritized board.
Jul 2 2016, 7:09 PM · Customer Impact, Restricted Project, Restricted Project, Prioritized, Differential
epriestley moved T8909: Provide ability to hide all inline or done comments from Backlog to Rugged Terrain on the Prioritized board.

I'm moving this to "Rugged Terrain", as the best available pathway forward leaves us with dramatically higher implementation cost than the reasonable value of the feature. We can pursue this after tasks with better aligned costs and value are addressed.

Jul 2 2016, 7:09 PM · Inline Comments, Prioritized, Restricted Project, Restricted Project, Differential, FreeBSD
epriestley removed a project from T4631: Allow Differential to raise warnings on the server side via Conduit: Prioritized.

Mooted from a prioritization standpoint by D16139.

Jul 2 2016, 7:01 PM · Restricted Project, Arcanist, Differential
epriestley moved T11159: Cluster detection of "replication OK" is unreasonably optimistic from Backlog to The Queue on the Prioritized board.
Jul 2 2016, 7:00 PM · Prioritized, Clusters
chad merged task T11261: Simplify Dashboard Panel Layouts into T11132: New Phabricator NUX.
Jul 2 2016, 6:36 PM · Dashboards
chad merged T11261: Simplify Dashboard Panel Layouts into T11132: New Phabricator NUX.
Jul 2 2016, 6:36 PM · Design, Phacility, NUX
chad added a comment to T11261: Simplify Dashboard Panel Layouts.

but yeah, a spacer could help too

Jul 2 2016, 6:35 PM · Dashboards
epriestley added a comment to T11261: Simplify Dashboard Panel Layouts.

That sounds good to me, should give us more clarity around the dashboard stuff.

Jul 2 2016, 6:35 PM · Dashboards
chad added a comment to T11261: Simplify Dashboard Panel Layouts.

Maybe I should just design out my dock idea, the dock then goes with Dashboards (admins can set the apps, etc).

Jul 2 2016, 6:34 PM · Dashboards
epriestley added a comment to T11261: Simplify Dashboard Panel Layouts.

When you're editing a homepage dashboard, just having a spacer bar there for the applications menu might help make it more clear that 66/33 is better in a lot of cases than 50/50 -- currently, you can't see what stuff will really look like until you're already done and installed.

Jul 2 2016, 6:33 PM · Dashboards
chad added a comment to T11261: Simplify Dashboard Panel Layouts.

I think some sort of logged out / open source public template would head off like the majority of these. I think I had a plan in another thread.

Jul 2 2016, 6:33 PM · Dashboards
epriestley added a comment to T11261: Simplify Dashboard Panel Layouts.

wow pretty sure I even picked numbers that add up to 100%

Jul 2 2016, 6:32 PM · Dashboards
epriestley added a comment to T11261: Simplify Dashboard Panel Layouts.

I definitely think we should skip the 3-column versions for now, I don't think we've seen any kind of need for them in practice. We could look at it again after Facts since I think some of these weirder things could make more sense on standalone dashboards, the app just isn't powerful enough yet for that to be paritcularly useful.

Jul 2 2016, 6:32 PM · Dashboards
chad added a comment to T11261: Simplify Dashboard Panel Layouts.

this guy is clueless: T5531#66370

Jul 2 2016, 6:28 PM · Dashboards
chad added a comment to T11261: Simplify Dashboard Panel Layouts.

T6444#81725

Jul 2 2016, 6:28 PM · Dashboards
chad added a comment to T11261: Simplify Dashboard Panel Layouts.

The other solution is just to make it so these are more hidden / advanced options. That the defaults we provide are good enough that users just modify those templates instead of making one from scratch.

Jul 2 2016, 6:24 PM · Dashboards
epriestley added a comment to T11261: Simplify Dashboard Panel Layouts.

Do you think 50/50 and 33/66 are always bad even for standalone (not-on-homepage) dashboards? e.g., if we had a "Phacility Metrics" dashboard in the future after Facts, a 50/50 of a "Users" column and "Revenue" column or whatever could never be a useful way to present the information?

Jul 2 2016, 6:22 PM · Dashboards
chad added a comment to T11261: Simplify Dashboard Panel Layouts.

My rough thinking for new homepage is roughly consisting of three things:

Jul 2 2016, 6:19 PM · Dashboards
epriestley added a comment to T11261: Simplify Dashboard Panel Layouts.

Haha, that's probably T10612.

Jul 2 2016, 6:16 PM · Dashboards
chad added a comment to T11261: Simplify Dashboard Panel Layouts.

Babel seems to have the same panel in both columns, too.

Jul 2 2016, 6:14 PM · Dashboards
chad added a comment to T11261: Simplify Dashboard Panel Layouts.

The problem with 50/50 layouts is they don't guide the user anywhere as to what is important. It basically says, everything is important and conversely, is difficult to follow the page since panels just appear everywhere.

Jul 2 2016, 6:06 PM · Dashboards
chad added a revision to T11261: Simplify Dashboard Panel Layouts: Unknown Object (Differential Revision).
Jul 2 2016, 6:05 PM · Dashboards
epriestley added a comment to T11261: Simplify Dashboard Panel Layouts.

Why do you think installs (offhand: Babel, WMF) are picking 1/2 + 1/2 if they're strictly worse than 2/3 + 1/3?

Jul 2 2016, 6:05 PM · Dashboards
chad created T11261: Simplify Dashboard Panel Layouts.
Jul 2 2016, 6:00 PM · Dashboards
epriestley triaged T11260: Deactivate "Adjust timezone" when user setting is UTC as Wishlist priority.

You're the only user we're currently aware of who wants to use the software like this, so I don't plan to address this use case at this time.

Jul 2 2016, 5:03 PM · User Preferences
jayvdb added a comment to T11260: Deactivate "Adjust timezone" when user setting is UTC.

But when I am looking at my calendar app, I want to see local times, for whatever timezone I am in.

What would you like Phabricator to do when you look at events in its Calendar application?

Jul 2 2016, 5:00 PM · User Preferences
epriestley added a comment to T11260: Deactivate "Adjust timezone" when user setting is UTC.

But when I am looking at my calendar app, I want to see local times, for whatever timezone I am in.

Jul 2 2016, 4:50 PM · User Preferences
jayvdb created T11260: Deactivate "Adjust timezone" when user setting is UTC.
Jul 2 2016, 4:46 PM · User Preferences
featherless added a comment to T11259: UNRECOVERABLE FATAL ERROR after upgrading to 2016 Week 27.

Got it. Thank you! I just started the reindexing operations recommended in this release so I may leave things be 'til that finishes.

Jul 2 2016, 4:41 PM · Bug Report
epriestley added a comment to T11259: UNRECOVERABLE FATAL ERROR after upgrading to 2016 Week 27.

they won't not exist

Jul 2 2016, 4:41 PM · Bug Report
epriestley added a comment to T11259: UNRECOVERABLE FATAL ERROR after upgrading to 2016 Week 27.

"Ignored" in the sense of: no publishing, no herald rules, no audit triggers. Not, like, "they won't not exist".

Jul 2 2016, 4:41 PM · Bug Report
epriestley added a comment to T11259: UNRECOVERABLE FATAL ERROR after upgrading to 2016 Week 27.

Only that any legitimate commits pushed until things catch up will also be ignored. The repository will automatically leave the mode once everything finishes.

Jul 2 2016, 4:40 PM · Bug Report
featherless added a comment to T11259: UNRECOVERABLE FATAL ERROR after upgrading to 2016 Week 27.

Ok; will let it do its thing then.

Jul 2 2016, 4:39 PM · Bug Report
epriestley added a comment to T11259: UNRECOVERABLE FATAL ERROR after upgrading to 2016 Week 27.

There was a relatively recent change to make Phabricator discover commits which are only reachable from tags and refs. As it discovers these commits, it will treat them like new commits and publish feed stories about them. This is a one-time event, but a few installs have a lot of tag-reachable commits for whatever reason. You can find a bit more discussion in T11188, and details about the change in T9028 and T6878.

Jul 2 2016, 4:35 PM · Bug Report
epriestley added a comment to T7916: Can't paste images from clipboard in Firefox / Safari.

This doesn't appear to even to be possible in Firefox. Here's a simple example:

Jul 2 2016, 4:31 PM · Restricted Project, Firefox, Safari, PHUI
featherless added a comment to T11259: UNRECOVERABLE FATAL ERROR after upgrading to 2016 Week 27.

Ah, I found the problem in our upgrade script. We run our upgrade in the following order:

Jul 2 2016, 4:30 PM · Bug Report
featherless added a comment to T11259: UNRECOVERABLE FATAL ERROR after upgrading to 2016 Week 27.

Thanks for quick response; hadn't seen this before so couldn't tell how serious the errors were.

Jul 2 2016, 4:29 PM · Bug Report
epriestley closed T11259: UNRECOVERABLE FATAL ERROR after upgrading to 2016 Week 27 as Resolved.

Oh, or Apache.

Jul 2 2016, 4:27 PM · Bug Report
featherless added a comment to T11259: UNRECOVERABLE FATAL ERROR after upgrading to 2016 Week 27.

Just rebooted apache once more and things are looking better.

Jul 2 2016, 4:26 PM · Bug Report
epriestley added a comment to T11259: UNRECOVERABLE FATAL ERROR after upgrading to 2016 Week 27.

Also likely fixed by restarting PHP-FPM, that method is defined here:

Jul 2 2016, 4:26 PM · Bug Report
epriestley added a comment to T11259: UNRECOVERABLE FATAL ERROR after upgrading to 2016 Week 27.

Did you restart everything after upgrading? See Restarting Phabricator.

Jul 2 2016, 4:25 PM · Bug Report
featherless added a comment to T11259: UNRECOVERABLE FATAL ERROR after upgrading to 2016 Week 27.

Getting the following when I visit differential/:

Jul 2 2016, 4:25 PM · Bug Report
featherless created T11259: UNRECOVERABLE FATAL ERROR after upgrading to 2016 Week 27.
Jul 2 2016, 4:23 PM · Bug Report
epriestley added a comment to T7916: Can't paste images from clipboard in Firefox / Safari.

From the Firefox bug and this linked one:

Jul 2 2016, 3:44 PM · Restricted Project, Firefox, Safari, PHUI
epriestley closed T9028: Phabricator handles commits which are published and later deleted poorly as Resolved.

This appears to be stable, and is now accounted for.

Jul 2 2016, 3:24 PM · Restricted Project, Prioritized, Audit, Restricted Project
epriestley closed T11098: Feedback and Errata for Global Settings as Resolved.

This seem quiet and is now accounted for.

Jul 2 2016, 3:22 PM · Prioritized, Restricted Project, User Preferences
epriestley closed T3025: Detect when browser and account timezone settings differ and prompt user to reconcile them as Resolved.

This appears functional and is now accounted for.

Jul 2 2016, 3:18 PM · Prioritized, Restricted Project, User Preferences
epriestley closed T10431: Allow arc alias to read aliases from system configuration file as Resolved.

Haven't seen issues; now accounted for.

Jul 2 2016, 3:15 PM · Prioritized, Restricted Project, Arcanist, Feature Request
epriestley closed T10785: Allow custom third-party auth providers to implement autologin as Resolved.

The upstream part of this is working in production, and this is now accounted for.

Jul 2 2016, 3:13 PM · Prioritized, Auth, Restricted Project, Feature Request
epriestley closed T10227: Provide an extension point for HTTP behaviors (proxies, SSL trust) as Resolved.

We haven't seen issues with this and it's now accounted for (I believe it's also in production).

Jul 2 2016, 3:10 PM · Prioritized, Restricted Project, Feature Request
epriestley removed a project from T7441: Support OAuth providers in the cluster: Prioritized.

A substantial number of users have linked Google accounts in production and we haven't heard about any issues, so I think that aspect of things is in good shape.

Jul 2 2016, 3:04 PM · Auth, Phacility
epriestley closed T11140: Support at-rest encryption in Files as Resolved.

This is accounted for and we haven't run into any issues upstream with it in production after using it for about two weeks, but let us know if anyone runs into issues deploying it.

Jul 2 2016, 3:02 PM · Prioritized, Files
epriestley added a comment to T11257: HTML in Diffusion not escaped in certain circumstances.

For reference, @Danny_B filed a copy of this on HackerOne here so I could award a bounty for it:

Jul 2 2016, 2:50 PM · Security, Wikimedia, Diffusion, Bug Report
epriestley added a comment to T11257: HTML in Diffusion not escaped in certain circumstances.

Also worth noting is that Content Security Policy (T4340) would have prevented this. CSP is obviously worth implementing, but I believe the last attack it would have prevented was D4534 in January 2013, so it's hard to prioritize if it only defuses one attack every 3-4 years.

Jul 2 2016, 2:49 PM · Security, Wikimedia, Diffusion, Bug Report
epriestley added a comment to T11256: Plan how to split apart instance permissions.

Instances currently have two capabilities:

Jul 2 2016, 2:33 PM · Phacility
epriestley added a revision to T11256: Plan how to split apart instance permissions: Unknown Object (Differential Revision).
Jul 2 2016, 2:13 PM · Phacility
Danny_B updated subscribers of T11257: HTML in Diffusion not escaped in certain circumstances.
Jul 2 2016, 1:25 PM · Security, Wikimedia, Diffusion, Bug Report
epriestley added a comment to T11257: HTML in Diffusion not escaped in certain circumstances.

The Phacility cluster has been patched.

Jul 2 2016, 1:12 PM · Security, Wikimedia, Diffusion, Bug Report
epriestley edited the content of 2016 Week 27 (Very Early July).
Jul 2 2016, 12:30 PM
epriestley added a comment to T11257: HTML in Diffusion not escaped in certain circumstances.

In the future, please report security issues via HackerOne:

Jul 2 2016, 12:27 PM · Security, Wikimedia, Diffusion, Bug Report
epriestley claimed T11257: HTML in Diffusion not escaped in certain circumstances.
Jul 2 2016, 12:27 PM · Security, Wikimedia, Diffusion, Bug Report
epriestley closed T11257: HTML in Diffusion not escaped in certain circumstances as Resolved by committing Restricted Diffusion Commit.
Jul 2 2016, 12:18 PM · Security, Wikimedia, Diffusion, Bug Report
epriestley closed T11258: Push to Remote repository instead of Mirror - How to sync 2 remotes using Phabricator Diffusion as Wontfix.

This is impossible.

Jul 2 2016, 11:59 AM · Diffusion, Feature Request
johnny-bit added a comment to T4788: Allow "Edit Dependencies" both ways (blocking and depending).

Current task graph is great for visual representation... but can we get somewhere very simple list of blocks/blocked by?

Jul 2 2016, 8:34 AM · Prioritized, FreeBSD, User Delight, Wikimedia, Maniphest
• niveshsaharan created T11258: Push to Remote repository instead of Mirror - How to sync 2 remotes using Phabricator Diffusion.
Jul 2 2016, 7:47 AM · Diffusion, Feature Request
Danny_B created T11257: HTML in Diffusion not escaped in certain circumstances.
Jul 2 2016, 7:28 AM · Security, Wikimedia, Diffusion, Bug Report
chad added a revision to T9360: Unbeta Phame: Unknown Object (Differential Revision).
Jul 2 2016, 4:34 AM · Phame
chad edited the content of 2016 Week 27 (Very Early July).
Jul 2 2016, 1:02 AM
chad edited the content of 2016 Week 27 (Very Early July).
Jul 2 2016, 1:02 AM

Jul 1 2016

epriestley edited the content of Changelog.
Jul 1 2016, 11:41 PM
epriestley created 2016 Week 27 (Very Early July).
Jul 1 2016, 11:41 PM