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

Codepen's Portfolio images are missing #12447

Closed
justincorrigible opened this issue Jan 9, 2017 · 3 comments
Closed

Codepen's Portfolio images are missing #12447

justincorrigible opened this issue Jan 9, 2017 · 3 comments
Assignees

Comments

@justincorrigible
Copy link
Member

There's been a new set of issues with the images on the portfolio pen @ codepen.
The problem roots in the hotlink protection I've implemented in my server to avoid broadband overcharge for the images I sometimes update on the portfolio.

The current htaccess file allows for the appropriate user accounts to get access to the images, but it seems codepen has implemented some sort of buffer or safety feature that masks the user name while making external requests, and this is resulting in conflict with the specific hotlink allowance.

I've already reached out to codepen, trying to get some assistance from them to solve this issue.
Will update as soon as I get an answer. Else, I'll host the images on a different server (github?) and have them be linked from there.

@raisedadead
Copy link
Member

I'll host the images on a different server (github?)

This should be a good idea in my opinion, instead of having a bandwidth usage on your personal server.

@texas2010
Copy link
Member

@hallaathrad, it is good idea to use github pages.

I agree with @raisedadead.

@justincorrigible
Copy link
Member Author

justincorrigible commented Jan 14, 2017

After putting some thought into this, I've remembered why I'd blocked hotlinking to begin with: Last year I've found some dude using my images as part of their actual portfolio (not FCC related).

The current setting lets any codepen user (mostly learners); that should be good enough. I do want FCC people to have them available for their learning process.

Thus, the problem is solved, at least in the interim.

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

No branches or pull requests

3 participants