Commit c526e9b
committed
fix(graphql-generation-transformer): grant InvokeModel on inference profile and underlying models
The @generation transformer builds an AppSync->Bedrock IAM role that
scoped bedrock:InvokeModel to a single regional foundation-model ARN
(arn:<partition>:bedrock:<region>::foundation-model/<modelId>). For a
cross-region inference profile id (us./eu./apac. prefix) that ARN is
wrong: invoking a profile requires InvokeModel on the inference-profile
ARN AND on the underlying regional foundation-model ARNs the profile
routes to. As a result, customers using a cross-region profile model id
with @generation hit runtime AccessDenied.
When the model id matches /^(us|eu|apac)\./ the role now grants
InvokeModel on both the inference-profile ARN
(arn:<partition>:bedrock:<region>:<account>:inference-profile/<modelId>)
and the underlying foundation-model ARN with a wildcard region
(arn:<partition>:bedrock:*::foundation-model/<foundationModelId>), where
the underlying model id is the profile id with the region prefix
stripped. Plain foundation-model ids keep the existing single-ARN
behavior. All ARNs use CDK account/region/partition tokens.
Added unit/snapshot tests proving a plain FM id grants only the
foundation-model ARN (unchanged) and an inference-profile id grants both
the profile ARN and the wildcard-region foundation-model ARN. The
generation transformer unit tests run offline and all 16 pass.
---
Prompt: Draft the OPTION A product fix as its own branch/PR. The
@generation transformer scopes bedrock:InvokeModel to a single
foundation-model ARN; for a cross-region inference profile id that ARN
is wrong, so @generation + cross-region profile hits AccessDenied.
Implement granting InvokeModel on both the inference-profile ARN and the
underlying regional foundation-model ARNs (wildcard region), add tests,
build, commit, push, and open a draft PR off main.1 parent 9a2c742 commit c526e9b
3 files changed
Lines changed: 139 additions & 1 deletion
File tree
- packages/amplify-graphql-generation-transformer/src
- __tests__
- __snapshots__
Lines changed: 56 additions & 0 deletions
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
148 | 148 | | |
149 | 149 | | |
150 | 150 | | |
| 151 | + | |
| 152 | + | |
| 153 | + | |
| 154 | + | |
| 155 | + | |
| 156 | + | |
| 157 | + | |
| 158 | + | |
| 159 | + | |
| 160 | + | |
| 161 | + | |
| 162 | + | |
| 163 | + | |
| 164 | + | |
| 165 | + | |
| 166 | + | |
| 167 | + | |
| 168 | + | |
| 169 | + | |
| 170 | + | |
| 171 | + | |
| 172 | + | |
| 173 | + | |
| 174 | + | |
| 175 | + | |
| 176 | + | |
| 177 | + | |
| 178 | + | |
| 179 | + | |
| 180 | + | |
| 181 | + | |
| 182 | + | |
| 183 | + | |
| 184 | + | |
| 185 | + | |
| 186 | + | |
| 187 | + | |
| 188 | + | |
| 189 | + | |
| 190 | + | |
| 191 | + | |
| 192 | + | |
| 193 | + | |
| 194 | + | |
| 195 | + | |
| 196 | + | |
| 197 | + | |
| 198 | + | |
| 199 | + | |
| 200 | + | |
| 201 | + | |
| 202 | + | |
| 203 | + | |
| 204 | + | |
| 205 | + | |
| 206 | + | |
151 | 207 | | |
152 | 208 | | |
153 | 209 | | |
| |||
Lines changed: 49 additions & 0 deletions
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
326 | 326 | | |
327 | 327 | | |
328 | 328 | | |
| 329 | + | |
| 330 | + | |
| 331 | + | |
| 332 | + | |
| 333 | + | |
| 334 | + | |
| 335 | + | |
| 336 | + | |
| 337 | + | |
| 338 | + | |
| 339 | + | |
| 340 | + | |
| 341 | + | |
| 342 | + | |
| 343 | + | |
| 344 | + | |
| 345 | + | |
| 346 | + | |
| 347 | + | |
| 348 | + | |
| 349 | + | |
| 350 | + | |
| 351 | + | |
| 352 | + | |
| 353 | + | |
| 354 | + | |
| 355 | + | |
| 356 | + | |
| 357 | + | |
| 358 | + | |
| 359 | + | |
| 360 | + | |
| 361 | + | |
| 362 | + | |
| 363 | + | |
| 364 | + | |
| 365 | + | |
| 366 | + | |
| 367 | + | |
| 368 | + | |
| 369 | + | |
| 370 | + | |
| 371 | + | |
| 372 | + | |
| 373 | + | |
| 374 | + | |
| 375 | + | |
| 376 | + | |
| 377 | + | |
329 | 378 | | |
330 | 379 | | |
331 | 380 | | |
| |||
Lines changed: 34 additions & 1 deletion
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
207 | 207 | | |
208 | 208 | | |
209 | 209 | | |
210 | | - | |
| 210 | + | |
211 | 211 | | |
212 | 212 | | |
213 | 213 | | |
214 | 214 | | |
215 | 215 | | |
216 | 216 | | |
217 | 217 | | |
| 218 | + | |
| 219 | + | |
| 220 | + | |
| 221 | + | |
| 222 | + | |
| 223 | + | |
| 224 | + | |
| 225 | + | |
| 226 | + | |
| 227 | + | |
| 228 | + | |
| 229 | + | |
| 230 | + | |
| 231 | + | |
| 232 | + | |
| 233 | + | |
| 234 | + | |
| 235 | + | |
| 236 | + | |
| 237 | + | |
| 238 | + | |
| 239 | + | |
| 240 | + | |
| 241 | + | |
| 242 | + | |
| 243 | + | |
| 244 | + | |
| 245 | + | |
| 246 | + | |
| 247 | + | |
| 248 | + | |
| 249 | + | |
| 250 | + | |
218 | 251 | | |
219 | 252 | | |
220 | 253 | | |
| |||
0 commit comments