Skip to content
This repository has been archived by the owner on Nov 15, 2019. It is now read-only.

Create a Project image field is text #6

Open
teodor-stoenescu opened this issue Sep 7, 2017 · 2 comments
Open

Create a Project image field is text #6

teodor-stoenescu opened this issue Sep 7, 2017 · 2 comments
Assignees

Comments

@teodor-stoenescu
Copy link

As seen below, the project image field is a text field. It would make more sense for it to be a file field. This way the user can upload his own project image file.
scr2

@aabherve
Copy link
Contributor

aabherve commented Sep 8, 2017

Currently, an URL of an externaly hosted image is required.
We could implement au upload service for user images.

@teodor-stoenescu
Copy link
Author

I would argue that hotlinking is really unsafe and unethical. On the one hand you are using someone else's bandwidth. While this may not be a problem today, it could lead to other unexpected behaviours, like the image not being accessible in certain network configurations (such as accessing the platform via VPN). On the other hand this leaks information outside the platform, like who has access to certain projects. Last but not least, hotlinking opens a door for third parties to potentially deliver XSS exploits.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

3 participants