Consulta sobre la clase PaymentClient

[fgonzalez-cespi]

Con mi equipo de desarrollo tenemos una integración en donde usamos PaymentClient para obtener datos de pagos y hacer posibles refunds.

Vemos que en el create de dicha clase está este codigo:

  /** Default constructor. Uses the default http client used by the SDK. */
  public PaymentClient() {
    this(MercadoPagoConfig.getHttpClient());
  }

  /**
   * Constructor used for providing a custom http client.
   *
   * @param httpClient httpClient
   */
  public PaymentClient(MPHttpClient httpClient) {
    super(httpClient);
    refundClient = new PaymentRefundClient(httpClient);
    StreamHandler streamHandler = getStreamHandler();
    streamHandler.setLevel(MercadoPagoConfig.getLoggingLevel());
    LOGGER.addHandler(streamHandler);
    LOGGER.setLevel(MercadoPagoConfig.getLoggingLevel());
  }

Esto genera que al crear múltiples PaymentClient se vayan agregando handlers al logger (LOGGER.addHandler(streamHandler);).

Siendo que LOGGER es static, esto genera que al crear muchos PaymentClient con el tiempo el LOGGER va creciendo en memoria no collectable.

Mi pregunta es si esto es un bug o es un missuse mío, y solo tengo que crear un PaymentClient global, aunque si esto fuera el caso, la API debería ocultarme este constructor y proveerme un singleton para no tener que lidiar con ello.

Básicamente quiero asegurarme de que usar un solo PaymentClient como singleton sea thread safe concurrentemente.