Page MenuHomePhabricator
Feed Advanced Search

May 6 2016

epriestley added a comment to T10931: Changing mysql.implementation setting doesn't work.

AphrontMySQLDatabaseConnectionBase was renamed but I plan to remove this option anyway.

May 6 2016, 9:57 PM · Setup, Database, Bug Report
Luke081515.2 added a comment to T10927: Can't load data from reposorities: 403.

Thanks for that quick fix. :)

May 6 2016, 9:53 PM · Diffusion, Bug Report
• kerscher added a comment to T10660: Php 7 installation.

To get @kornrunner’s suggestion to work on Ubuntu 16.04 with PHP7:

May 6 2016, 9:47 PM · Bug Report
hut created T10931: Changing mysql.implementation setting doesn't work.
May 6 2016, 9:17 PM · Setup, Database, Bug Report
avivey created T10930: Exception in editengine for Paste.
May 6 2016, 6:09 PM · ApplicationEditor, Bug Report
epriestley closed T10928: Remote branches of git mirror are deleted as Resolved by committing Restricted Diffusion Commit.
May 6 2016, 3:04 PM · Bug Report
sascha-egerer added a comment to T10928: Remote branches of git mirror are deleted.

Thanks! Maybe this can save some time for some people...

May 6 2016, 3:03 PM · Bug Report
epriestley added a revision to T10928: Remote branches of git mirror are deleted: Restricted Differential Revision.
May 6 2016, 2:59 PM · Bug Report
sascha-egerer added a comment to T10928: Remote branches of git mirror are deleted.

I already thought that while writing the ticket ;-) That's the reason i added the last sentence to adjust the documentation. Maybe it would be good to know that this feature is using the git mirror function and may overwrite data of the mirror.

May 6 2016, 2:46 PM · Bug Report
epriestley added a comment to T10928: Remote branches of git mirror are deleted.

This is expected, and consistent with git's own definition of "mirror". The exact command we use is:

May 6 2016, 2:44 PM · Bug Report
sascha-egerer updated the task description for T10928: Remote branches of git mirror are deleted.
May 6 2016, 10:28 AM · Bug Report
sascha-egerer updated the task description for T10928: Remote branches of git mirror are deleted.
May 6 2016, 10:23 AM · Bug Report
sascha-egerer created T10928: Remote branches of git mirror are deleted.
May 6 2016, 9:54 AM · Bug Report

May 5 2016

epriestley closed T10927: Can't load data from reposorities: 403 as Resolved.

D15856 appears to have fixed this

May 5 2016, 11:48 PM · Diffusion, Bug Report
epriestley added a revision to T10927: Can't load data from reposorities: 403: Restricted Differential Revision.
May 5 2016, 11:19 PM · Diffusion, Bug Report
Luke081515.2 created T10927: Can't load data from reposorities: 403.
May 5 2016, 10:06 PM · Diffusion, Bug Report
chad added a comment to T10926: Make it more likely to see assignee & reporter in task view without having to scroll down.

Actions - will get reduced in size and sidebar content will move up as a result. We're heavily prioritized right now so no current ETA. It is our intention to reduce these globally. This will raise content underneath significantly, we expect to remove about 1/3 to 1/2 the actions globally if possible.

May 5 2016, 2:36 PM · Transactions, Feature Request, Maniphest
qgil added a comment to T10926: Make it more likely to see assignee & reporter in task view without having to scroll down.

I'm happy with the new UI but only because I happen to work with a monitor in portrait mode. :)

May 5 2016, 10:33 AM · Transactions, Feature Request, Maniphest
aklapper created T10926: Make it more likely to see assignee & reporter in task view without having to scroll down.
May 5 2016, 9:15 AM · Transactions, Feature Request, Maniphest
chad closed T10925: Cannot create repository with latest HEAD as Resolved by committing Restricted Diffusion Commit.
May 5 2016, 4:13 AM · Bug Report
chad added a revision to T10925: Cannot create repository with latest HEAD: Restricted Differential Revision.
May 5 2016, 3:56 AM · Bug Report
chad added a comment to T10925: Cannot create repository with latest HEAD.

awesome, thanks for the report!

May 5 2016, 3:43 AM · Bug Report
thoughtpolice added a comment to T10925: Cannot create repository with latest HEAD.

Yeah, should be restarted with my custom dumb script:

May 5 2016, 3:42 AM · Bug Report
chad added a comment to T10925: Cannot create repository with latest HEAD.

Ah I see, did you manually go to /diffusion/create/ or did we miss a link somewhere?

May 5 2016, 3:38 AM · Bug Report
chad added a comment to T10925: Cannot create repository with latest HEAD.

Did you restart phabricator?

May 5 2016, 3:36 AM · Bug Report
thoughtpolice updated the task description for T10925: Cannot create repository with latest HEAD.
May 5 2016, 3:21 AM · Bug Report
thoughtpolice created T10925: Cannot create repository with latest HEAD.
May 5 2016, 3:21 AM · Bug Report

May 4 2016

eadler merged task T10919: Websocket notification not working anymore after upgrade from 2016 week 15 to 2016 week 18 (stable branch) into T10402: Add a client setup check to detect that the browser is using HTTPS but Phabricator does not recognize it.
May 4 2016, 4:03 PM · Bug Report
julioromano added a comment to T10919: Websocket notification not working anymore after upgrade from 2016 week 15 to 2016 week 18 (stable branch).

Although it's not a default setting,

proxy_set_header X-Forwarded-Proto $scheme;

is a common configuration option when using nginx to terminate SSL.

May 4 2016, 3:28 PM · Bug Report
epriestley added a comment to T10919: Websocket notification not working anymore after upgrade from 2016 week 15 to 2016 week 18 (stable branch).

I suppose we can trust X-Forwarded-Proto for the purposes of raising a setup issue, although I'm not sure if that would help here, since I think the header usually won't be set if you use nginx to terminate SSL instead of, say, an ELB.

May 4 2016, 3:13 PM · Bug Report
julioromano added a comment to T10919: Websocket notification not working anymore after upgrade from 2016 week 15 to 2016 week 18 (stable branch).

@epriestley you nailed it.

May 4 2016, 3:05 PM · Bug Report
epriestley added a comment to T10919: Websocket notification not working anymore after upgrade from 2016 week 15 to 2016 week 18 (stable branch).

I suspect you may not be setting $_SERVER['HTTPS'] so Phabricator believes you are connecting over HTTP. It looks for a client using the http protocol, fails to find one (only https is available), and does not emit a websocket setup instruction.

May 4 2016, 2:35 PM · Bug Report
julioromano added a comment to T10919: Websocket notification not working anymore after upgrade from 2016 week 15 to 2016 week 18 (stable branch).
  • I git pulled the 3 phabricator repos from the stable branch.
May 4 2016, 1:58 PM · Bug Report
chad added a comment to T10919: Websocket notification not working anymore after upgrade from 2016 week 15 to 2016 week 18 (stable branch).

Can you give precise reproduction steps, please? Specifically:

May 4 2016, 1:48 PM · Bug Report
qgil added a comment to T10912: Repositioning tasks within a Milestone column moves the task out of that milestone.

Thank you for the quick fix! As for the fate of the misplaced tasks, don't worry, I'll deal with them.

May 4 2016, 11:53 AM · Projects, Workboards, Bug Report
eadler added a comment to T10918: https://secure.phabricator.com/feed/query/9XVX7CRbpoer/#xss"><img src=xss onerror=alert(1)>.

Please open a new test instance on phacility for playing around. This is a real, live, production instance.

May 4 2016, 7:30 AM · Bug Report
eadler closed T10918: https://secure.phabricator.com/feed/query/9XVX7CRbpoer/#xss"><img src=xss onerror=alert(1)> as Spite.
May 4 2016, 7:30 AM · Bug Report
julioromano updated the task description for T10919: Websocket notification not working anymore after upgrade from 2016 week 15 to 2016 week 18 (stable branch).
May 4 2016, 7:14 AM · Bug Report
julioromano created T10919: Websocket notification not working anymore after upgrade from 2016 week 15 to 2016 week 18 (stable branch).
May 4 2016, 7:14 AM · Bug Report
katilthe added a comment to T10918: https://secure.phabricator.com/feed/query/9XVX7CRbpoer/#xss"><img src=xss onerror=alert(1)>.

http://googlе.com

May 4 2016, 6:45 AM · Bug Report
katilthe updated the task description for T10918: https://secure.phabricator.com/feed/query/9XVX7CRbpoer/#xss"><img src=xss onerror=alert(1)>.
May 4 2016, 6:41 AM · Bug Report
katilthe created T10918: https://secure.phabricator.com/feed/query/9XVX7CRbpoer/#xss"><img src=xss onerror=alert(1)>.
May 4 2016, 6:39 AM · Bug Report

May 3 2016

zacchiro added a comment to T10913: herald "Commit Hook: Commit Content" triggers on "git push --tags".

Grmbl. I cannot indeed reproduce it neither with your example, I'll investigate more what's wrong with my (other) setup, that still doesn't work. I'll reopen if/when I find out what's happening. Sorry for the noise.

May 3 2016, 7:37 PM · Herald, Bug Report
epriestley closed T10913: herald "Commit Hook: Commit Content" triggers on "git push --tags" as Resolved.

Per F1256292, presuming resolved until more details surface about what's up.

May 3 2016, 7:36 PM · Herald, Bug Report
epriestley added a comment to T10913: herald "Commit Hook: Commit Content" triggers on "git push --tags".

I can't reproduce this. Here's what I did. I created this Herald rule:

May 3 2016, 3:38 PM · Herald, Bug Report
zacchiro created T10913: herald "Commit Hook: Commit Content" triggers on "git push --tags".
May 3 2016, 3:28 PM · Herald, Bug Report
epriestley closed T10912: Repositioning tasks within a Milestone column moves the task out of that milestone as Resolved by committing Restricted Diffusion Commit.
May 3 2016, 2:59 PM · Projects, Workboards, Bug Report
epriestley added a comment to T10912: Repositioning tasks within a Milestone column moves the task out of that milestone.

It's also fine to not fix this, or just fix some of it -- the affected tasks shouldn't do anything bad or break anything, even though their state isn't totally consistent. They won't respond correctly to queries (e.g., querying for tasks in the milestone won't return affected tasks, since they aren't "really" part of the milestone anymore) but should behave normally other than that. So it's not critical to hunt down every single affected task.

May 3 2016, 12:11 PM · Projects, Workboards, Bug Report
epriestley added a comment to T10912: Repositioning tasks within a Milestone column moves the task out of that milestone.

Thanks for the report! This is a definitely a bug, and should be fixed later today once D15834 is reviewed.

May 3 2016, 12:06 PM · Projects, Workboards, Bug Report
epriestley added a revision to T10912: Repositioning tasks within a Milestone column moves the task out of that milestone: Restricted Differential Revision.
May 3 2016, 11:54 AM · Projects, Workboards, Bug Report
epriestley claimed T10912: Repositioning tasks within a Milestone column moves the task out of that milestone.
May 3 2016, 11:03 AM · Projects, Workboards, Bug Report
qgil created T10912: Repositioning tasks within a Milestone column moves the task out of that milestone.
May 3 2016, 10:50 AM · Projects, Workboards, Bug Report

May 2 2016

chad added a project to T10731: Script-and-regex linter documentation does not make it clear enough that you really have to do what it says: Lint.
May 2 2016, 7:56 PM · Lint, Bug Report
chad added a project to T10725: Can't access binary files of differential revision: Differential.
May 2 2016, 7:56 PM · Restricted Project, Differential, Bug Report
chad added a project to T10559: Lease Working Copy Hangs on lease failure, but should fail: Drydock.
May 2 2016, 7:55 PM · Drydock, Bug Report
chad added a project to T10510: Harbormaster failing to trigger Jenkins build through HTTP GET and producing abnormal HTTP error code : Harbormaster.
May 2 2016, 7:55 PM · Restricted Project, Harbormaster, Bug Report
chad closed T10397: twitter links on phacility.com do not link to individual tweets as Wontfix.

No plans to fix this, next Phacility site due shortly does not have twitter links.

May 2 2016, 7:52 PM · Phacility, Bug Report
chad merged T10661: Can't create milestone into T10779: Cannot manage base project milestones if base project has subprojects.
May 2 2016, 7:15 PM · Restricted Project, Projects, Bug Report
chad merged task T10661: Can't create milestone into T10779: Cannot manage base project milestones if base project has subprojects.
May 2 2016, 7:15 PM · Projects (v3), Bug Report
chad added a comment to T10661: Can't create milestone.

I'm going to guess this is the same as T10779

May 2 2016, 7:15 PM · Projects (v3), Bug Report
chad closed T10802: Unable to create new forms, even though I am an administrator as Invalid.

No further responses here and unable to reproduce as explained. Feel free to comment if we've missed something.

May 2 2016, 7:12 PM · Bug Report
chad added a project to T10818: Lint errors about filenames, which do not include any source context, render with awkward whitespace: Arcanist.
May 2 2016, 6:57 PM · Arcanist, Bug Report
chad closed T10834: Project Workboard Indentation as Invalid.

Closing, as there's nothing here that's reproducible, or reported by other installs. Nothing else for us to look into.

May 2 2016, 6:56 PM · Bug Report
chad added projects to T10866: Missing Information in Diffusion User Guide: Repo Hosting: Documentation, Diffusion.
May 2 2016, 6:55 PM · Diffusion, Documentation, Bug Report
chad added a project to T10893: repo update fails for repos with 1000s of closing branches: Diffusion.
May 2 2016, 6:54 PM · Diffusion, Bug Report
chad added a project to T10881: git diff parsing fails with --submodule=log: Arcanist.
May 2 2016, 6:54 PM · Contributor Onboarding, Arcanist, Bug Report
chad added projects to T10872: any(Project) does not include Milestones of Project: Projects, ApplicationSearch.
May 2 2016, 6:53 PM · Restricted Project, ApplicationSearch, Projects, Bug Report
chad edited projects for T10910: Creating an arc diff from a working directory that is not the repository root, added: Bug Report, Arcanist, Windows; removed Feature Request.
May 2 2016, 5:18 PM · Windows, Arcanist, Bug Report

May 1 2016

chad closed T10820: Missing left padding in "Auditors" field on Diffusion Commit as Resolved by committing Restricted Diffusion Commit.
May 1 2016, 7:52 PM · Bug Report
chad closed T10906: Differential StatusView icons for "rejected" vs "rejected older" are differentiated only by color as Resolved by committing Restricted Diffusion Commit.
May 1 2016, 7:52 PM · Bug Report
chad added a revision to T10820: Missing left padding in "Auditors" field on Diffusion Commit: Restricted Differential Revision.
May 1 2016, 7:52 PM · Bug Report
chad added a revision to T10906: Differential StatusView icons for "rejected" vs "rejected older" are differentiated only by color: Restricted Differential Revision.
May 1 2016, 7:35 PM · Bug Report
epriestley merged task T10241: Modifying an externally-hosted repository's URL/path can unintentionally change the credentials into T10748: Implement `diffusion.repository.edit`, for creating and editing repositories via the API.
May 1 2016, 4:42 PM · Passphrase, Diffusion, Bug Report
epriestley added a comment to T10241: Modifying an externally-hosted repository's URL/path can unintentionally change the credentials.

I'm going to merge this into T10748. It should technically be fixed by D15829 on its own, but will be fixed even harder once we cut over to the changes in T10748.

May 1 2016, 4:42 PM · Passphrase, Diffusion, Bug Report
epriestley added a revision to T10241: Modifying an externally-hosted repository's URL/path can unintentionally change the credentials: Restricted Differential Revision.
May 1 2016, 4:39 PM · Passphrase, Diffusion, Bug Report
chad renamed T10781: Conpherence new links are missing styles from Join/Create Room links in Conpherence have no style. to Conpherence new links are missing styles.
May 1 2016, 4:30 PM · Conpherence, Bug Report
chad merged task T10908: Conpherence "Create new Room" click area is too big into T10781: Conpherence new links are missing styles.
May 1 2016, 4:29 PM · Bug Report
chad merged T10908: Conpherence "Create new Room" click area is too big into T10781: Conpherence new links are missing styles.
May 1 2016, 4:29 PM · Conpherence, Bug Report
monsdar created T10908: Conpherence "Create new Room" click area is too big.
May 1 2016, 3:16 PM · Bug Report

Apr 30 2016

avivey added a project to T10900: Repository content search with `hg grep` returns matches found in old versions of files: Mercurial.
Apr 30 2016, 6:15 PM · Mercurial, Diffusion
epriestley added a comment to T10906: Differential StatusView icons for "rejected" vs "rejected older" are differentiated only by color.

Ah, sure, that makes sense.

Apr 30 2016, 1:26 AM · Bug Report
chad added a comment to T10907: Older project's icon changes back after a few days.

i_ain__t_afraid_of_no_ghost_by_neopren-d34c7y9.jpg (1×1 px, 1 MB)

Apr 30 2016, 1:26 AM · Projects, Bug Report (Needs Information)
chad added a comment to T10906: Differential StatusView icons for "rejected" vs "rejected older" are differentiated only by color.

At least circle and circle-o are similar enough that it doesn't feel out of place.

Apr 30 2016, 1:22 AM · Bug Report
epriestley added a comment to T10907: Older project's icon changes back after a few days.

This one is obviously ghosts. How could this possibly be anything but ghosts?

Apr 30 2016, 1:03 AM · Projects, Bug Report (Needs Information)
cspeckmim created T10907: Older project's icon changes back after a few days.
Apr 30 2016, 12:58 AM · Projects, Bug Report (Needs Information)
epriestley renamed T10906: Differential StatusView icons for "rejected" vs "rejected older" are differentiated only by color from Can't differentiate diff rejection icons to Differential StatusView icons for "rejected" vs "rejected older" are differentiated only by color.
Apr 30 2016, 12:57 AM · Bug Report
epriestley added a revision to T10906: Differential StatusView icons for "rejected" vs "rejected older" are differentiated only by color: Restricted Differential Revision.
Apr 30 2016, 12:54 AM · Bug Report
getaaron added a comment to T10906: Differential StatusView icons for "rejected" vs "rejected older" are differentiated only by color.

@epriestley Inverting the "old" (greyscale) icons would resolve my issue.

Apr 30 2016, 12:52 AM · Bug Report
chad added a comment to T10906: Differential StatusView icons for "rejected" vs "rejected older" are differentiated only by color.

Yeah that's all I could come up with as a possibility too.

Apr 30 2016, 12:51 AM · Bug Report
getaaron added a comment to T10906: Differential StatusView icons for "rejected" vs "rejected older" are differentiated only by color.

They're used in a diff. Red means someone has rejected my most recent diff. Black means they rejected an older version but haven't seen my newest revision.

Apr 30 2016, 12:51 AM · Bug Report
epriestley added a comment to T10906: Differential StatusView icons for "rejected" vs "rejected older" are differentiated only by color.

I think it's maybe in the PHUIStatusView here?

Apr 30 2016, 12:51 AM · Bug Report
chad added a comment to T10906: Differential StatusView icons for "rejected" vs "rejected older" are differentiated only by color.

I can't think of where these are used without text or tooltip.

Apr 30 2016, 12:44 AM · Bug Report
epriestley added a comment to T10905: Cannot triple-click to select task title in phabricator (due to icon in the title).

Try that one instead.

Apr 30 2016, 12:00 AM · Bug Report

Apr 29 2016

epriestley closed T10905: Cannot triple-click to select task title in phabricator (due to icon in the title) as Resolved by committing Restricted Diffusion Commit.
Apr 29 2016, 11:59 PM · Bug Report
epriestley added a revision to T10905: Cannot triple-click to select task title in phabricator (due to icon in the title): Restricted Differential Revision.
Apr 29 2016, 11:56 PM · Bug Report
epriestley reopened T10905: Cannot triple-click to select task title in phabricator (due to icon in the title) as "Open".

Actually, this doesn't behave well with very long titles. Let me see if I can find an alternate fix.

Apr 29 2016, 11:51 PM · Bug Report
epriestley added a comment to T10905: Cannot triple-click to select task title in phabricator (due to icon in the title).

Thanks for the report. This should be fixed in HEAD of master, and should promote to stable within 24 hours. Let us know if you run into anything else.

Apr 29 2016, 11:43 PM · Bug Report
epriestley closed T10905: Cannot triple-click to select task title in phabricator (due to icon in the title) as Resolved by committing Restricted Diffusion Commit.
Apr 29 2016, 11:43 PM · Bug Report
epriestley added a revision to T10905: Cannot triple-click to select task title in phabricator (due to icon in the title): Restricted Differential Revision.
Apr 29 2016, 11:40 PM · Bug Report
epriestley added a comment to T10906: Differential StatusView icons for "rejected" vs "rejected older" are differentiated only by color.

Can you point us at where in the UI these are used, exactly? Just want to make sure we're fixing the right thing.

Apr 29 2016, 11:32 PM · Bug Report