search

mikedeboer / jsDAV

jsDAV allows you to easily add WebDAV support to a NodeJS application. jsDAV is meant to cover the entire standard, and attempts to allow integration using an easy to understand API.
http://www.mikedeboer.nl
MIT License
681 stars 159 forks source link

fix calendarquery with windows phone #128

Closed aenario closed 8 years ago

aenario commented 9 years ago

Windows Phone does a 

http REPORT https://xxxxxx/public/sync/calendars/username/calendarname/ 
'Depth:0' 'Content-type:text/xml'
<?xml version="1.0" encoding="UTF-8"?><calendar-query xmlns="urn:ietf:params:xml:ns:caldav"><prop xmlns="DAV:"><getetag /></prop><filter><comp-filter name="VCALENDAR"><comp-filter name="VEVENT"><time-range start="20150512T000000Z" end="20151110T000000Z" /></comp-filter></comp-filter></filter></calendar-query>

, which is not caught by either condition. Moreover, this case is not handled, so request is stuck until it timeouts.

Ping @Olegas for review.

mikedeboer commented 8 years ago

Alright, this looks good to me - stealing review from @Olegas