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

Add SSR helper to get resources (images/files) from the server using the API headers #653

Merged
merged 3 commits into from
Feb 28, 2019

Conversation

sneridagh
Copy link
Member

No description provided.

@sneridagh sneridagh marked this pull request as ready for review February 27, 2019 10:42
@sneridagh
Copy link
Member Author

I wake up this morning with this idea... I had to implement it right away.

After this, I really think that SSR is the hidden secret weapon of Volto...

The only "issue" that I see is that we force the SSR server to serve all the images. Do you think that we shouldn't do it for anonymous sessions? It won't be difficult I guess...

Also, if someone requests the image in Volto and it's private, it redirects to Plone... something that we can also polish I guess.

@sneridagh sneridagh changed the title WIP on fix images requests that require API auth using a SSR helper Add SSR helper to get resources (images/files) from the server using the API headers Feb 27, 2019
Copy link
Member

@tisto tisto left a comment

Choose a reason for hiding this comment

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

Works like a charm!

@tisto tisto merged commit bf17025 into master Feb 28, 2019
@tisto tisto deleted the fixauthimagesrequests branch February 28, 2019 04:04
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