Skip to content

api!: remove provider-db handling and provider lookup APIs - #8437

Merged
hpk42 merged 5 commits into
mainfrom
hpk/remove-provider-db
Jul 24, 2026
Merged

api!: remove provider-db handling and provider lookup APIs#8437
hpk42 merged 5 commits into
mainfrom
hpk/remove-provider-db

Conversation

@hpk42

@hpk42 hpk42 commented Jul 20, 2026

Copy link
Copy Markdown
Contributor

this removes all provider-db related code and data because it's not needed and we do not want to care or maintain for provider-db anyway.

Note that 6dddb97 re-introduces very few exceptions. I verified with adb regarding nauta.cu that it's still needed to provide a working experience for a sizable number of users so i guess it makes sense to retain the exceptions. the other two (hermes and aco-connexion) are less clear but i kept them anyway for now.

provider-db is also preparing for archiving chatmail/provider-db#341

@hpk42
hpk42 force-pushed the hpk/remove-provider-db branch from 713db31 to 8261244 Compare July 20, 2026 17:17
@hpk42
hpk42 marked this pull request as draft July 20, 2026 17:21
@hpk42
hpk42 force-pushed the hpk/remove-provider-db branch from 3854b27 to d8956eb Compare July 21, 2026 10:30
@hpk42
hpk42 force-pushed the hpk/remove-provider-db branch 2 times, most recently from feea15b to f159d46 Compare July 22, 2026 06:18
@hpk42
hpk42 marked this pull request as ready for review July 22, 2026 06:34
Comment thread src/sql/migrations.rs
Comment thread src/provider.rs Outdated
worse_media_quality: true,
..Default::default()
},
_ if is_exact_or_subdomain(&domain, "hermes.radio")

@link2xt link2xt Jul 22, 2026

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

hermes.radio settings can likely be removed, cc @rafael2k

The problem is that with v2 encryption is always used unless you try really hard to send plaintext emails by typing email addresses and not using the QR codes, so Delta Chat is likely not working as is.

The project is likely using v1 in some form, so if we want to make current v2 versions usable for rhizomatica/hermes, there are other changes needed.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

i am fine to remove hermes.radio special case, and you are probably right, but i didn't want to make the call in the PR so far. If we remove it, then we can also inline apply_legacy*

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

I'm fine removing hermes.radio, we can deal with this in other ways I think.

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Same as with .aco-connexion.org.

@link2xt link2xt Jul 22, 2026

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

@rafael2k Do you use v2 versions of Delta Chat currently? If it works for you there is no problem keeping the code, but i thought your setup relies on disabling OpenPGP/E2E encryption and removing Autocrypt headers is no longer enough for this with v2.

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

By v2 you mean 2.xy.z? Yes.
We do strip all Autocrypt header. If that will not work anymore, I need to change our setup asap.

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

There are no unencrypted Autocrypt headers anymore normally, especially since https://delta.chat/en/2026-03-31-zero
When the users get in contact by scanning a QR code, the QR code already contains symmetric key for encryption, and Autocrypt header is sent in the encrypted part. So users get encrypted chats this way. And generally since v2 (https://delta.chat/en/2025-08-04-encryption-v2) encrypted chats never become unencrypted.

Settings like worse_media_quality: true still make sense to make the client send worse quality images, but you cannot disable E2E encryption by intercepting Autocrypt headers to do your own compression anymore.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

i removed hermes/aco-connexion special casing for now. Maybe it's best if we do a dedicated discussion and analysis what would be most helpful for you, @rafael2k

Comment thread src/login_param.rs Outdated
Comment thread deltachat-jsonrpc/src/api/types/login_param.rs Outdated
Comment thread src/configure.rs Outdated
Comment thread src/context.rs Outdated
Comment thread src/provider.rs Outdated
Comment thread src/transport.rs Outdated
Comment thread src/transport.rs Outdated
hpk42 added 4 commits July 22, 2026 20:52
BREAKING CHANGE: provider lookup APIs were removed from CFFI and JSON-RPC.

also removes offline provider database code and generated provider data,
provider-specific fields in configure/transport paths, and REPL providerinfo.
Comment thread src/provider.rs Outdated

@r10s r10s left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

i created issues for andoid/ios/desktop/ubuntutouch to stop using these apis. should be simple to do (i did on android/ios), so unless we expect and urgent core update these days, i would say this PR can be merged at any time. if we want to play save, give UIs a little time

@adbenitez

Copy link
Copy Markdown
Collaborator

about nauta.cu, there was also some logic to treat it as a chatmail server so when multi-device mode is disabled it deletes messages from the server immediately just as with chatmail, but I guess that was never influenced by provider-db but something hardcoded in core even before this PR

@link2xt

link2xt commented Jul 23, 2026

Copy link
Copy Markdown
Collaborator

There is no special nauta.cu logic for deletion, since #8240 messages are deleted automatically when should_delete_all_downloaded_messages says so (IMAP server is a chatmail and bcc_self is off) and there is no exception for nauta.cu. And since we are getting rid of is_chatmail anyway (#8448) we will likely make the logic the same for chatmail and non-chatmail anyway.

@adbenitez

Copy link
Copy Markdown
Collaborator

yikes that explains reports I just got from people with full inbox

@r10s

This comment was marked as outdated.

@hpk42
hpk42 merged commit 7ec888f into main Jul 24, 2026
54 of 55 checks passed
@hpk42
hpk42 deleted the hpk/remove-provider-db branch July 24, 2026 11:22
WofWca added a commit to deltachat/deltachat-desktop that referenced this pull request Jul 25, 2026
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

Successfully merging this pull request may close these issues.

5 participants