deck9 / input

A privacy-focused, no-code, open-source form builder designed for simplicity and brand consistency.
https://getinput.co
GNU Affero General Public License v3.0
162 stars 27 forks source link

Problems with uploading images #103

Closed VannaKarenina closed 11 months ago

VannaKarenina commented 11 months ago

When a user tries to add a brand image gets an error in the console:

image

And in the docker container logs i see that app cant find storage folder

2023/07/24 06:37:43 [error] 16#16: *824 open() "/var/www/html/public/images/lR76/cyxlzx.1690180663.png" failed (2: No such file or directory), client: 172.17.0.1, server: _, request: "GET /images/lR76/cyxlzx.1690180663.png?w=256&q=75 HTTP/1.0", host: "domain.com", referrer: "https://domain.com/forms/lR76/settings"

But i see that app saved image in docker volume. Maybe I didn 't set up something ?

PhilReinking commented 11 months ago

@VannaKarenina I will check on it later this day

PhilReinking commented 11 months ago

@VannaKarenina just saw in your log that the referrer is set to https://domain.com.

Do you own this domain? If you are using a proxy to access the app, than make sure to follow the instructions in the README. Right now I can not find any other cause for your error, other than it might be configured to a wrong domain.

VannaKarenina commented 11 months ago

@PhilReinking

Here domain.com specified to not show the real one. I started the application according to the instructions, maybe it's because I use my NGINX that is work on server ?

PhilReinking commented 11 months ago

ok thanks for clarification. I guess it might be possible that a symlink is not created accordingly during the image build. I will test it and let you know if I have found a fix :)

PhilReinking commented 11 months ago

@VannaKarenina I have found the issue in the nginx config we used. Thanks for bringing this to my attention, I have just pushed a fix to the codebase (bda9ec9bda7fc4e5faab1dd7cee9a0e489adcb9e) and an updated image is building right now.

Please let me know if it works for you now :)

VannaKarenina commented 11 months ago

@VannaKarenina I have found the issue in the nginx config we used. Thanks for bringing this to my attention, I have just pushed a fix to the codebase (bda9ec9) and an updated image is building right now.

Please let me know if it works for you now :)

@PhilReinking Yes, the pictures are working fine now. Thanks ❤

But i think there is another problem. App says thats we have image size limit 4MB but when i upload image with size more then 1MB we can see error:

image

And console log says that:

Failed to load resource: the server responded with a status of 413 (Request Entity Too Large)

PhilReinking commented 11 months ago

@VannaKarenina thanks for another catch. Will look into that as well! :)

PhilReinking commented 11 months ago

okay @VannaKarenina. I found the missing config and added it to the build process.

The new release is just building, so you should be able to try out the updated image very soon.

Please let me know if it works now. This time I won't close the issue so quickly 😄

VannaKarenina commented 11 months ago

okay @VannaKarenina. I found the missing config and added it to the build process.

The new release is just building, so you should be able to try out the updated image very soon.

Please let me know if it works now. This time I won't close the issue so quickly 😄

@PhilReinking tested new build.

Everything works fine. Thanks ❤

If I find something else, I will open a new issue :)