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

Can't add attachments in version 11.19.0 2022-08-20 using core5_filemanager #1120

Open
philmck opened this issue Aug 27, 2022 · 5 comments
Open

Comments

@philmck
Copy link

philmck commented Aug 27, 2022

Hello, it's me again with my usual problem with attachments after an upgrade.

This seems to be a new fault (different from #1108) so I'm raising it as a new issue. It's on the same site as before and the login credentials I sent should still work, if that helps.

What I did: Uploaded pro_dada-11_19_0.tar.gz, replaced the existing files, renamed the installer directory and ran the installer in "update" mode. Everything seemd to go smoothly.

I then created a new draft message, went to the Attachments tab and clicked Select a File....

A file manager popup appeared as expected, but there was no list of files, just a spinning cursor.

The following errors show in the Javascript console in the Chrome inspector:

Mixed Content: The page at 'https://www.skeptic.org.uk/dada_mail_support_files/core5_filemanager/index.html?custom_function=SetAttachmentUrl' was loaded over HTTPS, but requested an insecure stylesheet 'http://fonts.googleapis.com/css?family=Oxygen'. This request has been blocked; the content must be served over HTTPS.
/dada_mail_support_files/core5_filemanager/connectors/pl/filemanager.pl?path=%2Fdada_mail_support_files%2Ffile_uploads%2F&mode=getfolder&showThumbs=true&time=338:1 Failed to load resource: the server responded with a status of 500 ()
/dada_mail_support_files/core5_filemanager/connectors/pl/filemanager.pl?path=%2Fdada_mail_support_files%2Ffile_uploads%2F&mode=getfolder&showThumbs=true&time=359:1 Failed to load resource: the server responded with a status of 500 ()
index.html:1 Mixed Content: The page at 'https://www.skeptic.org.uk/dada_mail_support_files/core5_filemanager/index.html?custom_function=SetAttachmentUrl' was loaded over HTTPS, but requested an insecure stylesheet 'http://fonts.googleapis.com/css?family=Oxygen'. This request has been blocked; the content must be served over HTTPS.

The following errors show in the Dada error log:

[Sat Aug 27 21:24:39 2022] filemanager.pl: [Sat Aug 27 21:24:39 2022] filemanager.pl: BEGIN failed--compilation aborted at /home/skeptic/cgi-bin/paranormal/DADA/Template/Widgets.pm line 41.
[Sat Aug 27 21:24:39 2022] filemanager.pl: [Sat Aug 27 21:24:39 2022] filemanager.pl: Compilation failed in require at /home/skeptic/cgi-bin/paranormal/DADA/MailingList/Settings.pm line 7.
[Sat Aug 27 21:24:39 2022] filemanager.pl: [Sat Aug 27 21:24:39 2022] filemanager.pl: BEGIN failed--compilation aborted at /home/skeptic/cgi-bin/paranormal/DADA/MailingList/Settings.pm line 7.
[Sat Aug 27 21:24:39 2022] filemanager.pl: [Sat Aug 27 21:24:39 2022] filemanager.pl: Compilation failed in require at /home/skeptic/cgi-bin/paranormal/DADA/App/Session.pm line 13.
[Sat Aug 27 21:24:39 2022] filemanager.pl: [Sat Aug 27 21:24:39 2022] filemanager.pl: BEGIN failed--compilation aborted at /home/skeptic/cgi-bin/paranormal/DADA/App/Session.pm line 13.
[Sat Aug 27 21:24:39 2022] filemanager.pl: Compilation failed in require at /home/skeptic/cgi-bin/paranormal/DADA/App/Guts.pm line 2161.

@justingit
Copy link
Owner

I added a new path to your perllib - it wouldn't have worked without that, as you've changed the name of the, "dada" directory.

@philmck
Copy link
Author

philmck commented Aug 28, 2022

Many thanks, all seems to be working now.

Just to be clear, did you add it to $ADDITIONAL_PERLLIBS in .dada_config? Will it therefore propagate to future upgrades automatically?

I'm not clear why I need to add additional Perl locations at all, when the config file is already configured with the program, config and support file locations and the server is running bog standard Ubuntu.

I haven't changed the name of any directory that I'm aware of. The original installation was done a decade ago by someone else and hasn't been moved.

@justingit
Copy link
Owner

Set up a repo just for core5 filemanager to track any changes,

https://github.com/justingit/core5-filemanager-for-dada-mail

@justingit
Copy link
Owner

Just to be clear, did you add it to $ADDITIONAL_PERLLIBS in .dada_config? Will it therefore propagate to future upgrades automatically?

Yes.

I'm not clear why I need to add additional Perl locations at all, when the config file is already configured with the program, config and support file locations and the server is running bog standard Ubuntu.

The "dada" directory is hardcoded in a few places in the distro, and that directory name is changed in your install. That's the primary reason you need to set your additional perllibs.

@justingit
Copy link
Owner

Phil, I want to close this issue, but if/when you upgrade to v11.20.0, please let me know how the core5_filemanager seems to be fairing for you. I've done a fair bit of work with hacking it to perhaps work a little better. It now lives at,

https://github.com/justingit/core5-filemanager-for-dada-mail

(instead of on my Macbook), which should help with future issue resolution,

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

2 participants