mirror of
https://github.com/cypht-org/cypht.git
synced 2026-04-25 04:56:03 +03:00
[GH-ISSUE #555] Warn about missing dependency (was: Nothing sent with compose method "markdown") #403
Labels
No labels
2fa
I18N
PGP
Security
Security
account
advanced_search
advanced_search
announcement
api_login
authentication
awaiting feedback
blocker
bug
bug
bug
calendar
config
contacts
core
core
devops
docker
docs
duplicate
dynamic_login
enhancement
epic
feature
feeds
framework
github
github
gmail_contacts
good first issue
help wanted
history
history
imap
imap_folders
inline_message
installation
keyboard_shortcuts
keyboard_shortcuts
ldap_contacts
mobile
need-ssh-access
new module set
nux
pop3
profiles
pull-request
question
refactor
release
research
saved_searches
smtp
strategic
tags
tests
themes
website
wordpress
No milestone
No project
No assignees
1 participant
Notifications
Due date
No due date set.
Dependencies
No dependencies set.
Reference
starred/cypht#403
Loading…
Add table
Add a link
Reference in a new issue
No description provided.
Delete branch "%!s()"
Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?
Originally created by @smitgd on GitHub (Feb 23, 2022).
Original GitHub issue: https://github.com/cypht-org/cypht/issues/555
Originally assigned to: @IrAlfred on GitHub.
🐛 Bugreport
I sent a plain text message ok and it arrived at the destination. Then I wanted to see if the markdown setting under Site/General would work. After composing the message and clicking send, the screen goes completely blank and nothing is sent and nothing received. I went back to plain text mode and it sent again ok. Tried markdown mode sending couple more times and still wouldn't send and screen went blank. Didn't try html.
Version & Environment
Rev: [not sure, got it around Feb 7 using the install script]
OS: [linux]
@jasonmunro commented on GitHub (Feb 23, 2022):
Sounds like a bug, let me try to reproduce and I will let you know. Thanks for the report!
@jasonmunro commented on GitHub (Feb 24, 2022):
I was able to send with markdown without issue. I did find a bug after send causing a 500 error/blank page however I just pushed a fix for that and it was happening after send so probably not what you are seeing.
We use a third party library called parsedown to convert the message body markdown to html on send (clients don't normally natively render markdown), since your site is failing before sending I suspect something want wrong with the "composer install" step of the installation. When you get a blank page it's almost always a server side 500 error and there maybe a clue in your webserver log about what is going wrong. I would try rerunning the composer step and checking the logs if it still fails.
@kk6mrp commented on GitHub (Feb 24, 2022):
I'm getting this error when trying to reply to messages:
@smitgd commented on GitHub (Feb 24, 2022):
I'm assuming this is what you mean by "composer" and I'm pretty sure I did this before since the errors/warnings look familiar:
Not sure who to run it as if not sudo, anyhow, then I updated:
I didn't try to update to "composer 2". Do I need it? Anyhow, before and after this it still doesn't work and I get this error in /var/log/apache2/error.log:
Not sure any of this is relevant.
@jasonmunro commented on GitHub (Feb 26, 2022):
Last bit is the problem, you need the DOMDocument PHP class which I think is part of the php-xml package depending on your distro. On debian unstable I have:
@jasonmunro commented on GitHub (Feb 26, 2022):
@kk6mrp not sure about your error it seems to be cut off before the actual important bit.
@kk6mrp commented on GitHub (Feb 26, 2022):
@jasonmunro What do you suggest to see the important part?
@smitgd commented on GitHub (Feb 26, 2022):
Yes, installing php7.4-xml on ubuntu 20.04 fixed it. Now it accepts markdown and sends OK.
I guess the INSTALL file needs an addition:
On Debian based systems you can bring in the dependencies this way;
$ sudo apt install php php-fpm php-mbstring php-curl composer php-xml
Thanks!
@jasonmunro commented on GitHub (Feb 26, 2022):
noted that the install doc needs work! glad this fixed it for you!
@jasonmunro commented on GitHub (Feb 26, 2022):
@kk6mrp the final part of the traceback is cut off so we don't have the actual error here just a trail of crumbs that gets us close :) Some webservers like nginx cap log messages at a max character count, might be the situation here
@kk6mrp commented on GitHub (Feb 28, 2022):
I needed php-dom in Alpine Linux. My issue is resolved. Thank you!
@marclaporte commented on GitHub (May 9, 2022):
To do:
@ElvisAns commented on GitHub (Sep 9, 2022):
What else can i add here https://github.com/jasonmunro/cypht-website/pull/35?
@marclaporte commented on GitHub (May 6, 2024):
@smitgd
Please retest, as a lot has changed since you reported this issue. Notably, we now have 3 active branches and recently released Cypht 2.0.0
@marclaporte commented on GitHub (Sep 28, 2024):
@smitgd Please retest with latest stable: https://github.com/cypht-org/cypht/releases/
@marclaporte commented on GitHub (Sep 1, 2025):
@smitgd I am persistent because I want to make Cypht better!
FYI: The monthly Cypht community meetings are a great opportunity to show bugs: https://github.com/cypht-org/cypht/wiki/Monthly-Community-Meetings