Page MenuHomePhabricator
Feed Advanced Search

Mar 18 2015

rbalik added a comment to T7620: git unable to pull after upgrading to latest.

In case it's helpful, these are my apache logs when trying to do a pull:

104.130.174.45 - - [18/Mar/2015:17:25:05 +0000] "GET /diffusion/API/konekt-rest-api.git/info/refs?service=git-upload-pack HTTP/1.1" 401 2384 "-" "git/1.9.1"
104.130.174.45 - - [18/Mar/2015:17:25:09 +0000] "GET /diffusion/API/konekt-rest-api.git/info/refs?service=git-upload-pack HTTP/1.1" 401 555 "-" "git/1.9.1"
104.130.174.45 - reuben [18/Mar/2015:17:25:09 +0000] "GET /diffusion/API/konekt-rest-api.git/info/refs?service=git-upload-pack HTTP/1.0" 200 1179 "-" "git/1.9.1"
Mar 18 2015, 5:26 PM · Diffusion
rbalik created T7620: git unable to pull after upgrading to latest.
Mar 18 2015, 5:00 PM · Diffusion

Mar 17 2015

joshuaspence added a project to T7604: Remove "Arcanist Projects": Diffusion.
Mar 17 2015, 1:15 PM · Diffusion
joshuaspence created T7603: Remove the deprecated "Repositories" application.
Mar 17 2015, 1:13 PM · Repositories, Diffusion

Mar 14 2015

epriestley added a comment to T7537: In Branches Table also provide a link to the full repository.

There's (intentionally) no natural way to get to /repository/<branch>/. It's reachable by clicking the crumbs from /browse/<branch>/.

Mar 14 2015, 9:20 PM · Diffusion
chad added a comment to T7537: In Branches Table also provide a link to the full repository.

I couldn't figure out how to navigate to this page naturally, to be honest.

Mar 14 2015, 9:18 PM · Diffusion
epriestley added a comment to T7537: In Branches Table also provide a link to the full repository.

Why do you want to go to /repository/ instead of /browse/? See:

Mar 14 2015, 8:54 PM · Diffusion

Mar 12 2015

Mnkras renamed T7537: In Branches Table also provide a link to the full repository from In Branches Table also provide a like to the full repository to In Branches Table also provide a link to the full repository.
Mar 12 2015, 4:23 PM · Diffusion
chad updated subscribers of T5029: Fill in missing context for low-context diffs against tracked repositories.
Mar 12 2015, 6:18 AM · KDE, Restricted Project, Diffusion, Differential
chad renamed T7537: In Branches Table also provide a link to the full repository from In diffusion: http: //192.168.108.60/diffusion/MTF/, we click on the name of the branch 6.0_Story_1, we hope to enter the http://192.168.108.60/diffusion/MTF/repository/6.0_Story_1/, but entered http://192.168.108.60/diffusion/MTF/browse/6.0_Story_1/ to In Branches Table also provide a like to the full repository.
Mar 12 2015, 4:46 AM · Diffusion
chad added a comment to T7537: In Branches Table also provide a link to the full repository.

I understand, thanks for the clarification.

Mar 12 2015, 4:46 AM · Diffusion
• zhangzhenli added a comment to T7537: In Branches Table also provide a link to the full repository.

Yes,this is indeed a feature request.

Mar 12 2015, 4:39 AM · Diffusion
chad added a comment to T7537: In Branches Table also provide a link to the full repository.

Overall this sounds like a feature request, but I don't understand what /repository/ to you means in the URL.

Mar 12 2015, 2:31 AM · Diffusion
chad added a comment to T7537: In Branches Table also provide a link to the full repository.

I'm having trouble understanding your bug. Can you describe your current setup in more detail, for example what type of repository is this? Is this a feature request or do you think is is a bug? Does the issue reproduce on this server? Is your installation up to date? Can you provide screenshots to further explain your issue?

Mar 12 2015, 2:30 AM · Diffusion
• zhangzhenli created T7537: In Branches Table also provide a link to the full repository.
Mar 12 2015, 1:45 AM · Diffusion

Mar 7 2015

zzh added a comment to T7471: Unable to connect to SVN repo with invalid SSL cert.

And the Subversion server is install over other machine, I'm not the administrator

Mar 7 2015, 10:00 AM · Subversion, Diffusion
zzh added a comment to T7471: Unable to connect to SVN repo with invalid SSL cert.

I have add the ssl certs like "https://192.168.205.170/rZZH34382", but also have the same problem:

Mar 7 2015, 9:56 AM · Subversion, Diffusion

Mar 6 2015

Mnkras renamed T7471: Unable to connect to SVN repo with invalid SSL cert from Diffusion > Repository, SVN file cat has a problem to Unable to connect to SVN repo with invalid SSL cert.
Mar 6 2015, 3:30 PM · Subversion, Diffusion
Mnkras added a comment to T7471: Unable to connect to SVN repo with invalid SSL cert.

Take a look under the bold heading "Method": http://www.microhowto.info/howto/configure_subversion_to_trust_a_given_ssl_certificate.html

Mar 6 2015, 3:27 PM · Subversion, Diffusion
zzh added a comment to T7471: Unable to connect to SVN repo with invalid SSL cert.

how to set phabricator host ignore invalid ssl certs?

Mar 6 2015, 3:21 PM · Subversion, Diffusion
Mnkras added a comment to T7471: Unable to connect to SVN repo with invalid SSL cert.

Server SSL certificate verification failed: certificate issued for a different hostname, issuer is not trusted
Im pretty sure that is the issue, either connect over an unsecured connection, generate a valid certificate for the hostname, or have your phabricator host ignore invalid ssl certs

Mar 6 2015, 3:05 PM · Subversion, Diffusion
epriestley added a comment to T7472: Search repositories using a dedicated index for performance (and SVN support).

We could also implement a very sketchy version of this by maintaining an SVN working copy on disk and running normal grep against it. This is terrible, but maybe only moderately terrible.

Mar 6 2015, 1:01 PM · Diffusion
epriestley updated the task description for T7472: Search repositories using a dedicated index for performance (and SVN support).
Mar 6 2015, 12:41 PM · Diffusion
epriestley renamed T7472: Search repositories using a dedicated index for performance (and SVN support) from Full text svn search to Search repositories using a dedicated index for performance (and SVN support).
Mar 6 2015, 12:15 PM · Diffusion
epriestley updated subscribers of T7472: Search repositories using a dedicated index for performance (and SVN support).
Mar 6 2015, 12:13 PM · Diffusion
chad added projects to T7471: Unable to connect to SVN repo with invalid SSL cert: Diffusion, Subversion.
Mar 6 2015, 3:54 AM · Subversion, Diffusion
chad added a project to T7472: Search repositories using a dedicated index for performance (and SVN support): Subversion.
Mar 6 2015, 3:54 AM · Diffusion
chad removed a subtask for T4070: Feature Request: full text search of repositories: T7472: Search repositories using a dedicated index for performance (and SVN support).
Mar 6 2015, 3:44 AM · Diffusion
chad removed a parent task for T7472: Search repositories using a dedicated index for performance (and SVN support): T4070: Feature Request: full text search of repositories.
Mar 6 2015, 3:44 AM · Diffusion
chad placed T7472: Search repositories using a dedicated index for performance (and SVN support) up for grabs.
Mar 6 2015, 3:43 AM · Diffusion
jcdoll created T7472: Search repositories using a dedicated index for performance (and SVN support).
Mar 6 2015, 3:26 AM · Diffusion
epriestley closed T5179: Unearth Differential rendering technical debt, a subtask of T5644: Add controls to prevent `pygmentize` from saturating system resources, as Resolved.
Mar 6 2015, 12:31 AM · Differential, Diffusion

Mar 5 2015

chad renamed T7469: Handle folder copies/moves more gracefully from Handle folder copies/moves more gracefully. to Handle folder copies/moves more gracefully.
Mar 5 2015, 11:10 PM · Diffusion, Subversion
• sshannin updated the task description for T7469: Handle folder copies/moves more gracefully.
Mar 5 2015, 11:08 PM · Diffusion, Subversion
• sshannin created T7469: Handle folder copies/moves more gracefully.
Mar 5 2015, 11:08 PM · Diffusion, Subversion

Mar 4 2015

epriestley moved T5644: Add controls to prevent `pygmentize` from saturating system resources from Backlog to v2 (UI/Mobile) on the Differential board.
Mar 4 2015, 5:17 PM · Differential, Diffusion
epriestley added a project to T5644: Add controls to prevent `pygmentize` from saturating system resources: Differential.
Mar 4 2015, 5:17 PM · Differential, Diffusion
epriestley moved T2225: Render only the first 1,000 changed paths by default when viewing a commit in Diffusion from Backlog to v2 (UI/Mobile) on the Differential board.
Mar 4 2015, 5:16 PM · Differential, Diffusion
epriestley moved T2183: unify "table of contents" views between Differential and Diffusion from Backlog to v2 (UI/Mobile) on the Differential board.
Mar 4 2015, 5:16 PM · Owners, Differential, Diffusion
epriestley closed T7456: Syntax highlighting for VHDL files not working as Resolved by committing Restricted Diffusion Commit.
Mar 4 2015, 4:51 PM · Diffusion
timor added a comment to T7456: Syntax highlighting for VHDL files not working.

Adding the file type to syntax.filemap worked for me.
Thanks for the quick fix!

Mar 4 2015, 4:28 PM · Diffusion
epriestley added a comment to T7456: Syntax highlighting for VHDL files not working.
  • You can use syntax.filemap to configure how files are highlighted.
  • The attached diff adds support for VHDL by default.
Mar 4 2015, 3:41 PM · Diffusion
epriestley added a revision to T7456: Syntax highlighting for VHDL files not working: Restricted Differential Revision.
Mar 4 2015, 3:37 PM · Diffusion
timor created T7456: Syntax highlighting for VHDL files not working.
Mar 4 2015, 2:42 PM · Diffusion
epriestley added a comment to T6887: Emails sent for historical commits when importing repo to Diffusion.

See additional discussion in T6516. This is not fixed until D11964. The root issue is a discrepancy between the web and Conduit workflows which I missed until it did bad things.

Mar 4 2015, 2:27 PM · Wikimedia, Audit, Diffusion
epriestley added a comment to T6516: Diffusion pings accidentally tasks because the same task numbers are mentioned in cloned commit messages .

Those two patches should generally prevent problems here. With those applied, the behavior would have been:

Mar 4 2015, 2:24 PM · Wikimedia, Diffusion
epriestley added a revision to T6516: Diffusion pings accidentally tasks because the same task numbers are mentioned in cloned commit messages : Restricted Differential Revision.
Mar 4 2015, 2:16 PM · Wikimedia, Diffusion
epriestley added a revision to T6516: Diffusion pings accidentally tasks because the same task numbers are mentioned in cloned commit messages : Restricted Differential Revision.
Mar 4 2015, 12:53 PM · Wikimedia, Diffusion
epriestley added a comment to T6516: Diffusion pings accidentally tasks because the same task numbers are mentioned in cloned commit messages .

Some related discussion in T6887.

Mar 4 2015, 12:46 PM · Wikimedia, Diffusion
qgil added a comment to T6516: Diffusion pings accidentally tasks because the same task numbers are mentioned in cloned commit messages .

Today Wikimedia Phabricator has been hit badly when we started importing the Phabricator repository (having forgotten about this problem). We stopped the import as soon as we saw our mailboxes flooded with notifications of commits and tasks marked as resolved.

Mar 4 2015, 9:39 AM · Wikimedia, Diffusion
alex.n.james added a comment to T7449: Support ANSI escaped color sequences in Paste.

Sorry, maybe this isn't a diffusion issue. The issue is with whatever the
URL takes you to which results from arc paste.

Mar 4 2015, 2:55 AM · Paste

Mar 3 2015

epriestley renamed T7449: Support ANSI escaped color sequences in Paste from [feature request] Support ANSI escaped color sequences in diffusion to Support ANSI escaped color sequences in diffusion.
Mar 3 2015, 11:50 PM · Paste
epriestley added a comment to T7449: Support ANSI escaped color sequences in Paste.

To clarify, you're checking files which contain ANSI color sequences into a repository, and then viewing them in the repository browser in Diffusion?

Mar 3 2015, 11:50 PM · Paste
alex.n.james renamed T7449: Support ANSI escaped color sequences in Paste from [feature request] Support ANSI escaped color sequences in arc paste and diffusion to [feature request] Support ANSI escaped color sequences in diffusion.
Mar 3 2015, 11:45 PM · Paste
alex.n.james created T7449: Support ANSI escaped color sequences in Paste.
Mar 3 2015, 11:44 PM · Paste
epriestley closed T6887: Emails sent for historical commits when importing repo to Diffusion as Resolved.

I believe this was fixed by rP8599145 and rPf6915a7. Specifically:

Mar 3 2015, 11:44 AM · Wikimedia, Audit, Diffusion
devurandom updated the task description for T7339: Raise a setup warning when the "en_US.UTF-8" locale is unavailable.
Mar 3 2015, 9:54 AM · Diffusion (v3)
devurandom added a comment to T7339: Raise a setup warning when the "en_US.UTF-8" locale is unavailable.

@epriestley: Any progress with the config?

Mar 3 2015, 9:52 AM · Diffusion (v3)

Mar 2 2015

epriestley closed T7191: Can't import repositories on local machine as Invalid.

We don't have enough information to reproduce this.

Mar 2 2015, 3:13 PM · Diffusion

Feb 28 2015

techdragon added a comment to T4245: Make repository URIs slightly prettier.

After D11050 and rPf33e2de09286 what work remains to be done to provide this? I'd love to see this completed.

Feb 28 2015, 1:11 PM · Prioritized, Diffusion (v3), KDE, Restricted Project
joshuaspence claimed T1751: Consider ways to improve revert handling.
Feb 28 2015, 12:09 PM · Restricted Project, Differential, Diffusion

Feb 27 2015

epriestley closed T7315: Some VCS commands may bleed upward out of instances to affect `core/` as Invalid.

Something changed an origin URL for me locally, but I can't figure out how it happened so it might have been a fluke and/or me being dumb.

Feb 27 2015, 12:50 AM · Phacility, Diffusion
epriestley claimed T7315: Some VCS commands may bleed upward out of instances to affect `core/`.

bleh ugh yeuch

Feb 27 2015, 12:30 AM · Phacility, Diffusion

Feb 26 2015

joshuaspence added a project to T6803: Diffusion: sudo rules documentation refinement: Diffusion.
Feb 26 2015, 1:05 AM · Diffusion, Documentation

Feb 25 2015

chad added a comment to T7047: Add user preference for diffs in emails.

Adding T7378 here, any resolution of this task should include "diffs" in other applications as well (like Phriction).

Feb 25 2015, 10:04 PM · Restricted Project, Differential, Mail, Diffusion
chad updated subscribers of T7047: Add user preference for diffs in emails.
Feb 25 2015, 10:02 PM · Restricted Project, Differential, Mail, Diffusion
epriestley lowered the priority of T7375: Replace `hg locate` with `hg files` if `hg` is 3.2 or newer from Normal to Low.

Specifically, this isn't causing any problems yet (hg locate still works in modern hg at the time of writing), but at some point hg locate is presumably going to vanish and then we will have a problem.

Feb 25 2015, 4:15 PM · Diffusion, Mercurial
epriestley created T7375: Replace `hg locate` with `hg files` if `hg` is 3.2 or newer.
Feb 25 2015, 4:13 PM · Diffusion, Mercurial
epriestley claimed T7339: Raise a setup warning when the "en_US.UTF-8" locale is unavailable.

Sorry, my fault -- I only gave the VCS user sudo access to the git binaries when I set things up. I'll update the config when I have a chance.

Feb 25 2015, 4:00 PM · Diffusion (v3)
devurandom added a comment to T7339: Raise a setup warning when the "en_US.UTF-8" locale is unavailable.
In T7339#98164, @chad wrote:

rHGTEST exists for you to test our server / repository for the issue.

Feb 25 2015, 3:58 PM · Diffusion (v3)
joshuaspence added a subtask for T2453: Populate the blame cache when parsing commits: T2450: Build a blame cache.
Feb 25 2015, 9:49 AM · OS Students 2013, Diffusion
joshuaspence added a parent task for T2450: Build a blame cache: T2453: Populate the blame cache when parsing commits.
Feb 25 2015, 9:49 AM · Diffusion

Feb 24 2015

chad added a comment to T7339: Raise a setup warning when the "en_US.UTF-8" locale is unavailable.

rHGTEST exists for you to test our server / repository for the issue.

Feb 24 2015, 4:18 PM · Diffusion (v3)

Feb 20 2015

devurandom added a comment to T7339: Raise a setup warning when the "en_US.UTF-8" locale is unavailable.

Yep, I think after my initial try with the commit right on the default (aka master) branch, which arc land apparently dispises, I think I mostly hit Mercurial specific issues (landing bookmarks and --merge not being supported). I created T7342 on the documentation issue to not derail this task here.

Feb 20 2015, 8:20 PM · Diffusion (v3)
chad added a comment to T7339: Raise a setup warning when the "en_US.UTF-8" locale is unavailable.

arc land --help or https://secure.phabricator.com/book/phabricator/article/arcanist_diff/

Feb 20 2015, 7:00 PM · Diffusion (v3)
devurandom added a comment to T7339: Raise a setup warning when the "en_US.UTF-8" locale is unavailable.

Almost. hg generates bad commits (using nano or not), arc generates good diffs (using nano).

Feb 20 2015, 6:51 PM · Diffusion (v3)
chad added a comment to T7339: Raise a setup warning when the "en_US.UTF-8" locale is unavailable.

So to summarize, hg and nano generated commits = bad. arc = good?

Feb 20 2015, 6:13 PM · Diffusion (v3)
devurandom added a comment to T7339: Raise a setup warning when the "en_US.UTF-8" locale is unavailable.
  • Not using nano results in the same problem.
$ touch test
$ hg add test
$ hg commit -m 'Ümläüt Test'
Feb 20 2015, 6:10 PM · Diffusion (v3)
chad added a comment to T7339: Raise a setup warning when the "en_US.UTF-8" locale is unavailable.

We are going to set up test git, svn, and hg repositories on here for users to test diffs / commits against.

Feb 20 2015, 6:07 PM · Diffusion (v3)
chad added a project to T7339: Raise a setup warning when the "en_US.UTF-8" locale is unavailable: Diffusion.
Feb 20 2015, 5:34 PM · Diffusion (v3)
cspeckmim added a comment to T6118: Support for Mercurial extensions in hosted repositories.

Actually, changing the largefile extension from being defined in:
/etc/mercurial/hgrc.d/hgrc
instead moving to:
/etc/mercurial/hgrc.d/largefiles.rc

Feb 20 2015, 12:04 AM · Mercurial, Diffusion

Feb 19 2015

cspeckmim added a comment to T6118: Support for Mercurial extensions in hosted repositories.

We also utilize the largefiles extension as our repositories tend to have have numerous images, jars, other resources.

Feb 19 2015, 9:50 PM · Mercurial, Diffusion
jesseendahl added a comment to T5590: Change diffusion.allow-http-auth into an application policy and improve security features.

My mistake, I thought VCS password was referring to setting a password for the "vcs user," and then sharing that password with anyone who wanted to clone repos over HTTPS. I'm glad that's not how it works =)

Feb 19 2015, 8:53 PM · Diffusion, Security
epriestley added a comment to T5590: Change diffusion.allow-http-auth into an application policy and improve security features.

Maybe I'm misunderstanding, but per-user auth is already supported via HTTPS (Settings > VCS Password). Each user gets a username/password. Or do you mean something else?

Feb 19 2015, 8:03 PM · Diffusion, Security
jesseendahl added a comment to T5590: Change diffusion.allow-http-auth into an application policy and improve security features.

In addition to the things mentioned above, it'd be great if some form of per-user auth was supported via HTTPS. Perhaps this could be done by utilizing the existing user specific arc certificate, or generating a similar secondary certificate for the sole purpose of cloning repos.

Feb 19 2015, 8:02 PM · Diffusion, Security
epriestley closed T7298: Don't clog feed with old commits when importing a repo as Resolved by committing Restricted Diffusion Commit.
Feb 19 2015, 6:38 PM · Phacility, Diffusion
epriestley closed T6840: Repository imported with some commits missing Herald step as Resolved by committing Restricted Diffusion Commit.
Feb 19 2015, 6:36 PM · Phacility, Diffusion
epriestley closed T7310: Improve UX around bad repository credentials as Resolved by committing Restricted Diffusion Commit.
Feb 19 2015, 6:32 PM · Diffusion, Phacility
epriestley added a comment to T7298: Don't clog feed with old commits when importing a repo.

To surface the details of D11827 here for future users, once that lands we'll begin using a heuristic to detect whether a push is an import of an existing repository or the addition of new commits. At the time of writing, the rule identifies a push as an import if:

Feb 19 2015, 5:37 PM · Phacility, Diffusion
• sshannin added a comment to T7298: Don't clog feed with old commits when importing a repo.

Agreed on lack of value.

Feb 19 2015, 5:30 PM · Phacility, Diffusion
epriestley added a comment to T7310: Improve UX around bad repository credentials.

For posterity, see T6840 for the 99.99% issue.

Feb 19 2015, 5:29 PM · Diffusion, Phacility
epriestley merged task T7198: Surface repository syncing issues more prominently into T7310: Improve UX around bad repository credentials.
Feb 19 2015, 5:28 PM · Diffusion
epriestley updated subscribers of T7310: Improve UX around bad repository credentials.
Feb 19 2015, 5:28 PM · Diffusion, Phacility
epriestley added a comment to T7298: Don't clog feed with old commits when importing a repo.

I think the value of backdating feed is very small, and the current behavior is inconsistent anyway (not all commits get published). We could possibly consider backdating someday, but we'd probably want other use cases to support building the capability first.

Feb 19 2015, 5:27 PM · Phacility, Diffusion
epriestley updated subscribers of T6840: Repository imported with some commits missing Herald step.

@Mnkras, this is the issue you hit with the 99.9% repository.

Feb 19 2015, 5:25 PM · Phacility, Diffusion
epriestley closed T6017: Working copy check in Diffusion incorrectly reports that the repo has been deleted in Multiserver configuration as Resolved.

This was fixed by D11102.

Feb 19 2015, 5:25 PM · Conduit, Diffusion
epriestley added a revision to T7298: Don't clog feed with old commits when importing a repo: Restricted Differential Revision.
Feb 19 2015, 5:23 PM · Phacility, Diffusion
epriestley added a revision to T7298: Don't clog feed with old commits when importing a repo: Restricted Differential Revision.
Feb 19 2015, 5:18 PM · Phacility, Diffusion
epriestley added a revision to T6840: Repository imported with some commits missing Herald step: Restricted Differential Revision.
Feb 19 2015, 3:41 PM · Phacility, Diffusion
epriestley added a revision to T6840: Repository imported with some commits missing Herald step: Restricted Differential Revision.
Feb 19 2015, 3:32 PM · Phacility, Diffusion