Skip to content

Commit

Permalink
Merge pull request #1155 from lakridserne/remove_exception_logging
Browse files Browse the repository at this point in the history
Don't output errors about address - we will find another solutiaton
  • Loading branch information
lakridserne authored Jan 12, 2025
2 parents 41deffc + a1f7958 commit 26f278e
Showing 1 changed file with 2 additions and 7 deletions.
9 changes: 2 additions & 7 deletions members/models/person.py
Original file line number Diff line number Diff line change
Expand Up @@ -145,15 +145,12 @@ def update_dawa_data(self):
url = f"https://api.dataforsyningen.dk/adresser?q={quote_plus(self.addressWithZip())}"
response = requests.get(url)
if response.status_code != 200:
logger.error(
f"Failed to look up address for {self.name} ({response.status_code})"
)
self.address_invalid = True
self.save()
return None

data = response.json()
if not data:
logger.error(f"DAWA returned empty address result for {self.name}")
self.address_invalid = True
self.save()
return None
Expand Down Expand Up @@ -186,9 +183,7 @@ def update_dawa_data(self):
self.save()
logger.info(f"Updated address for {self.name}")

except Exception as error:
logger.error(f"Exception when looking up address for {self.name}")
logger.error(f"Error {error}")
except Exception:
self.address_invalid = True
self.save()

Expand Down

0 comments on commit 26f278e

Please sign in to comment.