Skip to content

Commit 1cfabc5

Browse files
committed
feat: Add logger to API calls
1 parent 12bf9cd commit 1cfabc5

File tree

6 files changed

+19
-13
lines changed

6 files changed

+19
-13
lines changed

backend/app/apis/managers/mvn_service.py

Lines changed: 5 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -4,6 +4,8 @@
44
from aiohttp import ClientConnectorError, ClientSession
55
from xmltodict import parse
66

7+
from app.logger import logger
8+
79

810
async def get_all_mvn_versions(
911
package_artifact_id: str, package_group_id: str
@@ -12,6 +14,7 @@ async def get_all_mvn_versions(
1214
async with ClientSession() as session:
1315
while True:
1416
try:
17+
logger.info(f"MAVEN - https://repo1.maven.org/maven2/{package_group_id.replace(".", "/")}/{package_artifact_id}/maven-metadata.xml")
1518
async with session.get(f"https://repo1.maven.org/maven2/{package_group_id.replace(".", "/")}/{package_artifact_id}/maven-metadata.xml") as response:
1619
xml_string = await response.text()
1720
break
@@ -38,16 +41,12 @@ async def requires_mvn_packages(
3841
async with ClientSession() as session:
3942
while True:
4043
try:
44+
logger.info(f"MAVEN - https://repo1.maven.org/maven2/{group_id}/{package_artifact_id}/{version_dist}/{package_artifact_id}-{version_dist}.pom")
4145
async with session.get(f"https://repo1.maven.org/maven2/{group_id}/{package_artifact_id}/{version_dist}/{package_artifact_id}-{version_dist}.pom") as response:
4246
xml_string = await response.text()
4347
break
4448
except (ClientConnectorError, TimeoutError):
45-
try:
46-
async with session.get(f"https://search.maven.org/remotecontent?filepath={group_id}/{package_artifact_id}/{version_dist}/{package_artifact_id}-{version_dist}.pom") as response:
47-
xml_string = await response.text()
48-
break
49-
except (ClientConnectorError, TimeoutError):
50-
await sleep(5)
49+
await sleep(5)
5150
try:
5251
pom_dict = parse(xml_string)
5352
except Exception as _:

backend/app/apis/managers/npm_service.py

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -4,11 +4,14 @@
44

55
from aiohttp import ClientConnectorError, ClientSession
66

7+
from app.logger import logger
8+
79

810
async def get_all_npm_versions(pkg_name: str) -> Any:
911
async with ClientSession() as session:
1012
while True:
1113
try:
14+
logger.info(f"NPM - https://registry.npmjs.org/{pkg_name}")
1215
async with session.get(f"https://registry.npmjs.org/{pkg_name}") as response:
1316
response = await response.json()
1417
break

backend/app/apis/managers/pip_service.py

Lines changed: 5 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -7,12 +7,15 @@
77

88
from app.utils import get_first_position, parse_pip_constraints
99

10+
from app.logger import logger
11+
1012

1113
# TODO: En las nuevas actualizaciones de la API JSON se debería devolver la info de forma diferente, estar atento a nuevas versiones.
1214
async def get_all_pip_versions(pkg_name: str) -> list[dict[str, Any]]:
1315
async with ClientSession() as session:
1416
while True:
1517
try:
18+
logger.info(f"PyPI - https://pypi.python.org/pypi/{pkg_name}/json")
1619
async with session.get(f"https://pypi.python.org/pypi/{pkg_name}/json") as response:
1720
response = await response.json()
1821
break
@@ -40,14 +43,15 @@ async def requires_pip_packages(
4043
async with ClientSession() as session:
4144
while True:
4245
try:
46+
logger.info(f"PyPI - https://pypi.python.org/pypi/{pkg_name}/{version_dist}/json")
4347
async with session.get(f"https://pypi.python.org/pypi/{pkg_name}/{version_dist}/json") as response:
4448
response = await response.json()
4549
break
4650
except (ClientConnectorError, TimeoutError):
4751
await sleep(5)
4852
except JSONDecodeError:
4953
return {}
50-
if response and "info" in response and "requires_dist" in response["info"]:
54+
if response and "info" in response and "requires_dist" in response["info"] and response["info"]["requires_dist"]:
5155
require_packages: dict[str, Any] = {}
5256
for dependency in response["info"]["requires_dist"]:
5357
data = dependency.split(";")

backend/app/exception_handler.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -27,7 +27,7 @@ async def unhandled_exception_handler(request: Request, exc: Exception) -> Plain
2727
host = getattr(getattr(request, "client", None), "host", None)
2828
port = getattr(getattr(request, "client", None), "port", None)
2929
url = f"{request.url.path}?{request.query_params}" if request.query_params else request.url.path
30-
exception_type, exception_value, exception_traceback = sys.exc_info()
30+
exception_type, exception_value, _ = sys.exc_info()
3131
exception_name = getattr(exception_type, "__name__", None)
3232
logger.error(
3333
f'{host}:{port} - "{request.method} {url}" 500 Internal Server Error <{exception_name}: {exception_value}>'

backend/requirements.txt

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -6,9 +6,9 @@ apscheduler==3.10.4
66
motor==3.6.0
77
python-dotenv==1.0.1
88
python-dateutil==2.9.0
9-
flamapy-smt==1.3.3
9+
flamapy-smt==1.3.5
1010
neo4j==5.20.0
11-
univers==30.11.0
11+
univers==30.12.1
1212
xmltodict==0.13.0
1313
GitPython==3.1.43
1414
toml==0.10.2

frontend/src/repositoriesPage/repositories.js

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -77,8 +77,8 @@ const RepositoriesPage = () => {
7777
})
7878
}
7979
fetchRepositories()
80-
const intervalId = setInterval(fetchRepositories, 10000)
81-
return () => clearInterval(intervalId)
80+
// const intervalId = setInterval(fetchRepositories, 10000)
81+
// return () => clearInterval(intervalId)
8282
}, [])
8383

8484
const handle_change_page = (newPage) => {
@@ -153,7 +153,7 @@ const RepositoriesPage = () => {
153153
>
154154
<Box className='text-gray-500 text-center border-blue-500 border-2 bg-white rounded-lg shadow'>
155155
<Typography id='modal-modal-title' variant='h6' component='h2'>
156-
Aply reasoning operations for requirement file {requirementFilesRow.name}
156+
Apply reasoning operations for requirement file {requirementFilesRow.name}
157157
</Typography>
158158
<OperationsPage
159159
id='modal-modal-description'

0 commit comments

Comments
 (0)