Commit 36c4540
committed
Fix azureblob keyless auth: always emit jclouds.credential (#27)
S3Proxy requires both jclouds.identity and jclouds.credential in every
backend properties file; only filesystem-nio2, transient-nio2 and
google-cloud-storage-sdk are exempt from that check in Main.java. The
azureblob backend defaults to the azureblob-sdk provider, which is NOT
exempt, so when it authenticates via Azure Workload Identity (account
set, no storage account key) the rendered properties file omitted
jclouds.credential entirely and S3Proxy exited on startup with:
Properties file must contain: jclouds.identity and jclouds.credential
Emit an empty `jclouds.credential=` whenever no storage account key is
provided. This keys off key.value alone (not key.value OR sasToken.value),
which also covers the SAS-token-only path: SAS auth carries the token via
jclouds.azureblob.sas but never sets jclouds.credential, so it hit the
same startup failure.
Verified with `helm template` for workload-identity, SAS-only, and
key-based configs. Chart version bumped 0.4.0 -> 0.4.1.
Fixes #271 parent 1e3d2d5 commit 36c4540
2 files changed
Lines changed: 9 additions & 3 deletions
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
15 | 15 | | |
16 | 16 | | |
17 | 17 | | |
18 | | - | |
| 18 | + | |
19 | 19 | | |
20 | 20 | | |
21 | 21 | | |
| |||
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
177 | 177 | | |
178 | 178 | | |
179 | 179 | | |
180 | | - | |
181 | | - | |
| 180 | + | |
| 181 | + | |
| 182 | + | |
| 183 | + | |
| 184 | + | |
| 185 | + | |
| 186 | + | |
| 187 | + | |
182 | 188 | | |
183 | 189 | | |
184 | 190 | | |
| |||
0 commit comments