Page MenuHomePhabricator
Feed Advanced Search

Feb 24 2017

Mnkras added a comment to T12296: Improve Phacility repository import performance.

@epriestley
https://test-3uoifmrptebb.phacility.com/R2:80233a27d7b72bb03710d358e12a531ce46808a9

Feb 24 2017, 6:25 PM · Customer Impact, Diffusion, Ops
epriestley added a comment to E1394: Dentist.

good luck w/ teeth 😁

Feb 24 2017, 6:07 PM
epriestley added a revision to T12296: Improve Phacility repository import performance: Restricted Differential Revision.
Feb 24 2017, 6:05 PM · Customer Impact, Diffusion, Ops
epriestley added a comment to T12296: Improve Phacility repository import performance.

Starting with repository update:

Feb 24 2017, 5:56 PM · Customer Impact, Diffusion, Ops
chad added a comment to E1394: Dentist.

so excite. much radiation.

Feb 24 2017, 5:54 PM
chad created E1394: Dentist.
Feb 24 2017, 5:44 PM
epriestley added a revision to T12298: Allow daemon pools to autoscale down to 0 processes: Unknown Object (Differential Revision).
Feb 24 2017, 5:14 PM · Daemons, Ops, Phacility
epriestley added a revision to T12298: Allow daemon pools to autoscale down to 0 processes: Unknown Object (Differential Revision).
Feb 24 2017, 5:06 PM · Daemons, Ops, Phacility
allixsenos updated the task description for T12313: Cloudflare leaked all HTTPS traffic on the internet into public caches ("Cloudbleed").
Feb 24 2017, 2:41 PM · Guides, Security
epriestley added a comment to T12313: Cloudflare leaked all HTTPS traffic on the internet into public caches ("Cloudbleed").

Broadly, there is nothing Phabricator could have done differently to anticipate or prevent this issue.

Feb 24 2017, 2:31 PM · Guides, Security
epriestley added a comment to T12313: Cloudflare leaked all HTTPS traffic on the internet into public caches ("Cloudbleed").

We could also provide some kind of bin/auth burn-everything-to-the-ground to automate these steps, but I worry that it would almost never be tested (I think it's very difficult to test comprehensively in an automated way) or run, and there's a good chance it might be forgotten about when authentication changes occur.

Feb 24 2017, 2:24 PM · Guides, Security
epriestley updated the task description for T12313: Cloudflare leaked all HTTPS traffic on the internet into public caches ("Cloudbleed").
Feb 24 2017, 2:12 PM · Guides, Security
epriestley added a comment to T12313: Cloudflare leaked all HTTPS traffic on the internet into public caches ("Cloudbleed").

Ah, you're right. It looks like request data may also have leaked. I'll edit my earlier posts to reflect that. Briefly, the change is: passwords and VCS passwords are potentially at risk.

Feb 24 2017, 2:12 PM · Guides, Security
allixsenos added a comment to T12313: Cloudflare leaked all HTTPS traffic on the internet into public caches ("Cloudbleed").

Has cloudflare explicitly said that only responses were leaked, and never
requests? I recall "POST data" being explicitly called out as showing up in
the leaks, but I can't remember if this was in an official write up or
media coverage.

Feb 24 2017, 1:43 PM · Guides, Security
epriestley added a comment to T12313: Cloudflare leaked all HTTPS traffic on the internet into public caches ("Cloudbleed").
NOTE: If Aphlict is served from the same domain as Phabricator (e.g., just on a different port) with Cloudflare as an intermediary, none of the rest of this comment applies: browsers could have sent credentials with the request and Cloudflare could have cached them.
Feb 24 2017, 12:50 PM · Guides, Security
epriestley added a comment to T12313: Cloudflare leaked all HTTPS traffic on the internet into public caches ("Cloudbleed").

To cycle passwords, you can do this:

Feb 24 2017, 12:43 PM · Guides, Security
allixsenos added a comment to T12313: Cloudflare leaked all HTTPS traffic on the internet into public caches ("Cloudbleed").

Only our websocket host is behind CF proxies, so I invalidated all auth sessions by changing the security.hmac-key. I wanted to also invalidate all Conduit tokens just as a fire drill but couldn't found a method of doing this.

Feb 24 2017, 10:56 AM · Guides, Security
20after4 added a comment to T12296: Improve Phacility repository import performance.

My impression is that there is a lot of room for optimization in the RepositoryPullLocalDaemon. I've wanted to tackle that for a while because it causes a fair delay between when a commit goes into git and when it actually shows up in Diffusion.

Feb 24 2017, 10:45 AM · Customer Impact, Diffusion, Ops
voltcode added a comment to T12310: Phd out of memory error.

I found that svn repo import was stuck on a very large branch import. I marked the repo as imported, suddenly the herald rules and notifications started to work properly (which was my other problem that I wanted to solve with my instance).

Feb 24 2017, 8:17 AM · Bug Report (Needs Information)
celia added a comment to T4382: Pholio inline comment "not submitted yet" message: add title to explain the message.

Hello I have a question that:
if there are several users written in comments in Pholio Mocks part, how can I see other person's comments?

Feb 24 2017, 6:48 AM · Pholio
celia added a project to T4382: Pholio inline comment "not submitted yet" message: add title to explain the message: Pholio.
Feb 24 2017, 6:46 AM · Pholio
celia removed a project from T4382: Pholio inline comment "not submitted yet" message: add title to explain the message: Pholio.
Feb 24 2017, 6:46 AM · Pholio
revi added a comment to T12313: Cloudflare leaked all HTTPS traffic on the internet into public caches ("Cloudbleed").

If you have configured your entire install behind Cloudflare (we do not encourage this in the documentation), you are at greater risk, as anything served by Phabricator could have been leaked. If you believe you are affected and would like help rotating sessions and invalidating credentials, let us know.

Feb 24 2017, 6:03 AM · Guides, Security
chad added a comment to T7190: Project templates/Copying tasks.

Events stuff isn't on any roadmap, so it also could be years before it was modified.

Feb 24 2017, 4:41 AM · Projects, Maniphest
chad added a revision to T6049: Implement Phurl, a URL shortener/bookmarking application: Unknown Object (Differential Revision).
Feb 24 2017, 4:25 AM · Phurl
• testqq added a comment to Q179: How do you configure an Almanac based Drydock resource (Answer 212).

First Part : Almanac - Create a service is now in

Feb 24 2017, 2:52 AM
isfs added a comment to T7190: Project templates/Copying tasks.

Thanks for the pointer. I wondered if there was something like that I could hook into.

Feb 24 2017, 2:42 AM · Projects, Maniphest
chad added a comment to T7190: Project templates/Copying tasks.

I think https://secure.phabricator.com/book/phabricator/article/events/#ui-did-render-actions still works, that you can build a "Clone Task" action on the side of your Maniphest links and have it use that task as a templateID. Might be worth fiddling with locally.

Feb 24 2017, 2:21 AM · Projects, Maniphest
isfs added a comment to T7190: Project templates/Copying tasks.

I don't have a problem grouping the use cases together, and it makes a lot of sense to solve as many use cases as possible per feature. I just wanted to point out the differences so that when this task progresses, it can progress with the different use cases in mind. At a first glance, I didn't think the use cases were very similar at all, as this is a lot about creating a bunch of tasks by template (or from a template project), which is a bulk action, really, whereas really our use case is about task micromanagement. However, on further consideration, there is significant overlap, especially from what T3320 brings to the table.

Feb 24 2017, 1:50 AM · Projects, Maniphest
chad added a comment to T7190: Project templates/Copying tasks.

Though this task also talks about Project Templates so maybe we should have two tasks.

Feb 24 2017, 12:51 AM · Projects, Maniphest
chad added a comment to T7190: Project templates/Copying tasks.

All of your use cases fall into this task, I believe. That being "I want a new task based on x, y, or z" We want to group similar feature requests together for planning reasons, so multiple root problems can be solved by a common, thought-out framework. I can't guess what the UI would look like, or if we'd offer anything else like you're mentioning. I appreciate the detail on the original task, but in general we're not likely to be able to get more into a discussion on a task thats several years away at best.

Feb 24 2017, 12:48 AM · Projects, Maniphest
chad added a comment to T7190: Project templates/Copying tasks.

How much time per day do you expect this feature to save you?

Feb 24 2017, 12:35 AM · Projects, Maniphest
isfs added a comment to T7190: Project templates/Copying tasks.

I see there is some similarity, but some important differences in the use cases:

  • We are interested in splitting a specific task, crafting the title and description in particular, manually (in fact, with quite a lot of care), whereas here the focus is on creating generic tasks, or templated tasks
  • We would not want subtasks to be copied, I don't think
  • We need to be able to edit the original task at the same time as creating the new ones, whereas there is no aspect of that in this use case
Feb 24 2017, 12:26 AM · Projects, Maniphest
epriestley added a comment to T12313: Cloudflare leaked all HTTPS traffic on the internet into public caches ("Cloudbleed").

Although this problem is severe, the absolute size of the leak appears to be small (my task title is hyperbolic). From the Cloudflare disclosure:

Feb 24 2017, 12:21 AM · Guides, Security
epriestley updated the task description for T12313: Cloudflare leaked all HTTPS traffic on the internet into public caches ("Cloudbleed").
Feb 24 2017, 12:15 AM · Guides, Security
epriestley updated the task description for T12313: Cloudflare leaked all HTTPS traffic on the internet into public caches ("Cloudbleed").
Feb 24 2017, 12:15 AM · Guides, Security
epriestley created T12313: Cloudflare leaked all HTTPS traffic on the internet into public caches ("Cloudbleed").
Feb 24 2017, 12:11 AM · Guides, Security

Feb 23 2017

epriestley awarded Bug Meister to recipient: nicolast.
Feb 23 2017, 11:38 PM
epriestley awarded Bug Meister to recipient: willplatts.
Feb 23 2017, 11:37 PM
epriestley added a comment to T12304: Herald fails to add auditors if the author is an auditor.

Thanks for the report! This should be fixed in HEAD of master, and promote to stable tomorrow, a little more than 24 hours from now. Let us know if you run into any issues.

Feb 23 2017, 11:37 PM · Herald, Bug Report
epriestley added a comment to T12302: Herald adding different auditors but only the last one is kept.

Thanks for the report! This should be fixed in HEAD of master, and promote to stable tomorrow, a little more than 24 hours from now. Let us know if you run into any issues.

Feb 23 2017, 11:36 PM · Herald, Bug Report
chad added a comment to T7190: Project templates/Copying tasks.

I merged this primarily for the "Create Similar Task" discussion above, which I think a similar enough feature.

Feb 23 2017, 11:31 PM · Projects, Maniphest
chad merged T12312: Workflow for splitting tasks into T7190: Project templates/Copying tasks.
Feb 23 2017, 11:29 PM · Projects, Maniphest
chad merged task T12312: Workflow for splitting tasks into T7190: Project templates/Copying tasks.
Feb 23 2017, 11:29 PM · Maniphest, Feature Request
epriestley closed T12304: Herald fails to add auditors if the author is an auditor as Resolved by committing Restricted Diffusion Commit.
Feb 23 2017, 11:19 PM · Herald, Bug Report
epriestley closed T12302: Herald adding different auditors but only the last one is kept as Resolved by committing Restricted Diffusion Commit.
Feb 23 2017, 11:15 PM · Herald, Bug Report
isfs added a project to T12312: Workflow for splitting tasks: Maniphest.
Feb 23 2017, 11:01 PM · Maniphest, Feature Request
isfs created T12312: Workflow for splitting tasks.
Feb 23 2017, 11:01 PM · Maniphest, Feature Request
epriestley added a revision to T12304: Herald fails to add auditors if the author is an auditor: Unknown Object (Differential Revision).
Feb 23 2017, 10:47 PM · Herald, Bug Report
epriestley added a revision to T12302: Herald adding different auditors but only the last one is kept: Unknown Object (Differential Revision).
Feb 23 2017, 10:44 PM · Herald, Bug Report
chad added a revision to T12270: Unbeta Badges: Unknown Object (Differential Revision).
Feb 23 2017, 7:47 PM · Badges (v0)
epriestley added a comment to T7954: Make it easier to upload XHProf profiles.

I didn't "ref" it here, but D17401 sort of technically did this. I'd like to clean it up a bit more before calling this resolved, though -- particularly, this is basically a secret that only I know about.

Feb 23 2017, 7:34 PM · Infrastructure, XHProf
epriestley added a comment to T12172: Better color scheme for diffs.

Also, above, I used the term "deuteranomaly" when I meant "deuteranopia". I mistakenly believed they were describing the same type of colorblindness, but they are two different types.

Feb 23 2017, 7:13 PM · Accessibility, Feature Request
epriestley added a comment to T12311: Allow users to choose both "Larger Fonts" and "High Contrast" at the same time.

Possibly this should really be three dimensions (Size, Contrast, Color Correction).

Feb 23 2017, 7:11 PM · Celerity, Accessibility
epriestley added a comment to T12172: Better color scheme for diffs.

See T12060#213081 some end notes and followups.

Feb 23 2017, 7:11 PM · Accessibility, Feature Request
epriestley added a comment to T12060: Colorblind accessibility for red/green highlighting in e-mails.

See some additional discussion in T12172. Briefly:

Feb 23 2017, 7:10 PM · Accessibility, Design, Feature Request, Restricted Project
epriestley added a project to T12311: Allow users to choose both "Larger Fonts" and "High Contrast" at the same time: Celerity.
Feb 23 2017, 7:03 PM · Celerity, Accessibility
epriestley created T12311: Allow users to choose both "Larger Fonts" and "High Contrast" at the same time.
Feb 23 2017, 7:03 PM · Celerity, Accessibility
epriestley closed T12060: Colorblind accessibility for red/green highlighting in e-mails as Resolved by committing Restricted Diffusion Commit.
Feb 23 2017, 6:57 PM · Accessibility, Design, Feature Request, Restricted Project
epriestley closed T12172: Better color scheme for diffs as Resolved by committing Restricted Diffusion Commit.
Feb 23 2017, 6:57 PM · Accessibility, Feature Request
epriestley added a comment to T12157: Make it visibly clearer that a Phabricator user account has been disabled.

@chad did another pass on user hovercards and they should be even more clear now (this screenshot doesn't demonstrate it clearly, but we also greyscale the profile photo);

Feb 23 2017, 6:46 PM · Profile, Feature Request
epriestley added a comment to T12288: Phabricator OAuth 2.0 initiating SSO login from a Third Party website.

Why do you have multiple authentication providers?

Feb 23 2017, 6:43 PM · Auth, Feature Request
• pasik added a comment to T12288: Phabricator OAuth 2.0 initiating SSO login from a Third Party website.

@epriestley: Yes, we have multiple authentication providers configured and needed in phabricator, and thus "allow auto login" won't work.

Feb 23 2017, 6:42 PM · Auth, Feature Request
epriestley added a revision to T12297: Make Conduit API calls on `admin.phacility.com` reasonably easy to profile: Unknown Object (Differential Revision).
Feb 23 2017, 6:41 PM · XHProf, Ops, Phacility
epriestley added a comment to T12196: PHP7, particularly before 7.1.2, segfaults a whole lot while running CLI scripts.

Yeah, I'm basically just keeping this open as a dumping ground for the moment and purely selfishly because my segfaults aren't resolved yet -- brew didn't have PHP 7.1.2 last I checked, although it seems like it now does. I've upgraded and I'll see if that improves things.

Feb 23 2017, 6:04 PM · Bug Report
epriestley added a revision to T12297: Make Conduit API calls on `admin.phacility.com` reasonably easy to profile: Unknown Object (Differential Revision).
Feb 23 2017, 5:59 PM · XHProf, Ops, Phacility
epriestley added a revision to T12297: Make Conduit API calls on `admin.phacility.com` reasonably easy to profile: Unknown Object (Differential Revision).
Feb 23 2017, 5:31 PM · XHProf, Ops, Phacility
• rajuonraj added a watcher for Bug Report (Needs Information): • rajuonraj.
Feb 23 2017, 4:53 PM
chad renamed T12310: Phd out of memory error from Phd out of memory error to Phd out of memory error.
Feb 23 2017, 4:50 PM · Bug Report (Needs Information)
epriestley edited projects for T12310: Phd out of memory error, added: Bug Report (Needs Information); removed Bug Report.

We need reproduction instructions which allow us to reproduce this issue in a clean environment. See Contributing Bug Reports and Providing Reproduction Steps.

Feb 23 2017, 3:56 PM · Bug Report (Needs Information)
voltcode created T12310: Phd out of memory error.
Feb 23 2017, 3:44 PM · Bug Report (Needs Information)
DragonBe edited the content of Organizations Using Phabricator.
Feb 23 2017, 2:42 PM
avivey added a comment to Q572: Database can not be initialized - bin-storage upgrade.

How did you end up in this state?
Is this part of a fresh install, or an upgrade? If upgrade, what version was the old code, and what version is the new code?

Feb 23 2017, 1:51 PM
AlexL777 added a comment to Q572: Database can not be initialized - bin-storage upgrade.

Database status can not be retrieved, because I am told, upgrade the database first.

Feb 23 2017, 12:14 PM
AlexL777 updated Q572: Database can not be initialized - bin-storage upgrade from to Database can not be initialized - bin-storage upgrade.
Feb 23 2017, 12:12 PM
chad updated the task description for T10390: Dashboards v2 (UX).
Feb 23 2017, 3:21 AM · Dashboards (v2)
cspeckmim updated the task description for T12309: Differential comment that inlines a Paste and has code block after it renders in strange fashion.
Feb 23 2017, 2:22 AM · Remarkup, Bug Report
cspeckmim closed T12309: Differential comment that inlines a Paste and has code block after it renders in strange fashion as Invalid.

Ooh just kidding. ⌃+Refresh fixed everything

Feb 23 2017, 2:21 AM · Remarkup, Bug Report
cspeckmim created T12309: Differential comment that inlines a Paste and has code block after it renders in strange fashion.
Feb 23 2017, 2:20 AM · Remarkup, Bug Report
epriestley closed T12307: Users unable to add new credit cards to Autopay? as Invalid.

This has been "resolved" by switching to bartering various sundry goods and foodstuffs instead of billing via credit card.

Feb 23 2017, 12:02 AM · Phortune, Phacility

Feb 22 2017

epriestley closed T12301: `Create Form` always leads to page creating forms for configuring forms as Resolved by committing Restricted Diffusion Commit.
Feb 22 2017, 11:00 PM · EditEngine, Bug Report
epriestley added a revision to T12301: `Create Form` always leads to page creating forms for configuring forms: Unknown Object (Differential Revision).
Feb 22 2017, 10:34 PM · EditEngine, Bug Report
epriestley closed T12306: Failing svn connection brings daemon down as Resolved by committing Restricted Diffusion Commit.
Feb 22 2017, 10:11 PM · Daemons, Bug Report
epriestley added a revision to T12306: Failing svn connection brings daemon down: Unknown Object (Differential Revision).
Feb 22 2017, 10:09 PM · Daemons, Bug Report
epriestley added a revision to T2794: Implement Phage (like Hypershell): Unknown Object (Differential Revision).
Feb 22 2017, 9:49 PM · Phage, Infrastructure
epriestley added a revision to T12218: Reduce the operational cost of a larger Phacility cluster: Unknown Object (Differential Revision).
Feb 22 2017, 9:49 PM · Ops, Phacility
chad added a comment to T12292: Diffusion commit details references HEAD -> master broken.

Feel free to use a Phacility Test Instance and record your steps, thanks!

Feb 22 2017, 9:43 PM · Bug Report (Needs Information)
chad added a project to T12308: Access forbidden on homepage if the first dashboard in menu is not visible to user: Dashboards.
Feb 22 2017, 9:38 PM · Dashboards, Profile Menu Item, Bug Report
chad edited projects for T12292: Diffusion commit details references HEAD -> master broken, added: Bug Report (Needs Information); removed Bug Report.
Feb 22 2017, 9:25 PM · Bug Report (Needs Information)
chad updated subscribers of T12292: Diffusion commit details references HEAD -> master broken.

We require all bug reports provide complete reproduction steps. The main reason is so that anyone can follow and reproduce the bug in a local environment. Skipping details makes it difficult for most (or anyone but @epriestley) people to provide you with help.

Feb 22 2017, 9:24 PM · Bug Report (Needs Information)
chad added a comment to T12292: Diffusion commit details references HEAD -> master broken.

Hi @rsmarples, unfortunately I don't know how to reproduce this based on your steps provided. Specifically, I do not see a broken "reference" link in the details of the commit. Do you know what additional steps I can take to see this bug locally? Everything looks formatted fine?

Feb 22 2017, 9:22 PM · Bug Report (Needs Information)
epriestley added a comment to T12298: Allow daemon pools to autoscale down to 0 processes.

No time like the present.

Feb 22 2017, 9:14 PM · Daemons, Ops, Phacility
chad added a project to T12288: Phabricator OAuth 2.0 initiating SSO login from a Third Party website: Auth.
Feb 22 2017, 9:02 PM · Auth, Feature Request
chad added projects to T12300: git push --mirror --force does not work on Phacility with repo that has refs/pull: Git, Phacility.
Feb 22 2017, 9:02 PM · Diffusion, Git
chad added a project to T12301: `Create Form` always leads to page creating forms for configuring forms: EditEngine.
Feb 22 2017, 9:01 PM · EditEngine, Bug Report
chad added a project to T12302: Herald adding different auditors but only the last one is kept: Herald.
Feb 22 2017, 8:59 PM · Herald, Bug Report
chad added a project to T12304: Herald fails to add auditors if the author is an auditor: Herald.
Feb 22 2017, 8:59 PM · Herald, Bug Report
chad closed T12303: Print View in Phriction shows Persistent Chat as Resolved by committing Restricted Diffusion Commit.
Feb 22 2017, 8:31 PM · Phriction, Bug Report
kristo.mario awarded T4096: Generated gif-macros are not animated a Doubloon token.
Feb 22 2017, 8:10 PM · Macros
chad added a revision to T12303: Print View in Phriction shows Persistent Chat: Unknown Object (Differential Revision).
Feb 22 2017, 5:44 PM · Phriction, Bug Report