Page MenuHomePhabricator
Feed Advanced Search

Feb 17 2017

achals updated the task description for T12290: `arc diff` should copy the differential URL to clipboard.
Feb 17 2017, 10:27 PM · Feature Request
achals created T12290: `arc diff` should copy the differential URL to clipboard.
Feb 17 2017, 10:19 PM · Feature Request
chad awarded T2794: Implement Phage (like Hypershell) a 100 token.
Feb 17 2017, 10:04 PM · Phage, Infrastructure
epriestley renamed T12289: When a user is attending multiple overlapping events, computation of availability state may not be accurate from calendar: away should be displayed first than busy to When a user is attending multiple overlapping events, computation of availability state may not be accurate.
Feb 17 2017, 9:08 PM · Calendar, Bug Report
epriestley added a comment to T2794: Implement Phage (like Hypershell).

This is still a shallow ghost of Hypershell, but I successfully deployed secure with it:

Feb 17 2017, 8:56 PM · Phage, Infrastructure
lvital updated subscribers of T12272: Allow users to accept on behalf of packages they control a containing package for.
Feb 17 2017, 7:52 PM · Owners, Audit, Restricted Project, Differential
chad added a comment to T12288: Phabricator OAuth 2.0 initiating SSO login from a Third Party website.

We generally don't consider a root problem of "Phabricator lacks feature" as a reason to accept a feature request. We still need to know the underlying problem.

Feb 17 2017, 7:09 PM · Auth, Feature Request
jcox awarded F905540: cat3.png a 100 token.
Feb 17 2017, 6:59 PM
jcox awarded F905540: cat3.png a Burninate token.
Feb 17 2017, 6:59 PM
lvital added a comment to T12272: Allow users to accept on behalf of packages they control a containing package for.

As far as "opt-in", are you mostly concerned about performance?

Feb 17 2017, 6:46 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.

As far as "opt-in", are you mostly concerned about performance?

Feb 17 2017, 6:14 PM · Owners, Audit, Restricted Project, Differential
epriestley added a comment to T12287: Crumb width on profile pages is extremely conservative in Safari (always chops off the last couple letters?).

(I'm not sure how "Compatibility Mode" and the DOM inspector are interacting, so maybe I'm just using it incorrectly.)

Feb 17 2017, 5:59 PM · Internet Explorer, Design
lvital added a comment to T12272: Allow users to accept on behalf of packages they control a containing package for.
NOTE: Since packages do not own paths exclusively, any user can create a package on / of every repository and be allowed to force-accept every package review because their "everything" package is now a containing package for all other packages. However, they could already just remove the reviewers, so I don't think this is important. We could add options to Owners (e.g., an optional whitelist of "Stronger" packages) or something to prevent this, but I don't plan to do this. Just fire anyone abusing it.
Feb 17 2017, 5:57 PM · Owners, Audit, Restricted Project, Differential
• pasik updated the task description for T12288: Phabricator OAuth 2.0 initiating SSO login from a Third Party website.
Feb 17 2017, 5:53 PM · Auth, Feature Request
epriestley added a comment to T12289: When a user is attending multiple overlapping events, computation of availability state may not be accurate.

Here's the note in PhabricatorPeopleQuery about this:

Feb 17 2017, 5:42 PM · Calendar, Bug Report
epriestley added a comment to T12288: Phabricator OAuth 2.0 initiating SSO login from a Third Party website.

I don't think this is worth bringing upstream. It very occasionally saves users one click on an otherwise straightforward workflow, but requires significant complexity and maintenance from the upstream.

Feb 17 2017, 5:40 PM · Auth, Feature Request
epriestley added a comment to T12287: Crumb width on profile pages is extremely conservative in Safari (always chops off the last couple letters?).

According to compatibility mode on my machine (Windows 7, IE 11, in IE10 mode), there is no crumb bug in IE10 if float: left is removed. This isn't consistent with T5902 or with other results I got (e.g., this thing says there is such a bug).

Feb 17 2017, 5:29 PM · Internet Explorer, Design
chad added a comment to T12287: Crumb width on profile pages is extremely conservative in Safari (always chops off the last couple letters?).

https://msdn.microsoft.com/en-us/library/dn255001(v=vs.85).aspx

Feb 17 2017, 5:24 PM · Internet Explorer, Design
chad added a comment to T12287: Crumb width on profile pages is extremely conservative in Safari (always chops off the last couple letters?).

why can't you test 8-10? I thought you could do that with the inspection tools?

Feb 17 2017, 5:22 PM · Internet Explorer, Design
Luke081515.2 created T12289: When a user is attending multiple overlapping events, computation of availability state may not be accurate.
Feb 17 2017, 5:22 PM · Calendar, Bug Report
epriestley added a comment to T12287: Crumb width on profile pages is extremely conservative in Safari (always chops off the last couple letters?).

Are you OK with this plan?

Feb 17 2017, 5:20 PM · Internet Explorer, Design
epriestley added a comment to T12287: Crumb width on profile pages is extremely conservative in Safari (always chops off the last couple letters?).

I made some effort to fix this, but without much headway:

Feb 17 2017, 5:16 PM · Internet Explorer, Design
chad added a comment to T12287: Crumb width on profile pages is extremely conservative in Safari (always chops off the last couple letters?).

just revert the change if you want... or fix

Feb 17 2017, 5:15 PM · Internet Explorer, Design
chad added a comment to T12287: Crumb width on profile pages is extremely conservative in Safari (always chops off the last couple letters?).

Thanks, Safari

Feb 17 2017, 5:13 PM · Internet Explorer, Design
• pasik created T12288: Phabricator OAuth 2.0 initiating SSO login from a Third Party website.
Feb 17 2017, 5:06 PM · Auth, Feature Request
epriestley added a comment to T12157: Make it visibly clearer that a Phabricator user account has been disabled.

D17374 makes hovercards a little bit more clear too, although that might get some design feedback still (see "Horse" here, for example):

Feb 17 2017, 4:08 PM · Profile, Feature Request
epriestley added a revision to T12157: Make it visibly clearer that a Phabricator user account has been disabled: Unknown Object (Differential Revision).
Feb 17 2017, 3:25 PM · Profile, Feature Request
epriestley added a revision to T12268: Make the grey "Unverified User" dot accurately reflect the new universal behavior: Unknown Object (Differential Revision).
Feb 17 2017, 3:25 PM · Mail, People
epriestley added a comment to T11172: Unable to configure Arcanist as per the quick start tutorial.

In the future (T550) we plan to allow arc commands to use an SSH key. I'm not sure if that will reduce or increase confusion.

Feb 17 2017, 3:16 PM · Documentation, Bug Report
jacksongabbard added a comment to T11172: Unable to configure Arcanist as per the quick start tutorial.

It is somewhat confusing to have both an SSH key access level that allows repo cloning and an arc certificate level of access that enables different features.

Feb 17 2017, 3:13 PM · Documentation, Bug Report
jacksongabbard added a comment to T11172: Unable to configure Arcanist as per the quick start tutorial.

Ah, yes -- this is where I got off track. In my head I had conflated being able to clone the repo with having the arc certificate installed. These steps are unrelated, I understand clearly now. As long as the SSH key is added in Phabricator and the Repo is activated in Diffusion, cloning it is no problem. In this case, I thought it was the lack of arc certificate blocking me, when in was actually just that I forgot to activate the repo. Whoops. Thanks for clarifying.

Feb 17 2017, 3:11 PM · Documentation, Bug Report
epriestley created T12287: Crumb width on profile pages is extremely conservative in Safari (always chops off the last couple letters?).
Feb 17 2017, 3:10 PM · Internet Explorer, Design
epriestley added a comment to T11172: Unable to configure Arcanist as per the quick start tutorial.

You should be able to can arc install-certificate --conduit-uri=https://whatever to install a certificate for an arbitrary install with no other configuration.

Feb 17 2017, 3:05 PM · Documentation, Bug Report
jacksongabbard added a comment to T11172: Unable to configure Arcanist as per the quick start tutorial.

Fwiw, if someone else lands here (which I just did by Googling the same error message), the problem for me was the same as for the original poster -- having not first initialized a repo locally one way or another. My intuition (which is probably wrong) is that there is an interesting order of operations problem here.

Feb 17 2017, 2:56 PM · Documentation, Bug Report
epriestley merged task T12286: Passphrase Conduit API does not specify secret for Note credential type into T5553: Allow SSL certificate to be stored in Passphrase.
Feb 17 2017, 2:35 PM · Bug Report
epriestley merged T12286: Passphrase Conduit API does not specify secret for Note credential type into T5553: Allow SSL certificate to be stored in Passphrase.
Feb 17 2017, 2:35 PM · Passphrase
pouyana added a comment to T12286: Passphrase Conduit API does not specify secret for Note credential type.

SSL Certificate, key and Ca files. Of course they can be saved as SSH keys but I don't want to mix them together.

Feb 17 2017, 2:34 PM · Bug Report
epriestley added a comment to T12286: Passphrase Conduit API does not specify secret for Note credential type.

What is your use case for API access to notes?

Feb 17 2017, 2:30 PM · Bug Report
pouyana created T12286: Passphrase Conduit API does not specify secret for Note credential type.
Feb 17 2017, 2:13 PM · Bug Report
epriestley closed T12284: Wrong notification label (subtask status update) as Resolved by committing Restricted Diffusion Commit.
Feb 17 2017, 2:09 PM · Bug Report
epriestley added a comment to T12284: Wrong notification label (subtask status update).

D17373 should fix the typo.

Feb 17 2017, 1:34 PM · Bug Report
stevex awarded T5427: Force a line break in a table cell a Like token.
Feb 17 2017, 1:28 PM · Remarkup
epriestley added a revision to T12284: Wrong notification label (subtask status update): Unknown Object (Differential Revision).
Feb 17 2017, 1:26 PM · Bug Report
rftfaria awarded T5427: Force a line break in a table cell a Like token.
Feb 17 2017, 12:12 PM · Remarkup
avivey merged T12285: uml support into T3591: Render PlantUML syntax as macro in remarkup areas. .
Feb 17 2017, 12:10 PM · Remarkup
avivey merged task T12285: uml support into T3591: Render PlantUML syntax as macro in remarkup areas. .
Feb 17 2017, 12:10 PM · Feature Request
avivey added a comment to T12285: uml support.

We used to have GraphViz support for this, but we shut it down for security reasons - see T9408.

Feb 17 2017, 12:09 PM · Feature Request
avivey changed the visibility for T3602: Provide some kind of "process this block with an interpreter" syntax to Remarkup.
Feb 17 2017, 12:04 PM · Remarkup
avivey changed the visibility for T3591: Render PlantUML syntax as macro in remarkup areas. .
Feb 17 2017, 11:59 AM · Remarkup
randuev created T12285: uml support.
Feb 17 2017, 11:06 AM · Feature Request
mcorteel created T12284: Wrong notification label (subtask status update).
Feb 17 2017, 10:55 AM · Bug Report
chad closed T12283: Rocket.Chat as Spite.

nyancat

Feb 17 2017, 6:32 AM · Feature Request
• prashant.mankar updated the task description for T12283: Rocket.Chat.
Feb 17 2017, 6:20 AM · Feature Request
• prashant.mankar updated the task description for T12283: Rocket.Chat.
Feb 17 2017, 6:16 AM · Feature Request
• prashant.mankar created T12283: Rocket.Chat.
Feb 17 2017, 6:15 AM · Feature Request
salvian added a comment to T12150: Restricting Push not working.

update: We used default mount options in /etc/fstab for the disk containing repositories

noexec,noatime,nodiratime

we fixes the push logs and herald commit content hooks issues by changing it to

noatime,nodiratime

thanks!

Feb 17 2017, 3:36 AM · Bug Report (Needs Information)

Feb 16 2017

epriestley closed T12281: Exception thrown when adding a form to the Favorites menu as Resolved by committing Restricted Diffusion Commit.
Feb 16 2017, 11:45 PM · Bug Report
epriestley added a comment to T12281: Exception thrown when adding a form to the Favorites menu.

Thanks! Your reproduction instructions were extremely clear and made hunting this down straightforward. D17372 should resolve this.

Feb 16 2017, 11:19 PM · Bug Report
epriestley added a revision to T12281: Exception thrown when adding a form to the Favorites menu: Unknown Object (Differential Revision).
Feb 16 2017, 11:17 PM · Bug Report
epriestley added a comment to T12282: "Phabricator Media" wiki page has old logos.

I gave the goat some leaves to eat because maybe goats eat leaves?

Feb 16 2017, 11:00 PM · Design
epriestley updated the task description for T12282: "Phabricator Media" wiki page has old logos.
Feb 16 2017, 10:59 PM · Design
epriestley created T12282: "Phabricator Media" wiki page has old logos.
Feb 16 2017, 10:58 PM · Design
epriestley added a comment to T9525: Provide a reasonable general use logo.

Those are old now I guess, I'll file something to see if @chad can fix it because I am not licensed to pixel.

Feb 16 2017, 10:57 PM · Public & Media Relations
epriestley added a comment to T12279: Timezone Asia/Jerusalem incorrectly listed under UTC-0.

Well by the time you divide false into 3600 pieces you can't have much left!

Feb 16 2017, 10:55 PM · Bug Report
avivey added a comment to T12279: Timezone Asia/Jerusalem incorrectly listed under UTC-0.

err, looks like it returns "FALSE on failure", and naturally in PHP, FALSE / 3600 is 0, so we just consider it as "UTC".

Feb 16 2017, 10:35 PM · Bug Report
avivey added a comment to T12279: Timezone Asia/Jerusalem incorrectly listed under UTC-0.

This information is obtained from your PHP, using datetimezone.getoffset(), so it's not all in our control.

Feb 16 2017, 10:27 PM · Bug Report
michaeljs1990 awarded T9525: Provide a reasonable general use logo a Like token.
Feb 16 2017, 10:05 PM · Public & Media Relations
Itms updated the task description for T12281: Exception thrown when adding a form to the Favorites menu.
Feb 16 2017, 9:57 PM · Bug Report
Itms created T12281: Exception thrown when adding a form to the Favorites menu.
Feb 16 2017, 9:50 PM · Bug Report
terirem added a comment to T12263: Javascript licence.

Thanks, Chad!

Feb 16 2017, 8:06 PM · Feature Request
zhiyuchen awarded T5474: Support workboard column triggers which activate when a task is dropped into a column a Like token.
Feb 16 2017, 7:44 PM · Workboards (v3), Restricted Project
mbenua-mparticle awarded T5474: Support workboard column triggers which activate when a task is dropped into a column a Love token.
Feb 16 2017, 7:43 PM · Workboards (v3), Restricted Project
zhiyuchen created T12280: mov and m4v videos don't play .
Feb 16 2017, 7:40 PM · Bug Report
chad added a comment to T12263: Javascript licence.

We wrote javelin

Feb 16 2017, 7:29 PM · Feature Request
epriestley added a comment to T12263: Javascript licence.

See LICENSE for license information:

Feb 16 2017, 7:26 PM · Feature Request
zhiyuchen created T12279: Timezone Asia/Jerusalem incorrectly listed under UTC-0.
Feb 16 2017, 7:03 PM · Bug Report
terirem added a comment to T12263: Javascript licence.

Well, upon browsing more through the source code, it appears that Phabricator is using "Javelin" javascript library which is not free software. Could you please give some more info about this?

Feb 16 2017, 7:00 PM · Feature Request
michaeljs1990 added Q568: Removing LDAP authentication from a Phabricator installation (Answer 511).
Feb 16 2017, 6:07 PM
terirem added a comment to T12263: Javascript licence.

If I want to add some code (like a script notice described in chapter 3.2.1 here: https://www.gnu.org/software/librejs/free-your-javascript.html) to the <head> before any script, where's the right place in the Phabricator code to do this?

Feb 16 2017, 5:36 PM · Feature Request
epriestley added a revision to T2794: Implement Phage (like Hypershell): Unknown Object (Differential Revision).
Feb 16 2017, 1:50 PM · Phage, Infrastructure
OCram added a comment to Q567: JSON Example for Conduit method maniphest.edit? (Answer 510).

It looks like a problem with simple JSON String and complex containing maps / array. I noticed it while using the conduit method pages itself seeing the different json syntaxes like:

$ echo '{
  "names": [
    "T1000",
    "T1002"
  ]
}' | arc call-conduit --conduit-uri https://secure.phabricator.com/ --conduit-token <conduit-token> phid.lookup

and

$ curl https://secure.phabricator.com/api/phid.lookup \
    -d api.token=api-token \
    -d names[0]=T1000 \
    -d names[1]=T1002

In addition to that it seems not quite clear which syntax to use with powershell (or other foreign clients) and which content type should be used. So for powershell it seems like there is a limitation (or issue) for JSON strings when using the default context type application/x-www-form-urlencoded.

Feb 16 2017, 1:11 PM
epriestley added a comment to T11989: Wage an endless war against HP Printer Fax Support.

Recently, we've started seeing a handful of attacks where human users appear to read a discussion, formulate a human-sounding, contextual reply (e.g., discussing the thread topic in what appears to be a human way, just with a very shallow understanding of the issue), and then include a link to a site offering various services (mostly essay writing?) in the footer.

Feb 16 2017, 12:29 PM · Abuse
epriestley removed a watcher for Feature Request: • ilmked0514.
Feb 16 2017, 12:09 PM
epriestley removed a watcher for Feature Request: ps625353.
Feb 16 2017, 12:09 PM
epriestley removed a watcher for Feature Request: • tawels74.
Feb 16 2017, 12:09 PM
• MarshalBlake added a comment to P1792 PhabricatorGermanTranslation.php.
Feb 16 2017, 11:48 AM · Localization
CodingNickNick awarded T2393: Adds a "Needs Verification" state to Audits a Love token.
Feb 16 2017, 11:40 AM · Restricted Project, Audit
milianw added a comment to T12276: Allow to configure arc commit message.

Regarding what modifications I would like to see:

Feb 16 2017, 11:02 AM · KDE, Feature Request
avivey updated subscribers of T12266: arc set-config encoding XXX not work.

This setting (In the Phabricator web interface) is very old, and should exist in all versions. Ask your admin to configure it (Assuming your admin created the repository).

Feb 16 2017, 10:46 AM · Bug Report
salvian added a comment to T12150: Restricting Push not working.

We migrated phabricator last months just realized somehow "Push Logs" are not generated anymore (the last push log was created just before the migration)
and I guess commit content hooks are somehow related on push logs (?)
May I ask what can I check if the push logs aren't being created?

Feb 16 2017, 10:44 AM · Bug Report (Needs Information)
avivey added a comment to Q567: JSON Example for Conduit method maniphest.edit? (Answer 510).

It sounds like a legitimate bug or feature request ("making conduit calls from powershell"). Did your Ponduit library used to work and then broke when Phabricator changed something?

Feb 16 2017, 10:32 AM
OCram added a comment to Q567: JSON Example for Conduit method maniphest.edit? (Answer 510).

No unfortunately not. Doing RestAPI like web request with Powershell is a real pain in the as. JSON Strings doesn't work with some structures. Powershell JSON Conduit Discussion.

Feb 16 2017, 9:40 AM
mirrom created T12278: Choose specific form on Create Subtask.
Feb 16 2017, 9:21 AM · Maniphest, Feature Request
mirrom added a project to T12277: Show the points of the sub and parent tasks in the Task Graph: Maniphest.
Feb 16 2017, 9:20 AM · Maniphest, Feature Request
mirrom created T12277: Show the points of the sub and parent tasks in the Task Graph.
Feb 16 2017, 9:18 AM · Maniphest, Feature Request
xian0617 added a comment to T12266: arc set-config encoding XXX not work.

thanks all your answers

Feb 16 2017, 8:29 AM · Bug Report
xian0617 updated subscribers of T12266: arc set-config encoding XXX not work.
Feb 16 2017, 8:28 AM · Bug Report
• HarryOtto added a comment to T12268: Make the grey "Unverified User" dot accurately reflect the new universal behavior.
Feb 16 2017, 7:46 AM · Mail, People
epriestley updated subscribers of T12276: Allow to configure arc commit message.

Also, please work with other KDE project members to discuss KDE priorities and needs downstream, then have a single point of contact (nominally, @bcooksley) bring them upstream once there is consensus on the nature and priority of problems. Currently, you, @abika, and @bcooksley are all independently interacting with this upstream.

Feb 16 2017, 12:24 AM · KDE, Feature Request
avivey awarded T12271: When users accept a revision, allow them to accept for a subset of package/project reviewers they could accept for a Yellow Medal token.
Feb 16 2017, 12:24 AM · Audit, Restricted Project, Differential