Differential D8287 Diff 30026 swh/lister/pubdev/lister.py

Changeset View

Standalone View

swh/lister/pubdev/lister.py

This file was added.

				# Copyright (C) 2022 The Software Heritage developers
				# See the AUTHORS file at the top-level directory of this distribution
				# License: GNU General Public License version 3, or any later version
				# See top-level LICENSE file for more information
				import logging
				from typing import Any, Dict, Iterator, List, Optional

				import requests
				from tenacity.before_sleep import before_sleep_log

				from swh.lister.utils import throttling_retry
				from swh.scheduler.interface import SchedulerInterface
				from swh.scheduler.model import ListedOrigin

				from .. import USER_AGENT
				from ..pattern import CredentialsType, StatelessLister

				logger = logging.getLogger(__name__)

				# Aliasing the page results returned by `get_pages` method from the lister.
				PubDevListerPage = List[str]


				class PubDevLister(StatelessLister[PubDevListerPage]):
				"""List pub.dev (Dart, Flutter) origins."""

				LISTER_NAME = "pubdev"
				VISIT_TYPE = "pubdev"
				INSTANCE = "pubdev"

				BASE_URL = "https://pub.dev/api/"
				PACKAGE_NAMES_URL_PATTERN = "{base_url}package-names"
				PACKAGE_INFO_URL_PATTERN = "{base_url}packages/{pkgname}"

				def __init__(
				self,
				scheduler: SchedulerInterface,
				credentials: Optional[CredentialsType] = None,
				):
				super().__init__(
				scheduler=scheduler,
				credentials=credentials,
				instance=self.INSTANCE,
				url=self.BASE_URL,
				)
				self.session = requests.Session()
				self.session.headers.update(
				{
				"Accept": "application/json",
				"User-Agent": USER_AGENT,
				}
				)

				@throttling_retry(before_sleep=before_sleep_log(logger, logging.WARNING))
				def page_request(self, url: str, params: Dict[str, Any]) -> requests.Response:

				logger.info("Fetching URL %s with params %s", url, params)

				response = self.session.get(url, params=params)
				if response.status_code != 200:
				logger.warning(
				"Unexpected HTTP status code %s on %s: %s",
				response.status_code,
				response.url,
				response.content,
				)
				response.raise_for_status()

				return response

				def get_pages(self) -> Iterator[PubDevListerPage]:
				"""Yield an iterator which returns 'page'

				It uses the api provided by https://pub.dev/api/ to find Dart and Flutter package
				origins.

				The http api call get "{base_url}package-names" to retrieve a sorted list
				of all package names.

				There is only one page that list all origins url based on "{base_url}packages/{pkgname}"
				"""
				response = self.page_request(
				url=self.PACKAGE_NAMES_URL_PATTERN.format(base_url=self.url), params={}
				)
				yield response.json()["packages"]

				def get_origins_from_page(self, page: PubDevListerPage) -> Iterator[ListedOrigin]:
				"""Iterate on all pages and yield ListedOrigin instances."""
				assert self.lister_obj.id is not None
				vlorentzUnsubmitted Done Inline Actions I feel it would make more sense to move this loop to `get_origins_from_page`, since we only have one page. (Not a big deal, in the end the result is the same) vlorentz: I feel it would make more sense to move this loop to `get_origins_from_page`, since we only…

				for pkgname in page:
				url = self.PACKAGE_INFO_URL_PATTERN.format(
				base_url=self.url, pkgname=pkgname
				)
				yield ListedOrigin(
				lister_id=self.lister_obj.id,
				visit_type=self.VISIT_TYPE,
				url=url,
				last_update=None,
				)
				vlorentzUnsubmitted Done Inline Actions `last_update` should be `None` when the lister doesn't know when the package was last updated. Setting a non-`None` value would make the scheduler believe we know for sure there is an update, so it would prioritize loading this origin over others. vlorentz: `last_update` should be `None` when the lister doesn't know when the package was last updated.