Skip to content

Return configured container names#4

Merged
dmorawetz merged 2 commits into
mainfrom
bug/container-listing-name
Mar 26, 2026
Merged

Return configured container names#4
dmorawetz merged 2 commits into
mainfrom
bug/container-listing-name

Conversation

@dmorawetz
Copy link
Copy Markdown
Member

@dmorawetz dmorawetz commented Mar 24, 2026

When listing available containers, the name of the folder on disk was returned, instead of the configured container name. Therefore listing and then accessing containers failed in the client. Furthermore, configuring a container in the appsettings with the same name as the folder on disk lead to an exception.

This is fixed now and the returned list looks as expected.

@saxx Is this change a problem for any of our clients already consuming the simulator?

PS: The broken E2E test is fixed in #3 .

Before:
image

After:
image

Used config:

 "Storage": {
    "LocalPath": "/tmp/storage",
    "Accounts": [
      {
        "Name": "devstoreaccount1",
        "Key": "..."
      }
    ],
    "Containers": [
      {
        "Name": "configured-name-of-test-container",
        "LocalPath": "/tmp/storage/test"
      }
    ]
  }

@dmorawetz dmorawetz requested a review from saxx March 24, 2026 15:46
@dmorawetz dmorawetz force-pushed the bug/container-listing-name branch from 5aa5778 to 6d59298 Compare March 26, 2026 08:43
@dmorawetz dmorawetz merged commit 87dff62 into main Mar 26, 2026
2 checks passed
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.

1 participant