-
Notifications
You must be signed in to change notification settings - Fork 19
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 Config-Option to not export contact photos. #29
base: master
Are you sure you want to change the base?
Conversation
First of all, thanks for your pull request. However, I do feel there must be a better/automatic solution than introducing yet another config option. during the last release cycle I already tried to implement some automatic checking if FTP-Upload is supported at all on the fritzbox carddav2fb tries to connect to. And IMHO this should also work for the 7270 device than having to tune a config option based on the device. So IMHO it would be better to try to identify your situation (ftp server rejects due to permission denied) and then skip uploading all together than having to manually tune the config option. So can you please try to implement it in that way instead? |
:/ I'm sorry, but I have absolutely no experience with php and ftp. I'm fully OK, when you don't want to merge this. I would leave it in my fork. Greetings to Dresden from Freiberg ;) |
Ok, then I will see if I could implement this. However, to better understand what exactly needs to be changed can you pleas show some output of carddav2fb if you have it running over your 7270 fritzbox so that I can see what exactly it complains about if you use my fork. |
Yes, I will do that, but since I am only at home at weekends, you unfortunately have to wait until Friday or Saturday. |
Here is the error message: FTP output:
Permissions: Output of carddav2fb:
I hope these information can help you. |
This pull request could be closed after pull request #40 was merged, because it includes an option to disable image processing and upload. |
Add option to not export contact photos
Our Fritz!Box 7270 rejects uploading images in the WebGUI with a message similar to "Upload is only supported, when external memory is plugged in".
The FTP server then gives "Permission denied" when attempting to upload the images.
The commit only wraps the extraction of the photo data and the ftp upload in a new configuration option, which per default does not change behaviour.