diff --git a/api/utils.py b/api/utils.py index 50ae798..be6d403 100644 --- a/api/utils.py +++ b/api/utils.py @@ -6,6 +6,7 @@ from datetime import datetime, timedelta import json import traceback from concurrent.futures import ThreadPoolExecutor, as_completed +from django.conf import settings logger = logging.getLogger(__name__) @@ -32,7 +33,7 @@ def scrape_fft_club_tournaments( logger.info(f"Starting Playwright scraping for {club_name}") try: with sync_playwright() as p: - browser = p.chromium.launch(headless=True) + browser = get_browser_for_environment(p) page_obj = browser.new_page() page_obj.set_extra_http_headers( @@ -43,10 +44,10 @@ def scrape_fft_club_tournaments( # Navigate to FFT target_url = "https://tenup.fft.fr/recherche/tournois" - logger.info(f"Navigating to: {target_url}") + # logger.info(f"Navigating to: {target_url}") page_obj.goto(target_url) - # page_obj.wait_for_timeout(7000) + page_obj.wait_for_timeout(2000) current_url = page_obj.url logger.info(f"Current URL: {current_url}") @@ -64,7 +65,7 @@ def scrape_fft_club_tournaments( return None form_build_id = form_input.get_attribute("value") - logger.info(f"Extracted form_build_id: {form_build_id}") + # logger.info(f"Extracted form_build_id: {form_build_id}") # Build parameters date_component = "" @@ -83,7 +84,7 @@ def scrape_fft_club_tournaments( params = f"recherche_type=club&club[autocomplete][value_container][value_field]={club_code_clean}&club[autocomplete][value_container][label_field]={club_name_encoded}&pratique=PADEL{date_component}&page={page}&sort=dateDebut+asc&form_build_id={form_build_id}&form_id=recherche_tournois_form&_triggering_element_name=submit_page&_triggering_element_value=Submit+page" - logger.info(f"AJAX Parameters: {params}") + # logger.info(f"AJAX Parameters: {params}") # Make AJAX request and capture the full response ajax_script = f""" @@ -118,24 +119,24 @@ def scrape_fft_club_tournaments( }} """ - logger.info("Making AJAX request...") + # logger.info("Making AJAX request...") result = page_obj.evaluate(ajax_script) browser.close() # Print the full response for debugging - logger.info(f"AJAX Response Status: {result.get('status')}") - logger.info(f"AJAX Response Success: {result.get('success')}") + # logger.info(f"AJAX Response Status: {result.get('status')}") + # logger.info(f"AJAX Response Success: {result.get('success')}") if result.get("success"): response_text = result.get("responseText", "") - logger.info(f"Raw Response Length: {len(response_text)}") + # logger.info(f"Raw Response Length: {len(response_text)}") # logger.info(f"Raw Response (first 500 chars): {response_text[:500]}") try: # Try to parse as JSON json_data = json.loads(response_text) - logger.info(f"JSON Response Type: {type(json_data)}") + # logger.info(f"JSON Response Type: {type(json_data)}") # logger.info(f"JSON Response: {json.dumps(json_data, indent=2, default=str)}") # Now try to parse it @@ -164,7 +165,7 @@ def scrape_fft_club_tournaments_all_pages( """ Scrapes all pages of FFT tournaments for a specific club """ - logger.info(f"Starting complete tournament scraping for {club_name}") + # logger.info(f"Starting complete tournament scraping for {club_name}") all_tournaments = [] page = 0 @@ -237,7 +238,7 @@ def _parse_ajax_response(commands): # Check for alert commands (maintenance mode) for command in commands: if command.get("command") == "alert": - logger.warning("Maintenance mode detected") + # logger.warning("Maintenance mode detected") return None # Find the command with results @@ -245,7 +246,7 @@ def _parse_ajax_response(commands): for command in commands: if command.get("command") == "recherche_tournois_update": result_command = command - logger.info("Found recherche_tournois_update command!") + # logger.info("Found recherche_tournois_update command!") break if result_command and result_command.get("results"): @@ -253,7 +254,7 @@ def _parse_ajax_response(commands): items = results.get("items", []) total_results = results.get("nb_results", 0) - logger.info(f"Processing {len(items)} tournaments from results") + # logger.info(f"Processing {len(items)} tournaments from results") for item in items: # Parse dates - they're already in the correct format @@ -331,9 +332,9 @@ def _parse_ajax_response(commands): tournaments.append(tournament) - logger.info( - f"Successfully parsed {len(tournaments)} tournaments from response" - ) + # logger.info( + # f"Successfully parsed {len(tournaments)} tournaments from response" + # ) return { "tournaments": tournaments, "total_results": total_results, @@ -371,7 +372,7 @@ def scrape_fft_all_tournaments( try: with sync_playwright() as p: - browser = p.chromium.launch(headless=True) + browser = get_browser_for_environment(p) page_obj = browser.new_page() page_obj.set_extra_http_headers( @@ -382,28 +383,28 @@ def scrape_fft_all_tournaments( # Navigate to FFT target_url = "https://tenup.fft.fr/recherche/tournois" - logger.info(f"Navigating to: {target_url}") + # logger.info(f"Navigating to: {target_url}") page_obj.goto(target_url) - # page_obj.wait_for_timeout(7000) + page_obj.wait_for_timeout(500) current_url = page_obj.url - logger.info(f"Current URL: {current_url}") + # logger.info(f"Current URL: {current_url}") if "queue-it.net" in current_url.lower(): - logger.warning("Still in Queue-It") + # logger.warning("Still in Queue-It") browser.close() return None # Extract form_build_id form_input = page_obj.query_selector('input[name="form_build_id"]') if not form_input: - logger.error("Could not find form_build_id") + # logger.error("Could not find form_build_id") browser.close() return None form_build_id = form_input.get_attribute("value") - logger.info(f"Extracted form_build_id: {form_build_id}") + # logger.info(f"Extracted form_build_id: {form_build_id}") # Build parameters - EXACT same pattern as club function date_component = "" @@ -422,32 +423,32 @@ def scrape_fft_all_tournaments( # Add categories filter if categories: - logger.info(f"Adding categories filter: {categories}") + # logger.info(f"Adding categories filter: {categories}") for category in categories: filter_params += f"&epreuve[{category}]={category}" # Add levels filter if levels: - logger.info(f"Adding levels filter: {levels}") + # logger.info(f"Adding levels filter: {levels}") for level in levels: filter_params += f"&categorie_tournoi[{level}]={level}" # Add ages filter if ages: - logger.info(f"Adding ages filter: {ages}") + # logger.info(f"Adding ages filter: {ages}") for age in ages: filter_params += f"&categorie_age[{age}]={age}" # Add types filter if tournament_types: - logger.info(f"Adding types filter: {tournament_types}") + # logger.info(f"Adding types filter: {tournament_types}") for t_type in tournament_types: capitalized_type = t_type.capitalize() filter_params += f"&type[{capitalized_type}]={capitalized_type}" # Add national cup filter if national_cup: - logger.info("Adding national cup filter") + # logger.info("Adding national cup filter") filter_params += "&tournoi_npc=1" # Fix the sorting parameter @@ -477,7 +478,7 @@ def scrape_fft_all_tournaments( # Default to ligue search if no city provided params = f"recherche_type=ligue&pratique=PADEL{date_component}{filter_params}&page={page}&sort={sorting_option or 'dateDebut+asc'}&form_build_id={form_build_id}&form_id=recherche_tournois_form&_triggering_element_name=submit_page&_triggering_element_value=Submit+page" - logger.info(f"AJAX Parameters: {params}") + # logger.info(f"AJAX Parameters: {params}") # Make AJAX request and capture the full response - EXACT same as club function ajax_script = f""" @@ -512,23 +513,23 @@ def scrape_fft_all_tournaments( }} """ - logger.info("Making AJAX request...") + # logger.info("Making AJAX request...") result = page_obj.evaluate(ajax_script) browser.close() # Print the full response for debugging - EXACT same as club function - logger.info(f"AJAX Response Status: {result.get('status')}") - logger.info(f"AJAX Response Success: {result.get('success')}") + # logger.info(f"AJAX Response Status: {result.get('status')}") + # logger.info(f"AJAX Response Success: {result.get('success')}") if result.get("success"): response_text = result.get("responseText", "") - logger.info(f"Raw Response Length: {len(response_text)}") + # logger.info(f"Raw Response Length: {len(response_text)}") try: # Try to parse as JSON json_data = json.loads(response_text) - logger.info(f"JSON Response Type: {type(json_data)}") + # logger.info(f"JSON Response Type: {type(json_data)}") # Now try to parse it - EXACT same as club function parsed_result = _parse_ajax_response(json_data) @@ -536,11 +537,11 @@ def scrape_fft_all_tournaments( return parsed_result except json.JSONDecodeError as json_error: - logger.error(f"JSON Parse Error: {json_error}") - logger.error(f"Response text: {response_text}") + # logger.error(f"JSON Parse Error: {json_error}") + # logger.error(f"Response text: {response_text}") return None else: - logger.error(f"AJAX request failed: {result.get('error')}") + # logger.error(f"AJAX request failed: {result.get('error')}") return None except Exception as e: @@ -551,73 +552,277 @@ def scrape_fft_all_tournaments( def get_umpire_data(tournament_id): """ - Fast umpire data extraction using Playwright (optimized for speed) + Umpire data extraction with improved Queue-It handling """ - logger.info(f"Getting umpire data for tournament {tournament_id}") + # logger.info(f"Getting umpire data for tournament {tournament_id}") try: with sync_playwright() as p: - browser = p.chromium.launch( - headless=True, - args=[ - "--no-sandbox", - "--disable-dev-shm-usage", - "--disable-images", # Don't load images - "--disable-javascript", # Disable JS for faster loading - "--disable-plugins", - "--disable-extensions", - ], - ) + browser = get_browser_for_environment(p) page = browser.new_page() - # Navigate to tournament page quickly + # Navigate to tournament page url = f"https://tenup.fft.fr/tournoi/{tournament_id}" - logger.info(f"Navigating to tournament page: {url}") + # logger.info(f"Navigating to tournament page: {url}") try: - # Fast navigation - don't wait for everything to load - page.goto(url, timeout=15000, wait_until="domcontentloaded") + # Navigate with reasonable timeout + page.goto(url, timeout=30000, wait_until="domcontentloaded") - # Quick Queue-It check + # Enhanced Queue-It handling (similar to tournament search) if "queue-it.net" in page.url.lower(): - logger.warning("Hit Queue-It on tournament page") - browser.close() - return None, None, None - - # Extract data using the fastest method - regex on HTML content + # logger.warning(f"Queue-It detected for tournament {tournament_id}") + + # Wait strategy based on environment + max_queue_wait = ( + 120000 if not settings.DEBUG else 30000 + ) # 2 min prod, 30s dev + check_interval = 1000 # 10 seconds + elapsed_time = 0 + + # logger.info( + # f"Waiting in queue for umpire data (max {max_queue_wait / 1000}s)..." + # ) + + while ( + elapsed_time < max_queue_wait + and "queue-it.net" in page.url.lower() + ): + # logger.info( + # f"Umpire queue wait: {elapsed_time / 1000}s elapsed" + # ) + page.wait_for_timeout(check_interval) + elapsed_time += check_interval + + # Check if we've been redirected + if "queue-it.net" not in page.url.lower(): + # logger.info( + # "Successfully passed through queue for umpire data!" + # ) + break + + # If still in queue, try fallback or return None + if "queue-it.net" in page.url.lower(): + # logger.warning( + # "Umpire queue timeout - trying requests fallback" + # ) + browser.close() + return _get_umpire_data_requests_fallback(tournament_id) + + # Wait for page to load properly + page.wait_for_load_state("networkidle", timeout=20000) + + # Extract data using multiple strategies html_content = page.content() - # Extract name - name_pattern = ( - r'tournoi-detail-page-inscription-responsable-title">\s*([^<]+)\s*<' - ) - name_match = re.search(name_pattern, html_content) - name = name_match.group(1).strip() if name_match else None - - # Extract email - email_pattern = r'mailto:([^"]+)"' - email_match = re.search(email_pattern, html_content) - email = email_match.group(1) if email_match else None + # Strategy 1: Try existing regex patterns + name, email, phone = _extract_umpire_with_regex(html_content) - # Extract phone - phone_pattern = r'