Skip to content

fix: check for the identifier alias for the storage backend#41538

Open
jvillafanez wants to merge 3 commits intomasterfrom
check_for_local_alias
Open

fix: check for the identifier alias for the storage backend#41538
jvillafanez wants to merge 3 commits intomasterfrom
check_for_local_alias

Conversation

@jvillafanez
Copy link
Copy Markdown
Member

Description

Prevent local storage to be used as external if it isn't explicitly allowed.

Related Issue

  • Fixes <issue_link>

Motivation and Context

How Has This Been Tested?

  • test environment:
  • test case 1:
  • test case 2:
  • ...

Screenshots (if appropriate):

Types of changes

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Database schema changes (next release will require increase of minor version instead of patch)
  • Breaking change (fix or feature that would cause existing functionality to change)
  • Technical debt
  • Tests only (no source changes)

Checklist:

  • Code changes
  • Unit tests added
  • Acceptance tests added
  • Documentation ticket raised:
  • Changelog item, see TEMPLATE

Notes:

This is likely the minimum change possible at the moment. There are no plans to add or modify the backend's aliases
or identifiers at the moment, but it could become unmanageable quickly.
We should consider to move and improve the check at some point.

@jvillafanez jvillafanez self-assigned this Apr 21, 2026
@update-docs
Copy link
Copy Markdown

update-docs Bot commented Apr 21, 2026

Thanks for opening this pull request! The maintainers of this repository would appreciate it if you would create a changelog item based on your changes.

Copy link
Copy Markdown
Member

@DeepDiver1975 DeepDiver1975 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

  • can you add unit tests?
  • please add changelog items

Comment thread changelog/unreleased/41538 Outdated
@jvillafanez jvillafanez force-pushed the check_for_local_alias branch from fa639a3 to 5080fc6 Compare April 22, 2026 06:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants