diff --git a/app/jobs/inboxes/fetch_imap_emails_job.rb b/app/jobs/inboxes/fetch_imap_emails_job.rb index 5722e76a7..7564468ca 100644 --- a/app/jobs/inboxes/fetch_imap_emails_job.rb +++ b/app/jobs/inboxes/fetch_imap_emails_job.rb @@ -12,8 +12,7 @@ class Inboxes::FetchImapEmailsJob < MutexApplicationJob process_email_for_channel(channel) end rescue *ExceptionList::IMAP_EXCEPTIONS => e - Rails.logger.error e - channel.authorization_error! + Rails.logger.error "Authorization error for email channel - #{channel.inbox.id} : #{e.message}" rescue EOFError, OpenSSL::SSL::SSLError, Net::IMAP::NoResponseError, Net::IMAP::BadResponseError, Net::IMAP::InvalidResponseError => e Rails.logger.error e rescue LockAcquisitionError diff --git a/spec/jobs/inboxes/fetch_imap_emails_job_spec.rb b/spec/jobs/inboxes/fetch_imap_emails_job_spec.rb index f40e39151..f3b3d45e3 100644 --- a/spec/jobs/inboxes/fetch_imap_emails_job_spec.rb +++ b/spec/jobs/inboxes/fetch_imap_emails_job_spec.rb @@ -60,16 +60,6 @@ RSpec.describe Inboxes::FetchImapEmailsJob do end end - context 'when IMAP connection errors out' do - it 'mark the connection for authorization required' do - allow(Imap::FetchEmailService).to receive(:new).with(channel: imap_email_channel).and_raise(Errno::ECONNREFUSED) - allow(Redis::Alfred).to receive(:incr) - - expect(Redis::Alfred).to receive(:incr).with("AUTHORIZATION_ERROR_COUNT:channel_email:#{imap_email_channel.id}") - described_class.perform_now(imap_email_channel) - end - end - context 'when the fetch service returns the email objects' do let(:inbound_mail) { create_inbound_email_from_fixture('welcome.eml').mail } let(:mailbox) { double }