@@ -1756,13 +1756,13 @@ def add_permission(
1756
1756
# TODO: re-evaluate data model to prevent this dirty hack just for bumping the revision id
1757
1757
# TODO: does that need a `with function.lock` for atomic updates of the policy + revision_id?
1758
1758
if api_utils .qualifier_is_alias (resolved_qualifier ):
1759
- latest_alias = resolved_fn .aliases [resolved_qualifier ]
1760
- resolved_fn .aliases [resolved_qualifier ] = dataclasses .replace (latest_alias )
1759
+ resolved_alias = resolved_fn .aliases [resolved_qualifier ]
1760
+ resolved_fn .aliases [resolved_qualifier ] = dataclasses .replace (resolved_alias )
1761
1761
# Assumes that a non-alias is a version
1762
1762
else :
1763
- latest_version = resolved_fn .versions [resolved_qualifier ]
1763
+ resolved_version = resolved_fn .versions [resolved_qualifier ]
1764
1764
resolved_fn .versions [resolved_qualifier ] = dataclasses .replace (
1765
- latest_version , config = dataclasses .replace (latest_version .config )
1765
+ resolved_version , config = dataclasses .replace (resolved_version .config )
1766
1766
)
1767
1767
return AddPermissionResponse (Statement = json .dumps (permission_statement ))
1768
1768
@@ -1819,13 +1819,13 @@ def remove_permission(
1819
1819
# TODO: re-evaluate data model to prevent this dirty hack just for bumping the revision id
1820
1820
# TODO: does that need a `with function.lock` for atomic updates of the policy + revision_id?
1821
1821
if api_utils .qualifier_is_alias (resolved_qualifier ):
1822
- latest_alias = resolved_fn .aliases [resolved_qualifier ]
1823
- resolved_fn .aliases [resolved_qualifier ] = dataclasses .replace (latest_alias )
1822
+ resolved_alias = resolved_fn .aliases [resolved_qualifier ]
1823
+ resolved_fn .aliases [resolved_qualifier ] = dataclasses .replace (resolved_alias )
1824
1824
# Assumes that a non-alias is a version
1825
1825
else :
1826
- latest_version = resolved_fn .versions [resolved_qualifier ]
1826
+ resolved_version = resolved_fn .versions [resolved_qualifier ]
1827
1827
resolved_fn .versions [resolved_qualifier ] = dataclasses .replace (
1828
- latest_version , config = dataclasses .replace (latest_version .config )
1828
+ resolved_version , config = dataclasses .replace (resolved_version .config )
1829
1829
)
1830
1830
1831
1831
# remove the policy as a whole when there's no statement left in it
9DFA
tr>@@ -1856,12 +1856,12 @@ def get_policy(
1856
1856
1857
1857
fn_revision_id = None
1858
1858
if api_utils .qualifier_is_alias (resolved_qualifier ):
1859
- latest_alias = resolved_fn .aliases [resolved_qualifier ]
1860
- fn_revision_id = latest_alias .revision_id
1859
+ resolved_alias = resolved_fn .aliases [resolved_qualifier ]
1860
+ fn_revision_id = resolved_alias .revision_id
1861
1861
# Assumes that a non-alias is a version
1862
1862
else :
1863
- latest_version = resolved_fn .versions [resolved_qualifier ]
1864
- fn_revision_id = latest_version .config .revision_id
1863
+ resolved_version = resolved_fn .versions [resolved_qualifier ]
1864
+ fn_revision_id = resolved_version .config .revision_id
1865
1865
1866
1866
return GetPolicyResponse (
1867
1867
Policy = json .dumps (dataclasses .asdict (function_permission .policy )),
0 commit comments