Skip to content

Revert "feat: refactor Function URL permissions (#3735)" #3761

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 2 commits into from
Apr 29, 2025

Conversation

roger-zhangg
Copy link
Member

This reverts commit 9ed1d64.

Issue #, if available

Description of changes

Description of how you validated changes

Checklist

Examples?

Please reach out in the comments if you want to add an example. Examples will be
added to sam init through aws/aws-sam-cli-app-templates.

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

@roger-zhangg roger-zhangg requested a review from a team as a code owner April 28, 2025 23:25
@roger-zhangg roger-zhangg reopened this Apr 28, 2025
@roger-zhangg
Copy link
Member Author

CFN lint failed for network reason?

@seshubaws
Copy link
Contributor

strange, #3737 failed for the same reason

File "/opt/hostedtoolcache/Python/3.11.12/x64/lib/python3.11/socket.py", line 839, in create_connection
    for res in getaddrinfo(host, port, 0, SOCK_STREAM):
               ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/opt/hostedtoolcache/Python/3.11.12/x64/lib/python3.11/socket.py", line 974, in getaddrinfo
    for res in _socket.getaddrinfo(host, port, family, type, proto, flags):
               ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
socket.gaierror: [Errno -3] Temporary failure in name resolution

@roger-zhangg
Copy link
Member Author

roger-zhangg commented Apr 29, 2025

created a issue for CFN-lint CI issue aws-cloudformation/cfn-lint#4126

@seshubaws seshubaws closed this Apr 29, 2025
@seshubaws seshubaws reopened this Apr 29, 2025
@roger-zhangg roger-zhangg merged commit 79cf186 into aws:develop Apr 29, 2025
7 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants