mirror of
https://github.com/jberkel/sms-backup-plus.git
synced 2026-04-25 17:05:59 +03:00
[GH-ISSUE #999] Incomplete email address in email headers #793
Labels
No labels
AM+RCS
FAQ
awaiting response
backup
bespoke
bug
calendar
call log
cannot reproduce
cloudless
device-specific
documentation
dual- & multi-SIM
duplicate
feature-request
fixed in beta
good first issue
half-missing
help wanted
helpful
meta
misattribution
mms
other message sources
pull-request
question
rejuvenation
restore
schedule
security
stale
task
thanks
v1.5.1
v1.5.10
v1.5.11
v1.5.2
v1.5.3
v1.5.3
v1.5.4
v1.5.4
v1.5.5
v1.5.5
v1.5.6
v1.5.7
v1.5.8
v1.5.9
v1.6β
xoauth
~$ bounty $~
No milestone
No project
No assignees
1 participant
Notifications
Due date
No due date set.
Dependencies
No dependencies set.
Reference
starred/sms-backup-plus-jberkel#793
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 @psss on GitHub (Dec 18, 2019).
Original GitHub issue: https://github.com/jberkel/sms-backup-plus/issues/999
Originally assigned to: @kurahaupo, @psss on GitHub.
After updating my settings to use apps password instead of xoath I've noticed that the email headers
toandfromcontain only the first part of my email address (before the@character). Other email addresses are stored correctly. It would be nice to use a complete email address for the account owner as well. It allows to easily use gmail filters likefrom:mewhen searching for messages.Expected behaviour
Complete email address included in message headers, e.g.
user@example.comActual behaviour
Only the login part included, e.g.
userSteps to reproduce the behaviour
Configure imap using password, backup messages.
Thanks for this great app!
@kurahaupo commented on GitHub (Dec 19, 2019):
Use your full email address as your Gmail login ID.
@kurahaupo commented on GitHub (Dec 19, 2019):
(This should probably be included in the documentation.)
@psss commented on GitHub (Dec 19, 2019):
I've already tried to use the full email address for the login in imap settings. But it did not help.
@kurahaupo commented on GitHub (Dec 19, 2019):
I won't say absolutely that this is a problem that's specific to your set-up, but it does seem odd that nobody else has reported this problem.
Tracing through the code, I see that the sender and recipient fields in a new email message are set in app/src/main/java/com/zegoggles/smssync/mail/MessageGenerator.java at lines 91-99:
where
userAddressis set in the constructor for classMessageGenerator, when called from app/src/main/java/com/zegoggles/smssync/mail/MessageConverter.java at lines 81-89:.where
userEmailis set in the constructor for classMessageConverter, when called fromapp/src/main/java/com/zegoggles/smssync/service/BackupTask.java at line 74:
where
getUserEmailis defined in app/src/main/java/com/zegoggles/smssync/preferences/AuthPreferences.java at lines 136-142:and
getImapUsernameis defined at lines 232-234Or in short, changing your IMAP auth username to your full Gmail address should have worked. Please try again, and ensure that you restart the app.
Note that this will not change any messages that have already been recorded, and will only affect subsequent messages.
@psss commented on GitHub (Dec 20, 2019):
Thanks for detailed investigation. It seems that during my second attempt the old messages with wrong headers have been restored from trash instead of new messages created with correct headers. After deleting the wrong messages and cleaning up the trash everything seems to be working fine. Thanks much for your help.
@kurahaupo commented on GitHub (Dec 20, 2019):
Glad I could help.