Page MenuHomePhabricator
Feed All Stories

Apr 7 2017

arielyang created T12517: Some strings are not outputted by pht() method.
Apr 7 2017, 4:28 AM · Contributor Onboarding, Localization, Bug Report
arielyang added a comment to Q607: How to set help link of self-host Phabricator to local url. (Answer 539).

That's OK.

Apr 7 2017, 4:22 AM
arielyang created T12516: 4 typos in code.
Apr 7 2017, 4:21 AM · Contributor Onboarding, Bug Report
chad added Q607: How to set help link of self-host Phabricator to local url. (Answer 539).
Apr 7 2017, 4:15 AM
arielyang updated Q607: How to set help link of self-host Phabricator to local url. from to How to set help link of self-host Phabricator to local url..
Apr 7 2017, 3:08 AM · Diviner

Apr 6 2017

epriestley closed T12470: Store integrity hashes for file blobs in Files as Resolved.

See T12515 for followup guidance.

Apr 6 2017, 11:25 PM · Files
epriestley created T12515: Upgrading: File Integrity Hashing and SHA1.
Apr 6 2017, 11:24 PM · Files, Guides, Security
epriestley added a revision to T4245: Make repository URIs slightly prettier: Unknown Object (Differential Revision).
Apr 6 2017, 10:53 PM · Prioritized, Diffusion (v3), KDE, Restricted Project
epriestley added a revision to T12217: Reduce the hardware cost of Phacility free instances: Unknown Object (Differential Revision).
Apr 6 2017, 10:40 PM · Ops, Phacility
epriestley added a revision to T12298: Allow daemon pools to autoscale down to 0 processes: Unknown Object (Differential Revision).
Apr 6 2017, 10:26 PM · Daemons, Ops, Phacility
epriestley merged task T12514: `refs/keep-around/` commits should be ignored into T11314: Observe Mode + GitHub has weird mail interactions due to pull request refs.
Apr 6 2017, 10:23 PM · Diffusion, Bug Report
epriestley merged T12514: `refs/keep-around/` commits should be ignored into T11314: Observe Mode + GitHub has weird mail interactions due to pull request refs.
Apr 6 2017, 10:23 PM · Diffusion (v3), Restricted Project, Mail, Haskell.org, Bug Report
epriestley added a comment to T12514: `refs/keep-around/` commits should be ignored.

I'm going to merge this into T11314, but I don't want to hard-code rules around specific names used by GitHub/GitLab. We can provide more tools to manage these refs, but I don't think the right tool is a bunch of hard-coded rules. These refs are real refs which exist in the repository, and Diffusion should be able to represent them. A driving use case for the original change was a desire to expose Gerrit refs.

Apr 6 2017, 10:23 PM · Diffusion, Bug Report
joshuaspence created T12514: `refs/keep-around/` commits should be ignored.
Apr 6 2017, 10:17 PM · Diffusion, Bug Report
lvital added a comment to T12272: Allow users to accept on behalf of packages they control a containing package for.

Just tested, works great. Thanks for the quick fix!

Apr 6 2017, 10:13 PM · Owners, Audit, Restricted Project, Differential
epriestley added a comment to T12272: Allow users to accept on behalf of packages they control a containing package for.

I think things should work now, let me know if you're still seeing issues. Thanks for the report!

Apr 6 2017, 10:03 PM · Owners, Audit, Restricted Project, Differential
epriestley added a revision to T12272: Allow users to accept on behalf of packages they control a containing package for: Unknown Object (Differential Revision).
Apr 6 2017, 9:59 PM · Owners, Audit, Restricted Project, Differential
epriestley added a comment to T12272: Allow users to accept on behalf of packages they control a containing package for.

Argh! I broke it while doing last-minute fixes to improve rendering, fix in a moment.

Apr 6 2017, 9:56 PM · Owners, Audit, Restricted Project, Differential
arpagon awarded T10339: Figure out the UI for Maniphest task search by points a Like token.
Apr 6 2017, 9:32 PM · Maniphest
lvital added a comment to T12272: Allow users to accept on behalf of packages they control a containing package for.

No prob! If it helps:

Apr 6 2017, 8:50 PM · Owners, Audit, Restricted Project, Differential
epriestley added a comment to T12272: Allow users to accept on behalf of packages they control a containing package for.

Thanks, that definitely looks like a bug.

Apr 6 2017, 8:48 PM · Owners, Audit, Restricted Project, Differential
lvital added a comment to T12272: Allow users to accept on behalf of packages they control a containing package for.

Not sure if this is a known bug / missing feature, but FYI just in case...

Apr 6 2017, 8:46 PM · Owners, Audit, Restricted Project, Differential
nickz created P2035 Error when upgrading Phabricator to the latest stable version.
Apr 6 2017, 7:48 PM
epriestley added a revision to T12509: Plan the path forward from HMAC-SHA1: Unknown Object (Differential Revision).
Apr 6 2017, 5:10 PM · Infrastructure, Security
kfsone added a comment to T12458: LDAP sign-in with "Trust Email Address" doesn't beat implicit "auth.require-email-verification".

@avivey I was hanging my head in RTFM shame, TBH.

Apr 6 2017, 5:08 PM · Bug Report
epriestley added a revision to T12509: Plan the path forward from HMAC-SHA1: Unknown Object (Differential Revision).
Apr 6 2017, 5:03 PM · Infrastructure, Security
epriestley added a revision to T12509: Plan the path forward from HMAC-SHA1: Unknown Object (Differential Revision).
Apr 6 2017, 4:38 PM · Infrastructure, Security
TafThorne added a comment to T12511: Add Support For User Definable Macro Or Template Markup to Phriction Similar To WikiMedia Templates.

No. You should list the (actual, real, day-to-day, not theoretical) things you want to be able to do that you currently can not, or which you do at significant time or complexity cost through cumbersome workarounds. Optionally, you might also say "in other cases, we currently use Wikimedia templates to solve these". Suggesting a solution is the least important part of a feature request.

Apr 6 2017, 3:16 PM · Phriction, Feature Request
epriestley added a comment to T12511: Add Support For User Definable Macro Or Template Markup to Phriction Similar To WikiMedia Templates.

I gave a trivial example as that is better than a complicated one.

Apr 6 2017, 2:51 PM · Phriction, Feature Request
TafThorne added a comment to T12511: Add Support For User Definable Macro Or Template Markup to Phriction Similar To WikiMedia Templates.

I gave a trivial example as that is better than a complicated one. In a huge wiki, like Wikipedia templates are used for all kinds of things besides making links simpler.

Apr 6 2017, 2:12 PM · Phriction, Feature Request
epriestley added a revision to T12470: Store integrity hashes for file blobs in Files: Unknown Object (Differential Revision).
Apr 6 2017, 2:01 PM · Files
epriestley added a comment to T12509: Plan the path forward from HMAC-SHA1.

(That said, the hash cost may become very relevant when we eventually ship "Phabricator Valuable Golden Coin Money", our blockchain-based virtual currency.)

Apr 6 2017, 2:01 PM · Infrastructure, Security
epriestley merged T12513: Failed to index task with too long title into T5679: Search daemon fails when document title >255 chars.
Apr 6 2017, 1:48 PM · Maniphest
epriestley merged task T12513: Failed to index task with too long title into T5679: Search daemon fails when document title >255 chars.
Apr 6 2017, 1:48 PM · Bug Report
eFrane created T12513: Failed to index task with too long title.
Apr 6 2017, 1:48 PM · Bug Report
epriestley added a comment to T3963: Templates in Phriction.

See T12511#218328.

Apr 6 2017, 1:47 PM · Phriction
epriestley added a comment to T12511: Add Support For User Definable Macro Or Template Markup to Phriction Similar To WikiMedia Templates.

There is basically zero chance we're going to implement parameterized templates which can call one another (easily dozens of hours of work) if the only real use case is making RFC5555 autolink (works today with InlineRule, takes a few minutes to make work).

Apr 6 2017, 1:42 PM · Phriction, Feature Request
TafThorne added a comment to T3963: Templates in Phriction.

T12511 is about a very different problem and use case. This request is about adding a feature that "will allow users to quickly create a wiki page with some specific base content.". T12511 is about adding a feature that lets users create a bit or markup that acts as a macro/function/template that generates a few lines of markup within a page.

Apr 6 2017, 1:34 PM · Phriction
TafThorne updated subscribers of T12511: Add Support For User Definable Macro Or Template Markup to Phriction Similar To WikiMedia Templates.

I do not think this is a duplicate. T3963 is about creating template Wiki pages that start off ready populated with some basic data. This request is about creating a Macro like tag (which is part of the Templates feature in WikiMedia markup) not about creating pre-canned pages.

Apr 6 2017, 1:28 PM · Phriction, Feature Request
epriestley added a comment to T12509: Plan the path forward from HMAC-SHA1.

I get these rough per-hash costs locally (Macbook Pro), with a 64-byte key:

Apr 6 2017, 1:05 PM · Infrastructure, Security
siepkes added a comment to T12509: Plan the path forward from HMAC-SHA1.

Piece of info (you guys might already be aware of it) which might be of interest when implementing this; SHA512 is often faster then SHA256 on x64. See for example: https://crypto.stackexchange.com/questions/26336/sha512-faster-than-sha256

Apr 6 2017, 12:54 PM · Infrastructure, Security
epriestley edited projects for T12510: Requesting changes as Reviewer and member of blocking Group Reviewer, makes Group Reviewer no longer blocking, added: Bug Report (Needs Information); removed Bug Report.
Apr 6 2017, 12:15 PM · Bug Report (Needs Information)
epriestley added a comment to T12510: Requesting changes as Reviewer and member of blocking Group Reviewer, makes Group Reviewer no longer blocking.

I'm having trouble reproducing this, but maybe I'm not fully understanding what you're running into?

Apr 6 2017, 12:14 PM · Bug Report (Needs Information)
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, but maybe we should just raise it as a setup issue if you have password auth enabled, and let users ignore it if they're VPN'd.

Apr 6 2017, 11:49 AM · Security
epriestley added a comment to T3963: Templates in Phriction.

T12511 discusses some possible use cases, but they are all in the form of {{name|123}} expanding to [[ something.com/123 | 123 ]] (external bug trackers, RFCs, etc).

Apr 6 2017, 11:43 AM · Phriction
epriestley merged T12511: Add Support For User Definable Macro Or Template Markup to Phriction Similar To WikiMedia Templates into T3963: Templates in Phriction.
Apr 6 2017, 11:41 AM · Phriction
epriestley merged task T12511: Add Support For User Definable Macro Or Template Markup to Phriction Similar To WikiMedia Templates into T3963: Templates in Phriction.
Apr 6 2017, 11:41 AM · Phriction, Feature Request
epriestley closed T12512: Bug in Upload URL's (Require user authentication over URL's ) as Resolved.

In the future, please report security issues via HackerOne: https://hackerone.com/phabricator -- notably, this allows us to award you a security bounty if you discover an issue.

Apr 6 2017, 11:39 AM · Security, Bug Report
wjhussain added a project to T12512: Bug in Upload URL's (Require user authentication over URL's ): Security.
Apr 6 2017, 11:10 AM · Security, Bug Report
wjhussain created T12512: Bug in Upload URL's (Require user authentication over URL's ).
Apr 6 2017, 10:12 AM · Security, Bug Report
TafThorne created T12511: Add Support For User Definable Macro Or Template Markup to Phriction Similar To WikiMedia Templates.
Apr 6 2017, 9:35 AM · Phriction, Feature Request
claassen created T12510: Requesting changes as Reviewer and member of blocking Group Reviewer, makes Group Reviewer no longer blocking.
Apr 6 2017, 7:50 AM · Bug Report (Needs Information)

Apr 5 2017

epriestley added a comment to P2034 aa.

yeah, sorry, in retrospect this is a real underwhelming test

Apr 5 2017, 8:42 PM
chad added a comment to P2034 aa.

am disappoint

Apr 5 2017, 8:32 PM
epriestley moved T8788: Allow querying for files by name from v3 to vNext on the Files board.
Apr 5 2017, 8:28 PM · Contributor Onboarding, Files
epriestley moved T10828: File deletion should be queued up and run by daemons from v3 to vNext on the Files board.
Apr 5 2017, 8:28 PM · Contributor Onboarding, Daemons, Files, Bug Report
epriestley added a comment to T12509: Plan the path forward from HMAC-SHA1.

I'm maybe going to try to do this in the short term:

Apr 5 2017, 8:26 PM · Infrastructure, Security
epriestley added a comment to T12508: Can't create pastes in this install.

I deployed that and created this masterpiece:

Apr 5 2017, 8:22 PM · Bug Report
epriestley created P2034 aa.
Apr 5 2017, 8:21 PM
epriestley created T12509: Plan the path forward from HMAC-SHA1.
Apr 5 2017, 8:21 PM · Infrastructure, Security
epriestley closed T12508: Can't create pastes in this install as Resolved by committing Restricted Diffusion Commit.
Apr 5 2017, 8:09 PM · Bug Report
epriestley added a revision to T12508: Can't create pastes in this install: Unknown Object (Differential Revision).
Apr 5 2017, 7:57 PM · Bug Report
epriestley claimed T12508: Can't create pastes in this install.

Almost certainly fallout from the increased strictness in D17616, thanks.

Apr 5 2017, 7:55 PM · Bug Report
epriestley closed T9979: Build support for ngram indexes for substring searches (e.g., file, paste, package, task titles) as Resolved.

I'm going to call this resolved; T9979 is the survivor for Files and the Maniphest case is probably more involved (T6721).

Apr 5 2017, 7:54 PM · Owners, Files, ApplicationSearch
epriestley closed T9979: Build support for ngram indexes for substring searches (e.g., file, paste, package, task titles), a subtask of T6721: Would like ability to search tasks by title, as Resolved.
Apr 5 2017, 7:54 PM · Search, Maniphest
avivey created T12508: Can't create pastes in this install.
Apr 5 2017, 7:52 PM · Bug Report
epriestley added a project to T8788: Allow querying for files by name: Contributor Onboarding.
Apr 5 2017, 7:52 PM · Contributor Onboarding, Files
epriestley added a project to T10828: File deletion should be queued up and run by daemons: Contributor Onboarding.
Apr 5 2017, 7:52 PM · Contributor Onboarding, Daemons, Files, Bug Report
epriestley added a comment to T8787: Add support for U2F MFA once browser implementations improve and compatible hardware is more widely available.

Just dropping this here since it's the first hit for "SAML" and makes it easier for me to contextualize things when I quote $50K for it, which I'm starting to think is too low:

Apr 5 2017, 7:44 PM · Haskell.org, Auth
cspeckmim updated the task description for T12507: User does not know why they received email based on Herald rule.
Apr 5 2017, 7:30 PM · Mail, Herald, Feature Request
epriestley added a comment to T12470: Store integrity hashes for file blobs in Files.

The HackerOne issue for this is here:

Apr 5 2017, 7:29 PM · Files
epriestley added a comment to T12470: Store integrity hashes for file blobs in Files.

The security part of this is fixed by D17625, but I'm planning to write a support script like bin/files integrity or similar to do things like "compute hashes for existing files" and "check that a file or set of files match their integrity hashes".

Apr 5 2017, 7:28 PM · Files
cspeckmim created T12507: User does not know why they received email based on Herald rule.
Apr 5 2017, 7:22 PM · Mail, Herald, Feature Request
epriestley closed T12219: Some MP4 videos do not play in Chrome as Resolved.

This appears to be resolved for me, now. Handling Range: bytes=123- and Range: bytes=0- (plus the fixes for the broader issues in T8266 and T12079) seemed to be the key issues.

Apr 5 2017, 6:59 PM · Files
epriestley added a revision to T12219: Some MP4 videos do not play in Chrome: Unknown Object (Differential Revision).
Apr 5 2017, 6:34 PM · Files
epriestley added a revision to T12219: Some MP4 videos do not play in Chrome: Unknown Object (Differential Revision).
Apr 5 2017, 6:05 PM · Files
epriestley closed T8266: Some audio isn't streaming correctly in Safari as Resolved.

F60036 now plays correctly for me in Safari.

Apr 5 2017, 5:13 PM · Safari, Files
epriestley added a comment to T12079: Files does not serve HTTP "Content-Range" requests correctly if the underlying blob data is encrypted.

The video in T12078 now plays correctly for me so I think this is, in fact, resolved.

Apr 5 2017, 5:12 PM · Files, Public & Media Relations, Bug Report
epriestley closed T12079: Files does not serve HTTP "Content-Range" requests correctly if the underlying blob data is encrypted as Resolved by committing Restricted Diffusion Commit.
Apr 5 2017, 4:56 PM · Files, Public & Media Relations, Bug Report
epriestley added a revision to T12470: Store integrity hashes for file blobs in Files: Unknown Object (Differential Revision).
Apr 5 2017, 4:56 PM · Files
epriestley added a revision to T8266: Some audio isn't streaming correctly in Safari: Unknown Object (Differential Revision).
Apr 5 2017, 1:53 PM · Safari, Files
cspeckmim added a comment to T12506: hackerone: possible issue with secrets sharing.

Thank you for explaining - I didn't mean to imply I thought it was a security concern, as it is very clear what giving someone Edit access means.

Apr 5 2017, 1:53 PM · Security
epriestley added a revision to T12079: Files does not serve HTTP "Content-Range" requests correctly if the underlying blob data is encrypted: Unknown Object (Differential Revision).
Apr 5 2017, 1:35 PM · Files, Public & Media Relations, Bug Report
epriestley added a comment to T12506: hackerone: possible issue with secrets sharing.

T4721 (which I also linked on the HackerOne report) discusses usability improvements to Passphrase, including better documentation and hinting about this use case and possibly the separation of the "Can Use Credential" and "Can View Secret" policies. These are reasonable usability concerns.

Apr 5 2017, 1:16 PM · Security
cspeckmim added a comment to T12506: hackerone: possible issue with secrets sharing.

It sounds like the user is confused that having View access doesn't let them have access to decrypt/see the secret - only that they can use the secret within Phabricator. So in order to allow others to decrypt/see the secret he gives them Edit access which does give them the ability to change the secret.

Apr 5 2017, 1:09 PM · Security
cxzzero added a comment to T12506: hackerone: possible issue with secrets sharing.

Thanks for the response Evan.

Apr 5 2017, 1:04 PM · Security
epriestley closed T12506: hackerone: possible issue with secrets sharing as Invalid.

You can review the HackerOne report, including my response, here:

Apr 5 2017, 12:51 PM · Security
cxzzero created T12506: hackerone: possible issue with secrets sharing.
Apr 5 2017, 12:44 PM · Security
remusvrm updated Q606: Is it possible to limit the user to use a specific project as tag? from to Is it possible to limit the user to use a specific project as tag?.
Apr 5 2017, 12:11 PM
saannjaay added a watcher for Bug Report: saannjaay.
Apr 5 2017, 11:41 AM
avivey renamed T12458: LDAP sign-in with "Trust Email Address" doesn't beat implicit "auth.require-email-verification" from Email verification still required for ldap user to LDAP sign-in with "Trust Email Address" doesn't beat implicit "auth.require-email-verification".
Apr 5 2017, 9:04 AM · Bug Report
avivey reopened T12458: LDAP sign-in with "Trust Email Address" doesn't beat implicit "auth.require-email-verification" as "Open".

mm, yeah, I didn't actually read it 😊 @kfsone said in the chat that it can be resolved, but this does actually sound like trust email address should win over require-email-verification.

Apr 5 2017, 9:02 AM · Bug Report
asherkin added a comment to T12458: LDAP sign-in with "Trust Email Address" doesn't beat implicit "auth.require-email-verification".

@avivey Trust Email Address is meant to handle that though, is it not? The reported "bug" here is that the email addresses are not being verified when coming from LDAP, not that email verification is required.

Apr 5 2017, 8:15 AM · Bug Report
chad awarded T12458: LDAP sign-in with "Trust Email Address" doesn't beat implicit "auth.require-email-verification" a Hungry Hippo token.
Apr 5 2017, 4:46 AM · Bug Report
avivey closed T12458: LDAP sign-in with "Trust Email Address" doesn't beat implicit "auth.require-email-verification" as Resolved.

auth.email-domains implies auth.require-email-verification.

Apr 5 2017, 3:34 AM · Bug Report
epriestley edited the content of 2017 Week 13 (Very Early April).
Apr 5 2017, 1:38 AM

Apr 4 2017

epriestley closed T12464: Move away from SHA1 in Files as Resolved by committing Restricted Diffusion Commit.
Apr 4 2017, 11:56 PM · Files
epriestley closed T8348: `arc download` doesn't support chunked downloads as Resolved by committing Restricted Diffusion Commit.
Apr 4 2017, 11:16 PM · Restricted Project, Files, Arcanist
epriestley closed T8348: `arc download` doesn't support chunked downloads, a subtask of T11357: Move Files to EditEngine and modern APIs, as Resolved.
Apr 4 2017, 11:16 PM · Restricted Project, Files
epriestley added a comment to T8348: `arc download` doesn't support chunked downloads.

After D17614, arc download uses file.search to retrieve a URI it can GET, then does a normal HTTP GET to that URI, retrieving the file content in the HTTP response body.

Apr 4 2017, 11:14 PM · Restricted Project, Files, Arcanist