Page MenuHomePhabricator
Feed All Stories

Feb 3 2019

logangorence awarded T8787: Add support for U2F MFA once browser implementations improve and compatible hardware is more widely available a Like token.
Feb 3 2019, 1:29 AM · Haskell.org, Auth

Feb 2 2019

avivey awarded T13245: Improve MFA security by using replacing SMS code digits with emoji a Party Time token.
Feb 2 2019, 9:11 PM · Security
epriestley updated the task description for T13244: 2019 Week 6 Bonus Content.
Feb 2 2019, 6:55 PM · Plans
epriestley edited the content of Changelog.
Feb 2 2019, 6:51 PM
epriestley edited the content of 2019 Week 5 (Very Early February).
Feb 2 2019, 6:49 PM
epriestley created 2019 Week 5 (Very Early February).
Feb 2 2019, 6:49 PM
epriestley renamed T13245: Improve MFA security by using replacing SMS code digits with emoji from Improve MFA security by using replacing digits with emoji to Improve MFA security by using replacing SMS code digits with emoji.
Feb 2 2019, 6:46 PM · Security
epriestley triaged T13245: Improve MFA security by using replacing SMS code digits with emoji as Low priority.
Feb 2 2019, 6:46 PM · Security
epriestley added a revision to T13098: Plans: Arcanist toolsets and extensions: Unknown Object (Differential Revision).
Feb 2 2019, 1:23 PM · Arcanist, Plans
epriestley added a revision to T8693: Arc Anoid crashes on a 30x15 terminal: Unknown Object (Differential Revision).
Feb 2 2019, 1:23 PM · Arcanist
epriestley closed T7767: Running `arc anoid` results in a python stacktrace as Wontfix.

Unsure how to reproduce this.

Feb 2 2019, 1:18 PM · Arcanist
epriestley closed T11515: `arc ade` should do something as Wontfix.

I think this is reasonably well-covered between arc anoid and arc weld. Once T5055 comes online, third-parties can presumably fill in more titles for the arc game library.

Feb 2 2019, 12:58 PM · Feature Request
epriestley updated the task description for T13244: 2019 Week 6 Bonus Content.
Feb 2 2019, 1:52 AM · Plans

Feb 1 2019

epriestley closed T12314: Support formal task subtypes (like "bug" vs "feature") as Resolved.

These have existed for a while and recently got support for customizing sub-object behaviors in 2018 Week 50 (Mid December) and are being extended to Projects in 2019 Week 5, so it looks like they're here to stay.

Feb 1 2019, 10:43 PM · EditEngine, Maniphest, Custom Fields, Feature Request
epriestley triaged T13244: 2019 Week 6 Bonus Content as Normal priority.
Feb 1 2019, 10:38 PM · Plans
epriestley closed T13242: 2019 Week 5 Bonus Content as Resolved.
Feb 1 2019, 10:34 PM · Plans
epriestley updated the task description for T13242: 2019 Week 5 Bonus Content.
Feb 1 2019, 10:34 PM · Plans
epriestley added a revision to T13243: Writing to streams can run into issues with EINTR: Unknown Object (Differential Revision).
Feb 1 2019, 6:40 PM · Infrastructure
epriestley added a comment to T13243: Writing to streams can run into issues with EINTR.

Here's a script which can hit these behaviors without cloning 9000 working copies in a loop:

Feb 1 2019, 6:31 PM · Infrastructure
epriestley added a comment to Image Macro "butwhy".

uhoh

Feb 1 2019, 6:30 PM
amckinley created Image Macro "butwhy".
Feb 1 2019, 6:12 PM
epriestley triaged T13243: Writing to streams can run into issues with EINTR as Low priority.
Feb 1 2019, 5:31 PM · Infrastructure
epriestley added a comment to T13242: 2019 Week 5 Bonus Content.

State of the world on the instance changes:

Feb 1 2019, 5:18 PM · Plans
epriestley closed T13192: Inactive repositories can cause "Repository Servers" to always report "Partial Sync?" as Resolved by committing Restricted Diffusion Commit.
Feb 1 2019, 6:12 AM · Clusters, Diffusion

Jan 31 2019

epriestley added a comment to T13192: Inactive repositories can cause "Repository Servers" to always report "Partial Sync?".

See PHI1015 for a slightly meatier explanation of this issue.

Jan 31 2019, 7:47 PM · Clusters, Diffusion
epriestley added a revision to T13192: Inactive repositories can cause "Repository Servers" to always report "Partial Sync?": Unknown Object (Differential Revision).
Jan 31 2019, 7:47 PM · Clusters, Diffusion
epriestley added a revision to T11413: Support renaming Phacility instances: Unknown Object (Differential Revision).
Jan 31 2019, 5:39 PM · Phacility
epriestley added a comment to T13242: 2019 Week 5 Bonus Content.

~o.o~

Jan 31 2019, 5:03 PM · Plans
epriestley updated the task description for T13242: 2019 Week 5 Bonus Content.
Jan 31 2019, 3:21 PM · Plans

Jan 30 2019

epriestley added a revision to T13232: Mac OS X Mojave, SIP, Code Signing, and Apache: Unknown Object (Differential Revision).
Jan 30 2019, 10:51 PM · Infrastructure
epriestley added a comment to T13232: Mac OS X Mojave, SIP, Code Signing, and Apache.

PHP 7.3.1 is exiting abruptly for me (no error message, return code 0) on preg_match('(\\G\s+)sS', '');. This occurs in the daemons. I can't reproduce this in a separate test script. I have no idea what's going on.

Jan 30 2019, 9:35 PM · Infrastructure
epriestley added a comment to T13232: Mac OS X Mojave, SIP, Code Signing, and Apache.

Let's compile Gentoo nine times (Gentoo flavor is macOS Mojave 10.14.2):

Jan 30 2019, 8:35 PM · Infrastructure
epriestley added a revision to T13242: 2019 Week 5 Bonus Content: Unknown Object (Differential Revision).
Jan 30 2019, 8:20 PM · Plans
epriestley added a revision to T13242: 2019 Week 5 Bonus Content: Unknown Object (Differential Revision).
Jan 30 2019, 8:19 PM · Plans
epriestley added a comment to T13242: 2019 Week 5 Bonus Content.

PHI1010 could use more detailed triage around whether we can squeeze anything from transaction.search without hitting a bunch of other messes. This is adjacent to a lot of other work.

Jan 30 2019, 8:16 PM · Plans
epriestley added a comment to T12297: Make Conduit API calls on `admin.phacility.com` reasonably easy to profile.

Success! D20046 worked to fix the "profiler not sticking across form posts" issue on secure. 🐈

Jan 30 2019, 7:56 PM · XHProf, Ops, Phacility
epriestley added a revision to T13156: Plans: Improve Phacility UI for managing instance managers and cards: Unknown Object (Differential Revision).
Jan 30 2019, 7:40 PM · Plans, Phacility
figroc added a comment to T4256: Support Azure as an auth provider.

@epriestley Sorry bringing up this ancient task. I am also interested in this Azure OAuth feature. Would you kindly point me to some third-party extension coding guideline/documents? I've dug around for the docs, but not found any.

Jan 30 2019, 6:19 AM · Auth
epriestley added a revision to T13231: Duo MFA Support: Unknown Object (Differential Revision).
Jan 30 2019, 5:18 AM · Auth
epriestley updated the task description for T13242: 2019 Week 5 Bonus Content.
Jan 30 2019, 2:08 AM · Plans
epriestley updated the task description for T13242: 2019 Week 5 Bonus Content.
Jan 30 2019, 12:34 AM · Plans
epriestley updated the task description for T13242: 2019 Week 5 Bonus Content.
Jan 30 2019, 12:34 AM · Plans

Jan 29 2019

epriestley added a revision to T13156: Plans: Improve Phacility UI for managing instance managers and cards: Unknown Object (Differential Revision).
Jan 29 2019, 11:44 PM · Plans, Phacility
epriestley added a revision to T13156: Plans: Improve Phacility UI for managing instance managers and cards: Unknown Object (Differential Revision).
Jan 29 2019, 10:32 PM · Plans, Phacility
epriestley added a revision to T13242: 2019 Week 5 Bonus Content: Unknown Object (Differential Revision).
Jan 29 2019, 10:32 PM · Plans
epriestley added a revision to T12921: Link to referenced object in transaction emails: Unknown Object (Differential Revision).
Jan 29 2019, 8:09 PM · Mail
tburial edited the content of Organizations Using Phabricator.
Jan 29 2019, 7:47 PM
epriestley added a comment to T6994: Write a general "Security guidelines" document.

Some guidance about "configure captchas if you're a public-facing, password-login install" would be good here too

Jan 29 2019, 7:21 PM · Security
avivey added a comment to T9044: arc diff eagerly detects git-svn repo, which makes arc-patch hang.

In https://discourse.phabricator-community.org/t/2334, thopre suggests checking for git config --get-regexp svn-remote to see if the repository is configured as git-svn repo, as an alternative to check if .git/svn/.metadata exists.

Jan 29 2019, 6:05 PM · Restricted Project, Arcanist
kerberizer awarded T8787: Add support for U2F MFA once browser implementations improve and compatible hardware is more widely available a Like token.
Jan 29 2019, 3:22 PM · Haskell.org, Auth
epriestley added a comment to T7408: Consider dropping support for older versions of PHP.

D20049 introduces a piece of code which could be simplified if the minimum version was 5.4.0 or newer, so PHP_QUERY_RFC3986 is available for http_build_query().

Jan 29 2019, 1:52 PM · Restricted Project, Infrastructure
nochum added a comment to T13224: Pygments Bash lexer has explosive complexity on unterminated strings with many backslashes.

@nochum, if you want to try replacing 'pygmentize ... with 'exec pygmentize ... near line 25 of PhutilPygmentsSyntaxHighlighter.php and that fixes it, I'll upstream it. exec will stop dash from leaving an sh -c process running.

Worked perfectly!

(Also: the time limit should be 15 seconds, not 15 minutes! If you're seeing the sh processes die after 15 minutes, something else is likely afoot.)

Yes, it's 15 seconds, I misspoke.

Jan 29 2019, 1:37 PM · Infrastructure
amckinley created Image Macro "yourewelcome".
Jan 29 2019, 3:22 AM
epriestley edited the content of Changelog.
Jan 29 2019, 2:36 AM
epriestley created 2019 Week 4 (Late January).
Jan 29 2019, 2:35 AM
epriestley added a revision to T13156: Plans: Improve Phacility UI for managing instance managers and cards: Unknown Object (Differential Revision).
Jan 29 2019, 2:15 AM · Plans, Phacility
epriestley added a revision to T13242: 2019 Week 5 Bonus Content: Unknown Object (Differential Revision).
Jan 29 2019, 2:15 AM · Plans

Jan 28 2019

epriestley added a comment to T13224: Pygments Bash lexer has explosive complexity on unterminated strings with many backslashes.

@nochum, if you want to try replacing 'pygmentize ... with 'exec pygmentize ... near line 25 of PhutilPygmentsSyntaxHighlighter.php and that fixes it, I'll upstream it. exec will stop dash from leaving an sh -c process running.

Jan 28 2019, 11:37 PM · Infrastructure
avivey added a comment to T13224: Pygments Bash lexer has explosive complexity on unterminated strings with many backslashes.

D212: Fix daemon issues caused by Ubuntu's surprising intermediary shell?

Jan 28 2019, 11:24 PM · Infrastructure
nochum added a comment to T13224: Pygments Bash lexer has explosive complexity on unterminated strings with many backslashes.

It seems the commit didn't fully fix the issue. When pygmentize runs for phabricator, there are processed that are forked from the initial process that are still never killed and they cause the cpu to spike. For example, when running into this issue on our install, we see something like:
#ps aux | grep pygm
user 13190 0.0 0.0 4456 680 ? S 16:46 0:00 sh -c pygmentize -O encoding=utf-8 -O stripnl=False -f html -l 'bash'
user 13191 0.0 0.0 4456 784 ? S 16:46 0:00 sh -c pygmentize -O encoding=utf-8 -O stripnl=False -f html -l 'bash'
user 13192 74.7 0.0 23996 9336 ? R 16:46 0:05 /usr/bin/python /usr/local/bin/pygmentize -O encoding=utf-8 -O stripnl=False -f html -l bash
user 13193 76.7 0.0 23636 9028 ? R 16:46 0:06 /usr/bin/python /usr/local/bin/pygmentize -O encoding=utf-8 -O stripnl=False -f html -l bash

Jan 28 2019, 9:51 PM · Infrastructure
epriestley added a revision to T13156: Plans: Improve Phacility UI for managing instance managers and cards: Unknown Object (Differential Revision).
Jan 28 2019, 8:06 PM · Plans, Phacility
epriestley added a revision to T13242: 2019 Week 5 Bonus Content: Unknown Object (Differential Revision).
Jan 28 2019, 8:06 PM · Plans
epriestley added a comment to T13242: 2019 Week 5 Bonus Content.

PHI979 is a specific issue where we could use better instance controls (around worker count) in the short term. An alternative approach is to hard-coded if ($instance === 'X') { ... } for now. admin could also probably use more workers, I think we get saturated by backups occasionally.

Jan 28 2019, 7:46 PM · Plans
epriestley added a revision to T13242: 2019 Week 5 Bonus Content: Unknown Object (Differential Revision).
Jan 28 2019, 7:31 PM · Plans
epriestley added a revision to T4369: Phabricator HTTP repository hosting has fairly severe scalability limits: Restricted Differential Revision.
Jan 28 2019, 7:09 PM · Diffusion (v3), Restricted Project
epriestley added a revision to T13242: 2019 Week 5 Bonus Content: Restricted Differential Revision.
Jan 28 2019, 7:09 PM · Plans
epriestley added a revision to T12297: Make Conduit API calls on `admin.phacility.com` reasonably easy to profile: Restricted Differential Revision.
Jan 28 2019, 7:09 PM · XHProf, Ops, Phacility
epriestley added a comment to T12297: Make Conduit API calls on `admin.phacility.com` reasonably easy to profile.

Yeah, I think the issue is:

Jan 28 2019, 6:41 PM · XHProf, Ops, Phacility
epriestley updated the task description for T13242: 2019 Week 5 Bonus Content.
Jan 28 2019, 6:30 PM · Plans
epriestley added a revision to T13242: 2019 Week 5 Bonus Content: Unknown Object (Differential Revision).
Jan 28 2019, 6:29 PM · Plans
epriestley added a revision to T13242: 2019 Week 5 Bonus Content: Unknown Object (Differential Revision).
Jan 28 2019, 6:06 PM · Plans
epriestley closed T13227: Figure out if Google Auth needs to be updated before Google+ shutdown on March 7, 2019 as Resolved by committing Restricted Diffusion Commit.
Jan 28 2019, 5:40 PM · Auth
epriestley added a revision to T13242: 2019 Week 5 Bonus Content: Restricted Differential Revision.
Jan 28 2019, 5:28 PM · Plans
epriestley added a comment to T12297: Make Conduit API calls on `admin.phacility.com` reasonably easy to profile.

The "keep the profiler on across form submissions" code isn't working on secure.phabricator.com, even though it works locally and __profile__=page appears on the "Request" tab.

Jan 28 2019, 4:58 PM · XHProf, Ops, Phacility
epriestley added a revision to T13242: 2019 Week 5 Bonus Content: Unknown Object (Differential Revision).
Jan 28 2019, 3:52 PM · Plans

Jan 27 2019

epriestley added a revision to T13242: 2019 Week 5 Bonus Content: Unknown Object (Differential Revision).
Jan 27 2019, 6:31 PM · Plans
epriestley added a revision to T13242: 2019 Week 5 Bonus Content: Unknown Object (Differential Revision).
Jan 27 2019, 4:59 PM · Plans

Jan 26 2019

epriestley added a comment to T13156: Plans: Improve Phacility UI for managing instance managers and cards.
  • PHI979 describes a case where we'd like to automatically increase the worker task limit based on instance size. admin should also get an increased limit.
  • Now that MFA can be disabled completely, I want to disable MFA on admin (pretty much everyone adding it is adding it by accident) by deprecating the provider.
  • T13178 is adjacent to a lot of this.
Jan 26 2019, 3:06 PM · Plans, Phacility
epriestley closed T13222: 2018 Week 48-51 Bonus Content as Resolved.

Continued in T13242.

Jan 26 2019, 3:00 PM · Plans
epriestley triaged T13242: 2019 Week 5 Bonus Content as Normal priority.
Jan 26 2019, 3:00 PM · Plans

Jan 25 2019

epriestley added a revision to T13231: Duo MFA Support: Unknown Object (Differential Revision).
Jan 25 2019, 9:22 PM · Auth
epriestley added a revision to T7667: Provide `auth lock` and `auth unlock` to restrict authentication provider management to the CLI: Unknown Object (Differential Revision).
Jan 25 2019, 6:45 PM · Auth, Security
epriestley added a revision to T13222: 2018 Week 48-51 Bonus Content: Unknown Object (Differential Revision).
Jan 25 2019, 6:45 PM · Plans
epriestley added a comment to T7667: Provide `auth lock` and `auth unlock` to restrict authentication provider management to the CLI.

After T13222, this is more relevant:

Jan 25 2019, 6:31 PM · Auth, Security
epriestley added a comment to T13138: Improve consistency of MFA requirements to invite/approve users.

From T13222, MFA on related flows should generally be updated.

Jan 25 2019, 6:28 PM · People, Security
epriestley added a comment to T13134: Give Conduit API tokens PHIDs, transactions, and other modern capabilities.

From T13222, this should also revisit MFA around Conduit tokens. In particular:

Jan 25 2019, 6:26 PM · Conduit
epriestley added a revision to T13222: 2018 Week 48-51 Bonus Content: Unknown Object (Differential Revision).
Jan 25 2019, 6:24 PM · Plans
epriestley added a revision to T13222: 2018 Week 48-51 Bonus Content: Unknown Object (Differential Revision).
Jan 25 2019, 6:19 PM · Plans
epriestley added a revision to T13222: 2018 Week 48-51 Bonus Content: Unknown Object (Differential Revision).
Jan 25 2019, 6:07 PM · Plans
epriestley added a revision to T13222: 2018 Week 48-51 Bonus Content: Unknown Object (Differential Revision).
Jan 25 2019, 5:52 PM · Plans
epriestley added a revision to T13222: 2018 Week 48-51 Bonus Content: Unknown Object (Differential Revision).
Jan 25 2019, 5:32 PM · Plans
epriestley updated the task description for T13241: Guide: SMS is Insecure.
Jan 25 2019, 3:57 PM · Security, Guides
epriestley updated the task description for T13241: Guide: SMS is Insecure.
Jan 25 2019, 3:39 PM · Security, Guides
epriestley created U44 SMS is Insecure.
Jan 25 2019, 3:39 PM
epriestley triaged T13241: Guide: SMS is Insecure as Wishlist priority.
Jan 25 2019, 3:38 PM · Security, Guides
epriestley added a revision to T13222: 2018 Week 48-51 Bonus Content: Unknown Object (Differential Revision).
Jan 25 2019, 3:12 PM · Plans
epriestley added a revision to T13222: 2018 Week 48-51 Bonus Content: Unknown Object (Differential Revision).
Jan 25 2019, 2:00 PM · Plans
epriestley added a comment to T13222: 2018 Week 48-51 Bonus Content.

"Kick everyone down to partial session" docs / tooling.

Jan 25 2019, 1:57 PM · Plans
epriestley added a comment to T13237: Stripping MFA configs with `bin/auth strip` should disable, not delete, the factor configurations.

For now, I'm continuing to allow users to "Remove" the last active factor on their account when security.require-multi-factor-auth is enabled. This will kick them to the MFA enrollment screen.

Jan 25 2019, 1:39 PM · Auth
epriestley added a revision to T13227: Figure out if Google Auth needs to be updated before Google+ shutdown on March 7, 2019: Restricted Differential Revision.
Jan 25 2019, 1:48 AM · Auth