T
Twenty2mo ago
G

Google Sync Fails: Invalid/Empty Emails Causing Domain Filter Error

Hi there, after a successful installation, I am getting the following error on the worker when syncing my work Gmail account. The error sometimes occurs at the beginning of the sync process or later on. It feels a bit random. Any support or feedback is appreciated. Error: Email is required 2025-02-28T13:59:04.114144081Z at getDomainNameByEmail (/app/packages/twenty-server/dist/src/utils/get-domain-name-by-email.js:13:15) 2025-02-28T13:59:04.114146028Z at /app/packages/twenty-server/dist/src/modules/messaging/message-import-manager/utils/filter-emails.util.js:46:133 2025-02-28T13:59:04.114147861Z at Array.every (<anonymous>) 2025-02-28T13:59:04.114149810Z at /app/packages/twenty-server/dist/src/modules/messaging/message-import-manager/utils/filter-emails.util.js:46:65 2025-02-28T13:59:04.114151658Z at Array.filter (<anonymous>) 2025-02-28T13:59:04.114153454Z at filterOutInternals (/app/packages/twenty-server/dist/src/modules/messaging/message-import-manager/utils/filter-emails.util.js:41:21) 2025-02-28T13:59:04.114158865Z at filterEmails (/app/packages/twenty-server/dist/src/modules/messaging/message-import-manager/utils/filter-emails.util.js:20:16) 2025-02-28T13:59:04.114160803Z at MessagingMessagesImportService.processMessageBatchImport (/app/packages/twenty-server/dist/src/modules/messaging/message-import-manager/services/messaging-messages-import.service.js:99:71) 2025-02-28T13:59:04.114163436Z at process.processTicksAndRejections (node:internal/process/task_queues:95:5) 2025-02-28T13:59:04.114165663Z at async MessagingMessagesImportJob.handle (/app/packages/twenty-server/dist/src/modules/messaging/message-import-manager/jobs/messaging-messages-import.job.js:63:9)
15 Replies
ɃØĦɆᵾS
ɃØĦɆᵾS2mo ago
@Guillaume could you help?
Guillaume
Guillaume2mo ago
Hi, Yep I will investigate this week. Thanks for reporting
G
GOP2mo ago
Thank you
Guillaume
Guillaume2mo ago
Should be fixed with this PR: https://github.com/twentyhq/twenty/pull/10625. I would recommend to wait for it to be tested, merged and deployed but if you are in a hurry, you can give it a try 🙂
GitHub
Build software better, together
GitHub is where people build software. More than 150 million people use GitHub to discover, fork, and contribute to over 420 million projects.
From An unknown user
From An unknown user
From An unknown user
G
GOP5w ago
thx @Guillaume Unfortunately the bug is present in v.43 @Guillaume Exception Captured undefined [ Error: Email is required at getDomainNameByEmail (/app/packages/twenty-server/dist/src/utils/get-domain-name-by-email.js:13:15) at /app/packages/twenty-server/dist/src/modules/messaging/message-import-manager/utils/filter-emails.util.js:46:133 at Array.every (<anonymous>) at /app/packages/twenty-server/dist/src/modules/messaging/message-import-manager/utils/filter-emails.util.js:46:65 at Array.filter (<anonymous>) at filterOutInternals (/app/packages/twenty-server/dist/src/modules/messaging/message-import-manager/utils/filter-emails.util.js:41:21) at filterEmails (/app/packages/twenty-server/dist/src/modules/messaging/message-import-manager/utils/filter-emails.util.js:20:16) at MessagingMessagesImportService.processMessageBatchImport (/app/packages/twenty-server/dist/src/modules/messaging/message-import-manager/services/messaging-messages-import.service.js:99:71) at process.processTicksAndRejections (node:internal/process/task_queues:95:5) at async MessagingMessagesImportJob.handle (/app/packages/twenty-server/dist/src/modules/messaging/message-import-manager/jobs/messaging-messages-import.job.js:63:9) ]
Guillaume
Guillaume5w ago
it's normal : the fix I did is now available on the github repository but not in production so far. we release features once every 2 weeks. so you will have to pull the main branch yourself to test it right now, or wait for the next release (around the 14th of march)
G
GOP5w ago
I see, cool. I though it was merged in today's release. Will test and report back
Guillaume
Guillaume5w ago
Yes please give your feedback on this bugfix to make sure it works better !
G
GOP5w ago
Is there any documentation on how I can build the branch with the fix?
Guillaume
Guillaume5w ago
No there isn’t specifically for this branch. But on our public documentation you will find guides for developers that will guide you. Start with a git clone of the main branch and it should be good
G
GOP4w ago
Alright, so I wasn't able to compile the image from the repo. I deleted all empty emails from the draft inbox, and the importing didn't stop this time. However, I am getting a constant importing message without any updates on the log of the worker container.
thomast
thomast7d ago
Hi, do you still have an issue here?
Sonntagsgebet
Sonntagsgebet7d ago
Hey @thomast 🙂 thanks to Guillaume, I got the calendar working and it shows that it is importing. Sadly the mails are still not synced and I'm running out of ideas on how to manage to import the mails. I'm using docker and would be very happy to share the logs etc with you
thomast
thomast7d ago
cc @Guillaume 🙏
Pierre Rama
Pierre Rama7d ago
Same here!

Did you find this page helpful?