Page MenuHomePhabricator
Feed All Stories

Mar 10 2017

lvital added a comment to T12372: Long list of reviewers breaks to: field when metamta.one-mail-per-recipient set to false.

Any suggestions to test this on staging without mass emailing everyone / patching prod? I am trying to test this on staging with bin/mail show-outbound --id xxxx (staging has mail disabled), but the recipients / To: list look fine with or without the patch.

Mar 10 2017, 10:23 PM · Mail, Bug Report
cspeckmim updated the task description for T12383: Accumulate Phabricator changelogs applicable from a current install.
Mar 10 2017, 9:55 PM · Feature Request, Installing & Upgrading
cspeckmim added a comment to T12383: Accumulate Phabricator changelogs applicable from a current install.

That's a good point about also accumulating any security notes

Mar 10 2017, 9:51 PM · Feature Request, Installing & Upgrading
avivey added a comment to T12383: Accumulate Phabricator changelogs applicable from a current install.

(There's no need for a token to get the changlog via conduit)

Mar 10 2017, 9:37 PM · Feature Request, Installing & Upgrading
chad added a revision to T12381: Profile link click area overflow?: Unknown Object (Differential Revision).
Mar 10 2017, 7:18 PM · Bug Report
cspeckmim updated subscribers of T12383: Accumulate Phabricator changelogs applicable from a current install.

Asking someone who is say, 16 months behind to read 80 changelogs seems detrimental to having people keep up to date.

This is partly why I would like this -- the Upgrade / Compatibility section of changelogs is probably the only near-mandatory sections to review before upgrading (things might behave differently until you do databasey things, the way a configuration behaves has changed, etc.). My process for upgrading right now begins with reviewing the changelogs to make sure I don't miss any pre/post-intall steps, and I'd like to automate part of that if possible. For a site that's upgrading weekly this isn't a challenge, but sites that are unable to upgrade as frequently would (should?) be sifting through this stuff anyways. I think this would be part of community resources and not officially part of Phabricator.

Mar 10 2017, 5:35 PM · Feature Request, Installing & Upgrading
chad added a comment to T12383: Accumulate Phabricator changelogs applicable from a current install.

I am however, all for making Phriction better and supportive at the CMS level for navigating / categorizing content.

Mar 10 2017, 5:29 PM · Feature Request, Installing & Upgrading
chad added a comment to T12383: Accumulate Phabricator changelogs applicable from a current install.

I don't think there is a lot of benefit here, and worry it might actually detract someone from upgrading. We already post a changelog here, via email, and on social media. Asking someone who is say, 16 months behind to read 80 changelogs seems detrimental to having people keep up to date.

Mar 10 2017, 5:27 PM · Feature Request, Installing & Upgrading
cspeckmim added a comment to T12383: Accumulate Phabricator changelogs applicable from a current install.

Did I really have an extra space in the title

Mar 10 2017, 5:21 PM · Feature Request, Installing & Upgrading
chad renamed T12383: Accumulate Phabricator changelogs applicable from a current install from Accumulate Phabricator changelogs applicable from a current install to Accumulate Phabricator changelogs applicable from a current install.
Mar 10 2017, 5:18 PM · Feature Request, Installing & Upgrading
chad added a comment to T12383: Accumulate Phabricator changelogs applicable from a current install.

Good job triggering the spam filter.

Mar 10 2017, 5:11 PM · Feature Request, Installing & Upgrading
cspeckmim created T12383: Accumulate Phabricator changelogs applicable from a current install.
Mar 10 2017, 5:09 PM · Feature Request, Installing & Upgrading
cinek updated the task description for T12382: ArcanistJSONLintRenderer fails to produce JSON output with non-UTF-8 encodings.
Mar 10 2017, 2:39 PM · Lint, Arcanist, Bug Report
mrybczyn added a comment to T12376: Export workboard columns for reporting.

The workaround works, thanks!

Mar 10 2017, 2:36 PM · Feature Request
cinek created T12382: ArcanistJSONLintRenderer fails to produce JSON output with non-UTF-8 encodings.
Mar 10 2017, 2:24 PM · Lint, Arcanist, Bug Report
stettberger updated the task description for T12375: Integrate Conduit API for Policy Object Creation.
Mar 10 2017, 2:14 PM · Policy, Feature Request
stettberger added a comment to T12375: Integrate Conduit API for Policy Object Creation.

The different edit/view policy is not the actual problem here. The problem with using spaces to solve the described scenario in an automatic fashion is that a) there is no spaces API b) the view policy for the space would also be a complex policy b.1) There is no API to create complex policies.

Mar 10 2017, 2:10 PM · Policy, Feature Request
chad added a comment to T12381: Profile link click area overflow?.

I going guess the here is that a small margin around the area of the hovercard is clickable? Or maybe your using a browser that displays a more nefarious issue? In Chrome/Mac I can't reproduce anything unexpected or can't find anything I'm prevented in doing, like copying text from the hovercard. Hovercards are meant to be small bits of information, and then click through to the profile. I'm not sure what "bug" there is in having a larger click area.

Mar 10 2017, 2:04 PM · Bug Report
chad added a comment to T12375: Integrate Conduit API for Policy Object Creation.

Why do you need separated edit/push policies if it's locked to a space with just the specific tutors and students? Is there an actual abuse case, or is this just anticipatory / hypothetical?

Mar 10 2017, 2:02 PM · Policy, Feature Request
chad added a comment to T12375: Integrate Conduit API for Policy Object Creation.
Mar 10 2017, 1:59 PM · Policy, Feature Request
stettberger added a comment to T12375: Integrate Conduit API for Policy Object Creation.

Ok, so root problem is more of a user story. Sorry to not getting this right. But this was my first feature request on Phabricator.

Mar 10 2017, 1:58 PM · Policy, Feature Request
chad added a comment to T12381: Profile link click area overflow?.

What is the bug that you're reporting here?

Mar 10 2017, 1:49 PM · Bug Report
chad added a comment to T12375: Integrate Conduit API for Policy Object Creation.

A root problem to us describes the issue you're trying to solve with the request, in full detail. In general we can't accept "feature doesn't exist" as a root problem. Here's an example of a root problem:

Mar 10 2017, 1:42 PM · Policy, Feature Request
stettberger added a comment to T12375: Integrate Conduit API for Policy Object Creation.

From an usability side, it would also be a possibility to create policy objects just in time, when the API user submits a transaction touching a policy:

Mar 10 2017, 11:30 AM · Policy, Feature Request
20after4 added a comment to T12375: Integrate Conduit API for Policy Object Creation.

What I would find especially useful would be a way to create named policies and then reference them when creating objects. That might warrant a separate feature-request task though.

Mar 10 2017, 10:40 AM · Policy, Feature Request
20after4 added a comment to T12375: Integrate Conduit API for Policy Object Creation.

I don't think this is related to wikimedia since we already have the functionality live in the WMF production install.

Mar 10 2017, 10:37 AM · Policy, Feature Request
stettberger added a comment to T12375: Integrate Conduit API for Policy Object Creation.

Root of the Problem: The conduit API does not provide the possibility to create custom policies. Therefore, the fine-grained policy model of phabricator cannot be used in automated workflows.

Mar 10 2017, 10:36 AM · Policy, Feature Request
fosskevin created T12381: Profile link click area overflow?.
Mar 10 2017, 9:51 AM · Bug Report
OCram closed Q567: JSON Example for Conduit method maniphest.edit? as resolved.
Mar 10 2017, 8:39 AM · Conduit
OCram added a comment to Q567: JSON Example for Conduit method maniphest.edit? (Answer 510).

Finally I found a workaround! Conduit requests from foreign clients have to use the cURL form!
For example if I use the maniphest.edit for adding comments via powershell:

Mar 10 2017, 8:37 AM
cspeckmim added a comment to T12380: Errors with Mercurial trust while browsing rHGTEST on this install.

Different parts of the repository page may or may not load indicating error. I refreshed on the page with the commit a few times and sometimes it says the commit is missing and other times it can't load the content, and other times it can't load the paths, but sometimes it can load them.

Mar 10 2017, 5:32 AM · Mercurial, Bug Report
cspeckmim created T12380: Errors with Mercurial trust while browsing rHGTEST on this install.
Mar 10 2017, 5:28 AM · Mercurial, Bug Report
epriestley created T12379: Instance reporting difficulty pushing the Linux kernel to a Phacility instance.
Mar 10 2017, 1:06 AM · Customer Impact, Diffusion, Ops
chad added a comment to T12374: Saved Workboard Queries.

I don't mean the project menu, I mean replacing the query menu on the work board with a confuigurable one like the favorites menu. I'd restrict it to task queires

Mar 10 2017, 12:59 AM · Workboards, Feature Request
jkrets added a comment to T12374: Saved Workboard Queries.

I am currently using bookmarks to workaround the issue. I never considered using the project menu for the workboard views. This would certainly suffice, but it doesn't feel like the right place for it.

Mar 10 2017, 12:46 AM · Workboards, Feature Request

Mar 9 2017

avivey closed Q587: Is it possible to create a static URL to a file download? as resolved.
Mar 9 2017, 11:57 PM
ftdysa added a comment to Q587: Is it possible to create a static URL to a file download? (Answer 528).

Ehh, that seems messy/a lot of maintenance for a single file. It'd be perfect if you could tie a paste to a file in diffusion and handle access control through the paste since that already supports policies, but I doubt upstream has any interest in that.

Mar 9 2017, 11:56 PM
chad added a comment to T12374: Saved Workboard Queries.

I think we have a basic query menu we could switch over to MenuItem, let users customize them per workboard like they can per project.

Mar 9 2017, 11:52 PM · Workboards, Feature Request
Mnkras added a comment to Q587: Is it possible to create a static URL to a file download? (Answer 528).

Use sub-modules and make the sub-module public? (or the parent public and the sub-module private)

Mar 9 2017, 11:27 PM
ftdysa added a comment to Q587: Is it possible to create a static URL to a file download? (Answer 528).

Yeah, good point. I guess what I really want is "the ability to generate a static download URL with custom policy controls to a file in a diffusion". Looks like I should host this file somewhere else because I don't want the entire repo to be public.

Mar 9 2017, 9:28 PM
avivey added a comment to Q587: Is it possible to create a static URL to a file download? (Answer 528).

The auth is trying to protect your code from outsiders. You can make the whole repo "public", and then there will be no auth requirement.

Mar 9 2017, 9:06 PM
ftdysa added a comment to Q587: Is it possible to create a static URL to a file download? (Answer 528).

This doesn't quite work because the redirect to the raw version requires auth while navigating directly to the temp file appears to be publicly accessible.

Mar 9 2017, 9:02 PM
avivey added Q587: Is it possible to create a static URL to a file download? (Answer 528).
Mar 9 2017, 8:54 PM
terrluhr added a watcher for Nuance: terrluhr.
Mar 9 2017, 8:49 PM
Sohojoe awarded T5474: Support workboard column triggers which activate when a task is dropped into a column a Love token.
Mar 9 2017, 8:28 PM · Workboards (v3), Restricted Project
ftdysa updated Q587: Is it possible to create a static URL to a file download? from to Is it possible to create a static URL to a file download?.
Mar 9 2017, 8:26 PM
epriestley added a revision to T12270: Unbeta Badges: Unknown Object (Differential Revision).
Mar 9 2017, 7:43 PM · Badges (v0)
epriestley merged T12376: Export workboard columns for reporting into T5024: Provide a dedicated page for viewing a single workboard column.
Mar 9 2017, 7:39 PM · Restricted Project, Workboards
epriestley merged task T12376: Export workboard columns for reporting into T5024: Provide a dedicated page for viewing a single workboard column.
Mar 9 2017, 7:39 PM · Feature Request
epriestley added a comment to T12376: Export workboard columns for reporting.

Here's a (not so great) workaround:

Mar 9 2017, 7:39 PM · Feature Request
mrybczyn added a comment to T12376: Export workboard columns for reporting.

This is for another project. If it were for KDE, I would have asked @bcooksley first.

Mar 9 2017, 7:33 PM · Feature Request
epriestley added a comment to T12270: Unbeta Badges.

I'll send you some kind of something-or-other for dropping "Quality" from Conduit for the moment.

Mar 9 2017, 7:29 PM · Badges (v0)
bcooksley added a comment to T12376: Export workboard columns for reporting.

@mrybczyn Is someone who is a member of the KDE Community yes, but I don't know if this task relates to stuff they do in KDE though. Based on their activity page on Phabricator i'm inclined to think this isn't KDE related but I could be wrong.

Mar 9 2017, 7:27 PM · Feature Request
chad closed T12326: Long tag names can break out of hovercards as Resolved by committing Restricted Diffusion Commit.
Mar 9 2017, 7:09 PM · Bug Report, Hovercards, PHUI
chad added a revision to T12326: Long tag names can break out of hovercards: Unknown Object (Differential Revision).
Mar 9 2017, 7:06 PM · Bug Report, Hovercards, PHUI
lvital added a comment to T12372: Long list of reviewers breaks to: field when metamta.one-mail-per-recipient set to false.

Thanks for the quick fix and the huge amount of detail under the hood, we'll give that patch a shot!

Mar 9 2017, 6:57 PM · Mail, Bug Report
epriestley added a comment to T12372: Long list of reviewers breaks to: field when metamta.one-mail-per-recipient set to false.

Internally, PHPMailer sends headers through two adjustment phases:

Mar 9 2017, 6:30 PM · Mail, Bug Report
epriestley added a comment to T12359: Registration does not accept valid email addresses.

I know someone who has a normalname@domain.horse address and frequently finds services which won't accept it (since they're hard-coded for 3-letter TLDs) so I'd guess users with ={^_^}=@nekoneko.com addresses run into more frequent problems. We should handle .horse, .technology, etc., properly.

Mar 9 2017, 6:18 PM · Bug Report
lambdafu added a comment to T12359: Registration does not accept valid email addresses.

I am sure these email addresses are valid, but they are from people who run their own mailserver and can easily add a reasonable address for themselves, so I agree with your conclusion. More importantly, I want to thank you for your quick response, for phabricator, and your passion!

Mar 9 2017, 6:15 PM · Bug Report
epriestley added a comment to T12372: Long list of reviewers breaks to: field when metamta.one-mail-per-recipient set to false.

Can you test if this fixes the issue in your environment?

Mar 9 2017, 6:15 PM · Mail, Bug Report
epriestley added a comment to T12372: Long list of reviewers breaks to: field when metamta.one-mail-per-recipient set to false.

I think this is the controlling specification -- 2008 was a simpler time when no sequence of non-whitespace characters could ever be longer than 78 bytes, and a thousand bytes of memory was an untold wealth of riches:

Mar 9 2017, 6:09 PM · Mail, Bug Report
epriestley added a comment to T12372: Long list of reviewers breaks to: field when metamta.one-mail-per-recipient set to false.

I think the actual issue is that we're not wrapping the header. When I generate a synthetic message locally with a "To:" header with a length of ~50,000 bytes, the raw output is an unbroken line ~50,00 bytes long. I suspect this is RFC-violating and that whatever the mail gets handed to next gets upset by it.

Mar 9 2017, 5:57 PM · Mail, Bug Report
epriestley added a comment to T12372: Long list of reviewers breaks to: field when metamta.one-mail-per-recipient set to false.

Perfect, thanks!

Mar 9 2017, 5:28 PM · Mail, Bug Report
jmeador added a comment to T12372: Long list of reviewers breaks to: field when metamta.one-mail-per-recipient set to false.

The default -- PhabricatorMailImplementationPHPMailerLiteAdapter

Mar 9 2017, 5:27 PM · Mail, Bug Report
epriestley added a comment to T12372: Long list of reviewers breaks to: field when metamta.one-mail-per-recipient set to false.

What is metamta.mail-adapter set to?

Mar 9 2017, 5:26 PM · Mail, Bug Report
epriestley added a comment to T12372: Long list of reviewers breaks to: field when metamta.one-mail-per-recipient set to false.

See also T12046 for why all open source is sketchy voodoo and the eventual pathway forward here is likely replacing it with first-party code.

Mar 9 2017, 5:25 PM · Mail, Bug Report
epriestley updated subscribers of T12374: Saved Workboard Queries.

Two workarounds you could use here are:

Mar 9 2017, 5:15 PM · Workboards, Feature Request
epriestley added a comment to T12375: Integrate Conduit API for Policy Object Creation.

(Are you associated with WMF, or just noting that they have an extension for this?)

Mar 9 2017, 4:56 PM · Policy, Feature Request
epriestley added a comment to T12376: Export workboard columns for reporting.

Or maybe a better question is: @mrybczyn, are you asking on behalf of KDE, or a different project?

Mar 9 2017, 4:55 PM · Feature Request
epriestley updated subscribers of T12376: Export workboard columns for reporting.

@bcooksley, is this another one of yours?

Mar 9 2017, 4:53 PM · Feature Request
epriestley awarded T12378: "This task has been locked" on all logged out tasks a Pterodactyl token.
Mar 9 2017, 4:52 PM · Maniphest, EditEngine
epriestley added a comment to T12378: "This task has been locked" on all logged out tasks.

Thanks for catching this, and for the clear reproduction steps.

Mar 9 2017, 4:52 PM · Maniphest, EditEngine
epriestley closed T12378: "This task has been locked" on all logged out tasks as Resolved by committing Restricted Diffusion Commit.
Mar 9 2017, 4:51 PM · Maniphest, EditEngine
d.maznekov added a comment to T12369: Users sometimes prompted to "Edit Locked Task" when attempting to create a new task.

Ok, so I have several custom fields in forms such Task or New bug. Could that lead to that de-effect ?

Mar 9 2017, 4:46 PM · Bug Report, EditEngine
epriestley added a revision to T12378: "This task has been locked" on all logged out tasks: Unknown Object (Differential Revision).
Mar 9 2017, 4:46 PM · Maniphest, EditEngine
epriestley added a comment to T12369: Users sometimes prompted to "Edit Locked Task" when attempting to create a new task.

I have no idea either. We need reproduction steps. Reinstalling will not resolve this.

Mar 9 2017, 4:44 PM · Bug Report, EditEngine
chad updated subscribers of T12369: Users sometimes prompted to "Edit Locked Task" when attempting to create a new task.

There is definitely a bug here, I just have no idea what configuration of forms leads to it. @epriestley might have other guesses.

Mar 9 2017, 4:43 PM · Bug Report, EditEngine
d.maznekov added a comment to T12369: Users sometimes prompted to "Edit Locked Task" when attempting to create a new task.

I'll wait Friday stable.... today I've switched from stable to master branch. The install is if I can say untouched and every Friday updated.
Just git pull in 3 dirs and that's all. Use well known bash script for that. All customization and setups are stored in SQL database.
What else steps to report - just pressing New task and appears that.
Is it an option to clean up folder and make clean git clone ?
If tomorrow stable don't resolve that nasty problem will proceed with clean install.

Mar 9 2017, 4:04 PM · Bug Report, EditEngine
epriestley added a comment to T12373: Support macros in Phame.

That is, the problem would be:

Mar 9 2017, 3:37 PM · Macros, Phame, Bug Report
epriestley added a comment to T12373: Support macros in Phame.

I guess we actually have to test this use case:

Mar 9 2017, 3:36 PM · Macros, Phame, Bug Report
chad added a comment to T12373: Support macros in Phame.

I can fix if it's one line, I just assumed it was something super hard with external urls or something.

Mar 9 2017, 3:32 PM · Macros, Phame, Bug Report
chad added a comment to T12373: Support macros in Phame.

I blame the fun police.

Mar 9 2017, 3:32 PM · Macros, Phame, Bug Report
epriestley added a comment to T12373: Support macros in Phame.

https://secure.phabricator.com/source/phabricator/browse/master/src/infrastructure/markup/PhabricatorMarkupEngine.php;4c7d464f8bf2627ef7455596fde2ae5667e67a57$356

Mar 9 2017, 3:27 PM · Macros, Phame, Bug Report
epriestley updated subscribers of T12373: Support macros in Phame.

Macros are disabled in Phame posts, via markup engine. This originated in D2202.

Mar 9 2017, 3:27 PM · Macros, Phame, Bug Report
cspeckmim added a comment to T12370: Maniphest menu items misalign if they have submenu items.

youreawesome

Mar 9 2017, 3:27 PM · Bug Report
chad closed T12370: Maniphest menu items misalign if they have submenu items as Resolved by committing Restricted Diffusion Commit.
Mar 9 2017, 3:25 PM · Bug Report
epriestley claimed T12378: "This task has been locked" on all logged out tasks.

I think the way the view gets built is a little odd and I didn't get the logic quite right. Definitely a bug, thanks.

Mar 9 2017, 3:16 PM · Maniphest, EditEngine
chad added projects to T12378: "This task has been locked" on all logged out tasks: EditEngine, Maniphest.
Mar 9 2017, 3:05 PM · Maniphest, EditEngine
chad added a comment to T12375: Integrate Conduit API for Policy Object Creation.

We'd want to know the root problem you're having with Phabricator specifically so that we could decide the best way to resolve the issue in the upstream. Can you go into more detail about the custom policies you're wanting to create, how often, what they do, etc?

Mar 9 2017, 3:03 PM · Policy, Feature Request
chad created T12378: "This task has been locked" on all logged out tasks.
Mar 9 2017, 2:48 PM · Maniphest, EditEngine
chad closed T12377: S3 cannot migrate / upload files as Wontfix.
Mar 9 2017, 2:45 PM · Bug Report
chad added a comment to T12369: Users sometimes prompted to "Edit Locked Task" when attempting to create a new task.

Thanks, but we really just need the steps to reproduce the issue locally. I don't know them, I have no guesses as to how to get an installation to do that. Without knowing how to reproduce your bug specifically, we have no idea what to look for or if we're fixing the correct bug. This is a huge time burden for us, and is outside what we cover under "free support". We require every bug report to provide these steps for us, since Phabricator is a free product.

Mar 9 2017, 2:42 PM · Bug Report, EditEngine
remusvrm closed Q586: Is there a reason way I can not add a subscriber to a question in Ponder? as resolved.
Mar 9 2017, 1:54 PM · Ponder
remusvrm added Q586: Is there a reason way I can not add a subscriber to a question in Ponder? (Answer 527).
Mar 9 2017, 1:54 PM
remusvrm updated Q586: Is there a reason way I can not add a subscriber to a question in Ponder? from to Is there a reason way I can not add a subscriber to a question in Ponder?.
Mar 9 2017, 1:41 PM · Ponder
rlex added a comment to T12377: S3 cannot migrate / upload files.

Ok, i'm an idiot.
I configured wrong endpoint.
It's working now.
Still, it might be good idea for mentioning this policy in install doc - it's not really obvious that you need

"Effect": "Allow",
"Action": "s3:ListAllMyBuckets",
"Resource": "arn:aws:s3:::*"

For IAM policy to work.

Mar 9 2017, 1:37 PM · Bug Report
rlex added a comment to T12377: S3 cannot migrate / upload files.

Probably worth mentioning that this creds are configured using IAM instead of root creds.
Policy for this user:

{
    "Version": "2012-10-17",
    "Statement": [
        {
            "Effect": "Allow",
            "Action": "s3:ListAllMyBuckets",
            "Resource": "arn:aws:s3:::*"
        },
        {
            "Effect": "Allow",
            "Action": "s3:*",
            "Resource": [
                "arn:aws:s3:::files.example.com",
                "arn:aws:s3:::files.example.com/*"
            ]
        }
    ]
}
Mar 9 2017, 1:29 PM · Bug Report
rlex created T12377: S3 cannot migrate / upload files.
Mar 9 2017, 1:27 PM · Bug Report
mrybczyn created T12376: Export workboard columns for reporting.
Mar 9 2017, 1:25 PM · Feature Request
stettberger created T12375: Integrate Conduit API for Policy Object Creation.
Mar 9 2017, 1:11 PM · Policy, Feature Request