Merge branch 'clipboardjs' into master

This replaces clippy.sfw with Javascript for issue #1241

Remove clippy.swf and update documentation

The `clippy.swf` Flash program is no longer needed and can be deleted.
The configuration property is now incorrectly named, but we keep the
name and update the documentation. Maybe it could be completely deleted
one day, when the clipboard.js solution is known to work and be universally
supported.

Replace SWF clippy with clipboard.js on ticket page

The ticket page also has some copy-to-clipboard buttons, which get
updated to work with JS instead of SWF.

Add a "Copied" tooltip to the copy-to-clipboard button

This is not the ideal version, since the height is too low for the tooltip
used for the drop-down menus. Probably has something to do with the
container or something. But at least something is there now, even if
not the most beautiful.

Replace SWF clippy with clipboardjs on repository page

Shockwave Flash is dead. But Gitblit still uses it to copy the repository
URLs to the clip board. Which doesn't work anymore since no browser uses
Flash anymore, so this has degraded disgracefully.

Instead, we can use JavaScript to copy directly to the clipboard, now
that there are APIs for it. So replace the use of clippy.swf on the
repository page with clipboard.js[1]. This right now only has the
functionality to copy to clipboard but now visual feedback, yet.

This addresses GH issue #1241.

[1] https://clipboardjs.com

Fix issue with not serialisable ed25519 SSH keys

Adding Ed25519 keys brings the problem that with the library currently
used, the PublicKey instance of that key is not serialisable. This
results in an exception when wicket tries to cache the UsersPage.
So change the SshKeysPanel so that the PublicKey object is removed
from the `SshKey` when the panel is detached. It can be regenerated from
the raw key data.

deps: Update various dependencies that had known CVEs

deps: Update Jetty to version 9.4.49.v20220914

This updates Jetty to the latest 9.x version as of writing. The 9.x is
still running on Java 8. The update needs two code changes.

`SessionManager` was replaced with `SessionHandler`. This was documented
in the Jetty documentation.

Adding the `GitblitContext` to the `WebAppContext` will result in two
instances getting created, because the code was changed that prevents
instantiation the same listener class multiple times. (The second time
is when the web.xml is read.) Instead, it must be added to the servlet
handler of the `WebAppContext`. This results in properly adhering to the
changed internal startup flow.

Updating Jetty also resolves #1409.

fix: Fix jetty crashing when redirecting HTTP to HTTPS

Jetty 9.3 changed the `setHandler` on the ServletContextHandler to no
longer automatically detect SecurityHandler, SessionHandler, etc. It
simply passes on the setHandler request to the ContextHandler class
(with a warning logged). So make sure to explicitly use the method
`setSecurityHandler` to set the ContraintSecurityHandler responsible
for the http -> https redirection.

Merge branch 'jvanhercke-jgit-default-charset' into master

This branch mostly records the commit from PR #1253, so we have it in
the code base. The changes are immediately undone, because the update
to JGit version 4.8.0, as proposed in PR #1252, renders the whole
patching of JGit obsolete.

The update is done now, against reservations in that PR, because we
guess that potential issues have been found and fixed by now. JGit
is updated to the last version on the major version 4 line: 4.11.9.
Also, since upstream JGit is at v6 by now, more updates will have to
happen later on anyways.

To make the protection against illegal character sets complete, an
additional fix for Bugtraq is included, that protects against exceptions
from bugtraq should it encounter an `encoding` that cannot be handled.

deps: Update JGit to 4.11.9.201909030838-r and other dependencies

Update JGit, and also update other dependencies where the 4.11 JGit
version uses newer versions than we do:
commond-codec updated to 1.9
commons-compress updated to 1.15
gson updated to 2.8.2

bugtraq: Catch exceptions from bugtraq and show message

Catch all exceptions, and not just IOExceptions, from bugtraq formatter.
If an exception is caught, ignore the bugtraq handling of the commit
message and show the plain message. Way better then not showing anything
just because something broke in bugtraq.

bugtraq: Fallback to UTF-8 if commit encoding is unsupported

Reading the encoding of a commit can result in a Unsupported- or
IllegalCharsetException. This happens when for whatever reason the
commit has an encoding recorded that the system doesn't understand.
Instead of completely failing, fallback to UTF-8.

deps: Update JGit to version 4.8.0.201706111038-r

Remove workaround for JGit crashing on 'utf-9' etc

Updating JGit fixed the issue that a commit in a repo with an unknown
character set throws an exception. This would crash the RepositoryManager.
The extra handling, which patches JGit classes during runtime is completely
removed.

Add wrapper class to return a default encoding

Unknown encodings may cause gitblit to fail to start. This modification
injects a wrapper class in the JGit internal to fake a valid return value.

ci: Delete nightly build artefact after testing the container

Leave the artefact until after the container has been built and tested.
In case the test fails the artefact can be still downloaded to analyse
what might have caused the container build or test to fail.

Update Bugtraq to v0.4

Sync with upstream @ https://github.com/mstrap/bugtraq

Merge branch 'pr-1214' into master

Fix conflict from earlier JGit update to 4.5
Add updated Eclipse and IntelliJ files.

Fix jgit version to one that is found in Maven Central

Merge branch 'pr-1213' into master

Fix conflict in updated slf4j version.
Add updated Eclipse and IntelliJ files.

Add missing jetty-xml dependency.

Merge branch 'sshkey_form_feedback' into master

This brings back in a change that was implemented in a pull request
from 2017, but got lost in the chaos of multiple pull requests from
intermingling branches.

This does not only provide feedback when a SSH key cannot be parsed,
but it also does so in a way that the warning goes away when a correct
key is added. Admittedly, I have no idea how to properly do this with
a Wicket FeedbackMessage, all I could find on Google was highly
complicated.

Not only does this bring back (or really in) the fix for issue #1226,
but it also fixes #984.

Replace key feedback with label instead of FeedbackMessage

Since it is a pest to get rid of a Wicket FeedbackMessage in an AJAX target,
change the code to use an extra label that can provide feedback if the
key could not be parsed or was empty.

Add feedback to SSH Key Form. #1226

if key is empty of can not be parsed, form did provide any feedback to
user before

Fix paths after update of checkout action

The Github 'checkout' action was updated from v1 to v3. But the behaviour
changed between the two which broke this workflow.
The old action would clone other repositories not into the workspace
but parallel to the workspace. The new version does every clone/checkout
relative to the workspace. That means that where previously the
gitblit-docker repository would be cloned in parallel to the workspace
which is gitblit/gitblit, it is now cloned into the gitblit/gitblit
directory path.

So remove all the references to `../gitblit-docker`. The files are in
the current directory now.

Merge pull request #1435 from flaix/fix-authority-np

Fix crash in Gitblit Authority for deleted users

authority: Fix null pointer crash for deleted users

When a user had a certificate, i.e. an entry in the Gitblit Authority
database, but the user was deleted from the Gitblit database, then the
Authority application crashes upon loading. This patch prevents the
crash. The deleted user is no longer shown in the Authority. But the
database entry still is kept. This should be improved to show deleted
users and give the possibility to delete them from the Authority's
database.

This fixes #1359

Merge pull request #1434 from flaix/update-guice

Update Guice version to 5.1.0

deps: Update Guice version to 5.1.0

Update Guice to 5.1.0. This version is compatible with Java 17.
The gitblit patch of the servlet extension was ported to Guice 5.1.0,
too.
The update of Guice requires an update of the Guava version, too.
Thus Guava is updated to 27.0.1-jar.

Merge pull request #1433 from flaix/np-fix

Fix null pointer exception in FileSettings

fix: Fix null pointer exception in FileSettings

Fix a null pointer access in the `toString` method for a freshly created
`FileSettings`.

Merge pull request #1430 from flaix/update-actions

Update workflows due to deprecations in actions

ci: Replace set-output command with echo to GITHUB_OUTPUT

The `set-output` command was deprecated. The new way to set an output
parameter for a step is to echo to the file stored in `$GITHUB_OUTPUT`

ci: Update action versions in nightly build workflow

Versions are updated for various actions that have a new version which
uses NodeJS 16.

One action has no updated version available yet:
`e1himself/goss-installation-action`
But an issue exists mentioning that it needs an update.

ci: Update action versions in push build workflow

The workflow runs started issuing warnings because NodeJS 12 was
deprecated. Actions need to get updated to newer versions that use
NodeJS 16. This commit updates the `actions/checkout` and the
`actions/setup-java` actions to the latest version in the workflow that
builds on every push.

The new setup-java action requires the distribution to be specified,
since multiple JDK distributions are supported now. We chose Eclipse's
Temurin here which is the successor to AdoptOpenJDK, which we use for
development.

ci: Build pull requests, too

Build pull requests, just like we build on every push.

Merge pull request #1429 from flaix/ssh-host-algs

Add new SSH host key types

Skip SSH host key files that do not exist

Since we now do not generate a DSA host key file anymore, but keep it in
the list of potential keys so that existing keys still work, it can
happen that the files for DSA (and Ed25519) are getting loaded but they
do not exist. This results in an error in the log.
So instead check if the file exists and only try to load files that
exist. This prevents from errors (which are none) being spammed in the
log.

Add SSH host keys with ECDSA and Ed25519

Create new host keys, one with ECDSA and one with Ed25519 algorithms.
For the Ed25519 currently the EdDSA library from i2p is used. This
requires some quirks, compared to a modern BouncyCastle. But the SSHD
library used cannot use BouncyCastle yet for Ed25519.

No DSA key is generated anymore, but we still support existing ones.

format: Cleanup formatting and whitespaces

Update BouncyCastle to version 1.69

The version 1.69 is chosen instead of 1.70, because the moxie build
would not download the jars, trying to download `...1.7.jar` instead.

Three class deprecations are fixed. `PEMWriter` and `X509Extension`
are replaced with their drop-in replacements `JcaPEMWriter` and
`Extension`. The `PasswordFinder` deprecation note says that "it is
no longer used". It also was never used in Gitblit's code, so it is
removed from the key par provider class.

Merge pull request #1428 from urkle/fix-pt

Correct is/is not usage in pt.py to not be used with the value is a literal

Merge pull request #1427 from flaix/ssh-upate

Update SSHD to version 1.7.0 and add support for EdDSA user keys

Correct is/is not usage in pt.py to not be used with the value is a literal

- new python 3 versions warn/errors about this usage

Update dependency MINA SSHD to version 1.7.0

upgrade to sshd 1.6.0, add eddsa dependency, upgrade jgit to 4.1.2

(cherry picked from commit d8fbdda2ab3fa48e92bdf37399d4b75c48409c5c@rpardini:master)

# Conflicts:
# .classpath
# build.moxie
# src/test/java/com/gitblit/tests/SshUnitTest.java

Merge branch 'oddeirik-disable-client-certs' into master

Use existing setting but with new values

Instead of adding another setting and having to explain how the new one
and the existing `requireClientCertificates` setting are interdependent,
let's use the existing setting and add new values.

It is changed from a boolean to a string, with the values `required`,
`optional` and `disabled`. To keep backward compatibility with the old
values, the `true` value is mapped to `required` and the `false` value
is mapped to `optional`.

Merge branch 'disable-client-certs' of github.com:oddeirik/gitblit into oddeirik-disable-client-certs

Merge branch 'felazuris-fixpermdisplay' into master

This pulls in the rebased branch from PR #1100.

Rewrite fix without the additional List

This also removes the additional Set. What changes is that the order of
the repositories is kept, but the resulting list will have user and team
permissions intertwined.

Add init test for getting repository permissions

fix user effective permission display if users is in multiple groups with different permissions

Merge branch 'QuentinC-fix-tab-escape' into master

The contribution branch was rebased to current master.
This should close #1065 as merged.

Add more tests for tab expansion

These show more clearly what the code is supposed to do.

Fix: if statement without curly braces

Tab to space conversion testing

Changed tab to space conversion

Tabs are not always 4 spaces large. It completes the line to the 4th
character.

Create SECURITY.md

This closes #1418

doc: Merge release 1.9.3 info into releases.moxie

ci: Add debug logging to nightly build

For some reason the secret gate doesn't work and the main Gitblit
repo also attempts to deploy the nightly to Docker which must fail.

pages: Fix link for GitExtension, which moved to Github

doc: Fix download link for fedclient ad api tar

It was still pointing to Google Code.

This closes #1408

doc: Fix links for dependencies that moved from Google Code

Merge branch '1410-vulnerability-userdb' into master

Merge fix branch from 1.9 mainenance line into
master branch.

fix: Fix StoredUserConfig not escaping control characters

The `StoredUserConfig` only escaped the escape character, i.e. backslash.
But it does not escape control characters like tab or newline. This
introduces a vulnerability where an attacker can create new entries
in their user account and create new accounts.
In addition, other characters are also not properly handled. Field values
with a comment character need to be quoted. This only happens for the
`#` character and only when the value starts with it. Also the quote
is note escaped in values.

This change completely rewrites the `escape` method of `StoredUserConfig`.
It takes care of properly escaping characters that need escaping for the
git configuration file format.

This fixes #1410

fix: Fix StoredUserConfig handling null subsections

Te `StoredUserConfig` did not handle sections without a subsection.
When the subsection did not exist, i.e. was `null`, then the subsection
name would be set to the string "null". This is not how the config file
format works. It should create a `[SECTIONNAME]` entry instead.

This fix handles a `null` subsection correctly, by handling it as a
section without a subsection.

test: Add exploit test for config user service

Add unit tests for exploiting the email address or display name
in the config user service by using newlines in the values.

Reset build identifiers for next point release cycle

ide: Adjust IDE project files to javax.mail 1.5.6

Update javax.mail to 1.5.6

Merge branch 'pr-1219' into master

This merges rebased and enhanced pull request #1219

ci: Remove Java 7 build

build: Comment out option used for Java 7

doc: Update documentation for new Java 8 minimum

Update readme for new min java version

Bump min java support to 1.8

Fixes #1218

ci: Transition to ICO image labels

ci: Allow to force build a nightly version

Enable a switch for manual workflow triggering that overrides the check
if a new commits were made since the last build. If the input is set to
'true', a new build is run even when no new commits are present.

This is mainly so that changes to the gitblit-docker repo can be included
in builds. This could also be automated in the workflow, if need be.

ci: Test nightly docker image with Goss before pushing to registry

ci: Add workflow for a nightly build of docker image

Readme: Fix link to nightly docker image

The link to the nightly docker images points to the actual image layer,
which is removed when a new nightly is pushed.
Instead, we should point to the images page, filtering on the tag 'nightly'.

Reset build identifiers for next minor release cycle

Prepare 1.9.2 release

site: Fix awful HTML indentation

site: Old packages moved from Bintray to Cloudsmith

Since Bintray seized operation, the packages for versions 1.4.0 - 1.8.0
have been uploaded to Cloudsmith. Adjust the links appropriately.

An attribution is added to comply with Cloudsmith's OSS hosting policy.

build: Determine release version with simple grep

To determine what the release version is to be, use a simple grep over
the moxie.build file and cut off the `-SNAPSHOT` part. The moxie
tag command doesn't do anything else, either. But the problem is that
when using the moxie tag command on the release branch, where the
version has already been set to a release version, it would create
a second release version, like so: 1.9.2-1.

build: Allow for updating an existing draft release

When creating a release draft and uploading assets, provision for the case
that a draft release already exists. In that case, instead of creating a
release, the existing release (changelog) is edited and the assets are
deleted before the new built ones are uploaded.

This commit also introduces the `${gh.repo}` property in build.xml,
so that the Github project could be chosen dynamically. Not really needed,
to be honest, but, yeah, whatever.

build: Add target to determine and report release version

Build: create macro for creating the release process script

Refactor the creation of the release process script, from
`release.template` into a `macrodef`.
Add a new target `releaseScript`, so that the release script can be
created after the `tagRelease` target was run.
This is helpful if the `tagRelease` target is run by a different,
earlier process than the actual release creation process.

doc: Update release notes

Update japanese translation

Detect and report if running in container

To help with analysis, try to detect if the instance is running inside
a container. Some containers are detected, but this is probably not
exhaustive. At least a Docker container should be detectable.
Report in the runtime manager to the log if a container was detected.

run: Put variables in quotes in shell scripts

readme: Add shields for license and nightly docker date

pages: Fix repo-relative reference links in markdown docs

As with explicit links, also for reference links in markdown documents
which point to repository-relative files the links are broken. They do
not take the path to the repository into account.
This fix is related to commit b23269 which fixed issue #1358
for explicit links.

Merge branch 'fix-864' into master

Merge fix for issue #864.

bug: Fix double encoding links in Markdown/Wiki pages

When parsing Markdown or Wiki pages, links get URL encoded. This happened
twice for links to other documents. Once explicitly and once by Wicket
when it creates a `urlFor` the page. That results in multi-byte
characters getting percent escaped, and then the percent character again
getting percent escaped.
The explicit encoding looks like a forgotten left over, so it gets
removed from the code. The Wicket encoding is smarter anyways, knowing
what is path and what is parameter.

This fixes #864.

Add tests for MarkupProcessor