home-assistant / core

:house_with_garden: Open source home automation that puts local control and privacy first.
https://www.home-assistant.io
Apache License 2.0
72.72k stars 30.45k forks source link

CalDAV - SyntaxWarning: invalid escape sequence #115697

Open rgerbranda opened 6 months ago

rgerbranda commented 6 months ago

The problem

Since a recent update I have SyntaxWarnings in the logs, probably caused by CalDAV component

What version of Home Assistant Core has the issue?

core-2024.4.3

What was the last working version of Home Assistant Core?

core-2024.3

What type of installation are you running?

Home Assistant Container

Integration causing the issue

CalDAV

Link to integration documentation on our website

https://www.home-assistant.io/integrations/caldav/

Diagnostics information

No response

Example YAML snippet

calendar:
  - platform: caldav
    username: !secret apple_id
    password: !secret apple_password
    url: https://caldav.icloud.com
    calendars:
      - "Private"
      - "Birthdays"

Anything in the logs that might be useful for us?

2024-04-16 11:19:36.733 WARNING (ImportExecutor_0) [py.warnings] /usr/local/lib/python3.12/site-packages/vobject/base.py:736: SyntaxWarning: invalid escape sequence '\-'
  patterns['name'] = '[a-zA-Z0-9\-_]+'

2024-04-16 11:19:36.743 WARNING (ImportExecutor_0) [py.warnings] /usr/local/lib/python3.12/site-packages/vobject/base.py:1219: SyntaxWarning: invalid escape sequence '\;'
  s = s.replace("\\", "\\\\").replace(";", "\;").replace(",", "\,")

2024-04-16 11:19:36.744 WARNING (ImportExecutor_0) [py.warnings] /usr/local/lib/python3.12/site-packages/vobject/base.py:1219: SyntaxWarning: invalid escape sequence '\,'
  s = s.replace("\\", "\\\\").replace(";", "\;").replace(",", "\,")

Additional information

No response

home-assistant[bot] commented 6 months ago

caldav documentation caldav source

jackwilsdon commented 5 months ago

Seems like this has already been fixed in vobject: https://github.com/py-vobject/vobject/pull/21

But there hasn't been a release since this fix (so caldav hasn't pulled it in yet either).

issue-triage-workflows[bot] commented 2 months ago

There hasn't been any activity on this issue recently. Due to the high number of incoming GitHub notifications, we have to clean some of the old issues, as many of them have already been resolved with the latest updates. Please make sure to update to the latest Home Assistant version and check if that solves the issue. Let us know if that works for you by adding a comment 👍 This issue has now been marked as stale and will be closed if no further activity occurs. Thank you for your contributions.

rwjack commented 2 months ago

bad bot