javihernant/img2pdf

Simple Spring Boot application that receives an image and converts it into a pdf document.

Img2Pdf

Description

Simple Spring Boot application that receives an image and converts it into a pdf document.

Usage

Maven Wrapper has been installed to this project, so there's no need to install maven on your machine. To run the application you only have to use a simple command on your terminal:

./mvnw spring-boot:run

NOTE: Use mvnw.cmd if you're running Windows

Generate a pdf from an image

Send a POST request to http://localhost:9080/img2pdf. You can use the form at http://localhost:9080/ to upload your image.

The service will respond with a json containing an id corresponding to the pdf document generated.

Download your pdf

Use the id provided in order to download your document.

Make a GET request to http://localhost:9080/pdf/<idDocument> to obtain your document. You can directly paste the url on your browser and your download will start immediately

img2pdf's Issues

Variable innecesaria

Podría devolverse directamente el resultado de la llamada a storage.storePdf(doc, pdfFileName)

https://github.com/javihernant/img2pdf/blob/2ff94553f6bc02ad0217f9bc7fb554ee136b2898/src/main/java/com/example/simple_spring_boot/pdf_converter/PdfConverterService.java#L41C33-L41C67

Generación incremental de ids.

La generación de ids usando AtomicInteger haría que dos ejecuciones del proyecto generan ids que distintos usuarios podrían tener repetidos, pero el contenido diferente. Se debería haber utilizado un identificador único UUID para evitar duplicidad entre ejecuciones.

img2pdf/src/main/java/com/example/simple_spring_boot/storage/PdfStorageService.java

Line 21 in 2ff9455

final private AtomicLong id = new AtomicLong();

Uso de hashmap en entorno concurrente.

Un hashmap en un entorno de concurrencia puede producir error de multipla acceso ya que no es threadsafe, la opción correcta en este caso habría sido utilizar un ConcurrentHashMap

img2pdf/src/main/java/com/example/simple_spring_boot/storage/PdfStorageService.java

Line 22 in 2ff9455

private HashMap<Long, Path> pdfPaths;

mismo nombre de variable en método y campo de clase.

Este campo id confunde la lectura al tener el mismo nombre la variable del método y la propiedad id.

img2pdf/src/main/java/com/example/simple_spring_boot/storage/PdfStorageService.java

Line 38 in 2ff9455

pdfPaths.put(id, path);