[PR #651] [MERGED] Dirty fix for 500 on outgoing mail #958

New issue

Closed

opened 2026-02-25 21:36:46 +03:00 by kerem · 0 comments

kerem commented

2026-02-25 21:36:46 +03:00

Owner

📋 Pull Request Information

Original PR: https://github.com/cypht-org/cypht/pull/651
Author: @capocasa
Created: 10/31/2022
Status: ✅ Merged
Merged: 9/24/2023
Merged by: @josaphatim

Base: master ← Head: master

📝 Commits (2)

a61fdcf Dirty fix for 500 on outgoing mail
253aca0 Update modules.php

📊 Changes

1 file changed (+3 additions, -0 deletions)

View changed files

📝 modules/smtp/modules.php (+3 -0)

📄 Description

Pullrequest

There is a hard-to-reproduce bug that was first reported on gitter and now by me here on github, #650

This is not a clean fix in that it does not addres root causes, but it allows me to use the software right now apparently without issue and might give additional information.

Issues

dirty fix for #650

Checklist

None

How2Test

Reproduce bug #650 and watch it go away with this patch.

Todo

Reproduce: Spin up a debian bullseye instance somewhere, install apache, php and cypht and install cypht using the install script.
Find root cause and fix properly

_{🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.}

## 📋 Pull Request Information **Original PR:** https://github.com/cypht-org/cypht/pull/651 **Author:** [@capocasa](https://github.com/capocasa) **Created:** 10/31/2022 **Status:** ✅ Merged **Merged:** 9/24/2023 **Merged by:** [@josaphatim](https://github.com/josaphatim) **Base:** `master` ← **Head:** `master` --- ### 📝 Commits (2) - [`a61fdcf`](https://github.com/cypht-org/cypht/commit/a61fdcff751e3b32b0d84c881560ba3d386dac7f) Dirty fix for 500 on outgoing mail - [`253aca0`](https://github.com/cypht-org/cypht/commit/253aca0c2b95835bdbc3ddd328c2089a84241383) Update modules.php ### 📊 Changes **1 file changed** (+3 additions, -0 deletions) <details> <summary>View changed files</summary> 📝 `modules/smtp/modules.php` (+3 -0) </details> ### 📄 Description ## Pullrequest There is a hard-to-reproduce bug that was first reported on gitter and now by me here on github, #650 This is not a clean fix in that it does not addres root causes, but it allows me to use the software right now apparently without issue and might give additional information. ### Issues - dirty fix for #650 ### Checklist - [X] None ### How2Test Reproduce bug #650 and watch it go away with this patch. ### Todo - [ ] Reproduce: Spin up a debian bullseye instance somewhere, install apache, php and cypht and install cypht using the install script. - [ ] Find root cause and fix properly --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>