Skip to content
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

Update CLOUDFRONT.md - per wr384955 #498

Open
wants to merge 1 commit into
base: DEPRECATED_master
Choose a base branch
from

Conversation

JoshAudette
Copy link
Contributor

Per request in wr384955, these changes resolved issues:

  1. The presignedurl_tests.php was giving only content hashes and not original file names -- resolved by changing the caching settings to the Legacy settings; and
  2. The presignedurl_tests.php iFrames were giving an error - resolved by disabling the X-Frame-Options in the response headers policy

Per request in wr384955, these changes resolved issues:
1) The presignedurl_tests.php was giving only content hashes and not original file names -- resolved by changing the caching settings to the Legacy settings; and
2) The presignedurl_tests.php iFrames were giving an error - resolved by disabling the X-Frame-Options in the response headers policy
16. Confirm, that _Status_ is _Enabled_ and _Last modified_ is changed from _Deploying_ to the date the distribution was created.
17. Open your distribution.
18. Write down _Distribution domain name_ (with https://).
10. _Cache key and origin requests_: Choose _Legacy cache settings_.
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Some of these worry me that we've missed steps somewhere else and so changing these settings is a workaround to getting it work again. I'm not across the detail so can't know for sure but I think this should be reviewed by others in the au inf team

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.

2 participants