summaryrefslogtreecommitdiffstats
path: root/src/main/java/com/gitblit/wicket/pages/SummaryPage.java
Commit message (Collapse)AuthorAgeFilesLines
* Issue #1011: do not serialize JGit commit objectsTom2021-11-031-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | JGit commit objects are a recursive data structure; they have links to their parent commits. Serializing a JGit commit will try to recursively serialize all reachable ancestors as faras they have been loaded. If that ancestor chain is too long, a StackOverflowError is thrown during Wicket's page serialization if a page has a reference to sucha JGit commit. Fixed by making sure that pages o not contain references to JGit commits. Use the (existing) wrapper object RepositoryCommit instead. * RepositoryCommit has a transient reference to the JGit commit and reads the commit from the repository upon de-serialization. * RefModel is a similar case (JGit tags/branches may also have links to the commits they point to). Solved a bit differently by making it a pure data object by transferring the interesting data from the JGit object in the constructor. * Change DataViews instantiated with RevCommit to use RepositoryCommit instead. * Change inner anonymous DataViews to ensure they do not have a synthesized field referencing the "allRefs" map. Such a synthesized field would also get serialized, and then serialize JGit commits again. Finally, remove non-transient logger instances in Wicket classes. Those might lead to NotSerializableException. These StackOverflowErrors have been reported in several places since 2014: * https://groups.google.com/forum/#!topic/gitblit/GH1d8WSlR6Q * https://bugs.chromium.org/p/gerrit/issues/detail?id=3316 * https://groups.google.com/d/msg/repo-discuss/Kcl0JIGNiGk/0DjH4mO8hA8J * https://groups.google.com/d/msg/repo-discuss/0_P6A3fjTec/2kcpVPIUAQAJ * https://github.com/gitblit/gitblit/issues/1011 * https://github.com/tomaswolf/gerrit-gitblit-plugin/issues/21
* Enforce relaxed XSS filtering on markup documentsJames Moger2014-09-071-1/+1
|
* Fix skip summary metrics regression since introducing flotr2Ting2014-04-161-12/+14
|
* Switch to self-hosted flotr2 graphs from Google Chartstryan2014-04-091-37/+45
|
* Fix potential NPE on getting Markdown contentJames Moger2013-12-301-2/+5
| | | | Change-Id: Ie245da271ad1e7c90b1fab46832f9ecc7b5d297a
* Eliminate nearly all direct GitBlit singleton references in WicketJames Moger2013-11-261-9/+8
| | | | | | | | | | This is the first step towards modularization and injection. All direct references to the GitBlit singleton within the Wicket pages and panels have been replaced to proxy methods in the GitBlitWebApp singleton. There are still two Wicket classes which rely on the GitBlit singleton; those require manual instantiation (servlet 3). Change-Id: I0cdbbcf87959d590c446c99abb09c07c87c737bc
* Display common repository documents as tabs on the docs pageJames Moger2013-11-141-1/+1
| | | | Change-Id: I6935fa45598da847936748b474a3da47e94efaac
* README files are not shown on the summary page by defaultJames Moger2013-11-141-11/+17
| | | | Change-Id: I421a6b8f8c8eaa5d34b7629858de76fc96458cd3
* Refactor markup processing in preparation for supporting other formatsJames Moger2013-11-121-48/+11
| | | | Change-Id: I0eb217064abc4f4b0f6bfbbc21302c470cc2f9c6
* Strip Markdown extension from rendered [[WikiLink]] (issue-324)James Moger2013-10-251-1/+1
| | | | Change-Id: I2f695f5d5d72d8c08272981f756334b45398a1f7
* Support for intra-Markdown linking using [[WikiLinks]] syntax (issue-324)James Moger2013-10-251-2/+2
| | | | | | | | | All WikiLinks must be specified relative to the root of the repository. The displayed link text is stripped to just the document name. Spaces in the document name are replaced with '-' characters; this is consistent with wiki syntax and Github. Change-Id: Id3fb1b5441352d9bacc4993a5b85882db113693b
* Remove "show readme" flag in favor of automatic detectionJames Moger2013-10-251-29/+38
| | | | | | | Automatic detection also will now also display a plain text "readme" or "readme.txt" file. Change-Id: Id6be729bdc469e7a5cfd1f4144df340a6b93475e
* Replaced MarkdownPapers with pegdownJames Moger2013-10-181-2/+2
| | | | Change-Id: I11eb50ba1ef0bef8ac47bf6f7b17e0f79ecd3f2d
* Trim trailing whitespace and organize importsJames Moger2013-09-301-5/+6
| | | | Change-Id: I9f91138b20219be6e3c4b28251487df262bff6cc
* Implemented optional page cachingJames Moger2013-07-191-0/+3
|
* Set author as tooltip of "last change" on repositories page (issue-238)James Moger2013-07-031-1/+1
|
* Retrieve summary and metric charts over https (issue-61)James Moger2013-07-031-2/+2
|
* Strike-through missing owners on summary/overview pageJames Moger2013-06-171-2/+6
|
* Overhauled navigation and flattened lookJames Moger2013-05-191-1/+2
|
* Implemented application menus for repository url panelJames Moger2013-05-091-27/+1
|
* Uber-cool repository panel overhaulJames Moger2013-05-071-13/+0
|
* Partially merged url panel with optional support for app clone urlsJames Moger2013-05-031-29/+5
|
* Added SparkleShare invite url panelJames Moger2013-05-021-0/+10
| | | | | This will probably be merged into a refined, single multi-protocol url panel.
* Improved the repository url panel and show git daemon url, appropriatelyJames Moger2013-05-021-7/+27
| | | | | This will probably be refined some more to merge the http/https and git urls into a single url panel.
* Support username substitution in web.otherUrls (issue 213)James Moger2013-03-291-1/+3
|
* Reorganized to Apache Standard Directory Layout & integrated MoxieJames Moger2013-03-271-0/+238
This is a massive commit which reorganizes the entire project structure (although it is still monolithic), removes the Build classes, and switches to Moxie, a smarter Ant build tookit based on the original Gitblit Build classes. The Ant build script will likely require additional fine-tuning, but this is big step forward.