Skip to content

Conversation

@tchapi
Copy link
Owner

@tchapi tchapi commented May 13, 2025

Issue: php-imap was removed as deprecated from PHP8.4

See #195

Using https://www.php-imap.com/ as a replacement

New config format

IMAP_AUTH_URL=imap.mydomain.com:993
IMAP_ENCRYPTION_METHOD=ssl
IMAP_CERTIFICATE_VALIDATION=true

🧪 Testing setup

Running a local Greenmail instance via Docker with

docker run -t -i \
    -p 3025:3025 -p 3110:3110 -p 3143:3143 -p 3465:3465 -p 3993:3993 -p 3995:3995 -p 8080:8080 \
    -e GREENMAIL_OPTS='-Dgreenmail.setup.test.all -Dgreenmail.hostname=0.0.0.0 -Dgreenmail.users=test:[email protected] -Dgreenmail.users.login=email -Dgreenmail.verbose' \
    greenmail/standalone:2.1.5

@tchapi tchapi self-assigned this May 13, 2025
@tchapi tchapi added the dependencies Pull requests that update a dependency file label May 13, 2025
@tchapi tchapi force-pushed the tchapi/imap-replacement branch 3 times, most recently from 4aff757 to 44b5b15 Compare May 13, 2025 21:32
@tchapi tchapi force-pushed the main branch 2 times, most recently from cfd457d to ae56070 Compare June 12, 2025 20:02
@tchapi tchapi force-pushed the tchapi/imap-replacement branch from bfd9552 to f667704 Compare August 31, 2025 14:25

// Set a reminder, if needed
if (strtolower($this->birthdayReminderOffset) !== "false") {
if ('false' !== strtolower($this->birthdayReminderOffset)) {
Copy link
Owner Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🏕️ Unrelated cs fix

<a class="github-link" href="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/tchapi/davis" data-bs-toggle="popover" data-bs-trigger="hover" data-bs-content="Project Github page" data-bs-placement="bottom"></a>
</li>
<li class="list-group-item list-group-item-secondary">{{ "dashboard.auth"|trans }} : <code>{{ authMethod }}</code> ({{ "dashboard.auth_realm"|trans }}: <code>{{ authRealm }}</code>)</li>
<li class="list-group-item list-group-item-secondary">{{ "dashboard.auth"|trans }} : <code>{{ authMethod }}</code>{% if authMethod == 'Basic' %} ({{ "dashboard.auth_realm"|trans }}: <code>{{ authRealm }}</code>){% endif %}</li>
Copy link
Owner Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🏕️ Not displaying the realm except for Basic auth

@tchapi tchapi force-pushed the tchapi/imap-replacement branch from d2675f5 to 6fecdb4 Compare September 14, 2025 16:57
Copy link
Contributor

@Ramblurr Ramblurr left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Working great on my end with php 8.4.13

@tchapi tchapi merged commit 0e79f2b into main Oct 30, 2025
8 checks passed
@tchapi tchapi deleted the tchapi/imap-replacement branch October 30, 2025 19:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

dependencies Pull requests that update a dependency file

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants