Skip to content

limit static apps to top level domains#181

Merged
mehalter merged 9 commits into
mainfrom
180-limit-static-apps-to-top-level-domains
Nov 8, 2024
Merged

limit static apps to top level domains#181
mehalter merged 9 commits into
mainfrom
180-limit-static-apps-to-top-level-domains

Conversation

@thecaffiend
Copy link
Copy Markdown
Member

TO TEST

NOTE: This has been deployed

  • get on the VPN
  • go to https://analysis-pipelines.cape-dev.org and poke around the site making sure things work as expected
  • manually add any valid path in the s3 bucket to the root URL above and make sure you get the file.
  • manually add any invalid path (not in the s3 bucket) to the root URL above and make sure you get an error.
  • manually add any valid directory in the s3 bucket and end with a / to make sure you are given an error about a missing index.html file

@thecaffiend thecaffiend linked an issue Nov 8, 2024 that may be closed by this pull request
@thecaffiend thecaffiend marked this pull request as draft November 8, 2024 17:39
@thecaffiend thecaffiend marked this pull request as ready for review November 8, 2024 18:32
Copy link
Copy Markdown
Member

@mehalter mehalter left a comment

Choose a reason for hiding this comment

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

:shipit:

@mehalter mehalter merged commit 5269783 into main Nov 8, 2024
@mehalter mehalter deleted the 180-limit-static-apps-to-top-level-domains branch November 8, 2024 18:42
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.

Limit static apps to top level domains

2 participants