Skip to content

Commit afaec20

Browse files
Merge pull request #134 from JunAishima/handle-bnlpeople-exceptions
bnlpeople query failures fail more gracefully
2 parents 897e7a1 + f0096ab commit afaec20

1 file changed

Lines changed: 6 additions & 1 deletion

File tree

src/nsls2api/services/bnlpeople_service.py

Lines changed: 6 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -35,7 +35,12 @@ async def get_username_by_id(lifenumber: str) -> Optional[str]:
3535

3636
url = f"{base_url}/api/BNLPeople?employeeNumber={lifenumber}"
3737
logger.debug(f"Calling URL: {url}")
38-
person = await _call_bnlpeople_webservice(url)
38+
try:
39+
person = await _call_bnlpeople_webservice(url)
40+
except Exception as error:
41+
message = f"BNL People API query failed for lifenumber {lifenumber}"
42+
logger.exception(message)
43+
return None
3944
logger.debug(person)
4045
if len(person) == 0 or len(person) > 1:
4146
logger.warning(

0 commit comments

Comments
 (0)