Subversion Repositories SmartDukaan

package com.spice.profitmandi.service.mail;

import com.spice.profitmandi.common.mail.MailQueue;

import com.spice.profitmandi.common.mail.MailRequest;

import org.springframework.beans.factory.annotation.Autowired;

import org.springframework.stereotype.Service;

/**

 * Publishes {@link MailOutboxService} as a {@link MailQueue} without letting

 * MailOutboxService implement the interface itself.

 *

 * That indirection is load-bearing. MailOutboxService has @Transactional methods,

 * so Spring must proxy it, and @EnableTransactionManagement here runs with the

 * default proxyTargetClass=false. While the class implements no interface Spring

 * proxies it with a CGLIB subclass, which is still a MailOutboxService. The moment

 * it implements one, Spring switches to a JDK proxy that implements only that

 * interface -- and the 39 sites which inject the concrete MailOutboxService all

 * fail with BeanNotOfRequiredTypeException, taking the whole context down with

 * them. That is exactly what happened in production on 2026-08-31.

 *

 * So the interface lives here instead: this adapter carries no transactional

 * annotations, and its consumers (MailQueueHolder, EmailServiceImpl) inject it by

 * the MailQueue interface, which a proxy of either kind satisfies.

 */

@Service

public class MailQueueAdapter implements MailQueue {

    private final MailOutboxService mailOutboxService;

    @Autowired

    public MailQueueAdapter(MailOutboxService mailOutboxService) {

        this.mailOutboxService = mailOutboxService;

    }

    @Override

    public void queue(MailRequest request) {

        mailOutboxService.queue(request);

    }

}