Merge pull request #17 from maxsonferovante/fix/sanitize-special-characters-certificate-names

fix: Corrige problema com caracteres especiais nos nomes dos certific…

Author: maxsonferovante

models/participant.py

@@ -2,10 +2,16 @@
 from typing import Optional
 from .certificate import Certificate
 from .event import Event
+import re
+import unicodedata
+import logging
 
 import random
 import string
 
+# Configure logger for this module
+logger = logging.getLogger(__name__)
+
 class Participant(BaseModel): 
     first_name: str
     last_name: str
@@ -32,13 +38,83 @@ def name_completed(self):
 
         return name_completed.title()
 
+    def _sanitize_filename(self, text: str) -> str:
+        """
+        Sanitiza uma string para ser usada como nome de arquivo no S3.
+        Remove ou substitui caracteres especiais que podem causar problemas em URLs.
+        
+        Args:
+            text (str): Texto a ser sanitizado
+            
+        Returns:
+            str: Texto sanitizado adequado para nomes de arquivo S3
+        """
+        # Normaliza caracteres unicode (remove acentos)
+        text = unicodedata.normalize('NFD', text)
+        text = ''.join(char for char in text if unicodedata.category(char) != 'Mn')
+        
+        # Substitui caracteres especiais comuns por versões seguras
+        special_chars = {
+            'º': 'o',
+            'ª': 'a', 
+            '×': 'x',
+            '@': '_at_',
+            '&': '_and_',
+            '+': '_plus_',
+            '=': '_equals_',
+            '%': '_percent_',
+            '#': '_hash_',
+            '?': '_question_',
+            '/': '_slash_',
+            '\\': '_backslash_',
+            ':': '_colon_',
+            ';': '_semicolon_',
+            '<': '_lt_',
+            '>': '_gt_',
+            '|': '_pipe_',
+            '*': '_star_',
+            '"': '_quote_',
+            "'": '_apostrophe_'
+        }
+        
+        # Aplica as substituições de caracteres especiais
+        for char, replacement in special_chars.items():
+            text = text.replace(char, replacement)
+        
+        # Remove caracteres que não são alfanuméricos, espaços, hífens ou underscores
+        text = re.sub(r'[^\w\s\-_]', '', text)
+        
+        # Substitui espaços múltiplos por um único espaço
+        text = re.sub(r'\s+', ' ', text)
+        
+        # Substitui espaços por underscores
+        text = text.replace(' ', '_')
+        
+        # Remove underscores múltiplos consecutivos
+        text = re.sub(r'_+', '_', text)
+        
+        # Remove underscores do início e fim
+        text = text.strip('_')
+        
+        return text
 
 
     def formated_validation_code(self):
         self.validation_code = self.validation_code.upper()
         return f"{self.validation_code[0:3]}-{self.validation_code[3:6]}-{self.validation_code[6:9]}"
 
     def create_name_certificate(self):        
-        name_certificate = self.name_completed() + self.event.product_name + "_" + self.formated_validation_code() + ".png"
-        name_certificate = name_certificate.replace(" ", "_")
+        # Sanitiza o nome do participante e o nome do produto separadamente
+        sanitized_name = self._sanitize_filename(self.name_completed())
+        sanitized_product = self._sanitize_filename(self.event.product_name)
+        sanitized_validation = self._sanitize_filename(self.formated_validation_code())
+        
+        # Combina os componentes sanitizados
+        name_certificate = f"{sanitized_name}{sanitized_product}_{sanitized_validation}.png"
+        
+        logger.info(f"Nome do participante antes da sanitização: {self.name_completed()}")
+        logger.info(f"Nome do produto antes da sanitização: {self.event.product_name}")
+        logger.info(f"Código de validação antes da sanitização: {self.formated_validation_code()}")
+        logger.info(f"Nome do certificado após a sanitização: {name_certificate}")
+        
         return name_certificate

tests/test_participant.py

@@ -125,5 +125,105 @@ def test_name_completed_with_multiple_names(mock_certificate, mock_event):
     completed_name = participant.name_completed()
     assert len(completed_name.split()) == 3
     assert completed_name == "Jardel Silva Santos"
+
+def test_sanitize_filename_special_characters(mock_certificate):
+    """Testa se a função de sanitização remove adequadamente caracteres especiais"""
+    # Cria um evento com nome contendo caracteres especiais
+    event_with_special_chars = Event(
+        order_id=452,
+        product_id=316,
+        product_name="87º Python Floripa × CODECON @ UNICESUSC",
+        date=datetime.strptime("2025-03-26 20:55:25", "%Y-%m-%d %H:%M:%S"),
+        time_checkin=datetime.strptime("2025-03-26 20:55:44", "%Y-%m-%d %H:%M:%S"),
+        checkin_latitude=-27.5460492,
+        checkin_longitude=-48.6227075
+    )
+    
+    participant = Participant(
+        first_name="Rodrigo",
+        last_name="Farah",
+        email="rodrigo@example.com",
+        phone="(48) 98866-7447",
+        cpf="000.000.000-00",
+        certificate=mock_certificate,
+        event=event_with_special_chars
+    )
+    
+    # Testa a função de sanitização diretamente
+    sanitized = participant._sanitize_filename("87º Python Floripa × CODECON @ UNICESUSC")
+    
+    # Verifica se caracteres especiais foram sanitizados
+    assert 'º' not in sanitized
+    assert '×' not in sanitized
+    assert '@' not in sanitized
+    assert sanitized == "87o_Python_Floripa_x_CODECON_at_UNICESUSC"
+
+def test_create_name_certificate_with_special_characters(mock_certificate):
+    """Testa se o nome do certificado é gerado corretamente com caracteres especiais"""
+    # Cria um evento com nome contendo caracteres especiais (igual ao exemplo do usuário)
+    event_with_special_chars = Event(
+        order_id=1397,
+        product_id=1324,
+        product_name="87º Python Floripa × CODECON @ UNICESUSC",
+        date=datetime.strptime("2025-06-19 11:15:31", "%Y-%m-%d %H:%M:%S"),
+        time_checkin=datetime.strptime("2025-06-19 11:15:31", "%Y-%m-%d %H:%M:%S"),
+        checkin_latitude=-27.5460492,
+        checkin_longitude=-48.6227075
+    )
+    
+    participant = Participant(
+        first_name="Rodrigo",
+        last_name="Farah",
+        email="rodrigo.farah@example.com",
+        phone="(48) 98866-7447",
+        cpf="000.000.000-00",
+        certificate=mock_certificate,
+        event=event_with_special_chars
+    )
+    
+    name_certificate = participant.create_name_certificate()
+    
+    # Verifica se o nome do certificado não contém caracteres especiais problemáticos
+    assert 'º' not in name_certificate
+    assert '×' not in name_certificate
+    assert '@' not in name_certificate
+    assert name_certificate.endswith(".png")
+    
+    # Verifica se contém elementos esperados (sanitizados)
+    assert "Rodrigo_Farah" in name_certificate
+    assert "87o_Python_Floripa_x_CODECON_at_UNICESUSC" in name_certificate
+    
+    # Verifica se o nome é válido para URL (não contém caracteres que precisam ser encoded)
+    import re
+    # Permite apenas caracteres alfanuméricos, hífens, underscores e pontos
+    assert re.match(r'^[\w\-_.]+$', name_certificate), f"Nome do certificado contém caracteres inválidos: {name_certificate}"
+
+def test_sanitize_filename_edge_cases(mock_certificate, mock_event):
+    """Testa casos extremos da função de sanitização"""
+    participant = Participant(
+        first_name="Test",
+        last_name="User",
+        email="test@example.com",
+        phone="(48) 98866-7447",
+        cpf="000.000.000-00",
+        certificate=mock_certificate,
+        event=mock_event
+    )
+    
+    # Testa strings com múltiplos caracteres especiais
+    test_cases = {
+        "": "",  # String vazia
+        "   ": "",  # Apenas espaços
+        "___": "",  # Apenas underscores
+        "Test @@@ Test": "Test_at_at_at_Test",  # Múltiplos símbolos
+        "Açaí & Café": "Acai_and_Cafe",  # Acentos e símbolos
+        "100% Success!!!": "100_percent_Success",  # Porcentagem e exclamações
+        "C++ Programming": "C_plus_plus_Programming",  # Símbolos de programação
+        "file/path\\name": "file_slash_path_backslash_name",  # Separadores de caminho
+    }
+    
+    for input_text, expected_output in test_cases.items():
+        result = participant._sanitize_filename(input_text)
+        assert result == expected_output, f"Para '{input_text}', esperado '{expected_output}', obtido '{result}'"