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

fix: frame preview headers #345

Merged
merged 1 commit into from
Mar 5, 2024
Merged

fix: frame preview headers #345

merged 1 commit into from
Mar 5, 2024

Conversation

kaaps
Copy link
Contributor

@kaaps kaaps commented Mar 5, 2024

No description provided.

Copy link

linear bot commented Mar 5, 2024

@kaaps
Copy link
Contributor Author

kaaps commented Mar 5, 2024

@tofran Since the changes that you have on this PR the preview mode broke.

Basically the x-frame-options deny was preventing running the website on storyblok, me and @plckr implemented a hook that runs on every request and checks if the draft mode is enabled, if so the header changes to ALLOW-FROM storyblok.com as referenced here.

I check the preview deployment and our header manipulation is overriding the vercel header

Copy link
Member

@tofran tofran left a comment

Choose a reason for hiding this comment

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

Sorry and thank you for this

@kaaps kaaps merged commit b6b605d into main Mar 5, 2024
2 checks passed
@kaaps kaaps deleted the SIGN-646 branch March 5, 2024 14:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

2 participants