Antoine Lambert authored 1 month ago and Antoine Lambert committed 1 month ago

A better dedicated lister has been implemented since so there is no
need to display that kind of visit type anymore.

Fixes #4829.

The has_visits field is only returned by the /origin/search endpoint.

When searching for an origin URL, the search interface redirects
to its browse interface if the origin URL exists in SWH database.

Nevertheless, it the origin could not be archived, the redirection
was still done leading to a 404 error so ensure to check the origin
has a snapshot before redirecting.

Fixes #4828.

Antoine Lambert authored 2 months ago and Antoine Lambert committed 2 months ago

Sort releases according to their version numbers instead of their names.

nodejs 18 is close to EOL and some swh-web dependencies (pdfjs for
instance) bump minimal required nodejs version to 20 so time to
upgrade.

Related to swh/infra/sysadm-environment#5534

The swh.web.settings.tests settings are now dedicated to run Python tests.

The swh.web.settings.cypress settings should be used instead in that
Makefile as they do not use postgres but sqlite3 instead.

- django rename_app is no longer needed,
- mypy execution is now properly configured (in pyproject.toml).

Antoine Lambert authored 2 months ago and Antoine Lambert committed 2 months ago

django-stubs package requires mypy >= 1.12 and its latest release
pins mypy to >= 1.14.0 which fails to validate some valid typings
related to the standard Python module email.

So bump and pin mypy to 1.13.0 to avoid typing errors in Jenkins CI.

There is a discrepancy between the config entry and the module name, so
deal with it...

This is a dict which will automatically raise a ConfigurationError when
trying to access a non-existent key.

But keep it activate for development mode (aka when using the
swh.web.settings.development config file).

Prevent from having the logic of computing this boolean value (from the
configuration) replicated in several places.

Prevent polluting the logs, especially in cypress tests.

In some test scenarios, task types may not be registered.

The correct behavior was depending on the order in which swh.web django
apps are registered. This should allow to have less entries in the
SWH_BASE_DJANGO_APPS so we can have an instance only serving /api for
example (which is currently not possible).

These default config values are mostly irrelevant for any kind of
deployment. Removing the default values make it clear and mandatory
these need to be explicitly configured.

This also prevent the swh.web.utils.archive module from
executing (backend instantiation) code at module loading time.

Introduces a new patch_backend fixture to help mocking a specific method
of a known configured backend (storage, vault, search etc.) to try to
provide a standard and unified process to mock these in all tests.

remove SWH_WEB_XXX_NAMES constants as well as 'xxx_server_names' config
entries. These are not used any more.

Add a new 'corner_ribbon_text' config entry to specify the text displayed
in said corner ribbon. It should allow to remove some very swh
deployment specific hardcoded constants (especially urls).

It makes tests a bit easier to read and maintain.

Replace usage of the hardcoded and swh specific
SWH_WEB_INTERNAL_SERVER_NAMES by a more generic UNAUTHENTICATED_HOSTS
config option to bypass authentication for the graph public APIs.

This uses the same validation logic as django's ALLOWED_HOSTS one.

Antoine Lambert authored 2 months ago and Antoine Lambert committed 2 months ago

Network issues might happen that can slow down the loading
of README byes so display a spinner by default in the card
holding README content.

Fixes #4827.