nextcloud / calendar

📆 Calendar app for Nextcloud
https://apps.nextcloud.com/apps/calendar
GNU Affero General Public License v3.0
947 stars 231 forks source link

Calender subscription URL not stripped #6061

Open p-fruck opened 3 weeks ago

p-fruck commented 3 weeks ago

Steps to reproduce

  1. Choose iCal URL to import
  2. Click "New subscription from link"
  3. Import the URL, but prefix it with a couple of spaces

Expected behavior

The URL should be stripped and written to the database

Actual behaviour

The URL is not stripped which can be seen by executing select * from oc_calendarsubscriptions in the database. This causes issues when syncing the calendar on Android using DAVx5/ICSx5 as shown in this discussion

Calendar app version

4.7.6

CalDAV-clients used

DAVx5/ICSx5

Browser

N/A

Client operating system

Android 14

Server operating system

No response

Web server

None

Database engine version

None

PHP engine version

None

Nextcloud version

No response

Updated from an older installed version or fresh install

None

List of activated apps

No response

Nextcloud configuration

No response

Web server error log

No response

Log file

No response

Browser log

No response

Additional info

No response

tcitworld commented 3 weeks ago

Needs to be fixed in both front-end and back-end (server repo).