Merge lp://staging/~jtv/gwacl/include-account-in-endpoint into lp://staging/gwacl
Proposed by
Jeroen T. Vermeulen
| Status: | Merged |
|---|---|
| Approved by: | Jeroen T. Vermeulen |
| Approved revision: | 216 |
| Merged at revision: | 214 |
| Proposed branch: | lp://staging/~jtv/gwacl/include-account-in-endpoint |
| Merge into: | lp://staging/gwacl |
| Diff against target: |
64 lines (+17/-12) 3 files modified
endpoints.go (+4/-7) endpoints_test.go (+11/-4) storage_base.go (+2/-1) |
| To merge this branch: | bzr merge lp://staging/~jtv/gwacl/include-account-in-endpoint |
| Related bugs: |
| Reviewer | Review Type | Date Requested | Status |
|---|---|---|---|
| Julian Edwards (community) | Approve | ||
|
Review via email:
|
|||
Commit message
Make BlobStorageAPI include the account name.
Description of the change
This is something Julian wanted: when dealing with the blob storage API, the storage account name is included in the API URL's hostname. In this branch, that account name becomes part of the endpoint rather than something that's added to it in an unusual way later.
To post a comment you must log in.
Preview Diff
[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
:-)