Merge pull request #4790 from WoltLab/package-server-tls

Enforce TLS for package servers

Merge branch 'master' into package-server-tls

Merge branch '5.5'

Merge pull request #4799 from WoltLab/wcf-force-logout

Immediately destroy the session for banned users

Merge pull request #4795 from WoltLab/mb-strpos

Remove unneeded usage of `mb_strpos`

Merge pull request #4786 from WoltLab/remove-x-frame-options

Always send `x-frame-options`

Merge pull request #4793 from WoltLab/error-500

Use HTTP 500 in error conditions

Merge pull request #4794 from WoltLab/wcfacp-initPackage

Remove obsolete WCFACP::initPackage()

Merge pull request #4796 from WoltLab/canonical-url-s-parameter

Remove the obsolete stripping of the `s=<sessionID>` when handling the canonical URL

Merge pull request #4788 from WoltLab/template-modifier-allowlist

Always restrict valid template modifiers to an allow list

Immediately destroy the session for banned users

The forced logout for banned users was introduced in
ab84d9cab2f864c23f0b18dbeb67e7ea79b1fe9f and only destroyed the session during
shutdown.

At the point where this check runs the request effectively is fully booted up
and in any case the NamedUserException would abort any further booting, thus it
is safe to simply destroy the session immediately to keep all the necessary
logic in a single location.

Use `->prepare()` instead of `->prepareStatement()` in SessionHandler

Merge branch '5.4' into 5.5

Merge pull request #4798 from pehbeh/patch-1

Update URL to WoltLab Community and Plugin-Store in acp/pageHeaderUser.tpl

Merge pull request #4797 from WoltLab/mysql-requirements

Increase minimum MySQL version to 8.0.29 / MariaDB 10.5.12

Increase minimum MySQL version to 8.0.29 / MariaDB 10.5.12

Update URL to WoltLab Community and Plugin-Store in acp/pageHeaderUser.tpl

Remove the obsolete stripping of the `s=<sessionID>` when handling the canonical URL

Remove unneeded usage of `mb_strpos`

If the return value of `mb_strpos` is only compared to `false` then there is no
need to use the multibyte engine, instead `str_contains()` does example the
same and is clearer.

The same applies if the return value is compared to `0`, in this case the size
of multibyte characters cannot have affected the offset. `str_starts_with()`
can be used instead.

Merge branch '5.5'

Fix incorrect use of `mb_strpos` in MysqlSearchEngine

The return value of `mb_strpos` needs to be checked type-safely, as both `0`
and `false` are falsy. In this case this likely was safe, as the inner join may
not appear at the start of the query, it was a questionable nonetheless.

Fix this by using `str_contains()` which makes the intent even clearer.

Simplify PackageUpdateServer::isTrustedServer()

Remove `UPDATE_SERVER_TRUSTED_MIRROR`

Merge remote-tracking branch 'origin/master'

Merge branch '5.5'

Merge branch '5.4' into 5.5

Update URL to WoltLab Ticket Support in acp/pageHeaderUser.tpl

Merge pull request #4792 from WoltLab/styleID-parameter

Remove the `$_REQUEST['styleID']` handling

Remove obsolete WCFACP::initPackage()

Ever since WoltLab Suite Core got its own frontend the PACKAGE_ID constant
should be defined at all times, making this method obsolete.

Remove the `$_REQUEST['styleID']` handling

Resolves #4533

Use HTTP 500 in error conditions

Sending a 500 Internal Server Error for unplanned errors is more appropriate
than a 503 Service Unavailable, as the latter is defined:

RFC 7231#6.6.4:

> The 503 (Service Unavailable) status code indicates that the server
> is currently unable to handle the request due to a temporary overload
> or scheduled maintenance, which will likely be alleviated after some
> delay.

It's not likely that the exception will resolve itself after some delay.

Simplify generation of random cookie prefix in WCFSetup

Drop unused constant `URL_TO_LOWERCASE` from default options.inc.php

The option was removed in f3aa502157efa11ac3c76e8471d69d0ac9f69f1f.

Merge pull request #4791 from WoltLab/wcfsetup-n

Remove the WCF_N input in WCFSetup

Remove the WCF_N input in WCFSetup

Users should install separate instances into separate databases for security
reasons. This also avoids issues with users running a non-standard number and
any existing guides / queries not working, because the database tables have a
different name.

Merge branch 'header-normalize'

Use `Header::normalize()` in `ImageUtil::browserSupportsWebp()`

see #4534

Use `Header::normalize()` in `RequestHandler::sendPsr7Response()`

see #4534

Merge branch '5.5'

Merge pull request #4784 from WoltLab/require-x64

Require 64-bit support

Merge pull request #4783 from WoltLab/ts-target

Target es2019 in tsconfig.json

Add update_com.woltlab.wcf_5.5_checkUpdateServers.php

Remove obsolete reordering of package servers in PackageUpdateDispatcher::refreshPackageDatabase()

Remove processing of the `wcf-update-server-ssl` header in PackageUpdateDispatcher

Remove calls to RemoteFile::supportsSSL()

Deprecate RemoteFile::supportsSSL() and RemoteFile::disableSSL()

Enforce the https scheme in PackageUpdateServer

Remove the https to http downgrade fallback in PackageUpdateDispatcher

Require package servers to use https on the default port in PackageUpdateServerAddForm

Update WoltLab update servers URLs to the `https` scheme in PackageUpdateServer

Adjust the default package server `serverURL` to use https in install.sql

Merge pull request #4787 from WoltLab/user-online-list

User online statistics were invisible when all active users were invisible

Merge branch '5.4' into 5.5

Merge branch '5.3' into 5.4

Add the `required` attribute to the recipientID select in contact.tpl

see a8490749c3ba7014380d55462fc45dd635c1d71c

Merge branch '5.4' into 5.5

Merge branch '5.3' into 5.4

Indicate that selecting a recipient is required in contact.tpl

Always restrict valid template modifiers to an allow list

Previously this allow list was only used in enterprise mode, but it is
generally a useful security feature. Thus the allow list previously known as
`$enterpriseFunctions` is applied in call cases.

This also makes it easier for developers, as there will be less differences
between the enterprise mode and the non-enterprise mode.

As before this allow list can easily be extended if a useful function is
missing from it.

Add 64-bit check to update_com.woltlab.wcf_5.5_checkSystemRequirements.php

Merge branch 'master' into require-x64

Add update_com.woltlab.wcf_5.5_checkSystemRequirements.php

see 48b47a4a8ba0260d52226c80063ebac081fa719b

Applied code suggestions

Show legend only when users are listed

Always send `x-frame-options`

Resolves #4484

User online statistics were invisible when all active users were invisible

Drop obsolete upgrade instructions from 5.4 to 5.5

Merge branch '5.5'

Merge branch '5.4' into 5.5

Drop obsolete update_com.woltlab.wcf_5.4.15_deleteDsStore.php

Drop obsolete fileDelete_5.5.xml

Check for 64-bit support in SystemCheckPage

Require 64-bit PHP in WCFSetup

see #4512

Merge branch '5.5'

Fix language items in recommended section of system requirements in WCFSetup

see 3445cbe2a005ead9843d9e17709a915631dd11b5
see e88d06dc88bc263b7424fbccfa47c13907413b8c

Target es2019 in tsconfig.json

Resolves #4595

Run php-cs-fixer using PHP 8.1

Merge pull request #4782 from WoltLab/system-requirements

Increase minimum PHP requirement

Release 5.5.0 Beta 2

Incorrect dialog position on smartphones

See https://www.woltlab.com/community/thread/295560-neues-thema-erstellen-au%C3%9Ferhalb-des-bildschirms/

Fix php.yml workflow

see 7cb8935ef0ef6707e0ed1cb8fe26ac17dd155400

Upgrade laminas/laminas-httphandlerrunner to 2.1.0

Tighten up version constraints in composer.json

Remove obsolete Symfony polyfills

Update PHP platform version in composer.json

Remove PHP < 8.1 from php.yml workflow

Increase minimum PHP version to 8.1.2

Add the 5.5 branch to GitHub workflows

Release 5.5.0 Beta 1

Removed the upgrade instructions from 5.4.*

Updating minified JavaScript files

Merge branch '5.4'

Fix English versions of `wcf.user.security.multifactor.backup.authenticationEmail.body.*`

The phrases contained broken template scripting, due to the use of the `'`
apostroph within a single quoted string.

Dialogs could become too wide with lots of text

Suppress the redundant loading indicator

See https://www.woltlab.com/community/thread/295539-doppelte-ladebalken/

Merge pull request #4781 from WoltLab/ci-check

Flip only the file name casing on AbstractFileDeletePackageInstallationPlugin::isFilesystemCaseSensitive()

Add fileDelete_5.5.xml

These files never were part of the git repository, but were accidentally
included in an upgrade package, because an unclean source directory was used
when building the upgrade.

Flip only the file name casing on AbstractFileDeletePackageInstallationPlugin::isFilesystemCaseSensitive()

Flipping the whole path does not provide any real benefit, because we only care
about whether the file system where WoltLab Suite resides is case sensitive or
not. It does however break when `open_basedir` is configured.

Merge branch '5.4'

Remove the codestyle workflow from branches that are not master