Skip to content
This repository has been archived by the owner on Aug 30, 2022. It is now read-only.

Create s3_cors.json #278

Merged
merged 1 commit into from
Jan 17, 2022
Merged

Create s3_cors.json #278

merged 1 commit into from
Jan 17, 2022

Conversation

giovanimachado
Copy link
Contributor

Type: update
S3`s CORS needs to be json

Type: update
S3`s CORS needs to be json
@jeffalloneword
Copy link

jeffalloneword commented Feb 20, 2021

[
    {
        "AllowedHeaders": [
            "*"
        ],
        "AllowedMethods": [
            "POST",
            "GET",
            "PUT",
            "DELETE",
            "HEAD"
        ],
        "AllowedOrigins": [
            "*"
        ]
    }
]

@Phil15634280
Copy link

Any way this can get merged into master? Might be able to delete the s3_cors.xml file as well since it is no longer valid.

@SudKul SudKul merged commit 31a8c14 into udacity:master Jan 17, 2022
@SudKul
Copy link
Contributor

SudKul commented Jan 17, 2022

Thanks @giovanimachado

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants