[WEB-4380] chore: work item link sentry error (#7316)
* chore: add warning param in log_exception:'' * fix: logging the exception
This commit is contained in:
parent
f40dda8fdc
commit
16332e0f6d
2 changed files with 9 additions and 6 deletions
|
|
@ -19,6 +19,7 @@ logger = logging.getLogger("plane.worker")
|
||||||
|
|
||||||
DEFAULT_FAVICON = "PHN2ZyB4bWxucz0iaHR0cDovL3d3dy53My5vcmcvMjAwMC9zdmciIHdpZHRoPSIyNCIgaGVpZ2h0PSIyNCIgdmlld0JveD0iMCAwIDI0IDI0IiBmaWxsPSJub25lIiBzdHJva2U9ImN1cnJlbnRDb2xvciIgc3Ryb2tlLXdpZHRoPSIyIiBzdHJva2UtbGluZWNhcD0icm91bmQiIHN0cm9rZS1saW5lam9pbj0icm91bmQiIGNsYXNzPSJsdWNpZGUgbHVjaWRlLWxpbmstaWNvbiBsdWNpZGUtbGluayI+PHBhdGggZD0iTTEwIDEzYTUgNSAwIDAgMCA3LjU0LjU0bDMtM2E1IDUgMCAwIDAtNy4wNy03LjA3bC0xLjcyIDEuNzEiLz48cGF0aCBkPSJNMTQgMTFhNSA1IDAgMCAwLTcuNTQtLjU0bC0zIDNhNSA1IDAgMCAwIDcuMDcgNy4wN2wxLjcxLTEuNzEiLz48L3N2Zz4=" # noqa: E501
|
DEFAULT_FAVICON = "PHN2ZyB4bWxucz0iaHR0cDovL3d3dy53My5vcmcvMjAwMC9zdmciIHdpZHRoPSIyNCIgaGVpZ2h0PSIyNCIgdmlld0JveD0iMCAwIDI0IDI0IiBmaWxsPSJub25lIiBzdHJva2U9ImN1cnJlbnRDb2xvciIgc3Ryb2tlLXdpZHRoPSIyIiBzdHJva2UtbGluZWNhcD0icm91bmQiIHN0cm9rZS1saW5lam9pbj0icm91bmQiIGNsYXNzPSJsdWNpZGUgbHVjaWRlLWxpbmstaWNvbiBsdWNpZGUtbGluayI+PHBhdGggZD0iTTEwIDEzYTUgNSAwIDAgMCA3LjU0LjU0bDMtM2E1IDUgMCAwIDAtNy4wNy03LjA3bC0xLjcyIDEuNzEiLz48cGF0aCBkPSJNMTQgMTFhNSA1IDAgMCAwLTcuNTQtLjU0bC0zIDNhNSA1IDAgMCAwIDcuMDcgNy4wN2wxLjcxLTEuNzEiLz48L3N2Zz4=" # noqa: E501
|
||||||
|
|
||||||
|
|
||||||
def crawl_work_item_link_title_and_favicon(url: str) -> Dict[str, Any]:
|
def crawl_work_item_link_title_and_favicon(url: str) -> Dict[str, Any]:
|
||||||
"""
|
"""
|
||||||
Crawls a URL to extract the title and favicon.
|
Crawls a URL to extract the title and favicon.
|
||||||
|
|
@ -118,7 +119,7 @@ def find_favicon_url(soup: Optional[BeautifulSoup], base_url: str) -> Optional[s
|
||||||
if response.status_code == 200:
|
if response.status_code == 200:
|
||||||
return fallback_url
|
return fallback_url
|
||||||
except requests.RequestException as e:
|
except requests.RequestException as e:
|
||||||
log_exception(e)
|
log_exception(e, warning=True)
|
||||||
return None
|
return None
|
||||||
|
|
||||||
return None
|
return None
|
||||||
|
|
|
||||||
|
|
@ -6,13 +6,15 @@ import traceback
|
||||||
from django.conf import settings
|
from django.conf import settings
|
||||||
|
|
||||||
|
|
||||||
def log_exception(e):
|
def log_exception(e, warning=False):
|
||||||
# Log the error
|
# Log the error
|
||||||
logger = logging.getLogger("plane.exception")
|
logger = logging.getLogger("plane.exception")
|
||||||
|
|
||||||
|
if warning:
|
||||||
|
logger.warning(str(e))
|
||||||
|
else:
|
||||||
logger.exception(e)
|
logger.exception(e)
|
||||||
|
|
||||||
if settings.DEBUG:
|
if settings.DEBUG:
|
||||||
# Print the traceback if in debug mode
|
logger.debug(traceback.format_exc())
|
||||||
print(traceback.format_exc())
|
|
||||||
|
|
||||||
return
|
return
|
||||||
|
|
|
||||||
Loading…
Add table
Add a link
Reference in a new issue