nextcloud / nextcloud/calendar
Calender subscription URL not stripped
Nobody has claimed this yet.
- Dominant language
- JavaScript
- Stars
- 1.2k
- Forks
- 332
- Avg merge
- 16h 13m
- Merged PRs (30d)
- 137
Description
Steps to reproduce
- Choose iCal URL to import
- Click "New subscription from link"
- 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
Contributor guide
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
Research direction
Reproduce the issue through the “New subscription from link” flow using a URL prefixed with spaces, then inspect the value stored in the oc_calendarsubscriptions table. Trace the subscription import entry point and verify that the persisted URL no longer contains the leading spaces, then test calendar syncing with DAVx5/ICSx5.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- javascript
- Domain
- backend, databases
- Issue type
- Bug
- Difficulty
- 2/5
- Estimated time
- 1-3 hours
- Activity status
- Stale
- Clarity
- Mostly clear
- Newbie friendliness
- 45/100