Merge pull request #2364 from shlinkio/develop

Release 4.4.3

.github/DISCUSSION_TEMPLATE/help-wanted.yml

@@ -0,0 +1,49 @@
+title: 'Help wanted'
+body:
+  - type: input
+    validations:
+      required: true
+    attributes:
+      label: Shlink version
+      placeholder: x.y.z
+  - type: input
+    validations:
+      required: true
+    attributes:
+      label: PHP version
+      placeholder: x.y.z
+  - type: dropdown
+    validations:
+      required: true
+    attributes:
+      label: How do you serve Shlink
+      options:
+        - Self-hosted Apache
+        - Self-hosted nginx
+        - Self-hosted RoadRunner
+        - Docker image
+        - Other (explain in summary)
+  - type: dropdown
+    validations:
+      required: true
+    attributes:
+      label: Database engine
+      options:
+        - MySQL
+        - MariaDB
+        - PostgreSQL
+        - MicrosoftSQL
+        - SQLite
+  - type: input
+    validations:
+      required: true
+    attributes:
+      label: Database version
+      placeholder: x.y.z
+  - type: textarea
+    validations:
+      required: true
+    attributes:
+      label: Summary
+      value: '<!-- Describe your issue, question or request here. -->'
+

.github/ISSUE_TEMPLATE.md

@@ -0,0 +1,7 @@
+<!--
+Before opening an issue, just take into account that this is a completely free of charge and open source project.
+I'm always happy to help and provide support, but some understanding will be expected.
+I do this in my own free time, so expect some delays when implementing new features and fixing bugs, and don't take it personally if an issue gets eventually closed.
+You may also be asked to provide tests or ways to reproduce reported bugs.
+Try to be polite, and understand it is impossible for an OSS project to cover all use cases.
+-->

.github/ISSUE_TEMPLATE/config.yml

@@ -2,4 +2,4 @@ blank_issues_enabled: true
 contact_links:
   - name: Question - Support
     about: Do you need help setting up or using Shlink?
-    url: https://github.com/orgs/shlinkio/discussions/new?category=help-wanted
+    url: https://github.com/shlinkio/shlink/discussions/new?category=help-wanted

.github/workflows/ci-db-tests.yml

@@ -13,11 +13,11 @@ jobs:
     runs-on: ubuntu-24.04
     strategy:
       matrix:
-        php-version: ['8.4', '8.5']
+        php-version: ['8.3', '8.4']
     env:
       LC_ALL: C
     steps:
-      - uses: actions/checkout@v5
+      - uses: actions/checkout@v4
       - name: Install MSSQL ODBC
         if: ${{ inputs.platform == 'ms' }}
         run: sudo ./data/infra/ci/install-ms-odbc.sh
@@ -35,8 +35,8 @@ jobs:
       - name: Run tests
         run: composer test:db:${{ inputs.platform }}
       - name: Upload code coverage
-        uses: actions/upload-artifact@v5
-        if: ${{ matrix.php-version == '8.4' && inputs.platform == 'sqlite:ci' }}
+        uses: actions/upload-artifact@v4
+        if: ${{ matrix.php-version == '8.3' && inputs.platform == 'sqlite:ci' }}
         with:
           name: coverage-db
           path: |

.github/workflows/ci-tests.yml

@@ -13,11 +13,11 @@ jobs:
     runs-on: ubuntu-24.04
     strategy:
       matrix:
-        php-version: ['8.4', '8.5']
+        php-version: ['8.3', '8.4']
     env:
       GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }} # rr get-binary picks this env automatically
     steps:
-      - uses: actions/checkout@v5
+      - uses: actions/checkout@v4
       - name: Start postgres database server
         if: ${{ inputs.test-group == 'api' }}
         run: docker compose -f docker-compose.yml -f docker-compose.ci.yml up -d shlink_db_postgres
@@ -32,8 +32,8 @@ jobs:
         if: ${{ inputs.test-group == 'api' }}
         run: ./vendor/bin/rr get --no-interaction --no-config --location bin/ && chmod +x bin/rr
       - run: composer test:${{ inputs.test-group }}:ci
-      - uses: actions/upload-artifact@v5
-        if: ${{ matrix.php-version == '8.4' }}
+      - uses: actions/upload-artifact@v4
+        if: ${{ matrix.php-version == '8.3' }}
         with:
           name: coverage-${{ inputs.test-group }}
           path: |

.github/workflows/ci.yml

@@ -27,10 +27,10 @@ jobs:
     runs-on: ubuntu-24.04
     strategy:
       matrix:
-        php-version: ['8.4']
+        php-version: ['8.3']
         command: ['cs', 'stan', 'openapi:validate']
     steps:
-      - uses: actions/checkout@v5
+      - uses: actions/checkout@v4
       - uses: './.github/actions/ci-setup'
         with:
           php-version: ${{ matrix.php-version }}
@@ -69,15 +69,16 @@ jobs:
     runs-on: ubuntu-24.04
     strategy:
       matrix:
-        php-version: ['8.4']
+        php-version: ['8.3']
     steps:
-      - uses: actions/checkout@v5
+      - name: Checkout code
+        uses: actions/checkout@v4
       - name: Use PHP
         uses: './.github/actions/ci-setup'
         with:
           php-version: ${{ matrix.php-version }}
           extensions-cache-key: tests-extensions-${{ matrix.php-version }}
-      - uses: actions/download-artifact@v6
+      - uses: actions/download-artifact@v4
         with:
           path: build
       - run: mv build/coverage-unit/coverage-unit.cov build/coverage-unit.cov
@@ -86,16 +87,16 @@ jobs:
       - run: mv build/coverage-cli/coverage-cli.cov build/coverage-cli.cov
       - run: vendor/bin/phpcov merge build --clover build/clover.xml
       - name: Publish coverage
-        uses: codecov/codecov-action@v5
+        uses: codecov/codecov-action@v4
         with:
-          files: ./build/clover.xml
+          file: ./build/clover.xml
 
   delete-artifacts:
     needs:
       - upload-coverage
     runs-on: ubuntu-24.04
     steps:
-      - uses: geekyeggo/delete-artifact@v5
+      - uses: geekyeggo/delete-artifact@v2
         with:
           name: |
             coverage-*

.github/workflows/publish-openapi-spec.yml

@@ -10,9 +10,9 @@ jobs:
     runs-on: ubuntu-24.04
     strategy:
       matrix:
-        php-version: ['8.4']
+        php-version: ['8.3']
     steps:
-      - uses: actions/checkout@v5
+      - uses: actions/checkout@v4
       - name: Determine version
         id: determine_version
         run: echo "version=${GITHUB_REF#refs/tags/}" >> $GITHUB_OUTPUT

.github/workflows/publish-release.yml

@@ -10,16 +10,16 @@ jobs:
     runs-on: ubuntu-24.04
     strategy:
       matrix:
-        php-version: ['8.4', '8.5']
+        php-version: ['8.3', '8.4']
     steps:
-      - uses: actions/checkout@v5
+      - uses: actions/checkout@v4
       - uses: './.github/actions/ci-setup'
         with:
           php-version: ${{ matrix.php-version }}
           extensions-cache-key: publish-swagger-spec-extensions-${{ matrix.php-version }}
           install-deps: 'no'
       - run: ./build.sh ${GITHUB_REF#refs/tags/v}
-      - uses: actions/upload-artifact@v5
+      - uses: actions/upload-artifact@v4
         with:
           name: dist-files-${{ matrix.php-version }}
           path: build
@@ -28,8 +28,8 @@ jobs:
     needs: ['build']
     runs-on: ubuntu-24.04
     steps:
-      - uses: actions/checkout@v5
-      - uses: actions/download-artifact@v6
+      - uses: actions/checkout@v4
+      - uses: actions/download-artifact@v4
         with:
           path: build
       - name: Publish release with assets
@@ -45,6 +45,6 @@ jobs:
     needs: ['publish']
     runs-on: ubuntu-24.04
     steps:
-      - uses: geekyeggo/delete-artifact@v5
+      - uses: geekyeggo/delete-artifact@v2
         with:
           name: dist-files-*

CHANGELOG.md

@@ -4,256 +4,6 @@ All notable changes to this project will be documented in this file.
 
 The format is based on [Keep a Changelog](https://keepachangelog.com), and this project adheres to [Semantic Versioning](https://semver.org).
 
-## [Unreleased]
-### Added
-* *Nothing*
-
-### Changed
-* [#2573](https://github.com/shlinkio/shlink/issues/2573) Update to PHPUnit 13
-
-### Deprecated
-* *Nothing*
-
-### Removed
-* *Nothing*
-
-### Fixed
-* *Nothing*
-
-
-## [5.0.0] - 2026-01-09
-### Added
-* [#2431](https://github.com/shlinkio/shlink/issues/2431) Add new date-based conditions for the dynamic rules redirections system, that allow to perform redirections based on an ISO-8601 date value.
-
-  * `before-date`: matches when current date and time is earlier than the defined threshold.
-  * `after-date`: matches when current date and time is later than the defined threshold.
-
-* [#2513](https://github.com/shlinkio/shlink/issues/2513) Add support for redis connections via unix socket (e.g. `REDIS_SERVERS=unix:/path/to/redis.sock`).
-* Visits generated in the command line can now be formatted in CSV, via `--format=csv`.
-
-### Changed
-* [#2522](https://github.com/shlinkio/shlink/issues/2522) Shlink no longer tries to detect trusted proxies automatically, when resolving the visitor's IP address, as this is a potential security issue.
-
-    Instead, if you have more than 1 proxy in front of Shlink, you should provide `TRUSTED_PROXIES` env var, with either a comma-separated list of the IP addresses of your proxies, or a number indicating how many proxies are there in front of Shlink.
-
-* [#2311](https://github.com/shlinkio/shlink/issues/2311) All visits-related commands now return more information, and columns are arranged slightly differently.
-
-    Among other things, they now always return the type of the visit, region, visited URL, redirected URL and whether the visit comes from a potential bot or not.
-
-* [#2540](https://github.com/shlinkio/shlink/issues/2540) Update Symfony packages to 8.0.
-* [#2512](https://github.com/shlinkio/shlink/issues/2512) Make all remaining console commands invokable.
-
-### Deprecated
-* *Nothing*
-
-### Removed
-* [#2507](https://github.com/shlinkio/shlink/issues/2507) Drop support for PHP 8.3.
-* [#2514](https://github.com/shlinkio/shlink/issues/2514) Remove support to generate QR codes. This functionality is now handled by Shlink Web Client and Shlink Dashboard.
-* [#2517](https://github.com/shlinkio/shlink/issues/2517) Remove `REDIRECT_APPEND_EXTRA_PATH` env var. Use `REDIRECT_EXTRA_PATH_MODE=append` instead.
-* [#2519](https://github.com/shlinkio/shlink/issues/2519) Disabling API keys by their plain-text key is no longer supported. When calling `api-key:disable`, the first argument is now always assumed to be the name.
-* [#2520](https://github.com/shlinkio/shlink/issues/2520) Remove deprecated `--including-all-tags` and `--show-api-key-name` options from `short-url:list` command. Use `--tags-all` and `--show-api-key` instead.
-* [#2521](https://github.com/shlinkio/shlink/issues/2521) Remove deprecated `--tags` option in all commands using it. Use `--tag` multiple times instead, one per tag.
-* [#2543](https://github.com/shlinkio/shlink/issues/2543) Remove support for `--order-by=field,dir` option `short-url:list` command. Use `--order-by=field-dir` instead.
-* Remove support to provide redis database index via URI path. Use `?database=3` query instead.
-* [#2565](https://github.com/shlinkio/shlink/issues/2565) Remove explicit dependency in ext-json, since it's part of PHP since v8.0
-
-### Fixed
-* [#2564](https://github.com/shlinkio/shlink/issues/2564) Fix error when trying to persist non-utf-8 title without being able to determine its original charset for parsing.
-
-  Now, when resolving a website's charset, two improvements have been introduced:
-
-  1. If the `Content-Type` header does not define the charset, we fall back to `<meta charset>` or `<meta http-equiv="Content-Type">`.
-  2. If it's still not possible to determine the charset, we ignore the auto-resolved title, to avoid other encoding errors further down the line.
-
-
-## [4.6.0] - 2025-11-01
-### Added
-* [#2327](https://github.com/shlinkio/shlink/issues/2327) Allow filtering short URL lists by those not including certain tags.
-
-    Now, the `GET /short-urls` endpoint accepts two new params: `excludeTags`, which is an array of strings with the tags that should not be included, and `excludeTagsMode`, which accepts the values `any` and `all`, and determines if short URLs should be filtered out if they contain any of the excluded tags, or all the excluded tags.
-
-    Additionally, the `short-url:list` command also supports the same feature via `--exclude-tag` option, which requires a value and can be provided multiple times, and `--exclude-tags-all`, which does not expect a value and determines if the mode should be `all`, or `any`.
-
-* [#2192](https://github.com/shlinkio/shlink/issues/2192) Allow filtering short URL lists by the API key that was used to create them.
-
-  Now, the `GET /short-urls` endpoint accepts a new `apiKeyName` param, which is ignored if the request is performed with a non-admin API key which name does not match the one provided here.
-
-  Additionally, the `short-url:list` command also supports the same feature via the `--api-key-name` option.
-
-* [#2330](https://github.com/shlinkio/shlink/issues/2330) Add support to serve Shlink with FrankenPHP, by providing a worker script in `bin/frankenphp-worker.php`.
-
-* [#2449](https://github.com/shlinkio/shlink/issues/2449) Add support to provide redis credentials separately when using redis sentinels, where provided servers are the sentinels and not the redis instances.
-
-    For this, Shlink supports two new env ras / config options, as `REDIS_SERVERS_USER` and `REDIS_SERVERS_PASSWORD`.
-
-* [#2498](https://github.com/shlinkio/shlink/issues/2498) Allow orphan visits, non-orphan visits and tag visits lists to be filtered by domain.
-
-    This is done via the `domain` query parameter in API endpoints, and via the `--domain` option in console commands.
-
-* [#2472](https://github.com/shlinkio/shlink/issues/2472) Add support for PHP 8.5
-* [#2291](https://github.com/shlinkio/shlink/issues/2291) Add `api-key:delete` console command to delete API keys.
-
-### Changed
-* [#2424](https://github.com/shlinkio/shlink/issues/2424) Make simple console commands invokable.
-
-### Deprecated
-* *Nothing*
-
-### Removed
-* *Nothing*
-
-### Fixed
-* *Nothing*
-
-
-## [4.5.3] - 2025-10-10
-### Added
-* *Nothing*
-
-### Changed
-* *Nothing*
-
-### Deprecated
-* *Nothing*
-
-### Removed
-* *Nothing*
-
-### Fixed
-* [#2488](https://github.com/shlinkio/shlink/issues/2488) Ensure `Access-Control-Allow-Credentials` is set in all cross-origin responses when `CORS_ALLOW_ORIGIN=true`.
-
-
-## [4.5.2] - 2025-08-27
-### Added
-* *Nothing*
-
-### Changed
-* *Nothing*
-
-### Deprecated
-* *Nothing*
-
-### Removed
-* *Nothing*
-
-### Fixed
-* [#2433](https://github.com/shlinkio/shlink/issues/2433) Try to mitigate memory leaks allowing RoadRunner to garbage collect memory after every request and every job, by setting `GC_COLLECT_CYCLES=true`.
-
-
-## [4.5.1] - 2025-08-24
-### Added
-* *Nothing*
-
-### Changed
-* *Nothing*
-
-### Deprecated
-* *Nothing*
-
-### Removed
-* *Nothing*
-
-### Fixed
-* [#2433](https://github.com/shlinkio/shlink/issues/2433) Try to mitigate memory leaks by restarting job and http workers every 250 executions when using RoadRunner.
-
-
-## [4.5.0] - 2025-07-24
-### Added
-* [#2438](https://github.com/shlinkio/shlink/issues/2438) Add `MERCURE_ENABLED` env var and corresponding config option, to more easily allow the mercure integration to be toggled.
-
-    For BC, if this env var is not present, we'll still consider the integration enabled if the `MERCURE_PUBLIC_HUB_URL` env var has a value. This is considered deprecated though, and next major version will rely only on `MERCURE_ENABLED`, so if you are using Mercure, make sure to set `MERCURE_ENABLED=true` to be ready.
-
-* [#2387](https://github.com/shlinkio/shlink/issues/2387) Add `REAL_TIME_UPDATES_TOPICS` env var and corresponding config option, to granularly decide which real-time updates topics should be enabled.
-* [#2418](https://github.com/shlinkio/shlink/issues/2418) Add more granular control over how Shlink handles CORS. It is now possible to customize the `Access-Control-Allow-Origin`, `Access-Control-Max-Age` and `Access-Control-Allow-Credentials` headers via env vars or config options.
-* [#2386](https://github.com/shlinkio/shlink/issues/2386) Add new `any-value-query-param` and `valueless-query-param` redirect rule conditions.
-
-    These new rules expand the existing `query-param`, which requires both a specific non-empty value in order to match the condition.
-
-    The new conditions match as soon as a query param exists with any or no value (in the case of `any-value-query-param`), or if a query param exists with no value at all (in the case of `valueless-query-param`).
-
-* [#2360](https://github.com/shlinkio/shlink/issues/2360) Add `TRUSTED_PROXIES` env var and corresponding config option, to configure a comma-separated list of all the proxies in front of Shlink, or simply the amount of trusted proxies in front of Shlink.
-
-    This is important to properly detect visitor's IP addresses instead of incorrectly matching one of the proxy's IP address, and if provided, it disables a workaround introduced in https://github.com/shlinkio/shlink/pull/2359.
-
-* [#2274](https://github.com/shlinkio/shlink/issues/2274) Add more supported device types for the `device` redirect condition:
-
-    * `linux`: Will match desktop devices with Linux.
-    * `windows`: Will match desktop devices with Windows.
-    * `macos`: Will match desktop devices with MacOS.
-    * `chromeos`: Will match desktop devices with ChromeOS.
-    * `mobile`: Will match any mobile devices with either Android or iOS.
-
-* [#2093](https://github.com/shlinkio/shlink/issues/2093) Add `REDIRECT_CACHE_LIFETIME` env var and corresponding config option, so that it is possible to set the `Cache-Control` visibility directive (`public` or `private`) when the `REDIRECT_STATUS_CODE` has been set to `301` or `308`.
-* [#2323](https://github.com/shlinkio/shlink/issues/2323) Add `LOGS_FORMAT` env var and corresponding config option, to allow the logs generated by Shlink to be in console or JSON formats.
-
-### Changed
-* [#2406](https://github.com/shlinkio/shlink/issues/2406) Remove references to bootstrap from error templates, and instead inline the very minimum required styles.
-
-### Deprecated
-* [#2408](https://github.com/shlinkio/shlink/issues/2408) Generating QR codes via `/{short-code}/qr-code` is now deprecated and will be removed in Shlink 5.0. Use the equivalent capability from web clients instead.
-
-### Removed
-* *Nothing*
-
-### Fixed
-* *Nothing*
-
-
-## [4.4.6] - 2025-03-20
-### Added
-* *Nothing*
-
-### Changed
-* *Nothing*
-
-### Deprecated
-* *Nothing*
-
-### Removed
-* *Nothing*
-
-### Fixed
-* [#2391](https://github.com/shlinkio/shlink/issues/2391) When sending visits to Matomo, send the country code, not the country name.
-* Fix error with new option introduced by `endroid/qr-code` 6.0.4.
-
-
-## [4.4.5] - 2025-03-01
-### Added
-* *Nothing*
-
-### Changed
-* *Nothing*
-
-### Deprecated
-* *Nothing*
-
-### Removed
-* *Nothing*
-
-### Fixed
-* [#2373](https://github.com/shlinkio/shlink/issues/2373) Ensure deprecation warnings do not end up escalated to `ErrorException`s by `ProblemDetailsMiddleware`.
-
-  In order to do this, Shlink will entirely ignore deprecation warnings when running in production, as those do not mean something is not working, but only that something will break in future versions.
-
-
-## [4.4.4] - 2025-02-19
-### Added
-* *Nothing*
-
-### Changed
-* *Nothing*
-
-### Deprecated
-* *Nothing*
-
-### Removed
-* *Nothing*
-
-### Fixed
-* [#2366](https://github.com/shlinkio/shlink/issues/2366) Fix error "Cannot use 'SCRIPT' with redis-cluster" thrown when creating a lock while using a redis cluster.
-* [#2368](https://github.com/shlinkio/shlink/issues/2368) Fix error when listing non-orphan visits using API key with `AUTHORED_SHORT_URLS` role.
-
-
 ## [4.4.3] - 2025-02-15
 ### Added
 * *Nothing*

Dockerfile

@@ -1,26 +1,26 @@
 FROM php:8.4-alpine3.21 AS base
 
 ARG SHLINK_VERSION=latest
-ENV SHLINK_VERSION=${SHLINK_VERSION}
+ENV SHLINK_VERSION ${SHLINK_VERSION}
 ARG SHLINK_RUNTIME=rr
-ENV SHLINK_RUNTIME=${SHLINK_RUNTIME}
+ENV SHLINK_RUNTIME ${SHLINK_RUNTIME}
 
-ENV USER_ID='1001'
-ENV PDO_SQLSRV_VERSION='5.12.0'
-ENV MS_ODBC_DOWNLOAD='7/6/d/76de322a-d860-4894-9945-f0cc5d6a45f8'
-ENV MS_ODBC_SQL_VERSION='18_18.4.1.1'
-ENV LC_ALL='C'
+ENV USER_ID '1001'
+ENV PDO_SQLSRV_VERSION 5.12.0
+ENV MS_ODBC_DOWNLOAD '7/6/d/76de322a-d860-4894-9945-f0cc5d6a45f8'
+ENV MS_ODBC_SQL_VERSION 18_18.4.1.1
+ENV LC_ALL 'C'
 
 WORKDIR /etc/shlink
 
 # Install required PHP extensions
 RUN \
-    # Temp install dev dependencies needed to compile the extensions \
-    apk add --no-cache --virtual .dev-deps sqlite-dev postgresql-dev icu-dev libzip-dev zlib-dev linux-headers && \
-    docker-php-ext-install -j"$(nproc)" pdo_mysql pdo_pgsql intl calendar sockets bcmath zip && \
+    # Temp install dev dependencies needed to compile the extensions
+    apk add --no-cache --virtual .dev-deps sqlite-dev postgresql-dev icu-dev libzip-dev zlib-dev libpng-dev linux-headers && \
+    docker-php-ext-install -j"$(nproc)" pdo_mysql pdo_pgsql intl calendar sockets bcmath zip gd && \
     apk add --no-cache sqlite-libs && \
     docker-php-ext-install -j"$(nproc)" pdo_sqlite && \
-    # Remove temp dev extensions, and install prod equivalents that are required at runtime \
+    # Remove temp dev extensions, and install prod equivalents that are required at runtime
     apk del .dev-deps && \
     apk add --no-cache postgresql icu libzip libpng
 

README.md

@@ -36,9 +36,10 @@ The idea is that you can just generate a container using the image and provide t
 
 First, make sure the host where you are going to run shlink fulfills these requirements:
 
-* PHP 8.4 or 8.5
+* PHP 8.3 or 8.4
 * The next PHP extensions: json, curl, pdo, intl, gd and gmp/bcmath.
     * apcu extension is recommended if you don't plan to use RoadRunner.
+    * xml extension is required if you want to generate QR codes in svg format.
     * sockets and bcmath extensions are required if you want to integrate with a RabbitMQ instance.
 * MySQL, MariaDB, PostgreSQL, MicrosoftSQL or SQLite.
     * You will also need the corresponding pdo variation for the database you are planning to use: `pdo_mysql`, `pdo_pgsql`, `pdo_sqlsrv` or `pdo_sqlite`.
@@ -98,12 +99,6 @@ Both the API and CLI allow you to do mostly the same operations, except for API
 
 If you are trying to find out how to run the project in development mode or how to provide contributions, read the [CONTRIBUTING](CONTRIBUTING.md) doc.
 
-## Powered by
-
-Thanks to [JetBrains](https://www.jetbrains.com/) for their continuous support to this project in the form of IDE licenses.
-
-![JetBrains logo](https://resources.jetbrains.com/storage/products/company/brand/logos/jetbrains.svg)
-
 ---
 
 > This product includes GeoLite2 data created by MaxMind, available from [https://www.maxmind.com](https://www.maxmind.com)

UPGRADE.md

@@ -1,26 +1,5 @@
 # Upgrading
 
-## From v4.x to v5.x
-
-### General
-
-* Generating QR codes by appending `/qr-code` to a short URL is no longer possible. Use external services to generate QR codes from a short URL, or the logic embedded in Shlink Web Client and Shlink Dashboard.
-* Shlink no longer tries to detect trusted proxies automatically, when resolving the visitor's IP address.
-    Instead, if you have more than 1 proxy in front of Shlink, you should provide `TRUSTED_PROXIES` env var, with either a comma-separated list of the IP addresses of your proxies, or a number indicating how many proxies are there in front of Shlink.
-* PHP 8.3 is no longer supported. Only 8.4 and 8.5 are officially supported as of Shlink 5.0.0.
-
-### Changes in CLI
-
-* Disabling API keys by their plain-text key is no longer supported. When calling `api-key:disable`, the first argument is now always assumed to be the name.
-* All visits-related commands (`short-url:visits`, `tag:visits`, `domain:visits`, `visit:orphan` and `visit:non-orphan`) now return more information, and columns are arranged slightly differently.
-* The `short-url:list` command no longer accepts `--including-all-tags` and `--show-api-key-name` options. Use `--tags-all` and `--show-api-key` instead.
-* The `short-url:list` command no longer allows ordering using the `--order-by=field,dir` format. Use `--order-by=field-dir` instead.
-* All commands which used to accept the `--tags` flag, no longer accept it. Pass `--tag` multiple times instead, one per tag.
-
-### Changes in env vars
-
-* The `REDIRECT_APPEND_EXTRA_PATH` env var is no longer supported. Use `REDIRECT_EXTRA_PATH_MODE=append` to enable the same behavior.
-
 ## From v3.x to v4.x
 
 ### General

bin/frankenphp-worker.php

@@ -1,36 +0,0 @@
-<?php
-
-declare(strict_types=1);
-
-namespace Shlinkio\Shlink;
-
-use Laminas\Diactoros\ServerRequestFactory;
-use Laminas\HttpHandlerRunner\Emitter\EmitterInterface;
-use Mezzio\Application;
-use Psr\Container\ContainerInterface;
-
-use function frankenphp_handle_request;
-use function gc_collect_cycles;
-
-(static function (): void {
-    /** @var ContainerInterface $container */
-    $container = include __DIR__ . '/../config/container.php';
-    $app = $container->get(Application::class);
-    $responseEmitter = $container->get(EmitterInterface::class);
-    $handler = static function () use ($app, $responseEmitter): void {
-        $response = $app->handle(ServerRequestFactory::fromGlobals());
-        $responseEmitter->emit($response);
-    };
-
-    $maxRequests = (int) ($_SERVER['MAX_REQUESTS'] ?? 0);
-    for ($nbRequests = 0; !$maxRequests || $nbRequests < $maxRequests; ++$nbRequests) {
-        $keepRunning = frankenphp_handle_request($handler);
-
-        // Call the garbage collector to reduce the chances of it being triggered in the middle of a page generation
-        gc_collect_cycles();
-
-        if (! $keepRunning) {
-            break;
-        }
-    }
-})();

bin/roadrunner-worker.php

@@ -2,22 +2,18 @@
 
 declare(strict_types=1);
 
-namespace Shlinkio\Shlink;
-
 use Mezzio\Application;
 use Psr\Container\ContainerInterface;
 use Shlinkio\Shlink\Common\Middleware\RequestIdMiddleware;
 use Shlinkio\Shlink\EventDispatcher\RoadRunner\RoadRunnerTaskConsumerToListener;
 use Spiral\RoadRunner\Http\PSR7Worker;
 
-use function gc_collect_cycles;
 use function Shlinkio\Shlink\Config\env;
 
 (static function (): void {
     /** @var ContainerInterface $container */
     $container = include __DIR__ . '/../config/container.php';
     $rrMode = env('RR_MODE');
-    $gcCollectCycles = env('GC_COLLECT_CYCLES', default: false);
 
     if ($rrMode === 'http') {
         // This was spin-up as a web worker
@@ -29,10 +25,6 @@ use function Shlinkio\Shlink\Config\env;
                 $worker->respond($app->handle($req));
             } catch (Throwable $e) {
                 $worker->getWorker()->error((string) $e);
-            } finally {
-                if ($gcCollectCycles) {
-                    gc_collect_cycles();
-                }
             }
         }
     } else {

composer.json

@@ -12,16 +12,19 @@
         }
     ],
     "require": {
-        "php": "^8.4",
+        "php": "^8.3",
         "ext-curl": "*",
+        "ext-gd": "*",
+        "ext-json": "*",
         "ext-mbstring": "*",
         "ext-pdo": "*",
         "akrabat/ip-address-middleware": "^2.6",
         "cakephp/chronos": "^3.1",
-        "doctrine/dbal": "^4.4",
-        "doctrine/migrations": "^3.9",
-        "doctrine/orm": "^3.6",
-        "donatj/phpuseragentparser": "^1.11",
+        "doctrine/dbal": "^4.2",
+        "doctrine/migrations": "^3.8",
+        "doctrine/orm": "^3.3",
+        "donatj/phpuseragentparser": "^1.10",
+        "endroid/qr-code": "^6.0",
         "friendsofphp/proxy-manager-lts": "^1.0",
         "geoip2/geoip2": "^3.1",
         "guzzlehttp/guzzle": "^7.9",
@@ -32,7 +35,6 @@
         "laminas/laminas-inputfilter": "^2.31",
         "laminas/laminas-servicemanager": "^3.23",
         "laminas/laminas-stdlib": "^3.20",
-        "league/csv": "^9.28",
         "matomo/matomo-php-tracker": "^3.3",
         "mezzio/mezzio": "^3.20",
         "mezzio/mezzio-fastroute": "^3.12",
@@ -41,38 +43,42 @@
         "pagerfanta/core": "^3.8",
         "ramsey/uuid": "^4.7",
         "shlinkio/doctrine-specification": "^2.2",
-        "shlinkio/shlink-common": "^8.0.0",
-        "shlinkio/shlink-config": "^4.1.0",
-        "shlinkio/shlink-event-dispatcher": "^4.4.0",
-        "shlinkio/shlink-importer": "^5.7.0",
-        "shlinkio/shlink-installer": "^10.0.0",
-        "shlinkio/shlink-ip-geolocation": "^5.0.0",
-        "shlinkio/shlink-json": "^1.3",
-        "spiral/roadrunner": "^2025.1",
-        "spiral/roadrunner-cli": "^2.7",
-        "spiral/roadrunner-http": "^3.6",
-        "spiral/roadrunner-jobs": "^4.7",
-        "symfony/console": "^8.0",
-        "symfony/filesystem": "^8.0",
-        "symfony/lock": "^8.0",
-        "symfony/process": "^8.0",
-        "symfony/string": "^8.0"
+        "shlinkio/shlink-common": "^7.0",
+        "shlinkio/shlink-config": "^4.0",
+        "shlinkio/shlink-event-dispatcher": "^4.2",
+        "shlinkio/shlink-importer": "^5.6",
+        "shlinkio/shlink-installer": "^9.5",
+        "shlinkio/shlink-ip-geolocation": "^4.3",
+        "shlinkio/shlink-json": "^1.2",
+        "spiral/roadrunner": "^2024.3",
+        "spiral/roadrunner-cli": "^2.6",
+        "spiral/roadrunner-http": "^3.5",
+        "spiral/roadrunner-jobs": "^4.6",
+        "symfony/console": "^7.2",
+        "symfony/filesystem": "^7.2",
+        "symfony/lock": "7.2.0",
+        "symfony/process": "^7.2",
+        "symfony/string": "^7.2"
     },
     "require-dev": {
         "devizzent/cebe-php-openapi": "^1.1.2",
         "devster/ubench": "^2.1",
         "phpstan/phpstan": "^2.1",
         "phpstan/phpstan-doctrine": "^2.0",
-        "phpstan/phpstan-phpunit": "^2.0.5",
+        "phpstan/phpstan-phpunit": "^2.0",
         "phpstan/phpstan-symfony": "^2.0",
-        "phpunit/php-code-coverage": "^13.0",
-        "phpunit/phpcov": "^12.0",
-        "phpunit/phpunit": "^13.0",
-        "shlinkio/php-coding-standard": "~2.5.0",
-        "shlinkio/shlink-test-utils": "^4.5",
-        "symfony/var-dumper": "^8.0",
+        "phpunit/php-code-coverage": "^12.0",
+        "phpunit/phpcov": "^11.0",
+        "phpunit/phpunit": "^12.0",
+        "roave/security-advisories": "dev-master",
+        "shlinkio/php-coding-standard": "~2.4.0",
+        "shlinkio/shlink-test-utils": "^4.3.1",
+        "symfony/var-dumper": "^7.2",
         "veewee/composer-run-parallel": "^1.4"
     },
+    "conflict": {
+        "symfony/var-exporter": ">=6.3.9,<=6.4.0"
+    },
     "autoload": {
         "psr-4": {
             "Shlinkio\\Shlink\\CLI\\": "module/CLI/src",
@@ -141,12 +147,12 @@
         "test:cli:ci": [
             "@putenv GENERATE_COVERAGE=yes",
             "@test:cli",
-            "@php -d memory_limit=-1 vendor/bin/phpcov merge build/coverage-cli --php build/coverage-cli.cov && rm build/coverage-cli/*.cov"
+            "vendor/bin/phpcov merge build/coverage-cli --php build/coverage-cli.cov && rm build/coverage-cli/*.cov"
         ],
         "test:cli:pretty": [
             "@putenv GENERATE_COVERAGE=yes",
             "@test:cli",
-            "@php -d memory_limit=-1 phpcov merge build/coverage-cli --html build/coverage-cli/coverage-html && rm build/coverage-cli/*.cov"
+            "phpcov merge build/coverage-cli --html build/coverage-cli/coverage-html && rm build/coverage-cli/*.cov"
         ],
         "openapi:validate": "php-openapi validate docs/swagger/swagger.json",
         "openapi:inline": "php-openapi inline docs/swagger/swagger.json docs/swagger/openapi-inlined.json",

config/autoload/cache.global.php

@@ -11,8 +11,6 @@ return (static function (): array {
         'redis' => [
             'servers' => $redisServers,
             'sentinel_service' => EnvVars::REDIS_SENTINEL_SERVICE->loadFromEnv(),
-            'username' => EnvVars::REDIS_SERVERS_USER->loadFromEnv(),
-            'password' => EnvVars::REDIS_SERVERS_PASSWORD->loadFromEnv(),
         ],
     ];
 

config/autoload/cors.global.php

@@ -0,0 +1,11 @@
+<?php
+
+declare(strict_types=1);
+
+return [
+
+    'cors' => [
+        'max_age' => 3600,
+    ],
+
+];

config/autoload/geolite2.global.php

@@ -8,7 +8,7 @@ return [
 
     'geolite2' => [
         'db_location' => __DIR__ . '/../../data/GeoLite2-City.mmdb',
-        'temp_dir' => __DIR__ . '/../../data/temp-geolite',
+        'temp_dir' => __DIR__ . '/../../data',
         'license_key' => EnvVars::GEOLITE_LICENSE_KEY->loadFromEnv(),
     ],
 

config/autoload/installer.global.php

@@ -13,7 +13,6 @@ return [
         'enabled_options' => [
             Option\Server\RuntimeConfigOption::class,
             Option\Server\MemoryLimitConfigOption::class,
-            Option\Server\LogsFormatConfigOption::class,
             Option\Database\DatabaseDriverConfigOption::class,
             Option\Database\DatabaseNameConfigOption::class,
             Option\Database\DatabaseHostConfigOption::class,
@@ -33,8 +32,6 @@ return [
             Option\Cache\CacheNamespaceConfigOption::class,
             Option\Redis\RedisServersConfigOption::class,
             Option\Redis\RedisSentinelServiceConfigOption::class,
-            Option\Redis\RedisServersUserConfigOption::class,
-            Option\Redis\RedisServersPasswordConfigOption::class,
             Option\Redis\RedisPubSubConfigOption::class,
             Option\UrlShortener\ShortCodeLengthOption::class,
             Option\Mercure\EnableMercureConfigOption::class,
@@ -44,7 +41,6 @@ return [
             Option\UrlShortener\GeoLiteLicenseKeyConfigOption::class,
             Option\UrlShortener\RedirectStatusCodeConfigOption::class,
             Option\UrlShortener\RedirectCacheLifeTimeConfigOption::class,
-            Option\UrlShortener\RedirectCacheVisibilityConfigOption::class,
             Option\UrlShortener\AutoResolveTitlesConfigOption::class,
             Option\UrlShortener\ExtraPathModeConfigOption::class,
             Option\UrlShortener\EnableMultiSegmentSlugsConfigOption::class,
@@ -60,6 +56,15 @@ return [
             Option\Tracking\DisableIpTrackingConfigOption::class,
             Option\Tracking\DisableReferrerTrackingConfigOption::class,
             Option\Tracking\DisableUaTrackingConfigOption::class,
+            Option\QrCode\DefaultSizeConfigOption::class,
+            Option\QrCode\DefaultMarginConfigOption::class,
+            Option\QrCode\DefaultFormatConfigOption::class,
+            Option\QrCode\DefaultErrorCorrectionConfigOption::class,
+            Option\QrCode\DefaultRoundBlockSizeConfigOption::class,
+            Option\QrCode\DefaultColorConfigOption::class,
+            Option\QrCode\DefaultBgColorConfigOption::class,
+            Option\QrCode\DefaultLogoUrlConfigOption::class,
+            Option\QrCode\EnabledForDisabledShortUrlsConfigOption::class,
             Option\RabbitMq\RabbitMqEnabledConfigOption::class,
             Option\RabbitMq\RabbitMqHostConfigOption::class,
             Option\RabbitMq\RabbitMqUseSslConfigOption::class,
@@ -71,11 +76,6 @@ return [
             Option\Matomo\MatomoBaseUrlConfigOption::class,
             Option\Matomo\MatomoSiteIdConfigOption::class,
             Option\Matomo\MatomoApiTokenConfigOption::class,
-            Option\RealTimeUpdates\RealTimeUpdatesTopicsConfigOption::class,
-            Option\Cors\CorsAllowOriginConfigOption::class,
-            Option\Cors\CorsAllowCredentialsConfigOption::class,
-            Option\Cors\CorsMaxAgeConfigOption::class,
-            Option\TrustedProxiesConfigOption::class,
         ],
 
         'installation_commands' => [

config/autoload/ip-address.global.php

@@ -2,47 +2,49 @@
 
 declare(strict_types=1);
 
+use Laminas\ServiceManager\AbstractFactory\ConfigAbstractFactory;
 use RKA\Middleware\IpAddress;
 use RKA\Middleware\Mezzio\IpAddressFactory;
-use Shlinkio\Shlink\Core\Config\EnvVars;
-
-use function Shlinkio\Shlink\Core\splitByComma;
+use Shlinkio\Shlink\Core\Middleware\ReverseForwardedAddressesMiddlewareDecorator;
 
 use const Shlinkio\Shlink\IP_ADDRESS_REQUEST_ATTRIBUTE;
 
-return (static function (): array {
-    $trustedProxies = EnvVars::TRUSTED_PROXIES->loadFromEnv();
-    $proxiesIsHopCount = is_numeric($trustedProxies);
+return [
 
-    return [
-
-        // Configuration for RKA\Middleware\IpAddress
-        'rka' => [
-            'ip_address' => [
-                'attribute_name' => IP_ADDRESS_REQUEST_ATTRIBUTE,
-                'check_proxy_headers' => true,
-                // List of trusted proxies
-                'trusted_proxies' => $proxiesIsHopCount ? [] : splitByComma($trustedProxies),
-                // Amount of addresses to skip from the right, before finding the visitor IP address
-                'hop_count' => $proxiesIsHopCount ? (int) $trustedProxies : 0,
-                'headers_to_inspect' => [
-                    'CF-Connecting-IP',
-                    'X-Forwarded-For',
-                    'X-Forwarded',
-                    'Forwarded',
-                    'True-Client-IP',
-                    'X-Real-IP',
-                    'X-Cluster-Client-Ip',
-                    'Client-Ip',
-                ],
+    // Configuration for RKA\Middleware\IpAddress
+    'rka' => [
+        'ip_address' => [
+            'attribute_name' => IP_ADDRESS_REQUEST_ATTRIBUTE,
+            'check_proxy_headers' => true,
+            'trusted_proxies' => [],
+            'headers_to_inspect' => [
+                'CF-Connecting-IP',
+                'X-Forwarded-For',
+                'X-Forwarded',
+                'Forwarded',
+                'True-Client-IP',
+                'X-Real-IP',
+                'X-Cluster-Client-Ip',
+                'Client-Ip',
             ],
         ],
+    ],
 
-        'dependencies' => [
-            'factories' => [
-                IpAddress::class => IpAddressFactory::class,
-            ],
+    'dependencies' => [
+        'factories' => [
+//            IpAddress::class => IpAddressFactory::class,
+            'actual_ip_address_middleware' => IpAddressFactory::class,
+            ReverseForwardedAddressesMiddlewareDecorator::class => ConfigAbstractFactory::class,
         ],
+        'aliases' => [
+            // Make sure the decorated middleware is resolved when getting IpAddress::class, to make this decoration
+            // transparent for other parts of the code
+            IpAddress::class => ReverseForwardedAddressesMiddlewareDecorator::class,
+        ],
+    ],
 
-    ];
-})();
+    ConfigAbstractFactory::class => [
+        ReverseForwardedAddressesMiddlewareDecorator::class => ['actual_ip_address_middleware'],
+    ],
+
+];

config/autoload/logger.global.php

@@ -23,16 +23,11 @@ use function Shlinkio\Shlink\Config\runningInRoadRunner;
 
 return (static function (): array {
     $isDev = EnvVars::isDevEnv();
-    $format = EnvVars::LOGS_FORMAT->loadFromEnv();
-    $buildCommonConfig = static fn (bool $addNewLine = false) => [
+    $common = [
         'level' => $isDev ? Level::Debug->value : Level::Info->value,
         'processors' => [RequestIdMiddleware::class],
-        'formatter' => [
-            'type' => $format,
-            'add_new_line' => $addNewLine,
-            'line_format' =>
-                '[%datetime%] [%extra.' . RequestIdMiddleware::ATTRIBUTE . '%] %channel%.%level_name% - %message%',
-        ],
+        'line_format' =>
+            '[%datetime%] [%extra.' . RequestIdMiddleware::ATTRIBUTE . '%] %channel%.%level_name% - %message%',
     ];
 
     // In dev env or the docker container, stream Shlink logs to stderr, otherwise send them to a file
@@ -44,15 +39,16 @@ return (static function (): array {
             'Shlink' => $useStreamForShlinkLogger ? [
                 'type' => LoggerType::STREAM->value,
                 'destination' => 'php://stderr',
-                ...$buildCommonConfig(),
+                ...$common,
             ] : [
                 'type' => LoggerType::FILE->value,
-                ...$buildCommonConfig(),
+                ...$common,
             ],
             'Access' => [
                 'type' => LoggerType::STREAM->value,
                 'destination' => 'php://stderr',
-                ...$buildCommonConfig(! runningInRoadRunner()),
+                'add_new_line' => ! runningInRoadRunner(),
+                ...$common,
             ],
         ],
 

config/autoload/mercure.global.php

@@ -12,7 +12,6 @@ return [
 
     // This config is used by shlink-common. Do not delete
     'mercure' => [
-        'enabled' => EnvVars::MERCURE_ENABLED->loadFromEnv(),
         'public_hub_url' => EnvVars::MERCURE_PUBLIC_HUB_URL->loadFromEnv(),
         'internal_hub_url' => EnvVars::MERCURE_INTERNAL_HUB_URL->loadFromEnv(),
         'jwt_secret' => EnvVars::MERCURE_JWT_SECRET->loadFromEnv(),

config/autoload/routes.config.php

@@ -94,6 +94,14 @@ return (static function (): array {
                 ],
                 'allowed_methods' => [RequestMethodInterface::METHOD_GET],
             ],
+            [
+                'name' => CoreAction\QrCodeAction::class,
+                'path' => '/{shortCode}/qr-code',
+                'middleware' => [
+                    CoreAction\QrCodeAction::class,
+                ],
+                'allowed_methods' => [RequestMethodInterface::METHOD_GET],
+            ],
             [
                 'name' => CoreAction\RedirectAction::class,
                 'path' => sprintf('/{shortCode}%s', $shortUrlRouteSuffix),

config/config.php

@@ -10,7 +10,7 @@ use Mezzio;
 use Mezzio\ProblemDetails;
 use Shlinkio\Shlink\Core\Config\EnvVars;
 
-return new ConfigAggregator\ConfigAggregator(
+return (new ConfigAggregator\ConfigAggregator(
     providers: [
         Mezzio\ConfigProvider::class,
         Mezzio\Router\ConfigProvider::class,
@@ -39,4 +39,4 @@ return new ConfigAggregator\ConfigAggregator(
         Core\Config\PostProcessor\MultiSegmentSlugProcessor::class,
         Core\Config\PostProcessor\ShortUrlMethodsProcessor::class,
     ],
-)->getMergedConfig();
+))->getMergedConfig();

config/constants.php

@@ -11,30 +11,15 @@ const DEFAULT_SHORT_CODES_LENGTH = 5;
 const MIN_SHORT_CODES_LENGTH = 4;
 const DEFAULT_REDIRECT_STATUS_CODE = RedirectStatus::STATUS_302;
 const DEFAULT_REDIRECT_CACHE_LIFETIME = 30;
-const DEFAULT_REDIRECT_CACHE_VISIBILITY = 'private';
 const LOCAL_LOCK_FACTORY = 'Shlinkio\Shlink\LocalLockFactory';
 const LOOSE_URI_MATCHER = '/(.+)\:(.+)/i'; // Matches anything starting with a schema.
+const DEFAULT_QR_CODE_SIZE = 300;
+const DEFAULT_QR_CODE_MARGIN = 0;
+const DEFAULT_QR_CODE_FORMAT = 'png';
+const DEFAULT_QR_CODE_ERROR_CORRECTION = 'l';
+const DEFAULT_QR_CODE_ROUND_BLOCK_SIZE = true;
+const DEFAULT_QR_CODE_ENABLED_FOR_DISABLED_SHORT_URLS = true;
+const DEFAULT_QR_CODE_COLOR = '#000000'; // Black
+const DEFAULT_QR_CODE_BG_COLOR = '#ffffff'; // White
 const IP_ADDRESS_REQUEST_ATTRIBUTE = 'remote_address';
 const REDIRECT_URL_REQUEST_ATTRIBUTE = 'redirect_url';
-
-/**
- * List of ISO 3166-1 alpha-2 two-letter country codes https://en.wikipedia.org/wiki/ISO_3166-1_alpha-2
- */
-const ISO_COUNTRY_CODES = [
-    'AF', 'AX', 'AL', 'DZ', 'AS', 'AD', 'AO', 'AI', 'AQ', 'AG', 'AR', 'AM', 'AW', 'AU', 'AT', 'AZ',
-    'BS', 'BH', 'BD', 'BB', 'BY', 'BE', 'BZ', 'BJ', 'BM', 'BT', 'BO', 'BQ', 'BA', 'BW', 'BV', 'BR',
-    'IO', 'BN', 'BG', 'BF', 'BI', 'CV', 'KH', 'CM', 'CA', 'KY', 'CF', 'TD', 'CL', 'CN', 'CX', 'CC',
-    'CO', 'KM', 'CG', 'CD', 'CK', 'CR', 'CI', 'HR', 'CU', 'CW', 'CY', 'CZ', 'DK', 'DJ', 'DM', 'DO',
-    'EC', 'EG', 'SV', 'GQ', 'ER', 'EE', 'SZ', 'ET', 'FK', 'FO', 'FJ', 'FI', 'FR', 'GF', 'PF', 'TF',
-    'GA', 'GM', 'GE', 'DE', 'GH', 'GI', 'GR', 'GL', 'GD', 'GP', 'GU', 'GT', 'GG', 'GN', 'GW', 'GY',
-    'HT', 'HM', 'VA', 'HN', 'HK', 'HU', 'IS', 'IN', 'ID', 'IR', 'IQ', 'IE', 'IM', 'IL', 'IT', 'JM',
-    'JP', 'JE', 'JO', 'KZ', 'KE', 'KI', 'KP', 'KR', 'KW', 'KG', 'LA', 'LV', 'LB', 'LS', 'LR', 'LY',
-    'LI', 'LT', 'LU', 'MO', 'MG', 'MW', 'MY', 'MV', 'ML', 'MT', 'MH', 'MQ', 'MR', 'MU', 'YT', 'MX',
-    'FM', 'MD', 'MC', 'MN', 'ME', 'MS', 'MA', 'MZ', 'MM', 'NA', 'NR', 'NP', 'NL', 'NC', 'NZ', 'NI',
-    'NE', 'NG', 'NU', 'NF', 'MK', 'MP', 'NO', 'OM', 'PK', 'PW', 'PS', 'PA', 'PG', 'PY', 'PE', 'PH',
-    'PN', 'PL', 'PT', 'PR', 'QA', 'RE', 'RO', 'RU', 'RW', 'BL', 'SH', 'KN', 'LC', 'MF', 'PM', 'VC',
-    'WS', 'SM', 'ST', 'SA', 'SN', 'RS', 'SC', 'SL', 'SG', 'SX', 'SK', 'SI', 'SB', 'SO', 'ZA', 'GS',
-    'SS', 'ES', 'LK', 'SD', 'SR', 'SJ', 'SE', 'CH', 'SY', 'TW', 'TJ', 'TZ', 'TH', 'TL', 'TG', 'TK',
-    'TO', 'TT', 'TN', 'TR', 'TM', 'TC', 'TV', 'UG', 'UA', 'AE', 'GB', 'US', 'UM', 'UY', 'UZ', 'VU',
-    'VE', 'VN', 'VG', 'VI', 'WF', 'EH', 'YE', 'ZM', 'ZW',
-];

config/container.php

@@ -11,7 +11,6 @@ use function Shlinkio\Shlink\Core\enumValues;
 
 use const Shlinkio\Shlink\LOCAL_LOCK_FACTORY;
 
-// Set current directory to the project's root directory
 chdir(dirname(__DIR__));
 
 require 'vendor/autoload.php';
@@ -22,11 +21,7 @@ loadEnvVarsFromConfig(
     enumValues(EnvVars::class),
 );
 
-// This is one of the first files loaded. Set global configuration here
-error_reporting(
-    // Set a less strict error reporting for prod, where deprecation warnings should be ignored
-    EnvVars::isProdEnv() ? E_ALL & ~E_DEPRECATED & ~E_USER_DEPRECATED : E_ALL,
-);
+// This is one of the first files loaded. Configure the timezone and memory limit here
 ini_set('memory_limit', EnvVars::MEMORY_LIMIT->loadFromEnv());
 date_default_timezone_set(EnvVars::TIMEZONE->loadFromEnv());
 

config/params/shlink_dev_env.php.dist

@@ -4,15 +4,13 @@ declare(strict_types=1);
 
 use Shlinkio\Shlink\Core\Config\EnvVars;
 
-use function Shlinkio\Shlink\Config\runningInRoadRunner;
-
 return [
 
     EnvVars::APP_ENV->value => 'dev',
 //    EnvVars::GEOLITE_LICENSE_KEY->value => '',
 
     // URL shortener
-    EnvVars::DEFAULT_DOMAIN->value => runningInRoadRunner() ? 'localhost:8800' : 'localhost:8008',
+    EnvVars::DEFAULT_DOMAIN->value => 'localhost:8800',
     EnvVars::IS_HTTPS_ENABLED->value => false,
 
     // Database - MySQL
@@ -60,7 +58,6 @@ return [
 //    EnvVars::MATOMO_API_TOKEN->value => ,
 
     // Mercure
-    EnvVars::MERCURE_ENABLED->value => true,
     EnvVars::MERCURE_PUBLIC_HUB_URL->value => 'http://localhost:8002',
     EnvVars::MERCURE_INTERNAL_HUB_URL->value => 'http://shlink_mercure_proxy',
     EnvVars::MERCURE_JWT_SECRET->value => 'mercure_jwt_key_long_enough_to_avoid_error',

config/roadrunner/.rr.dev.yml

@@ -30,17 +30,13 @@ jobs:
         prefetch: 10
 
 logs:
-  encoding: console
   mode: development
   channels:
     http:
       mode: 'off' # Disable logging as Shlink handles it internally
     server:
-      encoding: console
       level: info
     metrics:
-      encoding: console
       level: debug
     jobs:
-      encoding: console
       level: debug

config/roadrunner/.rr.test.yml

@@ -35,16 +35,15 @@ jobs:
                 prefetch: 10
 
 logs:
-    encoding: console
+    encoding: json
     mode: development
     channels:
         http:
             mode: 'off' # Disable logging as Shlink handles it internally
         server:
-            encoding: console
+            encoding: json
             level: info
         metrics:
             level: panic
         jobs:
-            encoding: console
             level: panic

config/roadrunner/.rr.yml

@@ -14,13 +14,11 @@ http:
     forbid: ['.php', '.htaccess']
   pool:
     num_workers: ${WEB_WORKER_NUM:-0}
-    max_jobs: 250 # Restart worker after processing this amount of requests to mitigate memory leaks
 
 jobs:
   timeout: 300 # 5 minutes
   pool:
     num_workers: ${TASK_WORKER_NUM:-0}
-    max_jobs: 250 # Restart worker after processing this amount of jobs to mitigate memory leaks
   consume: ['shlink']
   pipelines:
     shlink:
@@ -30,14 +28,11 @@ jobs:
         prefetch: 10
 
 logs:
-  encoding: ${LOGS_FORMAT:-console}
   mode: production
   channels:
     http:
       mode: 'off' # Disable logging as Shlink handles it internally
     server:
-      encoding: ${LOGS_FORMAT:-console}
       level: info
     jobs:
-      encoding: ${LOGS_FORMAT:-console}
       level: debug

config/test/constants.php

@@ -11,11 +11,5 @@ const ANDROID_USER_AGENT = 'Mozilla/5.0 (Linux; Android 13) AppleWebKit/537.36 (
     . 'Chrome/109.0.5414.86 Mobile Safari/537.36';
 const IOS_USER_AGENT = 'Mozilla/5.0 (iPhone; CPU iPhone OS 16_2 like Mac OS X) AppleWebKit/605.1.15 '
     . '(KHTML, like Gecko) FxiOS/109.0 Mobile/15E148 Safari/605.1.15';
-const WINDOWS_USER_AGENT = 'Mozilla/5.0 (Windows NT 10.0; Win64; x64) AppleWebKit/537.36 (KHTML, like Gecko) '
-    . 'Chrome/138.0.0.0 Safari/537.36 Edg/138.0.3351.95';
-const LINUX_USER_AGENT = 'Mozilla/5.0 (X11; Linux x86_64) AppleWebKit/537.36 (KHTML, like Gecko) '
-    . 'HeadlessChrome/81.0.4044.113 Safari/537.36';
-const MACOS_USER_AGENT = 'Mozilla/5.0 (Macintosh; Intel Mac OS X 15_5) AppleWebKit/605.1.15 (KHTML, like Gecko) '
-    . 'Version/18.4 Safari/605.1.15';
-const CHROMEOS_USER_AGENT = 'Mozilla/5.0 (X11; CrOS x86_64 16181.61.0) AppleWebKit/537.36 (KHTML, like Gecko) '
-    . 'Chrome/134.0.6998.198 Safari/537.36';
+const DESKTOP_USER_AGENT = 'Mozilla/5.0 (Windows NT 10.0; Win64; x64) AppleWebKit/537.36 (KHTML, like '
+    . 'Gecko) Chrome/109.0.0.0 Safari/537.36 Edg/109.0.1518.61';

data/infra/examples/nginx-vhost.conf

@@ -11,7 +11,7 @@ server {
 
     location ~ \.php$ {
         fastcgi_split_path_info ^(.+\.php)(/.+)$;
-        fastcgi_pass unix:/var/run/php/php8.4-fpm.sock;
+        fastcgi_pass unix:/var/run/php/php8.3-fpm.sock;
         fastcgi_index index.php;
         include fastcgi.conf;
     }

data/infra/frankenphp.Dockerfile

@@ -1,53 +0,0 @@
-FROM dunglas/frankenphp:1-php8.4-alpine
-MAINTAINER Alejandro Celaya <alejandro@alejandrocelaya.com>
-
-ENV PDO_SQLSRV_VERSION='5.12.0'
-ENV MS_ODBC_DOWNLOAD='7/6/d/76de322a-d860-4894-9945-f0cc5d6a45f8'
-ENV MS_ODBC_SQL_VERSION='18_18.4.1.1'
-
-RUN apk update
-
-# Install common php extensions
-RUN docker-php-ext-install pdo_mysql
-RUN docker-php-ext-install calendar
-
-RUN apk add --no-cache oniguruma-dev
-RUN docker-php-ext-install mbstring
-
-RUN apk add --no-cache sqlite-libs
-RUN apk add --no-cache sqlite-dev
-RUN docker-php-ext-install pdo_sqlite
-
-RUN apk add --no-cache icu-dev
-RUN docker-php-ext-install intl
-
-RUN apk add --no-cache postgresql-dev
-RUN docker-php-ext-install pdo_pgsql
-
-COPY --from=ghcr.io/php/pie:bin /pie /usr/bin/pie
-RUN apk add --no-cache libzip-dev zlib-dev && \
-    apk add --no-cache --virtual .phpize-deps $PHPIZE_DEPS linux-headers && \
-    docker-php-ext-install sockets && \
-    pie install xdebug/xdebug && \
-    pie install pecl/zip && \
-    apk del .phpize-deps
-RUN docker-php-ext-install bcmath
-
-# Install sqlsrv driver
-RUN apk add --update linux-headers && \
-    wget https://download.microsoft.com/download/${MS_ODBC_DOWNLOAD}/msodbcsql${MS_ODBC_SQL_VERSION}-1_amd64.apk && \
-    apk add --allow-untrusted msodbcsql${MS_ODBC_SQL_VERSION}-1_amd64.apk && \
-    apk add --no-cache --virtual .phpize-deps $PHPIZE_DEPS unixodbc-dev && \
-    pecl install pdo_sqlsrv-${PDO_SQLSRV_VERSION} && \
-    docker-php-ext-enable pdo_sqlsrv && \
-    apk del .phpize-deps && \
-    rm msodbcsql${MS_ODBC_SQL_VERSION}-1_amd64.apk
-
-# Install composer
-COPY --from=composer:2 /usr/bin/composer /usr/local/bin/composer
-
-# Make home directory writable by anyone
-RUN chmod 777 /home
-
-VOLUME /home/shlink
-WORKDIR /home/shlink

data/infra/frankenphp_caddy_config/.gitignore

@@ -1,2 +0,0 @@
-*
-!.gitignore

data/infra/frankenphp_caddy_data/.gitignore

@@ -1,2 +0,0 @@
-*
-!.gitignore

data/infra/php.Dockerfile

@@ -1,10 +1,10 @@
 FROM php:8.4-fpm-alpine3.21
 MAINTAINER Alejandro Celaya <alejandro@alejandrocelaya.com>
 
-ENV APCU_VERSION='5.1.24'
-ENV PDO_SQLSRV_VERSION='5.12.0'
-ENV MS_ODBC_DOWNLOAD='7/6/d/76de322a-d860-4894-9945-f0cc5d6a45f8'
-ENV MS_ODBC_SQL_VERSION='18_18.4.1.1'
+ENV APCU_VERSION 5.1.24
+ENV PDO_SQLSRV_VERSION 5.12.0
+ENV MS_ODBC_DOWNLOAD '7/6/d/76de322a-d860-4894-9945-f0cc5d6a45f8'
+ENV MS_ODBC_SQL_VERSION 18_18.4.1.1
 
 RUN apk update
 
@@ -22,26 +22,37 @@ RUN docker-php-ext-install pdo_sqlite
 RUN apk add --no-cache icu-dev
 RUN docker-php-ext-install intl
 
+RUN apk add --no-cache libzip-dev zlib-dev
+RUN docker-php-ext-install zip
+
+RUN apk add --no-cache libpng-dev
+RUN docker-php-ext-install gd
+
 RUN apk add --no-cache postgresql-dev
 RUN docker-php-ext-install pdo_pgsql
 
-COPY --from=ghcr.io/php/pie:bin /pie /usr/bin/pie
-RUN apk add --no-cache libzip-dev zlib-dev && \
-    apk add --no-cache --virtual .phpize-deps $PHPIZE_DEPS linux-headers && \
+RUN apk add --no-cache --virtual .phpize-deps $PHPIZE_DEPS linux-headers && \
     docker-php-ext-install sockets && \
-    pie install xdebug/xdebug && \
-    pie install pecl/zip && \
-    pie install apcu/apcu && \
     apk del .phpize-deps
 RUN docker-php-ext-install bcmath
 
-# Install sqlsrv driver
+# Install APCu extension
+ADD https://pecl.php.net/get/apcu-$APCU_VERSION.tgz /tmp/apcu.tar.gz
+RUN mkdir -p /usr/src/php/ext/apcu \
+  && tar xf /tmp/apcu.tar.gz -C /usr/src/php/ext/apcu --strip-components=1 \
+  && docker-php-ext-configure apcu \
+  && docker-php-ext-install apcu \
+  && rm /tmp/apcu.tar.gz \
+  && rm /usr/local/etc/php/conf.d/docker-php-ext-apcu.ini \
+  && echo extension=apcu.so > /usr/local/etc/php/conf.d/20-php-ext-apcu.ini
+
+# Install xdebug and sqlsrv driver
 RUN apk add --update linux-headers && \
     wget https://download.microsoft.com/download/${MS_ODBC_DOWNLOAD}/msodbcsql${MS_ODBC_SQL_VERSION}-1_amd64.apk && \
     apk add --allow-untrusted msodbcsql${MS_ODBC_SQL_VERSION}-1_amd64.apk && \
     apk add --no-cache --virtual .phpize-deps $PHPIZE_DEPS unixodbc-dev && \
-    pecl install pdo_sqlsrv-${PDO_SQLSRV_VERSION} && \
-    docker-php-ext-enable pdo_sqlsrv && \
+    pecl install pdo_sqlsrv-${PDO_SQLSRV_VERSION} xdebug && \
+    docker-php-ext-enable pdo_sqlsrv xdebug && \
     apk del .phpize-deps && \
     rm msodbcsql${MS_ODBC_SQL_VERSION}-1_amd64.apk
 

data/infra/php.ini

@@ -1,4 +1,5 @@
 display_errors=On
+error_reporting=-1
 log_errors_max_len=0
 zend.assertions=1
 assert.exception=1

data/infra/roadrunner.Dockerfile

@@ -1,9 +1,9 @@
 FROM php:8.4-alpine3.21
 MAINTAINER Alejandro Celaya <alejandro@alejandrocelaya.com>
 
-ENV PDO_SQLSRV_VERSION='5.12.0'
-ENV MS_ODBC_DOWNLOAD='7/6/d/76de322a-d860-4894-9945-f0cc5d6a45f8'
-ENV MS_ODBC_SQL_VERSION='18_18.4.1.1'
+ENV PDO_SQLSRV_VERSION 5.12.0
+ENV MS_ODBC_DOWNLOAD '7/6/d/76de322a-d860-4894-9945-f0cc5d6a45f8'
+ENV MS_ODBC_SQL_VERSION 18_18.4.1.1
 
 RUN apk update
 
@@ -21,25 +21,27 @@ RUN docker-php-ext-install pdo_sqlite
 RUN apk add --no-cache icu-dev
 RUN docker-php-ext-install intl
 
+RUN apk add --no-cache libzip-dev zlib-dev
+RUN docker-php-ext-install zip
+
+RUN apk add --no-cache libpng-dev
+RUN docker-php-ext-install gd
+
 RUN apk add --no-cache postgresql-dev
 RUN docker-php-ext-install pdo_pgsql
 
-COPY --from=ghcr.io/php/pie:bin /pie /usr/bin/pie
-RUN apk add --no-cache libzip-dev zlib-dev && \
-    apk add --no-cache --virtual .phpize-deps $PHPIZE_DEPS linux-headers && \
+RUN apk add --no-cache --virtual .phpize-deps $PHPIZE_DEPS linux-headers && \
     docker-php-ext-install sockets && \
-    pie install xdebug/xdebug && \
-    pie install pecl/zip && \
     apk del .phpize-deps
 RUN docker-php-ext-install bcmath
 
-# Install sqlsrv driver
+# Install xdebug and sqlsrv driver
 RUN apk add --update linux-headers && \
     wget https://download.microsoft.com/download/${MS_ODBC_DOWNLOAD}/msodbcsql${MS_ODBC_SQL_VERSION}-1_amd64.apk && \
     apk add --allow-untrusted msodbcsql${MS_ODBC_SQL_VERSION}-1_amd64.apk && \
     apk add --no-cache --virtual .phpize-deps $PHPIZE_DEPS unixodbc-dev && \
-    pecl install pdo_sqlsrv-${PDO_SQLSRV_VERSION} && \
-    docker-php-ext-enable pdo_sqlsrv && \
+    pecl install pdo_sqlsrv-${PDO_SQLSRV_VERSION} xdebug && \
+    docker-php-ext-enable pdo_sqlsrv xdebug && \
     apk del .phpize-deps && \
     rm msodbcsql${MS_ODBC_SQL_VERSION}-1_amd64.apk
 

data/temp-geolite/.gitignore

@@ -1,2 +0,0 @@
-*
-!.gitignore

docker-compose.yml

@@ -66,37 +66,6 @@ services:
         extra_hosts:
             - 'host.docker.internal:host-gateway'
 
-    shlink_frankenphp:
-      container_name: shlink_frankenphp
-      user: 1000:1000
-      build:
-        context: .
-        dockerfile: ./data/infra/frankenphp.Dockerfile
-      ports:
-        - "8008:8008"
-      volumes:
-        - ./:/home/shlink
-        - ./data/infra/php.ini:/usr/local/etc/php/php.ini
-        - ./data/infra/frankenphp_caddy_data:/data
-        - ./data/infra/frankenphp_caddy_config:/config
-      links:
-        - shlink_db_mysql
-        - shlink_db_postgres
-        - shlink_db_maria
-        - shlink_db_ms
-        - shlink_redis
-        - shlink_redis_acl
-        - shlink_mercure
-        - shlink_mercure_proxy
-        - shlink_rabbitmq
-        - shlink_matomo
-      environment:
-        FRANKENPHP_CONFIG: 'worker /home/shlink/bin/frankenphp-worker.php'
-        SERVER_NAME: ':8008 https:8009'
-      extra_hosts:
-        - 'host.docker.internal:host-gateway'
-      tty: true
-
     shlink_db_mysql:
         container_name: shlink_db_mysql
         user: 1000:1000

docker/docker-entrypoint.sh

@@ -4,7 +4,7 @@ set -e
 cd /etc/shlink
 
 # Create data directories if they do not exist. This allows data dir to be mounted as an empty dir if needed
-mkdir -p data/cache data/locks data/log data/proxies data/temp-geolite
+mkdir -p data/cache data/locks data/log data/proxies
 
 flags="--no-interaction --clear-db-cache"
 

docs/swagger/definitions/SetShortUrlRedirectRule.json

@@ -19,13 +19,9 @@
               "device",
               "language",
               "query-param",
-              "any-value-query-param",
-              "valueless-query-param",
               "ip-address",
               "geolocation-country-code",
-              "geolocation-city-name",
-              "before-date",
-              "after-date"
+              "geolocation-city-name"
             ],
             "description": "The type of the condition, which will determine the logic used to match it"
           },
@@ -33,7 +29,7 @@
             "type": ["string", "null"]
           },
           "matchValue":  {
-            "type": ["string", "null"]
+            "type": "string"
           }
         }
       }

docs/swagger/paths/v1_short-urls.json

@@ -31,7 +31,7 @@
             {
                 "name": "searchTerm",
                 "in": "query",
-                "description": "A query used to filter results by searching for it on the longUrl and shortCode fields.",
+                "description": "A query used to filter results by searching for it on the longUrl and shortCode fields. (Since v1.3.0)",
                 "required": false,
                 "schema": {
                     "type": "string"
@@ -40,7 +40,7 @@
             {
                 "name": "tags[]",
                 "in": "query",
-                "description": "A list of tags used to filter the result set. Only short URLs **with** these tags will be returned.",
+                "description": "A list of tags used to filter the result set. Only short URLs tagged with at least one of the provided tags will be returned. (Since v1.3.0)",
                 "required": false,
                 "schema": {
                     "type": "array",
@@ -52,29 +52,7 @@
             {
                 "name": "tagsMode",
                 "in": "query",
-                "description": "Tells how the filtering by `tags` should work, returning short URLs containing \"any\" of the tags, or \"all\" the tags. Defaults to \"any\".<br />It's ignored if `tags` is not provided.",
-                "required": false,
-                "schema": {
-                    "type": "string",
-                    "enum": ["any", "all"]
-                }
-            },
-            {
-                "name": "excludeTags[]",
-                "in": "query",
-                "description": "A list of tags used to filter the result set. Only short URLs **without** these tags will be returned.",
-                "required": false,
-                "schema": {
-                    "type": "array",
-                    "items": {
-                        "type": "string"
-                    }
-                }
-            },
-            {
-                "name": "excludeTagsMode",
-                "in": "query",
-                "description": "Tells how the filtering by `excludeTags` should work, returning short URLs not containing \"any\" of the tags, or not containing \"all\" the tags. Defaults to \"any\".<br />It's ignored if `excludeTags` is not provided.",
+                "description": "Tells how the filtering by tags should work, returning short URLs containing \"any\" of the tags, or \"all\" the tags. It's ignored if no tags are provided, and defaults to \"any\" if not provided.",
                 "required": false,
                 "schema": {
                     "type": "string",
@@ -156,15 +134,6 @@
                 "schema": {
                     "type": "string"
                 }
-            },
-            {
-                "name": "apiKeyName",
-                "in": "query",
-                "description": "Only get short URLs created with this API key.<br />This value is **ignored** if the request is performed with a non-admin API key that does not match this name.",
-                "required": false,
-                "schema": {
-                    "type": "string"
-                }
             }
         ],
         "security": [

docs/swagger/paths/v2_tags_{tag}_visits.json

@@ -64,10 +64,6 @@
                     "type": "string",
                     "enum": ["true"]
                 }
-            },
-            {
-                "$ref": "../parameters/domain.json",
-                "description": "Return visits for short URLs that belong to this domain. Use **DEFAULT** keyword to return visits from default domain."
             }
         ],
         "security": [

docs/swagger/paths/v2_visits_non-orphan.json

@@ -55,10 +55,6 @@
                     "type": "string",
                     "enum": ["true"]
                 }
-            },
-            {
-                "$ref": "../parameters/domain.json",
-                "description": "Return visits for short URLs that belong to this domain. Use **DEFAULT** keyword to return visits from default domain."
             }
         ],
         "security": [

docs/swagger/paths/v2_visits_orphan.json

@@ -65,10 +65,6 @@
                     "type": "string",
                     "enum": ["invalid_short_url", "base_url", "regular_404"]
                 }
-            },
-            {
-                "$ref": "../parameters/domain.json",
-                "description": "Return only visits for this domain. Use **DEFAULT** keyword to return visits from default domain."
             }
         ],
         "security": [

docs/swagger/paths/{shortCode}_qr-code.json

@@ -0,0 +1,120 @@
+{
+    "get": {
+        "operationId": "shortUrlQrCode",
+        "tags": [
+            "URL Shortener"
+        ],
+        "summary": "Short URL QR code",
+        "description": "Generates a QR code image pointing to a short URL.<br />Since this is not an API endpoint but an image one, when an invalid value is provided for any of the query params, they will fall to their default values instead of throwing an error.",
+        "parameters": [
+            {
+                "$ref": "../parameters/shortCode.json"
+            },
+            {
+                "name": "size",
+                "in": "query",
+                "description": "The size of the image to be returned.",
+                "required": false,
+                "schema": {
+                    "type": "integer",
+                    "minimum": 50,
+                    "maximum": 1000,
+                    "default": 300
+                }
+            },
+            {
+                "name": "format",
+                "in": "query",
+                "description": "The format for the QR code image, being valid values png and svg. Not providing the param or providing any other value will fall back to png.",
+                "required": false,
+                "schema": {
+                    "type": "string",
+                    "enum": ["png", "svg"],
+                    "default": "png"
+                }
+            },
+            {
+                "name": "margin",
+                "in": "query",
+                "description": "The margin around the QR code image.",
+                "required": false,
+                "schema": {
+                    "type": "integer",
+                    "minimum": 0,
+                    "default": 0
+                }
+            },
+            {
+                "name": "errorCorrection",
+                "in": "query",
+                "description": "The error correction level to apply to the QR code: **[L]**ow, **[M]**edium, **[Q]**uartile or **[H]**igh. See [docs](https://www.qrcode.com/en/about/error_correction.html).",
+                "required": false,
+                "schema": {
+                    "type": "string",
+                    "enum": ["L", "M", "Q", "H"],
+                    "default": "L"
+                }
+            },
+            {
+                "name": "roundBlockSize",
+                "in": "query",
+                "description": "Allows to disable block size rounding, which might reduce the readability of the QR code, but ensures no extra margin is added.",
+                "required": false,
+                "schema": {
+                    "type": "string",
+                    "enum": ["true", "false"],
+                    "default": "false"
+                }
+            },
+            {
+                "name": "color",
+                "in": "query",
+                "description": "The QR code foreground color. It should be an hex representation of a color, in 3 or 6 characters, optionally preceded by the \"#\" character.",
+                "required": false,
+                "schema": {
+                    "type": "string",
+                    "default": "#000000"
+                }
+            },
+            {
+                "name": "bgColor",
+                "in": "query",
+                "description": "The QR code background color. It should be an hex representation of a color, in 3 or 6 characters, optionally preceded by the \"#\" character.",
+                "required": false,
+                "schema": {
+                    "type": "string",
+                    "default": "#ffffff"
+                }
+            },
+            {
+                "name": "logo",
+                "in": "query",
+                "description": "Currently used to disable the logo that was set via configuration options. It may be used in future to dynamically choose from multiple logos.",
+                "required": false,
+                "schema": {
+                    "type": "string",
+                    "enum": ["disable"]
+                }
+            }
+        ],
+        "responses": {
+            "200": {
+                "description": "QR code in PNG format",
+                "content": {
+                    "image/png": {
+                        "schema": {
+                            "type": "string",
+                            "format": "binary"
+                        }
+                    },
+                    "image/svg+xml": {
+                        "schema": {
+                            "type": "string",
+                            "format": "binary"
+                        }
+                    }
+                }
+            }
+        }
+    }
+}

docs/swagger/swagger.json

@@ -133,6 +133,9 @@
         },
         "/{shortCode}/track": {
             "$ref": "paths/{shortCode}_track.json"
+        },
+        "/{shortCode}/qr-code": {
+            "$ref": "paths/{shortCode}_qr-code.json"
         }
     }
 }

module/CLI/config/cli.config.php

@@ -26,7 +26,6 @@ return [
 
             Command\Api\GenerateKeyCommand::NAME => Command\Api\GenerateKeyCommand::class,
             Command\Api\DisableKeyCommand::NAME => Command\Api\DisableKeyCommand::class,
-            Command\Api\DeleteKeyCommand::NAME => Command\Api\DeleteKeyCommand::class,
             Command\Api\ListKeysCommand::NAME => Command\Api\ListKeysCommand::class,
             Command\Api\InitialApiKeyCommand::NAME => Command\Api\InitialApiKeyCommand::class,
             Command\Api\RenameApiKeyCommand::NAME => Command\Api\RenameApiKeyCommand::class,

module/CLI/config/dependencies.config.php

@@ -32,7 +32,6 @@ return [
 
             RedirectRule\RedirectRuleHandler::class => InvokableFactory::class,
             Util\ProcessRunner::class => ConfigAbstractFactory::class,
-            Util\PhpProcessRunner::class => ConfigAbstractFactory::class,
 
             ApiKey\RoleResolver::class => ConfigAbstractFactory::class,
 
@@ -53,7 +52,6 @@ return [
 
             Command\Api\GenerateKeyCommand::class => ConfigAbstractFactory::class,
             Command\Api\DisableKeyCommand::class => ConfigAbstractFactory::class,
-            Command\Api\DeleteKeyCommand::class => ConfigAbstractFactory::class,
             Command\Api\ListKeysCommand::class => ConfigAbstractFactory::class,
             Command\Api\InitialApiKeyCommand::class => ConfigAbstractFactory::class,
             Command\Api\RenameApiKeyCommand::class => ConfigAbstractFactory::class,
@@ -80,7 +78,6 @@ return [
 
     ConfigAbstractFactory::class => [
         Util\ProcessRunner::class => [SymfonyCli\Helper\ProcessHelper::class],
-        Util\PhpProcessRunner::class => [Util\ProcessRunner::class, PhpExecutableFinder::class],
         ApiKey\RoleResolver::class => [DomainService::class, UrlShortenerOptions::class],
 
         Command\ShortUrl\CreateShortUrlCommand::class => [
@@ -107,11 +104,10 @@ return [
         ],
         Command\Visit\GetOrphanVisitsCommand::class => [Visit\VisitsStatsHelper::class],
         Command\Visit\DeleteOrphanVisitsCommand::class => [Visit\VisitsDeleter::class],
-        Command\Visit\GetNonOrphanVisitsCommand::class => [Visit\VisitsStatsHelper::class],
+        Command\Visit\GetNonOrphanVisitsCommand::class => [Visit\VisitsStatsHelper::class, ShortUrlStringifier::class],
 
         Command\Api\GenerateKeyCommand::class => [ApiKeyService::class, ApiKey\RoleResolver::class],
         Command\Api\DisableKeyCommand::class => [ApiKeyService::class],
-        Command\Api\DeleteKeyCommand::class => [ApiKeyService::class],
         Command\Api\ListKeysCommand::class => [ApiKeyService::class],
         Command\Api\InitialApiKeyCommand::class => [ApiKeyService::class],
         Command\Api\RenameApiKeyCommand::class => [ApiKeyService::class],
@@ -119,11 +115,11 @@ return [
         Command\Tag\ListTagsCommand::class => [TagService::class],
         Command\Tag\RenameTagCommand::class => [TagService::class],
         Command\Tag\DeleteTagsCommand::class => [TagService::class],
-        Command\Tag\GetTagVisitsCommand::class => [Visit\VisitsStatsHelper::class],
+        Command\Tag\GetTagVisitsCommand::class => [Visit\VisitsStatsHelper::class, ShortUrlStringifier::class],
 
         Command\Domain\ListDomainsCommand::class => [DomainService::class],
         Command\Domain\DomainRedirectsCommand::class => [DomainService::class],
-        Command\Domain\GetDomainVisitsCommand::class => [Visit\VisitsStatsHelper::class],
+        Command\Domain\GetDomainVisitsCommand::class => [Visit\VisitsStatsHelper::class, ShortUrlStringifier::class],
 
         Command\RedirectRule\ManageRedirectRulesCommand::class => [
             ShortUrl\ShortUrlResolver::class,
@@ -138,11 +134,16 @@ return [
 
         Command\Db\CreateDatabaseCommand::class => [
             LockFactory::class,
-            Util\PhpProcessRunner::class,
+            Util\ProcessRunner::class,
+            PhpExecutableFinder::class,
             'em',
             NoDbNameConnectionFactory::SERVICE_NAME,
         ],
-        Command\Db\MigrateDatabaseCommand::class => [LockFactory::class, Util\PhpProcessRunner::class],
+        Command\Db\MigrateDatabaseCommand::class => [
+            LockFactory::class,
+            Util\ProcessRunner::class,
+            PhpExecutableFinder::class,
+        ],
     ],
 
 ];

module/CLI/src/ApiKey/RoleResolver.php

@@ -4,13 +4,15 @@ declare(strict_types=1);
 
 namespace Shlinkio\Shlink\CLI\ApiKey;
 
-use Shlinkio\Shlink\CLI\Command\Api\Input\ApiKeyInput;
 use Shlinkio\Shlink\CLI\Exception\InvalidRoleConfigException;
 use Shlinkio\Shlink\Core\Config\Options\UrlShortenerOptions;
 use Shlinkio\Shlink\Core\Domain\DomainServiceInterface;
 use Shlinkio\Shlink\Rest\ApiKey\Model\RoleDefinition;
+use Shlinkio\Shlink\Rest\ApiKey\Role;
+use Symfony\Component\Console\Input\InputInterface;
+
+use function is_string;
 
-/** @deprecated API key roles are deprecated */
 readonly class RoleResolver implements RoleResolverInterface
 {
     public function __construct(
@@ -19,16 +21,16 @@ readonly class RoleResolver implements RoleResolverInterface
     ) {
     }
 
-    public function determineRoles(ApiKeyInput $input): iterable
+    public function determineRoles(InputInterface $input): iterable
     {
-        $domainAuthority = $input->domain;
-        $author = $input->authorOnly;
-        $noOrphanVisits = $input->noOrphanVisits;
+        $domainAuthority = $input->getOption(Role::DOMAIN_SPECIFIC->paramName());
+        $author = $input->getOption(Role::AUTHORED_SHORT_URLS->paramName());
+        $noOrphanVisits = $input->getOption(Role::NO_ORPHAN_VISITS->paramName());
 
         if ($author) {
             yield RoleDefinition::forAuthoredShortUrls();
         }
-        if ($domainAuthority !== null) {
+        if (is_string($domainAuthority)) {
             yield $this->resolveRoleForAuthority($domainAuthority);
         }
         if ($noOrphanVisits) {

module/CLI/src/ApiKey/RoleResolverInterface.php

@@ -4,14 +4,13 @@ declare(strict_types=1);
 
 namespace Shlinkio\Shlink\CLI\ApiKey;
 
-use Shlinkio\Shlink\CLI\Command\Api\Input\ApiKeyInput;
 use Shlinkio\Shlink\Rest\ApiKey\Model\RoleDefinition;
+use Symfony\Component\Console\Input\InputInterface;
 
-/** @deprecated API key roles are deprecated */
 interface RoleResolverInterface
 {
     /**
      * @return iterable<RoleDefinition>
      */
-    public function determineRoles(ApiKeyInput $input): iterable;
+    public function determineRoles(InputInterface $input): iterable;
 }

module/CLI/src/Command/Api/DeleteKeyCommand.php

@@ -1,94 +0,0 @@
-<?php
-
-declare(strict_types=1);
-
-namespace Shlinkio\Shlink\CLI\Command\Api;
-
-use Shlinkio\Shlink\Rest\Entity\ApiKey;
-use Shlinkio\Shlink\Rest\Exception\ApiKeyNotFoundException;
-use Shlinkio\Shlink\Rest\Service\ApiKeyServiceInterface;
-use Symfony\Component\Console\Attribute\Argument;
-use Symfony\Component\Console\Attribute\AsCommand;
-use Symfony\Component\Console\Command\Command;
-use Symfony\Component\Console\Input\InputInterface;
-use Symfony\Component\Console\Output\OutputInterface;
-use Symfony\Component\Console\Style\SymfonyStyle;
-
-use function Shlinkio\Shlink\Core\ArrayUtils\map;
-use function sprintf;
-
-#[AsCommand(
-    name: DeleteKeyCommand::NAME,
-    description: 'Deletes an API key by name',
-    help: <<<HELP
-    The <info>%command.name%</info> command allows you to delete an existing API key via its name.
-
-    If no arguments are provided, you will be prompted to select one of the existing API keys.
-
-        <info>%command.full_name%</info>
-
-    You can optionally pass the API key name to be disabled:
-
-        <info>%command.full_name% the_key_name</info>
-
-    HELP,
-)]
-class DeleteKeyCommand extends Command
-{
-    public const string NAME = 'api-key:delete';
-
-    public function __construct(private readonly ApiKeyServiceInterface $apiKeyService)
-    {
-        parent::__construct();
-    }
-
-    protected function interact(InputInterface $input, OutputInterface $output): void
-    {
-        $apiKeyName = $input->getArgument('name');
-
-        if ($apiKeyName === null) {
-            $apiKeys = $this->apiKeyService->listKeys();
-            $name = new SymfonyStyle($input, $output)->choice(
-                'What API key do you want to delete?',
-                map($apiKeys, static fn (ApiKey $apiKey) => $apiKey->name),
-            );
-
-            $input->setArgument('name', $name);
-        }
-    }
-
-    public function __invoke(
-        SymfonyStyle $io,
-        InputInterface $input,
-        #[Argument(description: 'The API key to delete.')]
-        string|null $name = null,
-    ): int {
-        if ($name === null) {
-            $io->warning('An API key name was not provided.');
-            return Command::INVALID;
-        }
-
-        if (! $this->shouldProceed($io, $input)) {
-            return Command::INVALID;
-        }
-
-        try {
-            $this->apiKeyService->deleteByName($name);
-            $io->success(sprintf('API key "%s" properly deleted', $name));
-            return Command::SUCCESS;
-        } catch (ApiKeyNotFoundException $e) {
-            $io->error($e->getMessage());
-            return Command::FAILURE;
-        }
-    }
-
-    private function shouldProceed(SymfonyStyle $io, InputInterface $input): bool
-    {
-        if (! $input->isInteractive()) {
-            return true;
-        }
-
-        $io->warning('You are about to delete an API key. This action cannot be undone.');
-        return $io->confirm('Are you sure you want to delete the API key?');
-    }
-}

module/CLI/src/Command/Api/DisableKeyCommand.php

@@ -4,35 +4,20 @@ declare(strict_types=1);
 
 namespace Shlinkio\Shlink\CLI\Command\Api;
 
+use Shlinkio\Shlink\CLI\Util\ExitCode;
 use Shlinkio\Shlink\Common\Exception\InvalidArgumentException;
 use Shlinkio\Shlink\Rest\Entity\ApiKey;
 use Shlinkio\Shlink\Rest\Service\ApiKeyServiceInterface;
-use Symfony\Component\Console\Attribute\Argument;
-use Symfony\Component\Console\Attribute\AsCommand;
 use Symfony\Component\Console\Command\Command;
+use Symfony\Component\Console\Input\InputArgument;
 use Symfony\Component\Console\Input\InputInterface;
+use Symfony\Component\Console\Input\InputOption;
 use Symfony\Component\Console\Output\OutputInterface;
 use Symfony\Component\Console\Style\SymfonyStyle;
 
 use function Shlinkio\Shlink\Core\ArrayUtils\map;
 use function sprintf;
 
-#[AsCommand(
-    name: DisableKeyCommand::NAME,
-    description: 'Disables an API key by name',
-    help: <<<HELP
-    The <info>%command.name%</info> command allows you to disable an existing API key.
-
-    If no arguments are provided, you will be prompted to select one of the existing non-disabled API keys.
-
-        <info>%command.full_name%</info>
-
-    You can optionally pass the API key name to be disabled:
-
-        <info>%command.full_name% the_key_name</info>
-
-    HELP,
-)]
 class DisableKeyCommand extends Command
 {
     public const string NAME = 'api-key:disable';
@@ -42,37 +27,82 @@ class DisableKeyCommand extends Command
         parent::__construct();
     }
 
+    protected function configure(): void
+    {
+        $help = <<<HELP
+        The <info>%command.name%</info> command allows you to disable an existing API key, via its name or the
+        plain-text key.
+
+        If no arguments are provided, you will be prompted to select one of the existing non-disabled API keys.
+
+            <info>%command.full_name%</info>
+
+        You can optionally pass the API key name to be disabled. In that case <comment>--by-name</comment> is also
+        required, to indicate the first argument is the API key name and not the plain-text key:
+
+            <info>%command.full_name% the_key_name --by-name</info>
+
+        You can pass the plain-text key to be disabled, but that is <options=bold>DEPRECATED</>. In next major version,
+        the argument will always be assumed to be the name:
+
+            <info>%command.full_name% d6b6c60e-edcd-4e43-96ad-fa6b7014c143</info>
+
+        HELP;
+
+        $this
+            ->setName(self::NAME)
+            ->setDescription('Disables an API key by name or plain-text key (providing a plain-text key is DEPRECATED)')
+            ->addArgument(
+                'keyOrName',
+                InputArgument::OPTIONAL,
+                'The API key to disable. Pass `--by-name` to indicate this value is the name and not the key.',
+            )
+            ->addOption(
+                'by-name',
+                mode: InputOption::VALUE_NONE,
+                description: 'Indicates the first argument is the API key name, not the plain-text key.',
+            )
+            ->setHelp($help);
+    }
+
     protected function interact(InputInterface $input, OutputInterface $output): void
     {
-        $name = $input->getArgument('name');
+        $keyOrName = $input->getArgument('keyOrName');
 
-        if ($name === null) {
+        if ($keyOrName === null) {
             $apiKeys = $this->apiKeyService->listKeys(enabledOnly: true);
-            $name = new SymfonyStyle($input, $output)->choice(
+            $name = (new SymfonyStyle($input, $output))->choice(
                 'What API key do you want to disable?',
                 map($apiKeys, static fn (ApiKey $apiKey) => $apiKey->name),
             );
 
-            $input->setArgument('name', $name);
+            $input->setArgument('keyOrName', $name);
+            $input->setOption('by-name', true);
         }
     }
 
-    public function __invoke(
-        SymfonyStyle $io,
-        #[Argument('The name of the API key to disable.')] string|null $name = null,
-    ): int {
-        if ($name === null) {
+    protected function execute(InputInterface $input, OutputInterface $output): int
+    {
+        $keyOrName = $input->getArgument('keyOrName');
+        $byName = $input->getOption('by-name');
+        $io = new SymfonyStyle($input, $output);
+
+        if (! $keyOrName) {
             $io->warning('An API key name was not provided.');
-            return Command::INVALID;
+            return ExitCode::EXIT_WARNING;
         }
 
         try {
-            $this->apiKeyService->disableByName($name);
-            $io->success(sprintf('API key "%s" properly disabled', $name));
-            return Command::SUCCESS;
+            if ($byName) {
+                $this->apiKeyService->disableByName($keyOrName);
+            } else {
+                $this->apiKeyService->disableByKey($keyOrName);
+            }
+            $io->success(sprintf('API key "%s" properly disabled', $keyOrName));
+            return ExitCode::EXIT_SUCCESS;
         } catch (InvalidArgumentException $e) {
             $io->error($e->getMessage());
-            return Command::FAILURE;
+            return ExitCode::EXIT_FAILURE;
         }
     }
 }

module/CLI/src/Command/Api/GenerateKeyCommand.php

@@ -4,40 +4,23 @@ declare(strict_types=1);
 
 namespace Shlinkio\Shlink\CLI\Command\Api;
 
+use Cake\Chronos\Chronos;
 use Shlinkio\Shlink\CLI\ApiKey\RoleResolverInterface;
-use Shlinkio\Shlink\CLI\Command\Api\Input\ApiKeyInput;
+use Shlinkio\Shlink\CLI\Util\ExitCode;
 use Shlinkio\Shlink\CLI\Util\ShlinkTable;
 use Shlinkio\Shlink\Rest\ApiKey\Model\ApiKeyMeta;
 use Shlinkio\Shlink\Rest\ApiKey\Role;
 use Shlinkio\Shlink\Rest\Entity\ApiKey;
 use Shlinkio\Shlink\Rest\Service\ApiKeyServiceInterface;
-use Symfony\Component\Console\Attribute\AsCommand;
-use Symfony\Component\Console\Attribute\MapInput;
 use Symfony\Component\Console\Command\Command;
 use Symfony\Component\Console\Input\InputInterface;
+use Symfony\Component\Console\Input\InputOption;
+use Symfony\Component\Console\Output\OutputInterface;
 use Symfony\Component\Console\Style\SymfonyStyle;
 
 use function Shlinkio\Shlink\Core\arrayToString;
-use function Shlinkio\Shlink\Core\normalizeOptionalDate;
 use function sprintf;
 
-#[AsCommand(
-    name: GenerateKeyCommand::NAME,
-    description: 'Generate a new valid API key',
-    help: <<<HELP
-        The <info>%command.name%</info> generates a new valid API key.
-
-            <info>%command.full_name%</info>
-
-        You can optionally set its name for tracking purposes with <comment>--name</comment> or <comment>-m</comment>:
-
-            <info>%command.full_name% --name Alice</info>
-
-        You can optionally set its expiration date with <comment>--expiration-date</comment> or <comment>-e</comment>:
-
-            <info>%command.full_name% --expiration-date 2020-01-01</info>
-        HELP,
-)]
 class GenerateKeyCommand extends Command
 {
     public const string NAME = 'api-key:generate';
@@ -49,13 +32,80 @@ class GenerateKeyCommand extends Command
         parent::__construct();
     }
 
-    public function __invoke(SymfonyStyle $io, InputInterface $input, #[MapInput] ApiKeyInput $inputData): int
+    protected function configure(): void
     {
-        $expirationDate = $inputData->expirationDate;
+        $authorOnly = Role::AUTHORED_SHORT_URLS->paramName();
+        $domainOnly = Role::DOMAIN_SPECIFIC->paramName();
+        $noOrphanVisits = Role::NO_ORPHAN_VISITS->paramName();
+
+        $help = <<<HELP
+        The <info>%command.name%</info> generates a new valid API key.
+
+            <info>%command.full_name%</info>
+
+        You can optionally set its name for tracking purposes with <comment>--name</comment> or <comment>-m</comment>:
+
+            <info>%command.full_name% --name Alice</info>
+
+        You can optionally set its expiration date with <comment>--expiration-date</comment> or <comment>-e</comment>:
+
+            <info>%command.full_name% --expiration-date 2020-01-01</info>
+
+        You can also set roles to the API key:
+
+            * Can interact with short URLs created with this API key: <info>%command.full_name% --{$authorOnly}</info>
+            * Can interact with short URLs for one domain: <info>%command.full_name% --{$domainOnly}=example.com</info>
+            * Cannot see orphan visits: <info>%command.full_name% --{$noOrphanVisits}</info>
+            * All: <info>%command.full_name% --{$authorOnly} --{$domainOnly}=example.com --{$noOrphanVisits}</info>
+        HELP;
+
+        $this
+            ->setName(self::NAME)
+            ->setDescription('Generate a new valid API key.')
+            ->addOption(
+                'name',
+                'm',
+                InputOption::VALUE_REQUIRED,
+                'The name by which this API key will be known.',
+            )
+            ->addOption(
+                'expiration-date',
+                'e',
+                InputOption::VALUE_REQUIRED,
+                'The date in which the API key should expire. Use any valid PHP format.',
+            )
+            ->addOption(
+                $authorOnly,
+                'a',
+                InputOption::VALUE_NONE,
+                sprintf('Adds the "%s" role to the new API key.', Role::AUTHORED_SHORT_URLS->value),
+            )
+            ->addOption(
+                $domainOnly,
+                'd',
+                InputOption::VALUE_REQUIRED,
+                sprintf(
+                    'Adds the "%s" role to the new API key, with the domain provided.',
+                    Role::DOMAIN_SPECIFIC->value,
+                ),
+            )
+            ->addOption(
+                $noOrphanVisits,
+                'o',
+                InputOption::VALUE_NONE,
+                sprintf('Adds the "%s" role to the new API key.', Role::NO_ORPHAN_VISITS->value),
+            )
+            ->setHelp($help);
+    }
+
+    protected function execute(InputInterface $input, OutputInterface $output): int
+    {
+        $io = new SymfonyStyle($input, $output);
+        $expirationDate = $input->getOption('expiration-date');
         $apiKeyMeta = ApiKeyMeta::fromParams(
-            name: $inputData->name,
-            expirationDate: isset($expirationDate) ? normalizeOptionalDate($expirationDate) : null,
-            roleDefinitions: $this->roleResolver->determineRoles($inputData),
+            name: $input->getOption('name'),
+            expirationDate: isset($expirationDate) ? Chronos::parse($expirationDate) : null,
+            roleDefinitions: $this->roleResolver->determineRoles($input),
         );
 
         $apiKey = $this->apiKeyService->create($apiKeyMeta);
@@ -73,6 +123,6 @@ class GenerateKeyCommand extends Command
             );
         }
 
-        return Command::SUCCESS;
+        return ExitCode::EXIT_SUCCESS;
     }
 }

module/CLI/src/Command/Api/InitialApiKeyCommand.php

@@ -4,16 +4,13 @@ declare(strict_types=1);
 
 namespace Shlinkio\Shlink\CLI\Command\Api;
 
+use Shlinkio\Shlink\CLI\Util\ExitCode;
 use Shlinkio\Shlink\Rest\Service\ApiKeyServiceInterface;
-use Symfony\Component\Console\Attribute\Argument;
-use Symfony\Component\Console\Attribute\AsCommand;
 use Symfony\Component\Console\Command\Command;
-use Symfony\Component\Console\Style\SymfonyStyle;
+use Symfony\Component\Console\Input\InputArgument;
+use Symfony\Component\Console\Input\InputInterface;
+use Symfony\Component\Console\Output\OutputInterface;
 
-#[AsCommand(
-    name: InitialApiKeyCommand::NAME,
-    description: 'Tries to create initial API key',
-)]
 class InitialApiKeyCommand extends Command
 {
     public const string NAME = 'api-key:initial';
@@ -23,16 +20,24 @@ class InitialApiKeyCommand extends Command
         parent::__construct();
     }
 
-    public function __invoke(
-        SymfonyStyle $io,
-        #[Argument('The initial API to create')] string $apiKey,
-    ): int {
-        $result = $this->apiKeyService->createInitial($apiKey);
+    protected function configure(): void
+    {
+        $this
+            ->setHidden()
+            ->setName(self::NAME)
+            ->setDescription('Tries to create initial API key')
+            ->addArgument('apiKey', InputArgument::REQUIRED, 'The initial API to create');
+    }
 
-        if ($result === null && $io->isVerbose()) {
-            $io->writeln('<comment>Other API keys already exist. Initial API key creation skipped.</comment>');
+    protected function execute(InputInterface $input, OutputInterface $output): int
+    {
+        $key = $input->getArgument('apiKey');
+        $result = $this->apiKeyService->createInitial($key);
+
+        if ($result === null && $output->isVerbose()) {
+            $output->writeln('<comment>Other API keys already exist. Initial API key creation skipped.</comment>');
         }
 
-        return Command::SUCCESS;
+        return ExitCode::EXIT_SUCCESS;
     }
 }

module/CLI/src/Command/Api/Input/ApiKeyInput.php

@@ -1,33 +0,0 @@
-<?php
-
-declare(strict_types=1);
-
-namespace Shlinkio\Shlink\CLI\Command\Api\Input;
-
-use Shlinkio\Shlink\Rest\ApiKey\Role;
-use Symfony\Component\Console\Attribute\Option;
-
-final class ApiKeyInput
-{
-    #[Option('The unique name by which this API key will be known', shortcut: 'm')]
-    public string|null $name = null;
-
-    #[Option('The date in which the API key should expire. Use any valid PHP format', shortcut: 'e')]
-    public string|null $expirationDate = null;
-
-    /** @deprecated */
-    #[Option('Adds the "' . Role::AUTHORED_SHORT_URLS->value . '" role to the new API key', shortcut: 'a')]
-    public bool $authorOnly = false;
-
-    /** @deprecated */
-    #[Option(
-        'Adds the "' . Role::DOMAIN_SPECIFIC->value . '" role to the new API key, with provided domain',
-        name: 'domain-only',
-        shortcut: 'd',
-    )]
-    public string|null $domain = null;
-
-    /** @deprecated */
-    #[Option('Adds the "' . Role::NO_ORPHAN_VISITS->value . '" role to the new API key', shortcut: 'o')]
-    public bool $noOrphanVisits = false;
-}

module/CLI/src/Command/Api/ListKeysCommand.php

@@ -4,24 +4,21 @@ declare(strict_types=1);
 
 namespace Shlinkio\Shlink\CLI\Command\Api;
 
+use Shlinkio\Shlink\CLI\Util\ExitCode;
 use Shlinkio\Shlink\CLI\Util\ShlinkTable;
 use Shlinkio\Shlink\Rest\ApiKey\Role;
 use Shlinkio\Shlink\Rest\Entity\ApiKey;
 use Shlinkio\Shlink\Rest\Service\ApiKeyServiceInterface;
-use Symfony\Component\Console\Attribute\AsCommand;
-use Symfony\Component\Console\Attribute\Option;
 use Symfony\Component\Console\Command\Command;
-use Symfony\Component\Console\Style\SymfonyStyle;
+use Symfony\Component\Console\Input\InputInterface;
+use Symfony\Component\Console\Input\InputOption;
+use Symfony\Component\Console\Output\OutputInterface;
 
 use function array_filter;
 use function array_map;
 use function implode;
 use function sprintf;
 
-#[AsCommand(
-    name: ListKeysCommand::NAME,
-    description: 'Lists all the available API keys.',
-)]
 class ListKeysCommand extends Command
 {
     private const string ERROR_STRING_PATTERN = '<fg=red>%s</>';
@@ -35,14 +32,23 @@ class ListKeysCommand extends Command
         parent::__construct();
     }
 
-    public function __invoke(
-        SymfonyStyle $io,
-        #[Option(
-            description: 'Tells if only enabled API keys should be returned.',
-            shortcut: 'e',
-        )]
-        bool $enabledOnly = false,
-    ): int {
+    protected function configure(): void
+    {
+        $this
+            ->setName(self::NAME)
+            ->setDescription('Lists all the available API keys.')
+            ->addOption(
+                'enabled-only',
+                'e',
+                InputOption::VALUE_NONE,
+                'Tells if only enabled API keys should be returned.',
+            );
+    }
+
+    protected function execute(InputInterface $input, OutputInterface $output): int
+    {
+        $enabledOnly = $input->getOption('enabled-only');
+
         $rows = array_map(function (ApiKey $apiKey) use ($enabledOnly) {
             $expiration = $apiKey->expirationDate;
             $messagePattern = $this->determineMessagePattern($apiKey);
@@ -60,14 +66,14 @@ class ListKeysCommand extends Command
             return $rowData;
         }, $this->apiKeyService->listKeys($enabledOnly));
 
-        ShlinkTable::withRowSeparators($io)->render(array_filter([
+        ShlinkTable::withRowSeparators($output)->render(array_filter([
             'Name',
             ! $enabledOnly ? 'Is enabled' : null,
             'Expiration date',
             'Roles',
         ]), $rows);
 
-        return Command::SUCCESS;
+        return ExitCode::EXIT_SUCCESS;
     }
 
     private function determineMessagePattern(ApiKey $apiKey): string

module/CLI/src/Command/Api/RenameApiKeyCommand.php

@@ -4,18 +4,19 @@ declare(strict_types=1);
 
 namespace Shlinkio\Shlink\CLI\Command\Api;
 
+use Shlinkio\Shlink\CLI\Util\ExitCode;
+use Shlinkio\Shlink\Core\Exception\InvalidArgumentException;
 use Shlinkio\Shlink\Core\Model\Renaming;
+use Shlinkio\Shlink\Rest\Entity\ApiKey;
 use Shlinkio\Shlink\Rest\Service\ApiKeyServiceInterface;
-use Symfony\Component\Console\Attribute\Argument;
-use Symfony\Component\Console\Attribute\AsCommand;
-use Symfony\Component\Console\Attribute\Ask;
 use Symfony\Component\Console\Command\Command;
+use Symfony\Component\Console\Input\InputArgument;
+use Symfony\Component\Console\Input\InputInterface;
+use Symfony\Component\Console\Output\OutputInterface;
 use Symfony\Component\Console\Style\SymfonyStyle;
 
-#[AsCommand(
-    name: RenameApiKeyCommand::NAME,
-    description: 'Renames an API key by name',
-)]
+use function Shlinkio\Shlink\Core\ArrayUtils\map;
+
 class RenameApiKeyCommand extends Command
 {
     public const string NAME = 'api-key:rename';
@@ -25,16 +26,52 @@ class RenameApiKeyCommand extends Command
         parent::__construct();
     }
 
-    public function __invoke(
-        SymfonyStyle $io,
-        #[Argument(description: 'Current name of the API key to rename'), Ask('What API key do you want to rename?')]
-        string $oldName,
-        #[Argument(description: 'New name to set to the API key'), Ask('What is the new name you want to set?')]
-        string $newName,
-    ): int {
+    protected function configure(): void
+    {
+        $this
+            ->setName(self::NAME)
+            ->setDescription('Renames an API key by name')
+            ->addArgument('oldName', InputArgument::REQUIRED, 'Current name of the API key to rename')
+            ->addArgument('newName', InputArgument::REQUIRED, 'New name to set to the API key');
+    }
+
+    protected function interact(InputInterface $input, OutputInterface $output): void
+    {
+        $io = new SymfonyStyle($input, $output);
+        $oldName = $input->getArgument('oldName');
+        $newName = $input->getArgument('newName');
+
+        if ($oldName === null) {
+            $apiKeys = $this->apiKeyService->listKeys();
+            $requestedOldName = $io->choice(
+                'What API key do you want to rename?',
+                map($apiKeys, static fn (ApiKey $apiKey) => $apiKey->name),
+            );
+
+            $input->setArgument('oldName', $requestedOldName);
+        }
+
+        if ($newName === null) {
+            $requestedNewName = $io->ask(
+                'What is the new name you want to set?',
+                validator: static fn (string|null $value): string => $value !== null
+                    ? $value
+                    : throw new InvalidArgumentException('The new name cannot be empty'),
+            );
+
+            $input->setArgument('newName', $requestedNewName);
+        }
+    }
+
+    protected function execute(InputInterface $input, OutputInterface $output): int
+    {
+        $io = new SymfonyStyle($input, $output);
+        $oldName = $input->getArgument('oldName');
+        $newName = $input->getArgument('newName');
+
         $this->apiKeyService->renameApiKey(Renaming::fromNames($oldName, $newName));
         $io->success('API key properly renamed');
 
-        return Command::SUCCESS;
+        return ExitCode::EXIT_SUCCESS;
     }
 }

module/CLI/src/Command/Config/ReadEnvVarCommand.php

@@ -5,13 +5,13 @@ declare(strict_types=1);
 namespace Shlinkio\Shlink\CLI\Command\Config;
 
 use Closure;
+use Shlinkio\Shlink\CLI\Util\ExitCode;
 use Shlinkio\Shlink\Core\Config\EnvVars;
-use Symfony\Component\Console\Attribute\Argument;
-use Symfony\Component\Console\Attribute\AsCommand;
-use Symfony\Component\Console\Attribute\Interact;
 use Symfony\Component\Console\Command\Command;
 use Symfony\Component\Console\Exception\InvalidArgumentException;
+use Symfony\Component\Console\Input\InputArgument;
 use Symfony\Component\Console\Input\InputInterface;
+use Symfony\Component\Console\Output\OutputInterface;
 use Symfony\Component\Console\Style\SymfonyStyle;
 
 use function Shlinkio\Shlink\Config\formatEnvVarValue;
@@ -19,11 +19,6 @@ use function Shlinkio\Shlink\Core\ArrayUtils\contains;
 use function Shlinkio\Shlink\Core\enumValues;
 use function sprintf;
 
-#[AsCommand(
-    name: ReadEnvVarCommand::NAME,
-    description: 'Display current value for an env var',
-    hidden: true,
-)]
 class ReadEnvVarCommand extends Command
 {
     public const string NAME = 'env-var:read';
@@ -37,11 +32,19 @@ class ReadEnvVarCommand extends Command
         parent::__construct();
     }
 
-    #[Interact]
-    public function askMissing(InputInterface $input, SymfonyStyle $io): void
+    protected function configure(): void
     {
-        /** @var string|null $envVar */
-        $envVar = $input->getArgument('env-var');
+        $this
+            ->setName(self::NAME)
+            ->setHidden()
+            ->setDescription('Display current value for an env var')
+            ->addArgument('envVar', InputArgument::REQUIRED, 'The env var to read');
+    }
+
+    protected function interact(InputInterface $input, OutputInterface $output): void
+    {
+        $io = new SymfonyStyle($input, $output);
+        $envVar = $input->getArgument('envVar');
         $validEnvVars = enumValues(EnvVars::class);
 
         if ($envVar === null) {
@@ -52,14 +55,14 @@ class ReadEnvVarCommand extends Command
             throw new InvalidArgumentException(sprintf('%s is not a valid Shlink environment variable', $envVar));
         }
 
-        $input->setArgument('env-var', $envVar);
+        $input->setArgument('envVar', $envVar);
     }
 
-    public function __invoke(
-        SymfonyStyle $io,
-        #[Argument(description: 'The env var to read')] string $envVar,
-    ): int {
-        $io->writeln(formatEnvVarValue(($this->loadEnvVar)($envVar)));
-        return Command::SUCCESS;
+    protected function execute(InputInterface $input, OutputInterface $output): int
+    {
+        $envVar = $input->getArgument('envVar');
+        $output->writeln(formatEnvVarValue(($this->loadEnvVar)($envVar)));
+
+        return ExitCode::EXIT_SUCCESS;
     }
 }

module/CLI/src/Command/Db/AbstractDatabaseCommand.php

@@ -0,0 +1,37 @@
+<?php
+
+declare(strict_types=1);
+
+namespace Shlinkio\Shlink\CLI\Command\Db;
+
+use Shlinkio\Shlink\CLI\Command\Util\AbstractLockedCommand;
+use Shlinkio\Shlink\CLI\Command\Util\LockedCommandConfig;
+use Shlinkio\Shlink\CLI\Util\ProcessRunnerInterface;
+use Symfony\Component\Console\Output\OutputInterface;
+use Symfony\Component\Lock\LockFactory;
+use Symfony\Component\Process\PhpExecutableFinder;
+
+abstract class AbstractDatabaseCommand extends AbstractLockedCommand
+{
+    private string $phpBinary;
+
+    public function __construct(
+        LockFactory $locker,
+        private readonly ProcessRunnerInterface $processRunner,
+        PhpExecutableFinder $phpFinder,
+    ) {
+        parent::__construct($locker);
+        $this->phpBinary = $phpFinder->find(false) ?: 'php';
+    }
+
+    protected function runPhpCommand(OutputInterface $output, array $command): void
+    {
+        $command = [$this->phpBinary, ...$command, '--no-interaction'];
+        $this->processRunner->run($output, $command);
+    }
+
+    protected function getLockConfig(): LockedCommandConfig
+    {
+        return LockedCommandConfig::blocking($this->getName() ?? static::class);
+    }
+}

module/CLI/src/Command/Db/CreateDatabaseCommand.php

@@ -7,65 +7,63 @@ namespace Shlinkio\Shlink\CLI\Command\Db;
 use Doctrine\DBAL\Connection;
 use Doctrine\ORM\EntityManagerInterface;
 use Doctrine\ORM\Mapping\ClassMetadata;
-use Shlinkio\Shlink\CLI\Command\Util\CommandUtils;
-use Shlinkio\Shlink\CLI\Command\Util\LockConfig;
+use Shlinkio\Shlink\CLI\Util\ExitCode;
 use Shlinkio\Shlink\CLI\Util\ProcessRunnerInterface;
-use Symfony\Component\Console\Attribute\AsCommand;
-use Symfony\Component\Console\Command\Command;
+use Symfony\Component\Console\Input\InputInterface;
+use Symfony\Component\Console\Output\OutputInterface;
 use Symfony\Component\Console\Style\SymfonyStyle;
 use Symfony\Component\Lock\LockFactory;
+use Symfony\Component\Process\PhpExecutableFinder;
 use Throwable;
 
 use function array_map;
 use function Shlinkio\Shlink\Core\ArrayUtils\contains;
 use function Shlinkio\Shlink\Core\ArrayUtils\some;
 
-#[AsCommand(
-    name: CreateDatabaseCommand::NAME,
-    description: 'Creates the database needed for shlink to work. It will do nothing if the database already exists',
-    hidden: true,
-)]
-class CreateDatabaseCommand extends Command
+class CreateDatabaseCommand extends AbstractDatabaseCommand
 {
     private readonly Connection $regularConn;
 
     public const string NAME = 'db:create';
-    public const string SCRIPT = 'bin/doctrine';
-    public const string COMMAND = 'orm:schema-tool:create';
+    public const string DOCTRINE_SCRIPT = 'bin/doctrine';
+    public const string DOCTRINE_CREATE_SCHEMA_COMMAND = 'orm:schema-tool:create';
 
     public function __construct(
-        private readonly LockFactory $locker,
-        private readonly ProcessRunnerInterface $processRunner,
+        LockFactory $locker,
+        ProcessRunnerInterface $processRunner,
+        PhpExecutableFinder $phpFinder,
         private readonly EntityManagerInterface $em,
         private readonly Connection $noDbNameConn,
     ) {
         $this->regularConn = $this->em->getConnection();
-        parent::__construct();
+        parent::__construct($locker, $processRunner, $phpFinder);
     }
 
-    public function __invoke(SymfonyStyle $io): int
+    protected function configure(): void
     {
-        return CommandUtils::executeWithLock(
-            $this->locker,
-            LockConfig::blocking(self::NAME),
-            $io,
-            fn () => $this->executeCommand($io),
-        );
+        $this
+            ->setName(self::NAME)
+            ->setHidden()
+            ->setDescription(
+                'Creates the database needed for shlink to work. It will do nothing if the database already exists',
+            );
     }
 
-    private function executeCommand(SymfonyStyle $io): int
+    protected function lockedExecute(InputInterface $input, OutputInterface $output): int
     {
+        $io = new SymfonyStyle($input, $output);
+
         if ($this->databaseTablesExist()) {
             $io->success('Database already exists. Run "db:migrate" command to make sure it is up to date.');
-            return self::SUCCESS;
+            return ExitCode::EXIT_SUCCESS;
         }
 
         // Create database
         $io->writeln('<fg=blue>Creating database tables...</>');
-        $this->processRunner->run($io, [self::SCRIPT, self::COMMAND, '--no-interaction']);
+        $this->runPhpCommand($output, [self::DOCTRINE_SCRIPT, self::DOCTRINE_CREATE_SCHEMA_COMMAND]);
         $io->success('Database properly created!');
 
-        return self::SUCCESS;
+        return ExitCode::EXIT_SUCCESS;
     }
 
     private function databaseTablesExist(): bool

module/CLI/src/Command/Db/MigrateDatabaseCommand.php

@@ -4,48 +4,33 @@ declare(strict_types=1);
 
 namespace Shlinkio\Shlink\CLI\Command\Db;
 
-use Shlinkio\Shlink\CLI\Command\Util\CommandUtils;
-use Shlinkio\Shlink\CLI\Command\Util\LockConfig;
-use Shlinkio\Shlink\CLI\Util\ProcessRunnerInterface;
-use Symfony\Component\Console\Attribute\AsCommand;
-use Symfony\Component\Console\Command\Command;
+use Shlinkio\Shlink\CLI\Util\ExitCode;
+use Symfony\Component\Console\Input\InputInterface;
+use Symfony\Component\Console\Output\OutputInterface;
 use Symfony\Component\Console\Style\SymfonyStyle;
-use Symfony\Component\Lock\LockFactory;
 
-#[AsCommand(
-    name: MigrateDatabaseCommand::NAME,
-    description: 'Runs database migrations, which will ensure the shlink database is up to date',
-    hidden: true,
-)]
-class MigrateDatabaseCommand extends Command
+class MigrateDatabaseCommand extends AbstractDatabaseCommand
 {
     public const string NAME = 'db:migrate';
-    public const string SCRIPT = 'vendor/doctrine/migrations/bin/doctrine-migrations.php';
-    public const string COMMAND = 'migrations:migrate';
+    public const string DOCTRINE_MIGRATIONS_SCRIPT = 'vendor/doctrine/migrations/bin/doctrine-migrations.php';
+    public const string DOCTRINE_MIGRATE_COMMAND = 'migrations:migrate';
 
-    public function __construct(
-        private readonly LockFactory $locker,
-        private readonly ProcessRunnerInterface $processRunner,
-    ) {
-        parent::__construct();
+    protected function configure(): void
+    {
+        $this
+            ->setName(self::NAME)
+            ->setHidden()
+            ->setDescription('Runs database migrations, which will ensure the shlink database is up to date.');
     }
 
-    public function __invoke(SymfonyStyle $io): int
+    protected function lockedExecute(InputInterface $input, OutputInterface $output): int
     {
-        return CommandUtils::executeWithLock(
-            $this->locker,
-            LockConfig::blocking(self::NAME),
-            $io,
-            fn () => $this->executeCommand($io),
-        );
-    }
+        $io = new SymfonyStyle($input, $output);
 
-    private function executeCommand(SymfonyStyle $io): int
-    {
         $io->writeln('<fg=blue>Migrating database...</>');
-        $this->processRunner->run($io, [self::SCRIPT, self::COMMAND, '--no-interaction']);
+        $this->runPhpCommand($output, [self::DOCTRINE_MIGRATIONS_SCRIPT, self::DOCTRINE_MIGRATE_COMMAND]);
         $io->success('Database properly migrated!');
 
-        return self::SUCCESS;
+        return ExitCode::EXIT_SUCCESS;
     }
 }

module/CLI/src/Command/Domain/DomainRedirectsCommand.php

@@ -4,14 +4,14 @@ declare(strict_types=1);
 
 namespace Shlinkio\Shlink\CLI\Command\Domain;
 
+use Shlinkio\Shlink\CLI\Util\ExitCode;
 use Shlinkio\Shlink\Core\Config\NotFoundRedirects;
 use Shlinkio\Shlink\Core\Domain\DomainServiceInterface;
 use Shlinkio\Shlink\Core\Domain\Model\DomainItem;
-use Symfony\Component\Console\Attribute\Argument;
-use Symfony\Component\Console\Attribute\AsCommand;
-use Symfony\Component\Console\Attribute\Interact;
 use Symfony\Component\Console\Command\Command;
+use Symfony\Component\Console\Input\InputArgument;
 use Symfony\Component\Console\Input\InputInterface;
+use Symfony\Component\Console\Output\OutputInterface;
 use Symfony\Component\Console\Style\SymfonyStyle;
 
 use function array_filter;
@@ -19,10 +19,6 @@ use function array_map;
 use function sprintf;
 use function str_contains;
 
-#[AsCommand(
-    name: DomainRedirectsCommand::NAME,
-    description: 'Set specific "not found" redirects for individual domains.',
-)]
 class DomainRedirectsCommand extends Command
 {
     public const string NAME = 'domain:redirects';
@@ -32,8 +28,19 @@ class DomainRedirectsCommand extends Command
         parent::__construct();
     }
 
-    #[Interact]
-    public function askDomain(InputInterface $input, SymfonyStyle $io): void
+    protected function configure(): void
+    {
+        $this
+            ->setName(self::NAME)
+            ->setDescription('Set specific "not found" redirects for individual domains.')
+            ->addArgument(
+                'domain',
+                InputArgument::REQUIRED,
+                'The domain authority to which you want to set the specific redirects',
+            );
+    }
+
+    protected function interact(InputInterface $input, OutputInterface $output): void
     {
         /** @var string|null $domain */
         $domain = $input->getArgument('domain');
@@ -41,6 +48,7 @@ class DomainRedirectsCommand extends Command
             return;
         }
 
+        $io = new SymfonyStyle($input, $output);
         $askNewDomain = static fn () => $io->ask('Domain authority for which you want to set specific redirects');
 
         /** @var string[] $availableDomains */
@@ -60,11 +68,10 @@ class DomainRedirectsCommand extends Command
         $input->setArgument('domain', str_contains($selectedOption, 'New domain') ? $askNewDomain() : $selectedOption);
     }
 
-    public function __invoke(
-        SymfonyStyle $io,
-        #[Argument('The domain authority to which you want to set the specific redirects', name: 'domain')]
-        string $domainAuthority,
-    ): int {
+    protected function execute(InputInterface $input, OutputInterface $output): int
+    {
+        $io = new SymfonyStyle($input, $output);
+        $domainAuthority = $input->getArgument('domain');
         $domain = $this->domainService->findByAuthority($domainAuthority);
 
         $ask = static function (string $message, string|null $current) use ($io): string|null {
@@ -88,20 +95,20 @@ class DomainRedirectsCommand extends Command
         $this->domainService->configureNotFoundRedirects($domainAuthority, NotFoundRedirects::withRedirects(
             $ask(
                 'URL to redirect to when a user hits this domain\'s base URL',
-                $domain?->baseUrlRedirect,
+                $domain?->baseUrlRedirect(),
             ),
             $ask(
                 'URL to redirect to when a user hits a not found URL other than an invalid short URL',
-                $domain?->regular404Redirect,
+                $domain?->regular404Redirect(),
             ),
             $ask(
                 'URL to redirect to when a user hits an invalid short URL',
-                $domain?->invalidShortUrlRedirect,
+                $domain?->invalidShortUrlRedirect(),
             ),
         ));
 
         $io->success(sprintf('"Not found" redirects properly set for "%s"', $domainAuthority));
 
-        return self::SUCCESS;
+        return ExitCode::EXIT_SUCCESS;
     }
 }

module/CLI/src/Command/Domain/GetDomainVisitsCommand.php

@@ -4,36 +4,50 @@ declare(strict_types=1);
 
 namespace Shlinkio\Shlink\CLI\Command\Domain;
 
-use Shlinkio\Shlink\CLI\Command\Visit\VisitsCommandUtils;
-use Shlinkio\Shlink\CLI\Input\VisitsListInput;
+use Shlinkio\Shlink\CLI\Command\Visit\AbstractVisitsListCommand;
+use Shlinkio\Shlink\Common\Paginator\Paginator;
+use Shlinkio\Shlink\Common\Util\DateRange;
+use Shlinkio\Shlink\Core\ShortUrl\Helper\ShortUrlStringifierInterface;
+use Shlinkio\Shlink\Core\Visit\Entity\Visit;
 use Shlinkio\Shlink\Core\Visit\Model\VisitsParams;
 use Shlinkio\Shlink\Core\Visit\VisitsStatsHelperInterface;
-use Symfony\Component\Console\Attribute\Argument;
-use Symfony\Component\Console\Attribute\AsCommand;
-use Symfony\Component\Console\Attribute\Ask;
-use Symfony\Component\Console\Attribute\MapInput;
-use Symfony\Component\Console\Command\Command;
-use Symfony\Component\Console\Style\SymfonyStyle;
+use Symfony\Component\Console\Input\InputArgument;
+use Symfony\Component\Console\Input\InputInterface;
 
-#[AsCommand(GetDomainVisitsCommand::NAME, 'Returns the list of visits for provided domain')]
-class GetDomainVisitsCommand extends Command
+class GetDomainVisitsCommand extends AbstractVisitsListCommand
 {
     public const string NAME = 'domain:visits';
 
-    public function __construct(private readonly VisitsStatsHelperInterface $visitsHelper)
-    {
-        parent::__construct();
+    public function __construct(
+        VisitsStatsHelperInterface $visitsHelper,
+        private readonly ShortUrlStringifierInterface $shortUrlStringifier,
+    ) {
+        parent::__construct($visitsHelper);
     }
 
-    public function __invoke(
-        SymfonyStyle $io,
-        #[Argument('The domain which visits we want to get'), Ask('For what domain do you want to get visits?')]
-        string $domain,
-        #[MapInput] VisitsListInput $input,
-    ): int {
-        $paginator = $this->visitsHelper->visitsForDomain($domain, new VisitsParams($input->dateRange()));
-        VisitsCommandUtils::renderOutput($io, $input, $paginator);
+    protected function configure(): void
+    {
+        $this
+            ->setName(self::NAME)
+            ->setDescription('Returns the list of visits for provided domain.')
+            ->addArgument('domain', InputArgument::REQUIRED, 'The domain which visits we want to get.');
+    }
 
-        return self::SUCCESS;
+    /**
+     * @return Paginator<Visit>
+     */
+    protected function getVisitsPaginator(InputInterface $input, DateRange $dateRange): Paginator
+    {
+        $domain = $input->getArgument('domain');
+        return $this->visitsHelper->visitsForDomain($domain, new VisitsParams($dateRange));
+    }
+
+    /**
+     * @return array<string, string>
+     */
+    protected function mapExtraFields(Visit $visit): array
+    {
+        $shortUrl = $visit->shortUrl;
+        return $shortUrl === null ? [] : ['shortUrl' => $this->shortUrlStringifier->stringify($shortUrl)];
     }
 }

module/CLI/src/Command/Domain/ListDomainsCommand.php

@@ -4,21 +4,18 @@ declare(strict_types=1);
 
 namespace Shlinkio\Shlink\CLI\Command\Domain;
 
+use Shlinkio\Shlink\CLI\Util\ExitCode;
 use Shlinkio\Shlink\CLI\Util\ShlinkTable;
 use Shlinkio\Shlink\Core\Config\NotFoundRedirectConfigInterface;
 use Shlinkio\Shlink\Core\Domain\DomainServiceInterface;
 use Shlinkio\Shlink\Core\Domain\Model\DomainItem;
-use Symfony\Component\Console\Attribute\AsCommand;
-use Symfony\Component\Console\Attribute\Option;
 use Symfony\Component\Console\Command\Command;
-use Symfony\Component\Console\Style\SymfonyStyle;
+use Symfony\Component\Console\Input\InputInterface;
+use Symfony\Component\Console\Input\InputOption;
+use Symfony\Component\Console\Output\OutputInterface;
 
 use function array_map;
 
-#[AsCommand(
-    name: ListDomainsCommand::NAME,
-    description: 'List all domains that have been ever used for some short URL',
-)]
 class ListDomainsCommand extends Command
 {
     public const string NAME = 'domain:list';
@@ -28,17 +25,25 @@ class ListDomainsCommand extends Command
         parent::__construct();
     }
 
-    public function __invoke(
-        SymfonyStyle $io,
-        #[Option(
-            'Will display an extra column with the information of the "not found" redirects for every domain.',
-            shortcut: 'r',
-        )]
-        bool $showRedirects = false,
-    ): int {
+    protected function configure(): void
+    {
+        $this
+            ->setName(self::NAME)
+            ->setDescription('List all domains that have been ever used for some short URL')
+            ->addOption(
+                'show-redirects',
+                'r',
+                InputOption::VALUE_NONE,
+                'Will display an extra column with the information of the "not found" redirects for every domain.',
+            );
+    }
+
+    protected function execute(InputInterface $input, OutputInterface $output): int
+    {
         $domains = $this->domainService->listDomains();
+        $showRedirects = $input->getOption('show-redirects');
         $commonFields = ['Domain', 'Is default'];
-        $table = $showRedirects ? ShlinkTable::withRowSeparators($io) : ShlinkTable::default($io);
+        $table = $showRedirects ? ShlinkTable::withRowSeparators($output) : ShlinkTable::default($output);
 
         $table->render(
             $showRedirects ? [...$commonFields, '"Not found" redirects'] : $commonFields,
@@ -49,19 +54,19 @@ class ListDomainsCommand extends Command
                     ? [
                         ...$commonValues,
                         $this->notFoundRedirectsToString($domain->notFoundRedirectConfig),
-                    ]
+                      ]
                     : $commonValues;
             }, $domains),
         );
 
-        return self::SUCCESS;
+        return ExitCode::EXIT_SUCCESS;
     }
 
     private function notFoundRedirectsToString(NotFoundRedirectConfigInterface $config): string
     {
-        $baseUrl = $config->baseUrlRedirect ?? 'N/A';
-        $regular404 = $config->regular404Redirect ?? 'N/A';
-        $invalidShortUrl = $config->invalidShortUrlRedirect ?? 'N/A';
+        $baseUrl = $config->baseUrlRedirect() ?? 'N/A';
+        $regular404 = $config->regular404Redirect() ?? 'N/A';
+        $invalidShortUrl = $config->invalidShortUrlRedirect() ?? 'N/A';
 
         return <<<EOL
         * Base URL: {$baseUrl}

module/CLI/src/Command/Integration/MatomoSendVisitsCommand.php

@@ -4,25 +4,38 @@ declare(strict_types=1);
 
 namespace Shlinkio\Shlink\CLI\Command\Integration;
 
+use Cake\Chronos\Chronos;
+use Shlinkio\Shlink\CLI\Util\ExitCode;
 use Shlinkio\Shlink\Core\Matomo\MatomoOptions;
 use Shlinkio\Shlink\Core\Matomo\MatomoVisitSenderInterface;
 use Shlinkio\Shlink\Core\Matomo\VisitSendingProgressTrackerInterface;
-use Symfony\Component\Console\Attribute\AsCommand;
-use Symfony\Component\Console\Attribute\Option;
 use Symfony\Component\Console\Command\Command;
 use Symfony\Component\Console\Input\InputInterface;
+use Symfony\Component\Console\Input\InputOption;
+use Symfony\Component\Console\Output\OutputInterface;
 use Symfony\Component\Console\Style\SymfonyStyle;
 use Throwable;
 
 use function Shlinkio\Shlink\Common\buildDateRange;
 use function Shlinkio\Shlink\Core\dateRangeToHumanFriendly;
-use function Shlinkio\Shlink\Core\normalizeOptionalDate;
 use function sprintf;
 
-#[AsCommand(
-    name: MatomoSendVisitsCommand::NAME,
-    description: 'Send existing visits to the configured matomo instance',
-    help: <<<HELP
+class MatomoSendVisitsCommand extends Command implements VisitSendingProgressTrackerInterface
+{
+    public const string NAME = 'integration:matomo:send-visits';
+
+    private readonly bool $matomoEnabled;
+    private SymfonyStyle $io;
+
+    public function __construct(MatomoOptions $matomoOptions, private readonly MatomoVisitSenderInterface $visitSender)
+    {
+        $this->matomoEnabled = $matomoOptions->enabled;
+        parent::__construct();
+    }
+
+    protected function configure(): void
+    {
+        $help = <<<HELP
         This command allows you to send existing visits from this Shlink instance to the configured Matomo server.
         
         Its intention is to allow you to configure Matomo at some point in time, and still have your whole visits 
@@ -42,40 +55,44 @@ use function sprintf;
 
         Send all visits created during 2022:
             <info>%command.name% --since 2022-01-01 --until 2022-12-31</info>
-        HELP,
-)]
-class MatomoSendVisitsCommand extends Command implements VisitSendingProgressTrackerInterface
-{
-    public const string NAME = 'integration:matomo:send-visits';
+        HELP;
 
-    private readonly bool $matomoEnabled;
-    private SymfonyStyle $io;
-
-    public function __construct(MatomoOptions $matomoOptions, private readonly MatomoVisitSenderInterface $visitSender)
-    {
-        $this->matomoEnabled = $matomoOptions->enabled;
-        parent::__construct();
+        $this
+            ->setName(self::NAME)
+            ->setDescription(sprintf(
+                '%sSend existing visits to the configured matomo instance',
+                $this->matomoEnabled ? '' : '[MATOMO INTEGRATION DISABLED] ',
+            ))
+            ->setHelp($help)
+            ->addOption(
+                'since',
+                's',
+                InputOption::VALUE_REQUIRED,
+                'Only visits created since this date, inclusively, will be sent to Matomo',
+            )
+            ->addOption(
+                'until',
+                'u',
+                InputOption::VALUE_REQUIRED,
+                'Only visits created until this date, inclusively, will be sent to Matomo',
+            );
     }
 
-    public function __invoke(
-        SymfonyStyle $io,
-        InputInterface $input,
-        #[Option('Only visits created since this date, inclusively, will be sent to Matomo', shortcut: 's')]
-        string|null $since = null,
-        #[Option('Only visits created until this date, inclusively, will be sent to Matomo', shortcut: 'u')]
-        string|null $until = null,
-    ): int {
-        $this->io = $io;
+    protected function execute(InputInterface $input, OutputInterface $output): int
+    {
+        $this->io = new SymfonyStyle($input, $output);
 
         if (! $this->matomoEnabled) {
             $this->io->warning('Matomo integration is not enabled in this Shlink instance');
-            return self::INVALID;
+            return ExitCode::EXIT_WARNING;
         }
 
         // TODO Validate provided date formats
+        $since = $input->getOption('since');
+        $until = $input->getOption('until');
         $dateRange = buildDateRange(
-            startDate: normalizeOptionalDate($since),
-            endDate: normalizeOptionalDate($until),
+            startDate: $since !== null ? Chronos::parse($since) : null,
+            endDate: $until !== null ? Chronos::parse($until) : null,
         );
 
         if ($input->isInteractive()) {
@@ -86,7 +103,7 @@ class MatomoSendVisitsCommand extends Command implements VisitSendingProgressTra
                 . 'you have verified only visits in the right date range are going to be sent.',
             ]);
             if (! $this->io->confirm('Continue?', default: false)) {
-                return self::INVALID;
+                return ExitCode::EXIT_WARNING;
             }
         }
 
@@ -105,7 +122,7 @@ class MatomoSendVisitsCommand extends Command implements VisitSendingProgressTra
             default => $this->io->info('There was no visits matching provided date range.'),
         };
 
-        return self::SUCCESS;
+        return ExitCode::EXIT_SUCCESS;
     }
 
     public function success(int $index): void

module/CLI/src/Command/RedirectRule/ManageRedirectRulesCommand.php

@@ -4,47 +4,55 @@ declare(strict_types=1);
 
 namespace Shlinkio\Shlink\CLI\Command\RedirectRule;
 
+use Shlinkio\Shlink\CLI\Input\ShortUrlIdentifierInput;
 use Shlinkio\Shlink\CLI\RedirectRule\RedirectRuleHandlerInterface;
+use Shlinkio\Shlink\CLI\Util\ExitCode;
 use Shlinkio\Shlink\Core\Exception\ShortUrlNotFoundException;
 use Shlinkio\Shlink\Core\RedirectRule\ShortUrlRedirectRuleServiceInterface;
-use Shlinkio\Shlink\Core\ShortUrl\Model\ShortUrlIdentifier;
 use Shlinkio\Shlink\Core\ShortUrl\ShortUrlResolverInterface;
-use Symfony\Component\Console\Attribute\Argument;
-use Symfony\Component\Console\Attribute\AsCommand;
-use Symfony\Component\Console\Attribute\Option;
 use Symfony\Component\Console\Command\Command;
+use Symfony\Component\Console\Input\InputInterface;
+use Symfony\Component\Console\Output\OutputInterface;
 use Symfony\Component\Console\Style\SymfonyStyle;
 
 use function sprintf;
 
-#[AsCommand(
-    name: ManageRedirectRulesCommand::NAME,
-    description: 'Set redirect rules for a short URL',
-)]
 class ManageRedirectRulesCommand extends Command
 {
     public const string NAME = 'short-url:manage-rules';
 
+    private readonly ShortUrlIdentifierInput $shortUrlIdentifierInput;
+
     public function __construct(
         protected readonly ShortUrlResolverInterface $shortUrlResolver,
         protected readonly ShortUrlRedirectRuleServiceInterface $ruleService,
         protected readonly RedirectRuleHandlerInterface $ruleHandler,
     ) {
         parent::__construct();
+        $this->shortUrlIdentifierInput = new ShortUrlIdentifierInput(
+            $this,
+            shortCodeDesc: 'The short code which rules we want to set.',
+            domainDesc: 'The domain for the short code.',
+        );
     }
 
-    public function __invoke(
-        SymfonyStyle $io,
-        #[Argument('The short code which rules we want to set')] string $shortCode,
-        #[Option('The domain of the short code', shortcut: 'd')] string|null $domain = null,
-    ): int {
-        $identifier = ShortUrlIdentifier::fromShortCodeAndDomain($shortCode, $domain);
+    protected function configure(): void
+    {
+        $this
+            ->setName(self::NAME)
+            ->setDescription('Set redirect rules for a short URL');
+    }
+
+    protected function execute(InputInterface $input, OutputInterface $output): int
+    {
+        $io = new SymfonyStyle($input, $output);
+        $identifier = $this->shortUrlIdentifierInput->toShortUrlIdentifier($input);
 
         try {
             $shortUrl = $this->shortUrlResolver->resolveShortUrl($identifier);
         } catch (ShortUrlNotFoundException) {
             $io->error(sprintf('Short URL for %s not found', $identifier->__toString()));
-            return self::FAILURE;
+            return ExitCode::EXIT_FAILURE;
         }
 
         $rulesToSave = $this->ruleHandler->manageRules($io, $shortUrl, $this->ruleService->rulesForShortUrl($shortUrl));
@@ -53,6 +61,6 @@ class ManageRedirectRulesCommand extends Command
             $io->success('Rules properly saved');
         }
 
-        return self::SUCCESS;
+        return ExitCode::EXIT_SUCCESS;
     }
 }

module/CLI/src/Command/ShortUrl/CreateShortUrlCommand.php

@@ -4,52 +4,125 @@ declare(strict_types=1);
 
 namespace Shlinkio\Shlink\CLI\Command\ShortUrl;
 
-use Shlinkio\Shlink\CLI\Command\ShortUrl\Input\ShortUrlCreationInput;
+use Shlinkio\Shlink\CLI\Input\ShortUrlDataInput;
+use Shlinkio\Shlink\CLI\Util\ExitCode;
 use Shlinkio\Shlink\Core\Config\Options\UrlShortenerOptions;
 use Shlinkio\Shlink\Core\Exception\NonUniqueSlugException;
 use Shlinkio\Shlink\Core\ShortUrl\Helper\ShortUrlStringifierInterface;
 use Shlinkio\Shlink\Core\ShortUrl\UrlShortenerInterface;
-use Symfony\Component\Console\Attribute\AsCommand;
-use Symfony\Component\Console\Attribute\MapInput;
 use Symfony\Component\Console\Command\Command;
+use Symfony\Component\Console\Input\InputInterface;
+use Symfony\Component\Console\Input\InputOption;
+use Symfony\Component\Console\Output\OutputInterface;
 use Symfony\Component\Console\Style\SymfonyStyle;
 
 use function sprintf;
 
-#[AsCommand(
-    name: CreateShortUrlCommand::NAME,
-    description: 'Generates a short URL for provided long URL and returns it',
-)]
 class CreateShortUrlCommand extends Command
 {
     public const string NAME = 'short-url:create';
 
+    private SymfonyStyle $io;
+    private readonly ShortUrlDataInput $shortUrlDataInput;
+
     public function __construct(
         private readonly UrlShortenerInterface $urlShortener,
         private readonly ShortUrlStringifierInterface $stringifier,
         private readonly UrlShortenerOptions $options,
     ) {
         parent::__construct();
+        $this->shortUrlDataInput = new ShortUrlDataInput($this);
     }
 
-    public function __invoke(SymfonyStyle $io, #[MapInput] ShortUrlCreationInput $inputData): int
+    protected function configure(): void
     {
+        $this
+            ->setName(self::NAME)
+            ->setDescription('Generates a short URL for provided long URL and returns it')
+            ->addOption(
+                'domain',
+                'd',
+                InputOption::VALUE_REQUIRED,
+                'The domain to which this short URL will be attached.',
+            )
+            ->addOption(
+                'custom-slug',
+                'c',
+                InputOption::VALUE_REQUIRED,
+                'If provided, this slug will be used instead of generating a short code',
+            )
+            ->addOption(
+                'short-code-length',
+                'l',
+                InputOption::VALUE_REQUIRED,
+                'The length for generated short code (it will be ignored if --custom-slug was provided).',
+            )
+            ->addOption(
+                'path-prefix',
+                'p',
+                InputOption::VALUE_REQUIRED,
+                'Prefix to prepend before the generated short code or provided custom slug',
+            )
+            ->addOption(
+                'find-if-exists',
+                'f',
+                InputOption::VALUE_NONE,
+                'This will force existing matching URL to be returned if found, instead of creating a new one.',
+            );
+    }
+
+    protected function interact(InputInterface $input, OutputInterface $output): void
+    {
+        $this->verifyLongUrlArgument($input, $output);
+    }
+
+    private function verifyLongUrlArgument(InputInterface $input, OutputInterface $output): void
+    {
+        $longUrl = $input->getArgument('longUrl');
+        if (! empty($longUrl)) {
+            return;
+        }
+
+        $io = $this->getIO($input, $output);
+        $longUrl = $io->ask('Which URL do you want to shorten?');
+        if (! empty($longUrl)) {
+            $input->setArgument('longUrl', $longUrl);
+        }
+    }
+
+    protected function execute(InputInterface $input, OutputInterface $output): int
+    {
+        $io = $this->getIO($input, $output);
+
         try {
-            $result = $this->urlShortener->shorten($inputData->toShortUrlCreation($this->options));
+            $result = $this->urlShortener->shorten($this->shortUrlDataInput->toShortUrlCreation(
+                $input,
+                $this->options,
+                customSlugField: 'custom-slug',
+                shortCodeLengthField: 'short-code-length',
+                pathPrefixField: 'path-prefix',
+                findIfExistsField: 'find-if-exists',
+                domainField: 'domain',
+            ));
 
             $result->onEventDispatchingError(static fn () => $io->isVerbose() && $io->warning(
                 'Short URL properly created, but the real-time updates cannot be notified when generating the '
-                    . 'short URL from the command line. Migrate to roadrunner in order to bypass this limitation.',
+                . 'short URL from the command line. Migrate to roadrunner in order to bypass this limitation.',
             ));
 
             $io->writeln([
                 sprintf('Processed long URL: <info>%s</info>', $result->shortUrl->getLongUrl()),
                 sprintf('Generated short URL: <info>%s</info>', $this->stringifier->stringify($result->shortUrl)),
             ]);
-            return self::SUCCESS;
+            return ExitCode::EXIT_SUCCESS;
         } catch (NonUniqueSlugException $e) {
             $io->error($e->getMessage());
-            return self::FAILURE;
+            return ExitCode::EXIT_FAILURE;
         }
     }
+
+    private function getIO(InputInterface $input, OutputInterface $output): SymfonyStyle
+    {
+        return $this->io ??= new SymfonyStyle($input, $output);
+    }
 }

module/CLI/src/Command/ShortUrl/DeleteExpiredShortUrlsCommand.php

@@ -4,20 +4,17 @@ declare(strict_types=1);
 
 namespace Shlinkio\Shlink\CLI\Command\ShortUrl;
 
+use Shlinkio\Shlink\CLI\Util\ExitCode;
 use Shlinkio\Shlink\Core\ShortUrl\DeleteShortUrlServiceInterface;
 use Shlinkio\Shlink\Core\ShortUrl\Model\ExpiredShortUrlsConditions;
-use Symfony\Component\Console\Attribute\AsCommand;
-use Symfony\Component\Console\Attribute\Option;
 use Symfony\Component\Console\Command\Command;
 use Symfony\Component\Console\Input\InputInterface;
+use Symfony\Component\Console\Input\InputOption;
+use Symfony\Component\Console\Output\OutputInterface;
 use Symfony\Component\Console\Style\SymfonyStyle;
 
 use function sprintf;
 
-#[AsCommand(
-    name: DeleteExpiredShortUrlsCommand::NAME,
-    description: 'Deletes all short URLs that are considered expired, because they have a validUntil date in the past',
-)]
 class DeleteExpiredShortUrlsCommand extends Command
 {
     public const string NAME = 'short-url:delete-expired';
@@ -27,17 +24,32 @@ class DeleteExpiredShortUrlsCommand extends Command
         parent::__construct();
     }
 
-    public function __invoke(
-        SymfonyStyle $io,
-        InputInterface $input,
-        #[Option('Also take into consideration short URLs which have reached their max amount of visits.')]
-        bool $evaluateMaxVisits = false,
-        #[Option('Delete short URLs with no confirmation', shortcut: 'f')] bool $force = false,
-        #[Option('Only check how many short URLs would be affected, without actually deleting them')]
-        bool $dryRun = false,
-    ): int {
-        $conditions = new ExpiredShortUrlsConditions(maxVisitsReached: $evaluateMaxVisits);
-        $force = $force || ! $input->isInteractive();
+    protected function configure(): void
+    {
+        $this
+            ->setName(self::NAME)
+            ->setDescription(
+                'Deletes all short URLs that are considered expired, because they have a validUntil date in the past',
+            )
+            ->addOption(
+                'evaluate-max-visits',
+                mode: InputOption::VALUE_NONE,
+                description: 'Also take into consideration short URLs which have reached their max amount of visits.',
+            )
+            ->addOption('force', 'f', InputOption::VALUE_NONE, 'Delete short URLs with no confirmation')
+            ->addOption(
+                'dry-run',
+                mode: InputOption::VALUE_NONE,
+                description: 'Delete short URLs with no confirmation',
+            );
+    }
+
+    protected function execute(InputInterface $input, OutputInterface $output): int
+    {
+        $io = new SymfonyStyle($input, $output);
+        $force = $input->getOption('force') || ! $input->isInteractive();
+        $dryRun = $input->getOption('dry-run');
+        $conditions = new ExpiredShortUrlsConditions(maxVisitsReached: $input->getOption('evaluate-max-visits'));
 
         if (! $force && ! $dryRun) {
             $io->warning([
@@ -46,19 +58,18 @@ class DeleteExpiredShortUrlsCommand extends Command
                 'This action cannot be undone. Proceed at your own risk',
             ]);
             if (! $io->confirm('Continue?', default: false)) {
-                return self::INVALID;
+                return ExitCode::EXIT_WARNING;
             }
         }
 
         if ($dryRun) {
             $result = $this->deleteShortUrlService->countExpiredShortUrls($conditions);
             $io->success(sprintf('There are %s expired short URLs matching provided conditions', $result));
-            return self::SUCCESS;
+            return ExitCode::EXIT_SUCCESS;
         }
 
         $result = $this->deleteShortUrlService->deleteExpiredShortUrls($conditions);
         $io->success(sprintf('%s expired short URLs have been deleted', $result));
-
-        return self::SUCCESS;
+        return ExitCode::EXIT_SUCCESS;
     }
 }

module/CLI/src/Command/ShortUrl/DeleteShortUrlCommand.php

@@ -4,47 +4,61 @@ declare(strict_types=1);
 
 namespace Shlinkio\Shlink\CLI\Command\ShortUrl;
 
+use Shlinkio\Shlink\CLI\Input\ShortUrlIdentifierInput;
+use Shlinkio\Shlink\CLI\Util\ExitCode;
 use Shlinkio\Shlink\Core\Exception;
 use Shlinkio\Shlink\Core\ShortUrl\DeleteShortUrlServiceInterface;
 use Shlinkio\Shlink\Core\ShortUrl\Model\ShortUrlIdentifier;
-use Symfony\Component\Console\Attribute\Argument;
-use Symfony\Component\Console\Attribute\AsCommand;
-use Symfony\Component\Console\Attribute\Option;
 use Symfony\Component\Console\Command\Command;
+use Symfony\Component\Console\Input\InputInterface;
+use Symfony\Component\Console\Input\InputOption;
+use Symfony\Component\Console\Output\OutputInterface;
 use Symfony\Component\Console\Style\SymfonyStyle;
 
 use function sprintf;
 
-#[AsCommand(name: DeleteShortUrlCommand::NAME, description: 'Deletes a short URL')]
 class DeleteShortUrlCommand extends Command
 {
     public const string NAME = 'short-url:delete';
 
+    private readonly ShortUrlIdentifierInput $shortUrlIdentifierInput;
+
     public function __construct(private readonly DeleteShortUrlServiceInterface $deleteShortUrlService)
     {
         parent::__construct();
+        $this->shortUrlIdentifierInput = new ShortUrlIdentifierInput(
+            $this,
+            shortCodeDesc: 'The short code for the short URL to be deleted',
+            domainDesc: 'The domain if the short code does not belong to the default one',
+        );
     }
 
-    public function __invoke(
-        SymfonyStyle $io,
-        #[Argument('The short code for the short URL to be deleted')] string $shortCode,
-        #[Option('The domain if the short code does not belong to the default one', shortcut: 'd')]
-        string|null $domain = null,
-        #[Option(
-            'Ignores the safety visits threshold check, which could make short URLs with many visits to be '
-            . 'accidentally deleted',
-            shortcut: 'i',
-        )]
-        bool $ignoreThreshold = false,
-    ): int {
-        $identifier = ShortUrlIdentifier::fromShortCodeAndDomain($shortCode, $domain);
+    protected function configure(): void
+    {
+        $this
+            ->setName(self::NAME)
+            ->setDescription('Deletes a short URL')
+            ->addOption(
+                'ignore-threshold',
+                'i',
+                InputOption::VALUE_NONE,
+                'Ignores the safety visits threshold check, which could make short URLs with many visits to be '
+                . 'accidentally deleted',
+            );
+    }
+
+    protected function execute(InputInterface $input, OutputInterface $output): int
+    {
+        $io = new SymfonyStyle($input, $output);
+        $identifier = $this->shortUrlIdentifierInput->toShortUrlIdentifier($input);
+        $ignoreThreshold = $input->getOption('ignore-threshold');
 
         try {
             $this->runDelete($io, $identifier, $ignoreThreshold);
-            return self::SUCCESS;
+            return ExitCode::EXIT_SUCCESS;
         } catch (Exception\ShortUrlNotFoundException $e) {
             $io->error($e->getMessage());
-            return self::FAILURE;
+            return ExitCode::EXIT_FAILURE;
         } catch (Exception\DeleteShortUrlException $e) {
             return $this->retry($io, $identifier, $e->getMessage());
         }
@@ -61,7 +75,7 @@ class DeleteShortUrlCommand extends Command
             $io->warning('Short URL was not deleted.');
         }
 
-        return $forceDelete ? self::SUCCESS : self::INVALID;
+        return $forceDelete ? ExitCode::EXIT_SUCCESS : ExitCode::EXIT_WARNING;
     }
 
     private function runDelete(SymfonyStyle $io, ShortUrlIdentifier $identifier, bool $ignoreThreshold): void

module/CLI/src/Command/ShortUrl/DeleteShortUrlVisitsCommand.php

@@ -4,52 +4,55 @@ declare(strict_types=1);
 
 namespace Shlinkio\Shlink\CLI\Command\ShortUrl;
 
-use Shlinkio\Shlink\CLI\Command\Util\CommandUtils;
+use Shlinkio\Shlink\CLI\Command\Visit\AbstractDeleteVisitsCommand;
+use Shlinkio\Shlink\CLI\Input\ShortUrlIdentifierInput;
+use Shlinkio\Shlink\CLI\Util\ExitCode;
 use Shlinkio\Shlink\Core\Exception\ShortUrlNotFoundException;
-use Shlinkio\Shlink\Core\ShortUrl\Model\ShortUrlIdentifier;
 use Shlinkio\Shlink\Core\ShortUrl\ShortUrlVisitsDeleterInterface;
-use Symfony\Component\Console\Attribute\Argument;
-use Symfony\Component\Console\Attribute\AsCommand;
-use Symfony\Component\Console\Attribute\Option;
-use Symfony\Component\Console\Command\Command;
+use Symfony\Component\Console\Input\InputInterface;
 use Symfony\Component\Console\Style\SymfonyStyle;
 
 use function sprintf;
 
-#[AsCommand(DeleteShortUrlVisitsCommand::NAME, 'Deletes visits from a short URL')]
-class DeleteShortUrlVisitsCommand extends Command
+class DeleteShortUrlVisitsCommand extends AbstractDeleteVisitsCommand
 {
     public const string NAME = 'short-url:visits-delete';
 
+    private readonly ShortUrlIdentifierInput $shortUrlIdentifierInput;
+
     public function __construct(private readonly ShortUrlVisitsDeleterInterface $deleter)
     {
         parent::__construct();
-    }
-
-    public function __invoke(
-        SymfonyStyle $io,
-        #[Argument('The short code for the short URL which visits will be deleted')] string $shortCode,
-        #[Option('The domain if the short code does not belong to the default one', shortcut: 'd')]
-        string|null $domain = null,
-    ): int {
-        $identifier = ShortUrlIdentifier::fromShortCodeAndDomain($shortCode, $domain);
-        return CommandUtils::executeWithWarning(
-            'You are about to delete all visits for a short URL. This operation cannot be undone',
-            $io,
-            fn () => $this->deleteVisits($io, $identifier),
+        $this->shortUrlIdentifierInput = new ShortUrlIdentifierInput(
+            $this,
+            shortCodeDesc: 'The short code for the short URL which visits will be deleted',
+            domainDesc: 'The domain if the short code does not belong to the default one',
         );
     }
 
-    private function deleteVisits(SymfonyStyle $io, ShortUrlIdentifier $identifier): int
+    protected function configure(): void
     {
+        $this
+            ->setName(self::NAME)
+            ->setDescription('Deletes visits from a short URL');
+    }
+
+    protected function doExecute(InputInterface $input, SymfonyStyle $io): int
+    {
+        $identifier = $this->shortUrlIdentifierInput->toShortUrlIdentifier($input);
         try {
             $result = $this->deleter->deleteShortUrlVisits($identifier);
             $io->success(sprintf('Successfully deleted %s visits', $result->affectedItems));
 
-            return self::SUCCESS;
+            return ExitCode::EXIT_SUCCESS;
         } catch (ShortUrlNotFoundException) {
             $io->warning(sprintf('Short URL not found for "%s"', $identifier->__toString()));
-            return self::INVALID;
+            return ExitCode::EXIT_WARNING;
         }
     }
+
+    protected function getWarningMessage(): string
+    {
+        return 'You are about to delete all visits for a short URL. This operation cannot be undone.';
+    }
 }

module/CLI/src/Command/ShortUrl/EditShortUrlCommand.php

@@ -4,53 +4,60 @@ declare(strict_types=1);
 
 namespace Shlinkio\Shlink\CLI\Command\ShortUrl;
 
-use Shlinkio\Shlink\CLI\Command\ShortUrl\Input\ShortUrlDataInput;
+use Shlinkio\Shlink\CLI\Input\ShortUrlDataInput;
+use Shlinkio\Shlink\CLI\Input\ShortUrlIdentifierInput;
+use Shlinkio\Shlink\CLI\Util\ExitCode;
 use Shlinkio\Shlink\Core\Exception\ShortUrlNotFoundException;
 use Shlinkio\Shlink\Core\ShortUrl\Helper\ShortUrlStringifierInterface;
-use Shlinkio\Shlink\Core\ShortUrl\Model\ShortUrlEdition;
-use Shlinkio\Shlink\Core\ShortUrl\Model\ShortUrlIdentifier;
 use Shlinkio\Shlink\Core\ShortUrl\ShortUrlServiceInterface;
-use Symfony\Component\Console\Attribute\Argument;
-use Symfony\Component\Console\Attribute\AsCommand;
-use Symfony\Component\Console\Attribute\MapInput;
-use Symfony\Component\Console\Attribute\Option;
 use Symfony\Component\Console\Command\Command;
+use Symfony\Component\Console\Input\InputInterface;
+use Symfony\Component\Console\Output\OutputInterface;
 use Symfony\Component\Console\Style\SymfonyStyle;
 
 use function sprintf;
 
-#[AsCommand(
-    name: EditShortUrlCommand::NAME,
-    description: 'Edit an existing short URL',
-)]
 class EditShortUrlCommand extends Command
 {
     public const string NAME = 'short-url:edit';
 
+    private readonly ShortUrlDataInput $shortUrlDataInput;
+    private readonly ShortUrlIdentifierInput $shortUrlIdentifierInput;
+
     public function __construct(
         private readonly ShortUrlServiceInterface $shortUrlService,
         private readonly ShortUrlStringifierInterface $stringifier,
     ) {
         parent::__construct();
+
+        $this->shortUrlDataInput = new ShortUrlDataInput($this, longUrlAsOption: true);
+        $this->shortUrlIdentifierInput = new ShortUrlIdentifierInput(
+            $this,
+            shortCodeDesc: 'The short code to edit',
+            domainDesc: 'The domain to which the short URL is attached.',
+        );
     }
 
-    public function __invoke(
-        SymfonyStyle $io,
-        #[MapInput] ShortUrlDataInput $data,
-        #[Argument('The short code to edit')] string $shortCode,
-        #[Option('The domain to which the short URL is attached', shortcut: 'd')] string|null $domain = null,
-        #[Option('The long URL to set', shortcut: 'l')] string|null $longUrl = null,
-    ): int {
-        $identifier = ShortUrlIdentifier::fromShortCodeAndDomain($shortCode, $domain);
+    protected function configure(): void
+    {
+        $this
+            ->setName(self::NAME)
+            ->setDescription('Edit an existing short URL');
+    }
+
+    protected function execute(InputInterface $input, OutputInterface $output): int
+    {
+        $io = new SymfonyStyle($input, $output);
+        $identifier = $this->shortUrlIdentifierInput->toShortUrlIdentifier($input);
 
         try {
             $shortUrl = $this->shortUrlService->updateShortUrl(
                 $identifier,
-                ShortUrlEdition::fromRawData($data->toArray()),
+                $this->shortUrlDataInput->toShortUrlEdition($input),
             );
 
             $io->success(sprintf('Short URL "%s" properly edited', $this->stringifier->stringify($shortUrl)));
-            return self::SUCCESS;
+            return ExitCode::EXIT_SUCCESS;
         } catch (ShortUrlNotFoundException $e) {
             $io->error(sprintf('Short URL not found for "%s"', $identifier->__toString()));
 
@@ -58,7 +65,7 @@ class EditShortUrlCommand extends Command
                 $this->getApplication()?->renderThrowable($e, $io);
             }
 
-            return self::FAILURE;
+            return ExitCode::EXIT_FAILURE;
         }
     }
 }

module/CLI/src/Command/ShortUrl/GetShortUrlVisitsCommand.php

@@ -4,43 +4,62 @@ declare(strict_types=1);
 
 namespace Shlinkio\Shlink\CLI\Command\ShortUrl;
 
-use Shlinkio\Shlink\CLI\Command\Visit\VisitsCommandUtils;
-use Shlinkio\Shlink\CLI\Input\VisitsListInput;
-use Shlinkio\Shlink\Core\ShortUrl\Model\ShortUrlIdentifier;
+use Shlinkio\Shlink\CLI\Command\Visit\AbstractVisitsListCommand;
+use Shlinkio\Shlink\CLI\Input\ShortUrlIdentifierInput;
+use Shlinkio\Shlink\Common\Paginator\Paginator;
+use Shlinkio\Shlink\Common\Util\DateRange;
+use Shlinkio\Shlink\Core\Visit\Entity\Visit;
 use Shlinkio\Shlink\Core\Visit\Model\VisitsParams;
-use Shlinkio\Shlink\Core\Visit\VisitsStatsHelperInterface;
-use Symfony\Component\Console\Attribute\Argument;
-use Symfony\Component\Console\Attribute\AsCommand;
-use Symfony\Component\Console\Attribute\Ask;
-use Symfony\Component\Console\Attribute\MapInput;
-use Symfony\Component\Console\Attribute\Option;
-use Symfony\Component\Console\Command\Command;
+use Symfony\Component\Console\Input\InputInterface;
+use Symfony\Component\Console\Output\OutputInterface;
 use Symfony\Component\Console\Style\SymfonyStyle;
 
-#[AsCommand(GetShortUrlVisitsCommand::NAME, 'Returns the detailed visits information for provided short code')]
-class GetShortUrlVisitsCommand extends Command
+class GetShortUrlVisitsCommand extends AbstractVisitsListCommand
 {
     public const string NAME = 'short-url:visits';
 
-    public function __construct(protected readonly VisitsStatsHelperInterface $visitsHelper)
+    private ShortUrlIdentifierInput $shortUrlIdentifierInput;
+
+    protected function configure(): void
     {
-        parent::__construct();
+        $this
+            ->setName(self::NAME)
+            ->setDescription('Returns the detailed visits information for provided short code');
+        $this->shortUrlIdentifierInput = new ShortUrlIdentifierInput(
+            $this,
+            shortCodeDesc: 'The short code which visits we want to get.',
+            domainDesc: 'The domain for the short code.',
+        );
     }
 
-    public function __invoke(
-        SymfonyStyle $io,
-        #[Argument('The short code which visits we want to get'), Ask('Which short code do you want to use?')]
-        string $shortCode,
-        #[MapInput] VisitsListInput $input,
-        #[Option('The domain for the short code', shortcut: 'd')]
-        string|null $domain = null,
-    ): int {
-        $identifier = ShortUrlIdentifier::fromShortCodeAndDomain($shortCode, $domain);
-        $dateRange = $input->dateRange();
-        $paginator = $this->visitsHelper->visitsForShortUrl($identifier, new VisitsParams($dateRange));
+    protected function interact(InputInterface $input, OutputInterface $output): void
+    {
+        $shortCode = $this->shortUrlIdentifierInput->shortCode($input);
+        if (! empty($shortCode)) {
+            return;
+        }
 
-        VisitsCommandUtils::renderOutput($io, $input, $paginator);
+        $io = new SymfonyStyle($input, $output);
+        $shortCode = $io->ask('A short code was not provided. Which short code do you want to use?');
+        if (! empty($shortCode)) {
+            $input->setArgument('shortCode', $shortCode);
+        }
+    }
 
-        return self::SUCCESS;
+    /**
+     * @return Paginator<Visit>
+     */
+    protected function getVisitsPaginator(InputInterface $input, DateRange $dateRange): Paginator
+    {
+        $identifier = $this->shortUrlIdentifierInput->toShortUrlIdentifier($input);
+        return $this->visitsHelper->visitsForShortUrl($identifier, new VisitsParams($dateRange));
+    }
+
+    /**
+     * @return array<string, string>
+     */
+    protected function mapExtraFields(Visit $visit): array
+    {
+        return [];
     }
 }

module/CLI/src/Command/ShortUrl/Input/ShortUrlCreationInput.php

@@ -1,57 +0,0 @@
-<?php
-
-declare(strict_types=1);
-
-namespace Shlinkio\Shlink\CLI\Command\ShortUrl\Input;
-
-use Shlinkio\Shlink\Core\Config\Options\UrlShortenerOptions;
-use Shlinkio\Shlink\Core\ShortUrl\Model\ShortUrlCreation;
-use Shlinkio\Shlink\Core\ShortUrl\Model\Validation\ShortUrlInputFilter;
-use Symfony\Component\Console\Attribute\Argument;
-use Symfony\Component\Console\Attribute\Ask;
-use Symfony\Component\Console\Attribute\MapInput;
-use Symfony\Component\Console\Attribute\Option;
-
-/**
- * Data used for short URL creation
- */
-final class ShortUrlCreationInput
-{
-    #[Argument('The long URL to set'), Ask('Which URL do you want to shorten?')]
-    public string $longUrl;
-
-    #[MapInput]
-    public ShortUrlDataInput $commonData;
-
-    #[Option('The domain to which this short URL will be attached', shortcut: 'd')]
-    public string|null $domain = null;
-
-    #[Option('If provided, this slug will be used instead of generating a short code', shortcut: 'c')]
-    public string|null $customSlug = null;
-
-    #[Option('The length for generated short code (it will be ignored if --custom-slug was provided)', shortcut: 'l')]
-    public int|null $shortCodeLength = null;
-
-    #[Option('Prefix to prepend before the generated short code or provided custom slug', shortcut: 'p')]
-    public string|null $pathPrefix = null;
-
-    #[Option(
-        'This will force existing matching URL to be returned if found, instead of creating a new one',
-        shortcut: 'f',
-    )]
-    public bool $findIfExists = false;
-
-    public function toShortUrlCreation(UrlShortenerOptions $options): ShortUrlCreation
-    {
-        $shortCodeLength = $this->shortCodeLength ?? $options->defaultShortCodesLength;
-        return ShortUrlCreation::fromRawData([
-            ShortUrlInputFilter::LONG_URL => $this->longUrl,
-            ShortUrlInputFilter::DOMAIN => $this->domain,
-            ShortUrlInputFilter::CUSTOM_SLUG => $this->customSlug,
-            ShortUrlInputFilter::SHORT_CODE_LENGTH => $shortCodeLength,
-            ShortUrlInputFilter::PATH_PREFIX => $this->pathPrefix,
-            ShortUrlInputFilter::FIND_IF_EXISTS => $this->findIfExists,
-            ...$this->commonData->toArray(),
-        ], $options);
-    }
-}

module/CLI/src/Command/ShortUrl/Input/ShortUrlDataInput.php

@@ -1,80 +0,0 @@
-<?php
-
-declare(strict_types=1);
-
-namespace Shlinkio\Shlink\CLI\Command\ShortUrl\Input;
-
-use Shlinkio\Shlink\Core\ShortUrl\Model\Validation\ShortUrlInputFilter;
-use Symfony\Component\Console\Attribute\Option;
-
-use function array_unique;
-
-/**
- * Common input used for short URL creation and edition
- */
-final class ShortUrlDataInput
-{
-    /** @var string[]|null */
-    #[Option('Tags to apply to the short URL', name: 'tag', shortcut: 't')]
-    public array|null $tags = null;
-
-    #[Option(
-        'The date from which this short URL will be valid. '
-        . 'If someone tries to access it before this date, it will not be found',
-        shortcut: 's',
-    )]
-    public string|null $validSince = null;
-
-    #[Option(
-        'The date until which this short URL will be valid. '
-        . 'If someone tries to access it after this date, it will not be found',
-        shortcut: 'u',
-    )]
-    public string|null $validUntil = null;
-
-    #[Option('This will limit the number of visits for this short URL', shortcut: 'm')]
-    public int|null $maxVisits = null;
-
-    #[Option('A descriptive title for the short URL')]
-    public string|null $title = null;
-
-    #[Option('Tells if this short URL will be included as "Allow" in Shlink\'s robots.txt', shortcut: 'r')]
-    public bool|null $crawlable = null;
-
-    #[Option(
-        'Disables the forwarding of the query string to the long URL, when the short URL is visited',
-        shortcut: 'w',
-    )]
-    public bool|null $noForwardQuery = null;
-
-    public function toArray(): array
-    {
-        $data = [];
-
-        // Avoid setting arguments that were not explicitly provided.
-        // This is important when editing short URLs and should not make a difference when creating.
-        if ($this->validSince !== null) {
-            $data[ShortUrlInputFilter::VALID_SINCE] = $this->validSince;
-        }
-        if ($this->validUntil !== null) {
-            $data[ShortUrlInputFilter::VALID_UNTIL] = $this->validUntil;
-        }
-        if ($this->maxVisits !== null) {
-            $data[ShortUrlInputFilter::MAX_VISITS] = $this->maxVisits;
-        }
-        if ($this->tags !== null) {
-            $data[ShortUrlInputFilter::TAGS] = array_unique($this->tags);
-        }
-        if ($this->title !== null) {
-            $data[ShortUrlInputFilter::TITLE] = $this->title;
-        }
-        if ($this->crawlable !== null) {
-            $data[ShortUrlInputFilter::CRAWLABLE] = $this->crawlable;
-        }
-        if ($this->noForwardQuery !== null) {
-            $data[ShortUrlInputFilter::FORWARD_QUERY] = !$this->noForwardQuery;
-        }
-
-        return $data;
-    }
-}

module/CLI/src/Command/ShortUrl/Input/ShortUrlsParamsInput.php

@@ -1,121 +0,0 @@
-<?php
-
-declare(strict_types=1);
-
-namespace Shlinkio\Shlink\CLI\Command\ShortUrl\Input;
-
-use Shlinkio\Shlink\CLI\Command\ShortUrl\ListShortUrlsCommand;
-use Shlinkio\Shlink\CLI\Input\InputUtils;
-use Shlinkio\Shlink\Common\Paginator\Paginator;
-use Shlinkio\Shlink\Core\Domain\Entity\Domain;
-use Shlinkio\Shlink\Core\ShortUrl\Model\TagsMode;
-use Shlinkio\Shlink\Core\ShortUrl\Model\Validation\ShortUrlsParamsInputFilter;
-use Symfony\Component\Console\Attribute\Option;
-use Symfony\Component\Console\Output\OutputInterface;
-
-use function array_unique;
-
-/**
- * Input arguments and options for short-url:list command
- * @see ListShortUrlsCommand
- */
-final class ShortUrlsParamsInput
-{
-    #[Option('The first page to list (10 items per page unless "--all" is provided).', shortcut: 'p')]
-    public int $page = 1;
-
-    #[Option(
-        'Disables pagination and just displays all existing URLs. Caution! If the amount of short URLs is big,this '
-        . 'may end up failing due to memory usage.',
-    )]
-    public bool $all = false;
-
-    #[Option('Only return short URLs older than this date', shortcut: 's')]
-    public string|null $startDate = null;
-
-    #[Option('Only return short URLs newer than this date', shortcut: 'e')]
-    public string|null $endDate = null;
-
-    #[Option('A query used to filter results by searching for it on the longUrl and shortCode fields', shortcut: 'st')]
-    public string|null $searchTerm = null;
-
-    #[Option(
-        'Used to filter results by domain. Use ' . Domain::DEFAULT_AUTHORITY . ' keyword to filter by default domain',
-        shortcut: 'd',
-    )]
-    public string|null $domain = null;
-
-    /** @var string[]|null */
-    #[Option('A list of tags that short URLs need to include', name: 'tag', shortcut: 't')]
-    public array|null $tags = null;
-
-    #[Option('If --tag is provided, returns only short URLs including ALL of them')]
-    public bool $tagsAll = false;
-
-    /** @var string[]|null */
-    #[Option('A list of tags that short URLs should NOT include', name: 'exclude-tag', shortcut: 'et')]
-    public array|null $excludeTags = null;
-
-    #[Option('If --exclude-tag is provided, returns only short URLs not including ANY of them')]
-    public bool $excludeTagsAll = false;
-
-    #[Option('Excludes short URLs which reached their max amount of visits')]
-    public bool $excludeMaxVisitsReached = false;
-
-    #[Option('Excludes short URLs which have a "validUntil" date in the past')]
-    public bool $excludePastValidUntil = false;
-
-    #[Option(
-        'Field name to order the list by. Set the dir by optionally passing ASC or DESC after "-": --orderBy=tags-ASC',
-        shortcut: 'o',
-    )]
-    public string|null $orderBy = null;
-
-    #[Option('List only short URLs created by the API key matching provided name', shortcut: 'kn')]
-    public string|null $apiKeyName = null;
-
-    #[Option('Whether to display the tags or not')]
-    public bool $showTags = false;
-
-    #[Option(
-        'Whether to display the domain or not. Those belonging to default domain will have value '
-        . '"' . Domain::DEFAULT_AUTHORITY . '"',
-    )]
-    public bool $showDomain = false;
-
-    #[Option('Whether to display the API key name from which the URL was generated or not', shortcut: 'k')]
-    public bool $showApiKey = false;
-
-    public function toArray(OutputInterface $output): array
-    {
-        $data = [
-            ShortUrlsParamsInputFilter::PAGE => $this->page,
-            ShortUrlsParamsInputFilter::SEARCH_TERM => $this->searchTerm,
-            ShortUrlsParamsInputFilter::DOMAIN => $this->domain,
-            ShortUrlsParamsInputFilter::ORDER_BY => $this->orderBy,
-            ShortUrlsParamsInputFilter::START_DATE => InputUtils::processDate('start-date', $this->startDate, $output),
-            ShortUrlsParamsInputFilter::END_DATE => InputUtils::processDate('end-date', $this->endDate, $output),
-            ShortUrlsParamsInputFilter::EXCLUDE_MAX_VISITS_REACHED => $this->excludeMaxVisitsReached,
-            ShortUrlsParamsInputFilter::EXCLUDE_PAST_VALID_UNTIL => $this->excludePastValidUntil,
-            ShortUrlsParamsInputFilter::API_KEY_NAME => $this->apiKeyName,
-        ];
-
-        if ($this->tags !== null) {
-            $tagsMode = $this->tagsAll ? TagsMode::ALL : TagsMode::ANY;
-            $data[ShortUrlsParamsInputFilter::TAGS_MODE] = $tagsMode->value;
-            $data[ShortUrlsParamsInputFilter::TAGS] = array_unique($this->tags);
-        }
-
-        if ($this->excludeTags !== null) {
-            $excludeTagsMode = $this->excludeTagsAll ? TagsMode::ALL : TagsMode::ANY;
-            $data[ShortUrlsParamsInputFilter::EXCLUDE_TAGS_MODE] = $excludeTagsMode->value;
-            $data[ShortUrlsParamsInputFilter::EXCLUDE_TAGS] = array_unique($this->excludeTags);
-        }
-
-        if ($this->all) {
-            $data[ShortUrlsParamsInputFilter::ITEMS_PER_PAGE] = Paginator::ALL_ITEMS;
-        }
-
-        return $data;
-    }
-}

module/CLI/src/Command/ShortUrl/ListShortUrlsCommand.php

@@ -4,7 +4,9 @@ declare(strict_types=1);
 
 namespace Shlinkio\Shlink\CLI\Command\ShortUrl;
 
-use Shlinkio\Shlink\CLI\Command\ShortUrl\Input\ShortUrlsParamsInput;
+use Shlinkio\Shlink\CLI\Input\EndDateOption;
+use Shlinkio\Shlink\CLI\Input\StartDateOption;
+use Shlinkio\Shlink\CLI\Util\ExitCode;
 use Shlinkio\Shlink\CLI\Util\ShlinkTable;
 use Shlinkio\Shlink\Common\Paginator\Paginator;
 use Shlinkio\Shlink\Common\Paginator\Util\PagerfantaUtils;
@@ -12,57 +14,169 @@ use Shlinkio\Shlink\Core\Domain\Entity\Domain;
 use Shlinkio\Shlink\Core\ShortUrl\Entity\ShortUrl;
 use Shlinkio\Shlink\Core\ShortUrl\Model\ShortUrlsParams;
 use Shlinkio\Shlink\Core\ShortUrl\Model\ShortUrlWithDeps;
+use Shlinkio\Shlink\Core\ShortUrl\Model\TagsMode;
 use Shlinkio\Shlink\Core\ShortUrl\Model\Validation\ShortUrlsParamsInputFilter;
 use Shlinkio\Shlink\Core\ShortUrl\ShortUrlListServiceInterface;
 use Shlinkio\Shlink\Core\ShortUrl\Transformer\ShortUrlDataTransformerInterface;
-use Symfony\Component\Console\Attribute\AsCommand;
-use Symfony\Component\Console\Attribute\MapInput;
 use Symfony\Component\Console\Command\Command;
 use Symfony\Component\Console\Input\InputInterface;
+use Symfony\Component\Console\Input\InputOption;
 use Symfony\Component\Console\Output\OutputInterface;
 use Symfony\Component\Console\Style\SymfonyStyle;
 
 use function array_keys;
+use function array_pad;
+use function explode;
 use function implode;
 use function Shlinkio\Shlink\Core\ArrayUtils\map;
 use function sprintf;
 
-#[AsCommand(name: ListShortUrlsCommand::NAME, description: 'List all short URLs')]
 class ListShortUrlsCommand extends Command
 {
     public const string NAME = 'short-url:list';
 
+    private readonly StartDateOption $startDateOption;
+    private readonly EndDateOption $endDateOption;
+
     public function __construct(
         private readonly ShortUrlListServiceInterface $shortUrlService,
         private readonly ShortUrlDataTransformerInterface $transformer,
     ) {
         parent::__construct();
+        $this->startDateOption = new StartDateOption($this, 'short URLs');
+        $this->endDateOption = new EndDateOption($this, 'short URLs');
     }
 
-    public function __invoke(
-        SymfonyStyle $io,
-        InputInterface $input,
-        #[MapInput] ShortUrlsParamsInput $paramsInput,
-    ): int {
-        $page = $paramsInput->page;
-        $data = $paramsInput->toArray($io);
+    protected function configure(): void
+    {
+        $this
+            ->setName(self::NAME)
+            ->setDescription('List all short URLs')
+            ->addOption(
+                'page',
+                'p',
+                InputOption::VALUE_REQUIRED,
+                'The first page to list (10 items per page unless "--all" is provided).',
+                '1',
+            )
+            ->addOption(
+                'search-term',
+                'st',
+                InputOption::VALUE_REQUIRED,
+                'A query used to filter results by searching for it on the longUrl and shortCode fields.',
+            )
+            ->addOption(
+                'domain',
+                'd',
+                InputOption::VALUE_REQUIRED,
+                'Used to filter results by domain. Use DEFAULT keyword to filter by default domain',
+            )
+            ->addOption(
+                'tags',
+                't',
+                InputOption::VALUE_REQUIRED,
+                'A comma-separated list of tags to filter results.',
+            )
+            ->addOption(
+                'including-all-tags',
+                'i',
+                InputOption::VALUE_NONE,
+                'If tags is provided, returns only short URLs having ALL tags.',
+            )
+            ->addOption(
+                'exclude-max-visits-reached',
+                null,
+                InputOption::VALUE_NONE,
+                'Excludes short URLs which reached their max amount of visits.',
+            )
+            ->addOption(
+                'exclude-past-valid-until',
+                null,
+                InputOption::VALUE_NONE,
+                'Excludes short URLs which have a "validUntil" date in the past.',
+            )
+            ->addOption(
+                'order-by',
+                'o',
+                InputOption::VALUE_REQUIRED,
+                'The field from which you want to order by. '
+                    . 'Define ordering dir by passing ASC or DESC after "-" or ",".',
+            )
+            ->addOption(
+                'show-tags',
+                null,
+                InputOption::VALUE_NONE,
+                'Whether to display the tags or not.',
+            )
+            ->addOption(
+                'show-domain',
+                null,
+                InputOption::VALUE_NONE,
+                'Whether to display the domain or not. Those belonging to default domain will have value "DEFAULT".',
+            )
+            ->addOption(
+                'show-api-key',
+                'k',
+                InputOption::VALUE_NONE,
+                'Whether to display the API key name from which the URL was generated or not.',
+            )
+            ->addOption('show-api-key-name', 'm', InputOption::VALUE_NONE, '[DEPRECATED] Use show-api-key')
+            ->addOption(
+                'all',
+                'a',
+                InputOption::VALUE_NONE,
+                'Disables pagination and just displays all existing URLs. Caution! If the amount of short URLs is big,'
+                . ' this may end up failing due to memory usage.',
+            );
+    }
 
+    protected function execute(InputInterface $input, OutputInterface $output): int
+    {
+        $io = new SymfonyStyle($input, $output);
+
+        $page = (int) $input->getOption('page');
+        $searchTerm = $input->getOption('search-term');
+        $domain = $input->getOption('domain');
+        $tags = $input->getOption('tags');
+        $tagsMode = $input->getOption('including-all-tags') === true ? TagsMode::ALL->value : TagsMode::ANY->value;
+        $tags = ! empty($tags) ? explode(',', $tags) : [];
+        $all = $input->getOption('all');
+        $startDate = $this->startDateOption->get($input, $output);
+        $endDate = $this->endDateOption->get($input, $output);
+        $orderBy = $this->processOrderBy($input);
         $columnsMap = $this->resolveColumnsMap($input);
+
+        $data = [
+            ShortUrlsParamsInputFilter::SEARCH_TERM => $searchTerm,
+            ShortUrlsParamsInputFilter::DOMAIN => $domain,
+            ShortUrlsParamsInputFilter::TAGS => $tags,
+            ShortUrlsParamsInputFilter::TAGS_MODE => $tagsMode,
+            ShortUrlsParamsInputFilter::ORDER_BY => $orderBy,
+            ShortUrlsParamsInputFilter::START_DATE => $startDate?->toAtomString(),
+            ShortUrlsParamsInputFilter::END_DATE => $endDate?->toAtomString(),
+            ShortUrlsParamsInputFilter::EXCLUDE_MAX_VISITS_REACHED => $input->getOption('exclude-max-visits-reached'),
+            ShortUrlsParamsInputFilter::EXCLUDE_PAST_VALID_UNTIL => $input->getOption('exclude-past-valid-until'),
+        ];
+
+        if ($all) {
+            $data[ShortUrlsParamsInputFilter::ITEMS_PER_PAGE] = Paginator::ALL_ITEMS;
+        }
+
         do {
             $data[ShortUrlsParamsInputFilter::PAGE] = $page;
-            $result = $this->renderPage($io, $columnsMap, ShortUrlsParams::fromRawData($data), $paramsInput->all);
+            $result = $this->renderPage($output, $columnsMap, ShortUrlsParams::fromRawData($data), $all);
             $page++;
 
             $continue = $result->hasNextPage() && $io->confirm(
                 sprintf('Continue with page <options=bold>%s</>?', $page),
-                default: false,
+                false,
             );
         } while ($continue);
 
         $io->newLine();
         $io->success('Short URLs properly listed');
 
-        return self::SUCCESS;
+        return ExitCode::EXIT_SUCCESS;
     }
 
     /**
@@ -91,6 +205,17 @@ class ListShortUrlsCommand extends Command
         return $shortUrls;
     }
 
+    private function processOrderBy(InputInterface $input): string|null
+    {
+        $orderBy = $input->getOption('order-by');
+        if (empty($orderBy)) {
+            return null;
+        }
+
+        [$field, $dir] = array_pad(explode(',', $orderBy), 2, null);
+        return $dir === null ? $field : sprintf('%s-%s', $field, $dir);
+    }
+
     /**
      * @return array<string, callable(array $serializedShortUrl, ShortUrl $shortUrl): ?string>
      */
@@ -112,7 +237,7 @@ class ListShortUrlsCommand extends Command
             $columnsMap['Domain'] = static fn (array $_, ShortUrl $shortUrl): string =>
                 $shortUrl->getDomain()->authority ?? Domain::DEFAULT_AUTHORITY;
         }
-        if ($input->getOption('show-api-key')) {
+        if ($input->getOption('show-api-key') || $input->getOption('show-api-key-name')) {
             $columnsMap['API Key Name'] = static fn (array $_, ShortUrl $shortUrl): string|null =>
                 $shortUrl->authorApiKey?->name;
         }

module/CLI/src/Command/ShortUrl/ResolveUrlCommand.php

@@ -4,46 +4,65 @@ declare(strict_types=1);
 
 namespace Shlinkio\Shlink\CLI\Command\ShortUrl;
 
+use Shlinkio\Shlink\CLI\Input\ShortUrlIdentifierInput;
+use Shlinkio\Shlink\CLI\Util\ExitCode;
 use Shlinkio\Shlink\Core\Exception\ShortUrlNotFoundException;
-use Shlinkio\Shlink\Core\ShortUrl\Model\ShortUrlIdentifier;
 use Shlinkio\Shlink\Core\ShortUrl\ShortUrlResolverInterface;
-use Symfony\Component\Console\Attribute\Argument;
-use Symfony\Component\Console\Attribute\AsCommand;
-use Symfony\Component\Console\Attribute\Ask;
-use Symfony\Component\Console\Attribute\Option;
 use Symfony\Component\Console\Command\Command;
+use Symfony\Component\Console\Input\InputInterface;
+use Symfony\Component\Console\Output\OutputInterface;
 use Symfony\Component\Console\Style\SymfonyStyle;
 
 use function sprintf;
 
-#[AsCommand(ResolveUrlCommand::NAME, 'Returns the long URL behind a short code')]
 class ResolveUrlCommand extends Command
 {
-    public const string NAME = 'short-url:resolve';
+    public const string NAME = 'short-url:parse';
+
+    private readonly ShortUrlIdentifierInput $shortUrlIdentifierInput;
 
     public function __construct(private readonly ShortUrlResolverInterface $urlResolver)
     {
         parent::__construct();
+        $this->shortUrlIdentifierInput = new ShortUrlIdentifierInput(
+            $this,
+            shortCodeDesc: 'The short code to parse',
+            domainDesc: 'The domain to which the short URL is attached.',
+        );
     }
 
-    public function __invoke(
-        SymfonyStyle $io,
-        #[
-            Argument('The short code to resolve'),
-            Ask('A short code was not provided. Which short code do you want to resolve?'),
-        ]
-        string $shortCode,
-        #[Option('The domain to which the short URL is attached', shortcut: 'd')] string|null $domain = null,
-    ): int {
-        $identifier = ShortUrlIdentifier::fromShortCodeAndDomain($shortCode, $domain);
+    protected function configure(): void
+    {
+        $this
+            ->setName(self::NAME)
+            ->setDescription('Returns the long URL behind a short code');
+    }
+
+    protected function interact(InputInterface $input, OutputInterface $output): void
+    {
+        $shortCode = $this->shortUrlIdentifierInput->shortCode($input);
+        if (! empty($shortCode)) {
+            return;
+        }
+
+        $io = new SymfonyStyle($input, $output);
+        $shortCode = $io->ask('A short code was not provided. Which short code do you want to parse?');
+        if (! empty($shortCode)) {
+            $input->setArgument('shortCode', $shortCode);
+        }
+    }
+
+    protected function execute(InputInterface $input, OutputInterface $output): int
+    {
+        $io = new SymfonyStyle($input, $output);
 
         try {
-            $url = $this->urlResolver->resolveShortUrl($identifier);
-            $io->writeln(sprintf('Long URL: <info>%s</info>', $url->getLongUrl()));
-            return self::SUCCESS;
+            $url = $this->urlResolver->resolveShortUrl($this->shortUrlIdentifierInput->toShortUrlIdentifier($input));
+            $output->writeln(sprintf('Long URL: <info>%s</info>', $url->getLongUrl()));
+            return ExitCode::EXIT_SUCCESS;
         } catch (ShortUrlNotFoundException $e) {
             $io->error($e->getMessage());
-            return self::FAILURE;
+            return ExitCode::EXIT_FAILURE;
         }
     }
 }

module/CLI/src/Command/Tag/DeleteTagsCommand.php

@@ -4,13 +4,14 @@ declare(strict_types=1);
 
 namespace Shlinkio\Shlink\CLI\Command\Tag;
 
+use Shlinkio\Shlink\CLI\Util\ExitCode;
 use Shlinkio\Shlink\Core\Tag\TagServiceInterface;
-use Symfony\Component\Console\Attribute\AsCommand;
-use Symfony\Component\Console\Attribute\Option;
 use Symfony\Component\Console\Command\Command;
+use Symfony\Component\Console\Input\InputInterface;
+use Symfony\Component\Console\Input\InputOption;
+use Symfony\Component\Console\Output\OutputInterface;
 use Symfony\Component\Console\Style\SymfonyStyle;
 
-#[AsCommand(name: DeleteTagsCommand::NAME, description: 'Deletes one or more tags.')]
 class DeleteTagsCommand extends Command
 {
     public const string NAME = 'tag:delete';
@@ -20,21 +21,31 @@ class DeleteTagsCommand extends Command
         parent::__construct();
     }
 
-    /**
-     * @param string[] $tagNames
-     */
-    public function __invoke(
-        SymfonyStyle $io,
-        #[Option('The name of the tags to delete', name: 'name', shortcut: 't')] array $tagNames = [],
-    ): int {
+    protected function configure(): void
+    {
+        $this
+            ->setName(self::NAME)
+            ->setDescription('Deletes one or more tags.')
+            ->addOption(
+                'name',
+                't',
+                InputOption::VALUE_REQUIRED | InputOption::VALUE_IS_ARRAY,
+                'The name of the tags to delete',
+            );
+    }
+
+    protected function execute(InputInterface $input, OutputInterface $output): int
+    {
+        $io = new SymfonyStyle($input, $output);
+        $tagNames = $input->getOption('name');
+
         if (empty($tagNames)) {
             $io->warning('You have to provide at least one tag name');
-            return self::INVALID;
+            return ExitCode::EXIT_WARNING;
         }
 
         $this->tagService->deleteTags($tagNames);
         $io->success('Tags properly deleted');
-
-        return self::SUCCESS;
+        return ExitCode::EXIT_SUCCESS;
     }
 }

module/CLI/src/Command/Tag/GetTagVisitsCommand.php

@@ -4,47 +4,50 @@ declare(strict_types=1);
 
 namespace Shlinkio\Shlink\CLI\Command\Tag;
 
-use Shlinkio\Shlink\CLI\Command\Visit\VisitsCommandUtils;
-use Shlinkio\Shlink\CLI\Input\VisitsListInput;
-use Shlinkio\Shlink\Core\Domain\Entity\Domain;
-use Shlinkio\Shlink\Core\Visit\Model\WithDomainVisitsParams;
+use Shlinkio\Shlink\CLI\Command\Visit\AbstractVisitsListCommand;
+use Shlinkio\Shlink\Common\Paginator\Paginator;
+use Shlinkio\Shlink\Common\Util\DateRange;
+use Shlinkio\Shlink\Core\ShortUrl\Helper\ShortUrlStringifierInterface;
+use Shlinkio\Shlink\Core\Visit\Entity\Visit;
+use Shlinkio\Shlink\Core\Visit\Model\VisitsParams;
 use Shlinkio\Shlink\Core\Visit\VisitsStatsHelperInterface;
-use Symfony\Component\Console\Attribute\Argument;
-use Symfony\Component\Console\Attribute\AsCommand;
-use Symfony\Component\Console\Attribute\Ask;
-use Symfony\Component\Console\Attribute\MapInput;
-use Symfony\Component\Console\Attribute\Option;
-use Symfony\Component\Console\Command\Command;
-use Symfony\Component\Console\Style\SymfonyStyle;
+use Symfony\Component\Console\Input\InputArgument;
+use Symfony\Component\Console\Input\InputInterface;
 
-#[AsCommand(GetTagVisitsCommand::NAME, 'Returns the list of visits for provided tag')]
-class GetTagVisitsCommand extends Command
+class GetTagVisitsCommand extends AbstractVisitsListCommand
 {
     public const string NAME = 'tag:visits';
 
-    public function __construct(private readonly VisitsStatsHelperInterface $visitsHelper)
-    {
-        parent::__construct();
+    public function __construct(
+        VisitsStatsHelperInterface $visitsHelper,
+        private readonly ShortUrlStringifierInterface $shortUrlStringifier,
+    ) {
+        parent::__construct($visitsHelper);
     }
 
-    public function __invoke(
-        SymfonyStyle $io,
-        #[Argument('The tag which visits we want to get'), Ask('For what tag do you want to get visits')] string $tag,
-        #[MapInput] VisitsListInput $input,
-        #[Option(
-            'Return visits that belong to this domain only. Use ' . Domain::DEFAULT_AUTHORITY . ' keyword for visits '
-            . 'in default domain',
-            shortcut: 'd',
-        )]
-        string|null $domain = null,
-    ): int {
-        $paginator = $this->visitsHelper->visitsForTag($tag, new WithDomainVisitsParams(
-            dateRange: $input->dateRange(),
-            domain: $domain,
-        ));
+    protected function configure(): void
+    {
+        $this
+            ->setName(self::NAME)
+            ->setDescription('Returns the list of visits for provided tag.')
+            ->addArgument('tag', InputArgument::REQUIRED, 'The tag which visits we want to get.');
+    }
 
-        VisitsCommandUtils::renderOutput($io, $input, $paginator);
+    /**
+     * @return Paginator<Visit>
+     */
+    protected function getVisitsPaginator(InputInterface $input, DateRange $dateRange): Paginator
+    {
+        $tag = $input->getArgument('tag');
+        return $this->visitsHelper->visitsForTag($tag, new VisitsParams($dateRange));
+    }
 
-        return self::SUCCESS;
+    /**
+     * @return array<string, string>
+     */
+    protected function mapExtraFields(Visit $visit): array
+    {
+        $shortUrl = $visit->shortUrl;
+        return $shortUrl === null ? [] : ['shortUrl' => $this->shortUrlStringifier->stringify($shortUrl)];
     }
 }

module/CLI/src/Command/Tag/ListTagsCommand.php

@@ -4,17 +4,17 @@ declare(strict_types=1);
 
 namespace Shlinkio\Shlink\CLI\Command\Tag;
 
+use Shlinkio\Shlink\CLI\Util\ExitCode;
 use Shlinkio\Shlink\CLI\Util\ShlinkTable;
 use Shlinkio\Shlink\Core\Tag\Model\TagInfo;
 use Shlinkio\Shlink\Core\Tag\Model\TagsParams;
 use Shlinkio\Shlink\Core\Tag\TagServiceInterface;
-use Symfony\Component\Console\Attribute\AsCommand;
 use Symfony\Component\Console\Command\Command;
-use Symfony\Component\Console\Style\SymfonyStyle;
+use Symfony\Component\Console\Input\InputInterface;
+use Symfony\Component\Console\Output\OutputInterface;
 
 use function array_map;
 
-#[AsCommand(ListTagsCommand::NAME, 'Lists existing tags.')]
 class ListTagsCommand extends Command
 {
     public const string NAME = 'tag:list';
@@ -24,10 +24,17 @@ class ListTagsCommand extends Command
         parent::__construct();
     }
 
-    public function __invoke(SymfonyStyle $io): int
+    protected function configure(): void
     {
-        ShlinkTable::default($io)->render(['Name', 'URLs amount', 'Visits amount'], $this->getTagsRows());
-        return self::SUCCESS;
+        $this
+            ->setName(self::NAME)
+            ->setDescription('Lists existing tags.');
+    }
+
+    protected function execute(InputInterface $input, OutputInterface $output): int
+    {
+        ShlinkTable::default($output)->render(['Name', 'URLs amount', 'Visits amount'], $this->getTagsRows());
+        return ExitCode::EXIT_SUCCESS;
     }
 
     private function getTagsRows(): array

module/CLI/src/Command/Tag/RenameTagCommand.php

@@ -4,16 +4,17 @@ declare(strict_types=1);
 
 namespace Shlinkio\Shlink\CLI\Command\Tag;
 
+use Shlinkio\Shlink\CLI\Util\ExitCode;
 use Shlinkio\Shlink\Core\Exception\TagConflictException;
 use Shlinkio\Shlink\Core\Exception\TagNotFoundException;
 use Shlinkio\Shlink\Core\Model\Renaming;
 use Shlinkio\Shlink\Core\Tag\TagServiceInterface;
-use Symfony\Component\Console\Attribute\Argument;
-use Symfony\Component\Console\Attribute\AsCommand;
 use Symfony\Component\Console\Command\Command;
+use Symfony\Component\Console\Input\InputArgument;
+use Symfony\Component\Console\Input\InputInterface;
+use Symfony\Component\Console\Output\OutputInterface;
 use Symfony\Component\Console\Style\SymfonyStyle;
 
-#[AsCommand(RenameTagCommand::NAME, 'Renames one existing tag.')]
 class RenameTagCommand extends Command
 {
     public const string NAME = 'tag:rename';
@@ -23,18 +24,28 @@ class RenameTagCommand extends Command
         parent::__construct();
     }
 
-    public function __invoke(
-        SymfonyStyle $io,
-        #[Argument('Current name of the tag.')] string $oldName,
-        #[Argument('New name of the tag.')] string $newName,
-    ): int {
+    protected function configure(): void
+    {
+        $this
+            ->setName(self::NAME)
+            ->setDescription('Renames one existing tag.')
+            ->addArgument('oldName', InputArgument::REQUIRED, 'Current name of the tag.')
+            ->addArgument('newName', InputArgument::REQUIRED, 'New name of the tag.');
+    }
+
+    protected function execute(InputInterface $input, OutputInterface $output): int
+    {
+        $io = new SymfonyStyle($input, $output);
+        $oldName = $input->getArgument('oldName');
+        $newName = $input->getArgument('newName');
+
         try {
             $this->tagService->renameTag(Renaming::fromNames($oldName, $newName));
             $io->success('Tag properly renamed.');
-            return Command::SUCCESS;
+            return ExitCode::EXIT_SUCCESS;
         } catch (TagNotFoundException | TagConflictException $e) {
             $io->error($e->getMessage());
-            return Command::FAILURE;
+            return ExitCode::EXIT_FAILURE;
         }
     }
 }

module/CLI/src/Command/Util/AbstractLockedCommand.php

@@ -0,0 +1,44 @@
+<?php
+
+declare(strict_types=1);
+
+namespace Shlinkio\Shlink\CLI\Command\Util;
+
+use Shlinkio\Shlink\CLI\Util\ExitCode;
+use Symfony\Component\Console\Command\Command;
+use Symfony\Component\Console\Input\InputInterface;
+use Symfony\Component\Console\Output\OutputInterface;
+use Symfony\Component\Lock\LockFactory;
+
+use function sprintf;
+
+abstract class AbstractLockedCommand extends Command
+{
+    public function __construct(private readonly LockFactory $locker)
+    {
+        parent::__construct();
+    }
+
+    final protected function execute(InputInterface $input, OutputInterface $output): int
+    {
+        $lockConfig = $this->getLockConfig();
+        $lock = $this->locker->createLock($lockConfig->lockName, $lockConfig->ttl, $lockConfig->isBlocking);
+
+        if (! $lock->acquire($lockConfig->isBlocking)) {
+            $output->writeln(
+                sprintf('<comment>Command "%s" is already in progress. Skipping.</comment>', $lockConfig->lockName),
+            );
+            return ExitCode::EXIT_WARNING;
+        }
+
+        try {
+            return $this->lockedExecute($input, $output);
+        } finally {
+            $lock->release();
+        }
+    }
+
+    abstract protected function lockedExecute(InputInterface $input, OutputInterface $output): int;
+
+    abstract protected function getLockConfig(): LockedCommandConfig;
+}

module/CLI/src/Command/Util/CommandUtils.php

@@ -1,58 +0,0 @@
-<?php
-
-declare(strict_types=1);
-
-namespace Shlinkio\Shlink\CLI\Command\Util;
-
-use Symfony\Component\Console\Command\Command;
-use Symfony\Component\Console\Style\SymfonyStyle;
-use Symfony\Component\Lock\LockFactory;
-
-use function sprintf;
-
-class CommandUtils
-{
-    /**
-     * Displays a warning and confirmation message before running a callback. If the response to the confirmation is
-     * positive, the callback is executed normally.
-     *
-     * @param callable(): int $callback
-     */
-    public static function executeWithWarning(string $warning, SymfonyStyle $io, callable $callback): int
-    {
-        $io->warning($warning);
-        if (! $io->confirm('<comment>Do you want to proceed?</comment>', default: false)) {
-            $io->info('Operation aborted');
-            return Command::SUCCESS;
-        }
-
-        return $callback();
-    }
-
-    /**
-     * Runs a callback with a lock, making sure the lock is released after running the callback, and the callback does
-     * not run if the lock is already acquired.
-     *
-     * @param callable(): int $callback
-     */
-    public static function executeWithLock(
-        LockFactory $locker,
-        LockConfig $lockConfig,
-        SymfonyStyle $io,
-        callable $callback,
-    ): int {
-        $lock = $locker->createLock($lockConfig->lockName, $lockConfig->ttl, $lockConfig->isBlocking);
-        if (! $lock->acquire($lockConfig->isBlocking)) {
-            $io->writeln(
-                sprintf('<comment>Command "%s" is already in progress. Skipping.</comment>', $lockConfig->lockName),
-            );
-            return Command::INVALID;
-        }
-
-        try {
-            return $callback();
-        } finally {
-            $lock->release();
-        }
-    }
-}

module/CLI/src/Command/Util/LockedCommandConfig.php

@@ -4,24 +4,24 @@ declare(strict_types=1);
 
 namespace Shlinkio\Shlink\CLI\Command\Util;
 
-final readonly class LockConfig
+final class LockedCommandConfig
 {
     public const float DEFAULT_TTL = 600.0; // 10 minutes
 
     private function __construct(
-        public string $lockName,
-        public bool $isBlocking,
-        public float $ttl = self::DEFAULT_TTL,
+        public readonly string $lockName,
+        public readonly bool $isBlocking,
+        public readonly float $ttl = self::DEFAULT_TTL,
     ) {
     }
 
     public static function blocking(string $lockName): self
     {
-        return new self($lockName, isBlocking: true);
+        return new self($lockName, true);
     }
 
     public static function nonBlocking(string $lockName): self
     {
-        return new self($lockName, isBlocking: false);
+        return new self($lockName, false);
     }
 }

module/CLI/src/Command/Visit/AbstractDeleteVisitsCommand.php

@@ -0,0 +1,35 @@
+<?php
+
+declare(strict_types=1);
+
+namespace Shlinkio\Shlink\CLI\Command\Visit;
+
+use Shlinkio\Shlink\CLI\Util\ExitCode;
+use Symfony\Component\Console\Command\Command;
+use Symfony\Component\Console\Input\InputInterface;
+use Symfony\Component\Console\Output\OutputInterface;
+use Symfony\Component\Console\Style\SymfonyStyle;
+
+abstract class AbstractDeleteVisitsCommand extends Command
+{
+    final protected function execute(InputInterface $input, OutputInterface $output): int
+    {
+        $io = new SymfonyStyle($input, $output);
+        if (! $this->confirm($io)) {
+            $io->info('Operation aborted');
+            return ExitCode::EXIT_SUCCESS;
+        }
+
+        return $this->doExecute($input, $io);
+    }
+
+    private function confirm(SymfonyStyle $io): bool
+    {
+        $io->warning($this->getWarningMessage());
+        return $io->confirm('<comment>Continue deleting visits?</comment>', false);
+    }
+
+    abstract protected function doExecute(InputInterface $input, SymfonyStyle $io): int;
+
+    abstract protected function getWarningMessage(): string;
+}

module/CLI/src/Command/Visit/AbstractVisitsListCommand.php

@@ -0,0 +1,89 @@
+<?php
+
+declare(strict_types=1);
+
+namespace Shlinkio\Shlink\CLI\Command\Visit;
+
+use Shlinkio\Shlink\CLI\Input\EndDateOption;
+use Shlinkio\Shlink\CLI\Input\StartDateOption;
+use Shlinkio\Shlink\CLI\Util\ExitCode;
+use Shlinkio\Shlink\CLI\Util\ShlinkTable;
+use Shlinkio\Shlink\Common\Paginator\Paginator;
+use Shlinkio\Shlink\Common\Util\DateRange;
+use Shlinkio\Shlink\Core\Visit\Entity\Visit;
+use Shlinkio\Shlink\Core\Visit\VisitsStatsHelperInterface;
+use Symfony\Component\Console\Command\Command;
+use Symfony\Component\Console\Input\InputInterface;
+use Symfony\Component\Console\Output\OutputInterface;
+
+use function array_keys;
+use function array_map;
+use function Shlinkio\Shlink\Common\buildDateRange;
+use function Shlinkio\Shlink\Core\ArrayUtils\select_keys;
+use function Shlinkio\Shlink\Core\camelCaseToHumanFriendly;
+
+abstract class AbstractVisitsListCommand extends Command
+{
+    private readonly StartDateOption $startDateOption;
+    private readonly EndDateOption $endDateOption;
+
+    public function __construct(protected readonly VisitsStatsHelperInterface $visitsHelper)
+    {
+        parent::__construct();
+        $this->startDateOption = new StartDateOption($this, 'visits');
+        $this->endDateOption = new EndDateOption($this, 'visits');
+    }
+
+    final protected function execute(InputInterface $input, OutputInterface $output): int
+    {
+        $startDate = $this->startDateOption->get($input, $output);
+        $endDate = $this->endDateOption->get($input, $output);
+        $paginator = $this->getVisitsPaginator($input, buildDateRange($startDate, $endDate));
+        [$rows, $headers] = $this->resolveRowsAndHeaders($paginator);
+
+        ShlinkTable::default($output)->render($headers, $rows);
+
+        return ExitCode::EXIT_SUCCESS;
+    }
+
+    /**
+     * @param Paginator<Visit> $paginator
+     */
+    private function resolveRowsAndHeaders(Paginator $paginator): array
+    {
+        $extraKeys = [];
+        $rows = array_map(function (Visit $visit) use (&$extraKeys) {
+            $extraFields = $this->mapExtraFields($visit);
+            $extraKeys = array_keys($extraFields);
+
+            $rowData = [
+                'referer' => $visit->referer,
+                'date' => $visit->date->toAtomString(),
+                'userAgent' => $visit->userAgent,
+                'potentialBot' => $visit->potentialBot,
+                'country' => $visit->getVisitLocation()->countryName ?? 'Unknown',
+                'city' => $visit->getVisitLocation()->cityName ?? 'Unknown',
+                ...$extraFields,
+            ];
+
+            // Filter out unknown keys
+            return select_keys($rowData, ['referer', 'date', 'userAgent', 'country', 'city', ...$extraKeys]);
+        }, [...$paginator->getCurrentPageResults()]);
+        $extra = array_map(camelCaseToHumanFriendly(...), $extraKeys);
+
+        return [
+            $rows,
+            ['Referer', 'Date', 'User agent', 'Country', 'City', ...$extra],
+        ];
+    }
+
+    /**
+     * @return Paginator<Visit>
+     */
+    abstract protected function getVisitsPaginator(InputInterface $input, DateRange $dateRange): Paginator;
+
+    /**
+     * @return array<string, string>
+     */
+    abstract protected function mapExtraFields(Visit $visit): array;
+}

module/CLI/src/Command/Visit/DeleteOrphanVisitsCommand.php

@@ -4,16 +4,14 @@ declare(strict_types=1);
 
 namespace Shlinkio\Shlink\CLI\Command\Visit;
 
-use Shlinkio\Shlink\CLI\Command\Util\CommandUtils;
+use Shlinkio\Shlink\CLI\Util\ExitCode;
 use Shlinkio\Shlink\Core\Visit\VisitsDeleterInterface;
-use Symfony\Component\Console\Attribute\AsCommand;
-use Symfony\Component\Console\Command\Command;
+use Symfony\Component\Console\Input\InputInterface;
 use Symfony\Component\Console\Style\SymfonyStyle;
 
 use function sprintf;
 
-#[AsCommand(DeleteOrphanVisitsCommand::NAME, 'Deletes all orphan visits')]
-class DeleteOrphanVisitsCommand extends Command
+class DeleteOrphanVisitsCommand extends AbstractDeleteVisitsCommand
 {
     public const string NAME = 'visit:orphan-delete';
 
@@ -22,20 +20,23 @@ class DeleteOrphanVisitsCommand extends Command
         parent::__construct();
     }
 
-    public function __invoke(SymfonyStyle $io): int
+    protected function configure(): void
     {
-        return CommandUtils::executeWithWarning(
-            'You are about to delete all orphan visits. This operation cannot be undone',
-            $io,
-            fn () => $this->deleteVisits($io),
-        );
+        $this
+            ->setName(self::NAME)
+            ->setDescription('Deletes all orphan visits');
     }
 
-    private function deleteVisits(SymfonyStyle $io): int
+    protected function doExecute(InputInterface $input, SymfonyStyle $io): int
     {
         $result = $this->deleter->deleteOrphanVisits();
         $io->success(sprintf('Successfully deleted %s visits', $result->affectedItems));
 
-        return self::SUCCESS;
+        return ExitCode::EXIT_SUCCESS;
+    }
+
+    protected function getWarningMessage(): string
+    {
+        return 'You are about to delete all orphan visits. This operation cannot be undone.';
     }
 }

module/CLI/src/Command/Visit/DownloadGeoLiteDbCommand.php

@@ -4,21 +4,19 @@ declare(strict_types=1);
 
 namespace Shlinkio\Shlink\CLI\Command\Visit;
 
+use Shlinkio\Shlink\CLI\Util\ExitCode;
 use Shlinkio\Shlink\Core\Exception\GeolocationDbUpdateFailedException;
 use Shlinkio\Shlink\Core\Geolocation\GeolocationDbUpdaterInterface;
 use Shlinkio\Shlink\Core\Geolocation\GeolocationDownloadProgressHandlerInterface;
 use Shlinkio\Shlink\Core\Geolocation\GeolocationResult;
-use Symfony\Component\Console\Attribute\AsCommand;
 use Symfony\Component\Console\Command\Command;
 use Symfony\Component\Console\Helper\ProgressBar;
+use Symfony\Component\Console\Input\InputInterface;
+use Symfony\Component\Console\Output\OutputInterface;
 use Symfony\Component\Console\Style\SymfonyStyle;
 
 use function sprintf;
 
-#[AsCommand(
-    DownloadGeoLiteDbCommand::NAME,
-    'Checks if the GeoLite2 db file is too old or it does not exist, and tries to download an up-to-date copy if so',
-)]
 class DownloadGeoLiteDbCommand extends Command implements GeolocationDownloadProgressHandlerInterface
 {
     public const string NAME = 'visit:download-db';
@@ -31,26 +29,36 @@ class DownloadGeoLiteDbCommand extends Command implements GeolocationDownloadPro
         parent::__construct();
     }
 
-    public function __invoke(SymfonyStyle $io): int
+    protected function configure(): void
     {
-        $this->io = $io;
+        $this
+            ->setName(self::NAME)
+            ->setDescription(
+                'Checks if the GeoLite2 db file is too old or it does not exist, and tries to download an up-to-date '
+                . 'copy if so.',
+            );
+    }
+
+    protected function execute(InputInterface $input, OutputInterface $output): int
+    {
+        $this->io = new SymfonyStyle($input, $output);
 
         try {
             $result = $this->dbUpdater->checkDbUpdate($this);
 
             if ($result === GeolocationResult::LICENSE_MISSING) {
                 $this->io->warning('It was not possible to download GeoLite2 db, because a license was not provided.');
-                return self::INVALID;
+                return ExitCode::EXIT_WARNING;
             }
 
             if ($result === GeolocationResult::MAX_ERRORS_REACHED) {
                 $this->io->warning('Max consecutive errors reached. Cannot retry for a couple of days.');
-                return self::INVALID;
+                return ExitCode::EXIT_WARNING;
             }
 
             if ($result === GeolocationResult::UPDATE_IN_PROGRESS) {
                 $this->io->warning('A geolocation db is already being downloaded by another process.');
-                return self::INVALID;
+                return ExitCode::EXIT_WARNING;
             }
 
             if ($this->progressBar === null) {
@@ -60,7 +68,7 @@ class DownloadGeoLiteDbCommand extends Command implements GeolocationDownloadPro
                 $this->io->success('GeoLite2 db file properly downloaded.');
             }
 
-            return self::SUCCESS;
+            return ExitCode::EXIT_SUCCESS;
         } catch (GeolocationDbUpdateFailedException $e) {
             return $this->processGeoLiteUpdateError($e, $this->io);
         }
@@ -82,7 +90,7 @@ class DownloadGeoLiteDbCommand extends Command implements GeolocationDownloadPro
             $this->getApplication()?->renderThrowable($e, $io);
         }
 
-        return $olderDbExists ? self::INVALID : self::FAILURE;
+        return $olderDbExists ? ExitCode::EXIT_WARNING : ExitCode::EXIT_FAILURE;
     }
 
     public function beforeDownload(bool $olderDbExists): void

module/CLI/src/Command/Visit/GetNonOrphanVisitsCommand.php

@@ -4,42 +4,46 @@ declare(strict_types=1);
 
 namespace Shlinkio\Shlink\CLI\Command\Visit;
 
-use Shlinkio\Shlink\CLI\Input\VisitsListInput;
-use Shlinkio\Shlink\Core\Domain\Entity\Domain;
-use Shlinkio\Shlink\Core\Visit\Model\WithDomainVisitsParams;
+use Shlinkio\Shlink\Common\Paginator\Paginator;
+use Shlinkio\Shlink\Common\Util\DateRange;
+use Shlinkio\Shlink\Core\ShortUrl\Helper\ShortUrlStringifierInterface;
+use Shlinkio\Shlink\Core\Visit\Entity\Visit;
+use Shlinkio\Shlink\Core\Visit\Model\VisitsParams;
 use Shlinkio\Shlink\Core\Visit\VisitsStatsHelperInterface;
-use Symfony\Component\Console\Attribute\AsCommand;
-use Symfony\Component\Console\Attribute\MapInput;
-use Symfony\Component\Console\Attribute\Option;
-use Symfony\Component\Console\Command\Command;
-use Symfony\Component\Console\Style\SymfonyStyle;
+use Symfony\Component\Console\Input\InputInterface;
 
-#[AsCommand(GetNonOrphanVisitsCommand::NAME, 'Returns the list of non-orphan visits')]
-class GetNonOrphanVisitsCommand extends Command
+class GetNonOrphanVisitsCommand extends AbstractVisitsListCommand
 {
     public const string NAME = 'visit:non-orphan';
 
-    public function __construct(private readonly VisitsStatsHelperInterface $visitsHelper)
-    {
-        parent::__construct();
+    public function __construct(
+        VisitsStatsHelperInterface $visitsHelper,
+        private readonly ShortUrlStringifierInterface $shortUrlStringifier,
+    ) {
+        parent::__construct($visitsHelper);
     }
 
-    public function __invoke(
-        SymfonyStyle $io,
-        #[MapInput] VisitsListInput $input,
-        #[Option(
-            'Return visits that belong to this domain only. Use ' . Domain::DEFAULT_AUTHORITY . ' keyword for visits '
-            . 'in default domain',
-            shortcut: 'd',
-        )]
-        string|null $domain = null,
-    ): int {
-        $paginator = $this->visitsHelper->nonOrphanVisits(new WithDomainVisitsParams(
-            dateRange: $input->dateRange(),
-            domain: $domain,
-        ));
-        VisitsCommandUtils::renderOutput($io, $input, $paginator);
+    protected function configure(): void
+    {
+        $this
+            ->setName(self::NAME)
+            ->setDescription('Returns the list of non-orphan visits.');
+    }
 
-        return self::SUCCESS;
+    /**
+     * @return Paginator<Visit>
+     */
+    protected function getVisitsPaginator(InputInterface $input, DateRange $dateRange): Paginator
+    {
+        return $this->visitsHelper->nonOrphanVisits(new VisitsParams($dateRange));
+    }
+
+    /**
+     * @return array<string, string>
+     */
+    protected function mapExtraFields(Visit $visit): array
+    {
+        $shortUrl = $visit->shortUrl;
+        return $shortUrl === null ? [] : ['shortUrl' => $this->shortUrlStringifier->stringify($shortUrl)];
     }
 }

module/CLI/src/Command/Visit/GetOrphanVisitsCommand.php

@@ -4,45 +4,47 @@ declare(strict_types=1);
 
 namespace Shlinkio\Shlink\CLI\Command\Visit;
 
-use Shlinkio\Shlink\CLI\Input\VisitsListInput;
-use Shlinkio\Shlink\Core\Domain\Entity\Domain;
+use Shlinkio\Shlink\Common\Paginator\Paginator;
+use Shlinkio\Shlink\Common\Util\DateRange;
+use Shlinkio\Shlink\Core\Visit\Entity\Visit;
 use Shlinkio\Shlink\Core\Visit\Model\OrphanVisitsParams;
 use Shlinkio\Shlink\Core\Visit\Model\OrphanVisitType;
-use Shlinkio\Shlink\Core\Visit\VisitsStatsHelperInterface;
-use Symfony\Component\Console\Attribute\AsCommand;
-use Symfony\Component\Console\Attribute\MapInput;
-use Symfony\Component\Console\Attribute\Option;
-use Symfony\Component\Console\Command\Command;
-use Symfony\Component\Console\Style\SymfonyStyle;
+use Symfony\Component\Console\Input\InputInterface;
+use Symfony\Component\Console\Input\InputOption;
 
-#[AsCommand(GetOrphanVisitsCommand::NAME, 'Returns the list of orphan visits')]
-class GetOrphanVisitsCommand extends Command
+use function Shlinkio\Shlink\Core\enumToString;
+use function sprintf;
+
+class GetOrphanVisitsCommand extends AbstractVisitsListCommand
 {
     public const string NAME = 'visit:orphan';
 
-    public function __construct(private readonly VisitsStatsHelperInterface $visitsHelper)
+    protected function configure(): void
     {
-        parent::__construct();
+        $this
+            ->setName(self::NAME)
+            ->setDescription('Returns the list of orphan visits.')
+            ->addOption('type', 't', InputOption::VALUE_REQUIRED, sprintf(
+                'Return visits only with this type. One of %s',
+                enumToString(OrphanVisitType::class),
+            ));
     }
 
-    public function __invoke(
-        SymfonyStyle $io,
-        #[MapInput] VisitsListInput $input,
-        #[Option(
-            'Return visits that belong to this domain only. Use ' . Domain::DEFAULT_AUTHORITY . ' keyword for visits '
-            . 'in default domain',
-            shortcut: 'd',
-        )]
-        string|null $domain = null,
-        #[Option('Return visits only with this type', shortcut: 't')] OrphanVisitType|null $type = null,
-    ): int {
-        $paginator = $this->visitsHelper->orphanVisits(new OrphanVisitsParams(
-            dateRange: $input->dateRange(),
-            domain: $domain,
-            type: $type,
-        ));
-        VisitsCommandUtils::renderOutput($io, $input, $paginator);
+    /**
+     * @return Paginator<Visit>
+     */
+    protected function getVisitsPaginator(InputInterface $input, DateRange $dateRange): Paginator
+    {
+        $rawType = $input->getOption('type');
+        $type = $rawType !== null ? OrphanVisitType::from($rawType) : null;
+        return $this->visitsHelper->orphanVisits(new OrphanVisitsParams(dateRange: $dateRange, type: $type));
+    }
 
-        return self::SUCCESS;
+    /**
+     * @return array<string, string>
+     */
+    protected function mapExtraFields(Visit $visit): array
+    {
+        return ['type' => $visit->type->value];
     }
 }

module/CLI/src/Command/Visit/LocateVisitsCommand.php

@@ -4,8 +4,9 @@ declare(strict_types=1);
 
 namespace Shlinkio\Shlink\CLI\Command\Visit;
 
-use Shlinkio\Shlink\CLI\Command\Util\CommandUtils;
-use Shlinkio\Shlink\CLI\Command\Util\LockConfig;
+use Shlinkio\Shlink\CLI\Command\Util\AbstractLockedCommand;
+use Shlinkio\Shlink\CLI\Command\Util\LockedCommandConfig;
+use Shlinkio\Shlink\CLI\Util\ExitCode;
 use Shlinkio\Shlink\Common\Util\IpAddress;
 use Shlinkio\Shlink\Core\Exception\IpCannotBeLocatedException;
 use Shlinkio\Shlink\Core\Visit\Entity\Visit;
@@ -15,22 +16,18 @@ use Shlinkio\Shlink\Core\Visit\Geolocation\VisitLocatorInterface;
 use Shlinkio\Shlink\Core\Visit\Geolocation\VisitToLocationHelperInterface;
 use Shlinkio\Shlink\Core\Visit\Model\UnlocatableIpType;
 use Shlinkio\Shlink\IpGeolocation\Model\Location;
-use Symfony\Component\Console\Attribute\AsCommand;
-use Symfony\Component\Console\Attribute\Option;
-use Symfony\Component\Console\Command\Command;
 use Symfony\Component\Console\Exception\RuntimeException;
 use Symfony\Component\Console\Input\ArrayInput;
+use Symfony\Component\Console\Input\InputInterface;
+use Symfony\Component\Console\Input\InputOption;
+use Symfony\Component\Console\Output\OutputInterface;
 use Symfony\Component\Console\Style\SymfonyStyle;
 use Symfony\Component\Lock\LockFactory;
 use Throwable;
 
 use function sprintf;
 
-#[AsCommand(
-    name: LocateVisitsCommand::NAME,
-    description: 'Resolves visits origin locations. It implicitly downloads/updates the GeoLite2 db file if needed',
-)]
-class LocateVisitsCommand extends Command implements VisitGeolocationHelperInterface
+class LocateVisitsCommand extends AbstractLockedCommand implements VisitGeolocationHelperInterface
 {
     public const string NAME = 'visit:locate';
 
@@ -39,30 +36,46 @@ class LocateVisitsCommand extends Command implements VisitGeolocationHelperInter
     public function __construct(
         private readonly VisitLocatorInterface $visitLocator,
         private readonly VisitToLocationHelperInterface $visitToLocation,
-        private readonly LockFactory $locker,
+        LockFactory $locker,
     ) {
-        parent::__construct();
+        parent::__construct($locker);
     }
 
-    public function __invoke(
-        SymfonyStyle $io,
-        #[Option(
-            'Will retry the location of visits that were located with a not-found location, in case it was due to '
-            . 'a temporal issue.',
-            shortcut: 'r',
-        )]
-        bool $retry = false,
-        #[Option(
-            'When provided together with --retry, will locate all existing visits, regardless the fact that they '
-            . 'have already been located.',
-            shortcut: 'a',
-        )]
-        bool $all = false,
-    ): int {
-        $this->io = $io;
+    protected function configure(): void
+    {
+        $this
+            ->setName(self::NAME)
+            ->setDescription(
+                'Resolves visits origin locations. It implicitly downloads/updates the GeoLite2 db file if needed.',
+            )
+            ->addOption(
+                'retry',
+                'r',
+                InputOption::VALUE_NONE,
+                'Will retry the location of visits that were located with a not-found location, in case it was due to '
+                . 'a temporal issue.',
+            )
+            ->addOption(
+                'all',
+                'a',
+                InputOption::VALUE_NONE,
+                'When provided together with --retry, will locate all existing visits, regardless the fact that they '
+                . 'have already been located.',
+            );
+    }
+
+    protected function initialize(InputInterface $input, OutputInterface $output): void
+    {
+        $this->io = new SymfonyStyle($input, $output);
+    }
+
+    protected function interact(InputInterface $input, OutputInterface $output): void
+    {
+        $retry = $input->getOption('retry');
+        $all = $input->getOption('all');
 
         if ($all && !$retry) {
-            $io->writeln(
+            $this->io->writeln(
                 '<comment>The <fg=yellow;options=bold>--all</> flag has no effect on its own. You have to provide it '
                 . 'together with <fg=yellow;options=bold>--retry</>.</comment>',
             );
@@ -71,13 +84,6 @@ class LocateVisitsCommand extends Command implements VisitGeolocationHelperInter
         if ($all && $retry && ! $this->warnAndVerifyContinue()) {
             throw new RuntimeException('Execution aborted');
         }
-
-        return CommandUtils::executeWithLock(
-            $this->locker,
-            LockConfig::nonBlocking(self::NAME),
-            $io,
-            fn () => $this->locateVisits($retry, $all),
-        );
     }
 
     private function warnAndVerifyContinue(): bool
@@ -92,8 +98,11 @@ class LocateVisitsCommand extends Command implements VisitGeolocationHelperInter
         return $this->io->confirm('Do you want to proceed?', false);
     }
 
-    private function locateVisits(bool $retry, bool $all): int
+    protected function lockedExecute(InputInterface $input, OutputInterface $output): int
     {
+        $retry = $input->getOption('retry');
+        $all = $retry && $input->getOption('all');
+
         try {
             $this->checkDbUpdate();
 
@@ -107,14 +116,14 @@ class LocateVisitsCommand extends Command implements VisitGeolocationHelperInter
             }
 
             $this->io->success('Finished locating visits');
-            return self::SUCCESS;
+            return ExitCode::EXIT_SUCCESS;
         } catch (Throwable $e) {
             $this->io->error($e->getMessage());
             if ($this->io->isVerbose()) {
                 $this->getApplication()?->renderThrowable($e, $this->io);
             }
 
-            return self::FAILURE;
+            return ExitCode::EXIT_FAILURE;
         }
     }
 
@@ -162,8 +171,13 @@ class LocateVisitsCommand extends Command implements VisitGeolocationHelperInter
         $downloadDbCommand = $cliApp->find(DownloadGeoLiteDbCommand::NAME);
         $exitCode = $downloadDbCommand->run(new ArrayInput([]), $this->io);
 
-        if ($exitCode === self::FAILURE) {
+        if ($exitCode === ExitCode::EXIT_FAILURE) {
             throw new RuntimeException('It is not possible to locate visits without a GeoLite2 db file.');
         }
     }
+
+    protected function getLockConfig(): LockedCommandConfig
+    {
+        return LockedCommandConfig::nonBlocking(self::NAME);
+    }
 }

module/CLI/src/Command/Visit/VisitsCommandUtils.php

@@ -1,117 +0,0 @@
-<?php
-
-declare(strict_types=1);
-
-namespace Shlinkio\Shlink\CLI\Command\Visit;
-
-use League\Csv\Writer;
-use Shlinkio\Shlink\CLI\Input\VisitsListFormat;
-use Shlinkio\Shlink\CLI\Input\VisitsListInput;
-use Shlinkio\Shlink\CLI\Util\ShlinkTable;
-use Shlinkio\Shlink\Common\Paginator\Paginator;
-use Shlinkio\Shlink\Common\Paginator\Util\PagerfantaUtils;
-use Shlinkio\Shlink\Core\Visit\Entity\Visit;
-use Symfony\Component\Console\Output\OutputInterface;
-
-use function array_map;
-
-class VisitsCommandUtils
-{
-    /**
-     * @param Paginator<Visit> $paginator
-     */
-    public static function renderOutput(
-        OutputInterface $output,
-        VisitsListInput $inputData,
-        Paginator $paginator,
-        callable|null $mapExtraFields = null,
-    ): void {
-        if ($inputData->format !== VisitsListFormat::FULL) {
-            // Avoid running out of memory by loading visits in chunks
-            $paginator->setMaxPerPage(1000);
-        }
-
-        match ($inputData->format) {
-            VisitsListFormat::CSV => self::renderCSVOutput($output, $paginator),
-            default => self::renderHumanFriendlyOutput($output, $paginator),
-        };
-    }
-
-    /**
-     * @param Paginator<Visit> $paginator
-     */
-    private static function renderCSVOutput(OutputInterface $output, Paginator $paginator): void
-    {
-        $page = 1;
-        do {
-            $paginator->setCurrentPage($page);
-
-            [$rows, $headers] = self::resolveRowsAndHeaders($paginator);
-            $csv = Writer::fromString();
-            if ($page === 1) {
-                $csv->insertOne($headers);
-            }
-
-            $csv->insertAll($rows);
-            $output->write($csv->toString());
-
-            $page++;
-        } while ($paginator->hasNextPage());
-    }
-
-    /**
-     * @param Paginator<Visit> $paginator
-     */
-    private static function renderHumanFriendlyOutput(OutputInterface $output, Paginator $paginator): void
-    {
-        $page = 1;
-        do {
-            $paginator->setCurrentPage($page);
-            $page++;
-
-            [$rows, $headers] = self::resolveRowsAndHeaders($paginator);
-            ShlinkTable::default($output)->render(
-                $headers,
-                $rows,
-                footerTitle: PagerfantaUtils::formatCurrentPageMessage($paginator, 'Page %s of %s'),
-            );
-        } while ($paginator->hasNextPage());
-    }
-
-    /**
-     * @param Paginator<Visit> $paginator
-     */
-    private static function resolveRowsAndHeaders(Paginator $paginator): array
-    {
-        $headers = [
-            'Date',
-            'Potential bot',
-            'User agent',
-            'Referer',
-            'Country',
-            'Region',
-            'City',
-            'Visited URL',
-            'Redirect URL',
-            'Type',
-        ];
-        $rows = array_map(function (Visit $visit) {
-            $visitLocation = $visit->visitLocation;
-
-            return [
-                'date' => $visit->date->toAtomString(),
-                'potentialBot' => $visit->potentialBot ? 'Potential bot' : '',
-                'userAgent' => $visit->userAgent,
-                'referer' => $visit->referer,
-                'country' => $visitLocation->countryName ?? 'Unknown',
-                'region' => $visitLocation->regionName ?? 'Unknown',
-                'city' => $visitLocation->cityName ?? 'Unknown',
-                'visitedUrl' => $visit->visitedUrl ?? 'Unknown',
-                'redirectUrl' => $visit->redirectUrl ?? 'Unknown',
-                'type' => $visit->type->value,
-            ];
-        }, [...$paginator->getCurrentPageResults()]);
-
-        return [$rows, $headers];
-    }
-}

module/CLI/src/Input/DateOption.php

@@ -0,0 +1,47 @@
+<?php
+
+declare(strict_types=1);
+
+namespace Shlinkio\Shlink\CLI\Input;
+
+use Cake\Chronos\Chronos;
+use Symfony\Component\Console\Command\Command;
+use Symfony\Component\Console\Input\InputInterface;
+use Symfony\Component\Console\Input\InputOption;
+use Symfony\Component\Console\Output\OutputInterface;
+use Throwable;
+
+use function is_string;
+use function sprintf;
+
+readonly class DateOption
+{
+    public function __construct(private Command $command, private string $name, string $shortcut, string $description)
+    {
+        $command->addOption($name, $shortcut, InputOption::VALUE_REQUIRED, $description);
+    }
+
+    public function get(InputInterface $input, OutputInterface $output): Chronos|null
+    {
+        $value = $input->getOption($this->name);
+        if (empty($value) || ! is_string($value)) {
+            return null;
+        }
+
+        try {
+            return Chronos::parse($value);
+        } catch (Throwable $e) {
+            $output->writeln(sprintf(
+                '<comment>> Ignored provided "%s" since its value "%s" is not a valid date. <</comment>',
+                $this->name,
+                $value,
+            ));
+
+            if ($output->isVeryVerbose()) {
+                $this->command->getApplication()?->renderThrowable($e, $output);
+            }
+
+            return null;
+        }
+    }
+}

module/CLI/src/Input/EndDateOption.php

@@ -0,0 +1,30 @@
+<?php
+
+declare(strict_types=1);
+
+namespace Shlinkio\Shlink\CLI\Input;
+
+use Cake\Chronos\Chronos;
+use Symfony\Component\Console\Command\Command;
+use Symfony\Component\Console\Input\InputInterface;
+use Symfony\Component\Console\Output\OutputInterface;
+
+use function sprintf;
+
+final readonly class EndDateOption
+{
+    private DateOption $dateOption;
+
+    public function __construct(Command $command, string $descriptionHint)
+    {
+        $this->dateOption = new DateOption($command, 'end-date', 'e', sprintf(
+            'Allows to filter %s, returning only those newer than provided date.',
+            $descriptionHint,
+        ));
+    }
+
+    public function get(InputInterface $input, OutputInterface $output): Chronos|null
+    {
+        return $this->dateOption->get($input, $output);
+    }
+}

module/CLI/src/Input/InputUtils.php

@@ -1,37 +0,0 @@
-<?php
-
-declare(strict_types=1);
-
-namespace Shlinkio\Shlink\CLI\Input;
-
-use Symfony\Component\Console\Output\OutputInterface;
-use Throwable;
-
-use function Shlinkio\Shlink\Core\normalizeOptionalDate;
-use function sprintf;
-
-final class InputUtils
-{
-    /**
-     * Process a date provided via input params, and format it as ATOM.
-     * A warning is printed if the date cannot be parsed, returning `null` in that case.
-     */
-    public static function processDate(string $name, string|null $value, OutputInterface $output): string|null
-    {
-        if ($value === null || $value === '') {
-            return null;
-        }
-
-        try {
-            return normalizeOptionalDate($value)->toAtomString();
-        } catch (Throwable) {
-            $output->writeln(sprintf(
-                '<comment>> Ignored provided "%s" since its value "%s" is not a valid date. <</comment>',
-                $name,
-                $value,
-            ));
-
-            return null;
-        }
-    }
-}

module/CLI/src/Input/ShortUrlDataInput.php

@@ -0,0 +1,136 @@
+<?php
+
+declare(strict_types=1);
+
+namespace Shlinkio\Shlink\CLI\Input;
+
+use Shlinkio\Shlink\Core\Config\Options\UrlShortenerOptions;
+use Shlinkio\Shlink\Core\ShortUrl\Model\ShortUrlCreation;
+use Shlinkio\Shlink\Core\ShortUrl\Model\ShortUrlEdition;
+use Shlinkio\Shlink\Core\ShortUrl\Model\Validation\ShortUrlInputFilter;
+use Symfony\Component\Console\Command\Command;
+use Symfony\Component\Console\Input\InputArgument;
+use Symfony\Component\Console\Input\InputInterface;
+use Symfony\Component\Console\Input\InputOption;
+
+use function array_map;
+use function array_unique;
+use function Shlinkio\Shlink\Core\ArrayUtils\flatten;
+use function Shlinkio\Shlink\Core\splitByComma;
+
+final readonly class ShortUrlDataInput
+{
+    public function __construct(Command $command, private bool $longUrlAsOption = false)
+    {
+        if ($longUrlAsOption) {
+            $command->addOption('long-url', 'l', InputOption::VALUE_REQUIRED, 'The long URL to set');
+        } else {
+            $command->addArgument('longUrl', InputArgument::REQUIRED, 'The long URL to set');
+        }
+
+        $command
+            ->addOption(
+                ShortUrlDataOption::TAGS->value,
+                ShortUrlDataOption::TAGS->shortcut(),
+                InputOption::VALUE_IS_ARRAY | InputOption::VALUE_REQUIRED,
+                'Tags to apply to the short URL',
+            )
+            ->addOption(
+                ShortUrlDataOption::VALID_SINCE->value,
+                ShortUrlDataOption::VALID_SINCE->shortcut(),
+                InputOption::VALUE_REQUIRED,
+                'The date from which this short URL will be valid. '
+                . 'If someone tries to access it before this date, it will not be found.',
+            )
+            ->addOption(
+                ShortUrlDataOption::VALID_UNTIL->value,
+                ShortUrlDataOption::VALID_UNTIL->shortcut(),
+                InputOption::VALUE_REQUIRED,
+                'The date until which this short URL will be valid. '
+                . 'If someone tries to access it after this date, it will not be found.',
+            )
+            ->addOption(
+                ShortUrlDataOption::MAX_VISITS->value,
+                ShortUrlDataOption::MAX_VISITS->shortcut(),
+                InputOption::VALUE_REQUIRED,
+                'This will limit the number of visits for this short URL.',
+            )
+            ->addOption(
+                ShortUrlDataOption::TITLE->value,
+                ShortUrlDataOption::TITLE->shortcut(),
+                InputOption::VALUE_REQUIRED,
+                'A descriptive title for the short URL.',
+            )
+            ->addOption(
+                ShortUrlDataOption::CRAWLABLE->value,
+                ShortUrlDataOption::CRAWLABLE->shortcut(),
+                InputOption::VALUE_NONE,
+                'Tells if this short URL will be included as "Allow" in Shlink\'s robots.txt.',
+            )
+            ->addOption(
+                ShortUrlDataOption::NO_FORWARD_QUERY->value,
+                ShortUrlDataOption::NO_FORWARD_QUERY->shortcut(),
+                InputOption::VALUE_NONE,
+                'Disables the forwarding of the query string to the long URL, when the short URL is visited.',
+            );
+    }
+
+    public function toShortUrlEdition(InputInterface $input): ShortUrlEdition
+    {
+        return ShortUrlEdition::fromRawData($this->getCommonData($input));
+    }
+
+    public function toShortUrlCreation(
+        InputInterface $input,
+        UrlShortenerOptions $options,
+        string $customSlugField,
+        string $shortCodeLengthField,
+        string $pathPrefixField,
+        string $findIfExistsField,
+        string $domainField,
+    ): ShortUrlCreation {
+        $shortCodeLength = $input->getOption($shortCodeLengthField) ?? $options->defaultShortCodesLength;
+        return ShortUrlCreation::fromRawData([
+            ...$this->getCommonData($input),
+            ShortUrlInputFilter::CUSTOM_SLUG => $input->getOption($customSlugField),
+            ShortUrlInputFilter::SHORT_CODE_LENGTH => $shortCodeLength,
+            ShortUrlInputFilter::PATH_PREFIX => $input->getOption($pathPrefixField),
+            ShortUrlInputFilter::FIND_IF_EXISTS => $input->getOption($findIfExistsField),
+            ShortUrlInputFilter::DOMAIN => $input->getOption($domainField),
+        ], $options);
+    }
+
+    private function getCommonData(InputInterface $input): array
+    {
+        $longUrl = $this->longUrlAsOption ? $input->getOption('long-url') : $input->getArgument('longUrl');
+        $data = [ShortUrlInputFilter::LONG_URL => $longUrl];
+
+        // Avoid setting arguments that were not explicitly provided.
+        // This is important when editing short URLs and should not make a difference when creating.
+        if (ShortUrlDataOption::VALID_SINCE->wasProvided($input)) {
+            $data[ShortUrlInputFilter::VALID_SINCE] = $input->getOption('valid-since');
+        }
+        if (ShortUrlDataOption::VALID_UNTIL->wasProvided($input)) {
+            $data[ShortUrlInputFilter::VALID_UNTIL] = $input->getOption('valid-until');
+        }
+        if (ShortUrlDataOption::MAX_VISITS->wasProvided($input)) {
+            $maxVisits = $input->getOption('max-visits');
+            $data[ShortUrlInputFilter::MAX_VISITS] = $maxVisits !== null ? (int) $maxVisits : null;
+        }
+        if (ShortUrlDataOption::TAGS->wasProvided($input)) {
+            $tags = array_unique(flatten(array_map(splitByComma(...), $input->getOption('tags'))));
+            $data[ShortUrlInputFilter::TAGS] = $tags;
+        }
+        if (ShortUrlDataOption::TITLE->wasProvided($input)) {
+            $data[ShortUrlInputFilter::TITLE] = $input->getOption('title');
+        }
+        if (ShortUrlDataOption::CRAWLABLE->wasProvided($input)) {
+            $data[ShortUrlInputFilter::CRAWLABLE] = $input->getOption('crawlable');
+        }
+        if (ShortUrlDataOption::NO_FORWARD_QUERY->wasProvided($input)) {
+            $data[ShortUrlInputFilter::FORWARD_QUERY] = !$input->getOption('no-forward-query');
+        }
+
+        return $data;
+    }
+}

module/CLI/src/Input/ShortUrlDataOption.php

@@ -0,0 +1,41 @@
+<?php
+
+declare(strict_types=1);
+
+namespace Shlinkio\Shlink\CLI\Input;
+
+use Symfony\Component\Console\Input\InputInterface;
+
+use function sprintf;
+
+enum ShortUrlDataOption: string
+{
+    case TAGS = 'tags';
+    case VALID_SINCE = 'valid-since';
+    case VALID_UNTIL = 'valid-until';
+    case MAX_VISITS = 'max-visits';
+    case TITLE = 'title';
+    case CRAWLABLE = 'crawlable';
+    case NO_FORWARD_QUERY = 'no-forward-query';
+
+    public function shortcut(): string|null
+    {
+        return match ($this) {
+            self::TAGS => 't',
+            self::VALID_SINCE => 's',
+            self::VALID_UNTIL => 'u',
+            self::MAX_VISITS => 'm',
+            self::TITLE => null,
+            self::CRAWLABLE => 'r',
+            self::NO_FORWARD_QUERY => 'w',
+        };
+    }
+
+    public function wasProvided(InputInterface $input): bool
+    {
+        $option = sprintf('--%s', $this->value);
+        $shortcut = $this->shortcut();
+
+        return $input->hasParameterOption($shortcut === null ? $option : [$option, sprintf('-%s', $shortcut)]);
+    }
+}

module/CLI/src/Input/ShortUrlIdentifierInput.php

@@ -0,0 +1,34 @@
+<?php
+
+declare(strict_types=1);
+
+namespace Shlinkio\Shlink\CLI\Input;
+
+use Shlinkio\Shlink\Core\ShortUrl\Model\ShortUrlIdentifier;
+use Symfony\Component\Console\Command\Command;
+use Symfony\Component\Console\Input\InputArgument;
+use Symfony\Component\Console\Input\InputInterface;
+use Symfony\Component\Console\Input\InputOption;
+
+final readonly class ShortUrlIdentifierInput
+{
+    public function __construct(Command $command, string $shortCodeDesc, string $domainDesc)
+    {
+        $command
+            ->addArgument('shortCode', InputArgument::REQUIRED, $shortCodeDesc)
+            ->addOption('domain', 'd', InputOption::VALUE_REQUIRED, $domainDesc);
+    }
+
+    public function shortCode(InputInterface $input): string|null
+    {
+        return $input->getArgument('shortCode');
+    }
+
+    public function toShortUrlIdentifier(InputInterface $input): ShortUrlIdentifier
+    {
+        $shortCode = $input->getArgument('shortCode');
+        $domain = $input->getOption('domain');
+
+        return ShortUrlIdentifier::fromShortCodeAndDomain($shortCode, $domain);
+    }
+}