[Fixes #14381] Relax Remote Service URL uniqueness

[etj]

This PR changes the identity model of remote services from endpoint URL uniqueness to service record uniqueness.

Before:

• A remote endpoint (base_url) implicitly identified one service.
• Duplicate endpoint registrations were rejected/assumed invalid.
• Some lookups and cache entries could collide when the same URL was reused.

After:

• base_url is no longer a unique identifier.
• Multiple Service objects can intentionally reference the same endpoint (e.g., different owners/use-cases/credentials).
• Service handler caching is keyed by service context (including auth fingerprint), so same-URL services with different credentials are isolated.
• Map layer dataset resolution is stricter: ambiguous URL-based fallback no longer fails silently.
• Registering the same remote endpoint multiple times no longer risks name collisions in harvester/service records; naming is now uniqueness-safe across both models.

What this PR changes

• Removes the uniqueness constraint from Service.base_url.
• Allows registering multiple Service records pointing to the same remote endpoint.
• Updates service registration/lookup logic and related tests to support duplicate base_url values.
• Adds a DB migration to alter the Service.base_url field.
• Makes service-handler cache keys credential-aware to avoid collisions across different auth contexts.
• Makes geoserver_pre_save_maplayer fallback strict on ambiguous dataset matches (logs + raises on MultipleObjectsReturned).
• Extracts service-handler caching into a ServiceHandlerCache class (serviceprocessors/cache.py), mirroring the existing ServiceTypeRegistry pattern, instead of loose module-level functions and a duplicated cache handle in views.py.
• Fixes a pre-existing bug in GeoNodeServiceHandler.parsed_service (predates this PR) that read auth/type/id off a variable that is actually just the raw service= query-string value, not a Service model -- this would raise AttributeError the moment the property ran (e.g. registering or probing a GeoNode-type remote service). Auth now resolves the same way WmsServiceHandler.parsed_service does.

Solved Note: this PR may conflict with #14328

Checklist

For all pull requests:

• Confirm you have read the contribution guidelines
• You have sent a Contribution Licence Agreement (CLA) as necessary (not required for small changes, e.g., fixing typos in the documentation)
• Make sure the first PR targets the master branch, eventual backports will be managed later. This can be ignored if the PR is fixing an issue that only happens in a specific branch, but not in newer ones.

The following are required only for core and extension modules (they are welcomed, but not required, for contrib modules):

• There is a ticket in https://github.com/GeoNode/geonode/issues describing the issue/improvement/feature (a notable exemption is, changes not visible to end-users)
• The issue connected to the PR must have Labels and Milestone assigned
• PR for bug fixes and small new features are presented as a single commit
• PR title must be in the form "[Fixes #<issue_number>] Title of the PR"
• New unit tests have been added covering the changes, unless there is an explanation on why the tests are not necessary/implemented

Submitting the PR does not require you to check all items, but by the time it gets merged, they should be either satisfied or inapplicable.

[etj]

Waiting for #14328 to be merged.

[Copilot]

Pull request overview

Copilot reviewed 12 out of 12 changed files in this pull request and generated 3 comments.

Comments suppressed due to low confidence (1)

geonode/services/serviceprocessors/init.py:61

• logger.exception(msg=...) is an invalid call signature for the stdlib logger (the message must be the first positional argument). In the exception handler this will raise a TypeError and can mask the original parsing error. Log the message positionally (optionally with %s formatting) and keep the original exception context.

    except Exception as e:
        logger.exception(e)
        logger.exception(msg=f"Could not parse service {base_url}")
        raise

[Copilot]

Pull request overview

Copilot reviewed 12 out of 12 changed files in this pull request and generated 4 comments.

[Copilot]

Pull request overview

Copilot reviewed 12 out of 12 changed files in this pull request and generated 2 comments.

[mattiagiupponi]

@etj i tried to use it, but when i try to import a resource from another service with the same URL, the following error is raised

The above exception was the direct cause of the following exception:

Traceback (most recent call last):
  File "/opt/core/geonode/geonode/harvesting/tasks.py", line 348, in _harvest_resource
    worker.update_geonode_resource(
  File "/opt/core/geonode/geonode/harvesting/harvesters/base.py", line 223, in update_geonode_resource
    geonode_resource = self._create_new_geonode_resource(geonode_resource_type, defaults)
                       ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/opt/core/geonode/geonode/harvesting/harvesters/base.py", line 253, in _create_new_geonode_resource
    geonode_resource = resource_manager_registry.get_for_model(geonode_resource_type).create(
                       ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/opt/core/geonode/geonode/resource/manager.py", line 353, in create
    _resource, _created = resource_type.objects.get_or_create(uuid=uuid, defaults=resource_dict)
                          ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/home/mattia/.virtualenvs/gn5/lib/python3.12/site-packages/django/db/models/manager.py", line 87, in manager_method
    return getattr(self.get_queryset(), name)(*args, **kwargs)
           ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/home/mattia/.virtualenvs/gn5/lib/python3.12/site-packages/django/db/models/query.py", line 955, in get_or_create
    return self.create(**params), True
           ^^^^^^^^^^^^^^^^^^^^^
  File "/home/mattia/.virtualenvs/gn5/lib/python3.12/site-packages/django/db/models/query.py", line 665, in create
    obj.save(force_insert=True, using=self.db)
  File "/opt/core/geonode/geonode/base/models.py", line 1090, in save
    super().save(*args, **kwargs)
  File "/home/mattia/.virtualenvs/gn5/lib/python3.12/site-packages/polymorphic/models.py", line 86, in save
    return super().save(*args, **kwargs)
           ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/home/mattia/.virtualenvs/gn5/lib/python3.12/site-packages/django/db/models/base.py", line 902, in save
    self.save_base(
  File "/home/mattia/.virtualenvs/gn5/lib/python3.12/site-packages/django/db/models/base.py", line 1008, in save_base
    updated = self._save_table(
              ^^^^^^^^^^^^^^^^^
  File "/home/mattia/.virtualenvs/gn5/lib/python3.12/site-packages/django/db/models/base.py", line 1169, in _save_table
    results = self._do_insert(
              ^^^^^^^^^^^^^^^^
  File "/home/mattia/.virtualenvs/gn5/lib/python3.12/site-packages/django/db/models/base.py", line 1210, in _do_insert
    return manager._insert(
           ^^^^^^^^^^^^^^^^
  File "/home/mattia/.virtualenvs/gn5/lib/python3.12/site-packages/django/db/models/manager.py", line 87, in manager_method
    return getattr(self.get_queryset(), name)(*args, **kwargs)
           ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/home/mattia/.virtualenvs/gn5/lib/python3.12/site-packages/django/db/models/query.py", line 1873, in _insert
    return query.get_compiler(using=using).execute_sql(returning_fields)
           ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/home/mattia/.virtualenvs/gn5/lib/python3.12/site-packages/django/db/models/sql/compiler.py", line 1882, in execute_sql
    cursor.execute(sql, params)
  File "/home/mattia/.virtualenvs/gn5/lib/python3.12/site-packages/django/db/backends/utils.py", line 122, in execute
    return super().execute(sql, params)
           ^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/home/mattia/.virtualenvs/gn5/lib/python3.12/site-packages/django/db/backends/utils.py", line 79, in execute
    return self._execute_with_wrappers(
           ^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/home/mattia/.virtualenvs/gn5/lib/python3.12/site-packages/django/db/backends/utils.py", line 92, in _execute_with_wrappers
    return executor(sql, params, many, context)
           ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/home/mattia/.virtualenvs/gn5/lib/python3.12/site-packages/django/db/backends/utils.py", line 100, in _execute
    with self.db.wrap_database_errors:
  File "/home/mattia/.virtualenvs/gn5/lib/python3.12/site-packages/django/db/utils.py", line 91, in __exit__
    raise dj_exc_value.with_traceback(traceback) from exc_value
  File "/home/mattia/.virtualenvs/gn5/lib/python3.12/site-packages/django/db/backends/utils.py", line 105, in _execute
    return self.cursor.execute(sql, params)
           ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
django.db.utils.IntegrityError: duplicate key value violates unique constraint "layers_dataset_store_workspace_name_0b4228a1_uniq"
DETAIL:  Key (store, workspace, name)=(httpswmsigngobargeoserveriderabahrawms, remoteWorkspace, bahra) already exists.