Cavemanon
/
cavecomm
8
0
Fork 0
Code Issues 22 Pull Requests 1 Packages Projects Releases Wiki Activity

NewSMTP Implementation #29

Merged
MichaelYick merged 4 commits from NewSMTP into master 2023-06-01 06:33:40 +00:00
Conversation 0 Commits 4 Files Changed 7 +215 -63
t_a commented 2023-05-29 13:24:16 +00:00
Collaborator
Copy Link

I gave vmime and libesmtp a try.
vmime is unnecessarily "bloated" for our use case and at that point there is no reason not to use the far better supported libcurl, additionally while testing vmime requires us to juggle certificates which (may just be my problem) made running it reliably not possible.
libesmtp similarly is not very active considering the last activity was last year too (and i couldn't get it to run).

Now libcurl functionality wise is quite extensive similar to vmime (which is effectively a full on email client) but library wise there is little more reliable than curl.

I gave vmime and libesmtp a try. vmime is unnecessarily "bloated" for our use case and at that point there is no reason not to use the far better supported libcurl, additionally while testing vmime requires us to juggle certificates which (may just be my problem) made running it reliably not possible. libesmtp similarly is not very active considering the last activity was last year too (and i couldn't get it to run). Now libcurl functionality wise is quite extensive similar to vmime (which is effectively a full on email client) but library wise there is little more reliable than curl.
t_a added 3 commits 2023-05-29 13:24:24 +00:00
8102ea41dc README Overhaul 
Config SSL made optional
Minor Refactoring
3237b2af38 RFC 28822 compliance changes. 
Move Mail Functionality into smtpManager.cpp
t_a added 1 commit 2023-05-29 13:25:55 +00:00
MichaelYick requested review from MichaelYick 2023-05-31 10:29:47 +00:00
MichaelYick approved these changes 2023-06-01 06:33:22 +00:00
MichaelYick merged commit 60a281084a into master 2023-06-01 06:33:39 +00:00
Sign in to join this conversation.
Reviewers
No reviewers
MichaelYick
Labels
Clear labels
  
bug

Something is not working

  
Critical Priority

Absolutely urgent issues that must be delt with before others can even be considered.

  
duplicate

This issue or pull request already exists

  
enhancement

New feature

  
help wanted

Need some help

  
High Priority

Bugs that cause crashes, major story errors, or other massively negative issues. Features that are a must-have inclusion or else the game won't be as good.

  
invalid

Something is wrong

  
Low Priority

Minor or easy to miss errors. Features that'd be nice, but can be done with out.

  
Medium Priority

Errors that stick out, but aren't fatal to gameplay. Features that are in decent demand or would improve gameplay by a signifigant amount

  
Meta

for all the other fun stuff about the project thats not code

  
question

More information is needed

  
wontfix

This won't be fixed

No Label
bug
Critical Priority
duplicate
enhancement
help wanted
High Priority
invalid
Low Priority
Medium Priority
Meta
question
wontfix
Milestone
Clear milestone
No items
No Milestone
Projects
Clear projects
No project
Assignees
Clear assignees
No Assignees
2 Participants
Notifications
Due Date
The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.

No due date set.

Dependencies

No dependencies set.

Reference: Cavemanon/cavecomm#29
No description provided.
Delete Branch "NewSMTP"

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?