fix(storage): return service properties XML by aniketshukla1 · Pull Request #144 · floci-io/floci-az · GitHub
Skip to content

fix(storage): return service properties XML - #144

Merged
hectorvent merged 1 commit into
floci-io:mainfrom
aniketshukla1:fix/storage-service-properties
Jul 31, 2026
Merged

fix(storage): return service properties XML#144
hectorvent merged 1 commit into
floci-io:mainfrom
aniketshukla1:fix/storage-service-properties

Conversation

@aniketshukla1

@aniketshukla1 aniketshukla1 commented Jul 28, 2026

Copy link
Copy Markdown
Contributor

Summary

  • Return real StorageServiceProperties XML for Blob and Queue storage instead of the XmlBuilder@... object identity string.
  • Route Table storage account-root ?restype=service&comp=properties requests to an Azure-compatible XML response.
  • Add regression tests for Blob, Queue, and Table service-properties responses.

Closes #131
Closes #132

Type of change

  • Bug fix (fix:)
  • New feature (feat:)
  • Breaking change (feat!: or fix!:)
  • Docs / chore

Azure Compatibility

Azure Storage clients expect StorageServiceProperties XML for account-root restype=service&comp=properties requests. Blob and Queue previously returned the builder object's identity string, while Table returned list-table JSON. This change returns the expected XML response shape for all three storage services.

Checklist

  • ./mvnw test passes locally on JDK 25
  • New or updated automated tests added
  • Commit messages follow Conventional Commits

@greptile-apps

greptile-apps Bot commented Jul 28, 2026

Copy link
Copy Markdown

@hectorvent hectorvent added bug Something isn't working blob Azure Blob Storage queue Azure Queue Storage table Azure Table Storage labels Jul 30, 2026
@hectorvent
hectorvent merged commit aa3e378 into floci-io:main Jul 31, 2026
9 checks passed
hectorvent added a commit that referenced this pull request Jul 31, 2026
…#144 (#152)

PRs #133 and #144 both added getBlobServicePropertiesReturnsXml and
getQueueServicePropertiesReturnsXml; the merges were textually clean but
left duplicate methods, breaking test compilation on main since aa3e378.
Each pair is now one method carrying both copies' assertions (XML structure
from #144, the StaticWebsite element and XmlBuilder@ toString regression
guard from #133).
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

blob Azure Blob Storage bug Something isn't working queue Azure Queue Storage table Azure Table Storage

Projects

None yet

2 participants