v33 第 33 批：DAV（Thunderbird）三处真根因修复
用户原话： 看看 dav 功能，还是不好用，thunderbird 提示这两个地址下没有任何联系人和日程。 关键教训：v31 那个探针（复刻 SyncCalendar 的 DavClient）线上 28/0 全过， 但用户还是空的 —— 因为 Thunderbird 的请求形状不一样：它真正取数据用的是 REPORT（calendar-multiget / addressbook-multiget），而 v31 探针只走 PROPFIND + GET。复刻了 A 客户端全绿，证明不了 B 客户端能看见东西。 三处真根因（都靠新探针 _fb_dav_tb.py 逐步 dump 原始 XML 抓出）： ① CardDAV 的 REPORT 回的是草案名 <card:address-data>，而 RFC 6352 规定是 <card:addressbook-data>。Thunderbird 要后者 → 拿到 0 个数据块 → 通讯录永远空。 修：新增 $cardDataXml 闭包，照客户端点名的属性回正文（没点名默认正式名）。 注：第一版写成两个都发 → 正文重复一份、e2e 的 BEGIN:VCARD 计数翻倍，改成按需回。 ② home-set 的 Depth:1 响应里没有集合自身。RFC 4918 要求 PROPFIND 响应必须包含 被请求的资源本身，原来只在非 home 请求时才输出 → PROPFIND /dav/calendars/ 只回子本、没有自己 → 严格客户端判 home 不存在 → 一本都看不见。 修：dav.php / carddav.php 的 home 分支始终先输出集合自身。 ③ 站点根 PROPFIND / 被 Apache 交给 index.php → 302 跳登录页 → Thunderbird 判定 这个域名下没有日历（RFC 6764 允许从域名根发现）。 修：router.php（本地）+ .htaccess / 部署脚本 5c 段独立标记块（线上）： RewriteCond 限住 REQUEST_METHOD + RewriteRule ^$ dav.php； 必须限方法，否则 GET / 被 ^$ 吞掉、首页 404（已加反向守卫）。 并让根响应与两个 principal 都给出两套 home-set（calendar + addressbook）。 验证：线上真机 _fb_dav_tb.py 25/0（站点根通 / home-set 含自身 / CardDAV REPORT 真带回 3 个数据块，修复前是 0）；逐本 REPORT 所有集合都能取回数据； 旧诊断（SyncCalendar 序列）28/0 无回归；GET / 仍是 302 未被吞。 本地 e2e 1503/0、v33 DAV 反向验证 7/0（三处各自改回旧行为 → 断言全变红）； 部署脚本静态断言段本地空跑通过。
This commit is contained in:
+52
-6
@@ -196,8 +196,27 @@ if ($method === 'PROPFIND') {
|
||||
// ⚠️ `/carddav/` 也走这条路：用户与老客户端都直接把 /carddav/ 当 home 用。
|
||||
if ($isHome || $bookId === 0) {
|
||||
$parts = [];
|
||||
// 首次请求 `/carddav/`（不是 home 路径）时，把发现链的答案一并给出 ——
|
||||
// 有些客户端在这里**同时**问 current-user-principal，少给一个它就又断了。
|
||||
// 🔴 v33：**始终**先输出 home 集合自身。
|
||||
// RFC 4918 规定：PROPFIND 的响应**必须包含被请求的资源本身**，
|
||||
// 哪怕 Depth:1 也一样 —— 只有先有"这个集合存在"的证据，
|
||||
// 客户端才会继续读它的成员。原来只在 `!$isHome`（即请求 `/carddav/`）时
|
||||
// 才输出，于是 `PROPFIND /carddav/addressbooks/`（Thunderbird 走的就是这条）
|
||||
// 只回子集、没有自己 ⇒ 严格客户端判定"这个 home 不存在" ⇒ 通讯录空。
|
||||
// `/carddav/`（不是 home 路径）那一次附带发现链三件套（保持原样）。
|
||||
$parts[] = '<d:response><d:href>' . CardDav::x(CardDav::HOME_PATH) . '</d:href>'
|
||||
. '<d:propstat><d:prop>'
|
||||
. '<d:resourcetype><d:collection/></d:resourcetype>'
|
||||
. '<d:displayname>' . CardDav::x('家书通讯录') . '</d:displayname>'
|
||||
. '<d:current-user-principal><d:href>' . CardDav::x(CardDav::PRINCIPAL_PATH)
|
||||
. '</d:href></d:current-user-principal>'
|
||||
. '<d:principal-URL><d:href>' . CardDav::x(CardDav::PRINCIPAL_PATH)
|
||||
. '</d:href></d:principal-URL>'
|
||||
. '<card:addressbook-home-set><d:href>' . CardDav::x(CardDav::HOME_PATH)
|
||||
. '</d:href></card:addressbook-home-set>'
|
||||
. $readOnlyPriv
|
||||
. '</d:prop></d:propstat></d:response>';
|
||||
// 首次请求 `/carddav/`（不是 home 路径）时，另加一条指向 `/carddav/` 自身的响应，
|
||||
// 客户端会拿它当后续请求的 base。
|
||||
if (!$isHome) {
|
||||
$parts[] = '<d:response><d:href>/carddav/</d:href>'
|
||||
. '<d:propstat><d:prop>'
|
||||
@@ -327,6 +346,31 @@ if ($method === 'REPORT') {
|
||||
}
|
||||
$bodyRaw = (string)file_get_contents('php://input');
|
||||
|
||||
/* 🔴 v33：vCard 正文的属性名要**照客户端点名的那个**回。
|
||||
· `<card:addressbook-data>` 是 **RFC 6352** 的正式名（Thunderbird / Evolution 要它）；
|
||||
· `<card:address-data>` 是早期草案名（老客户端可能还在用）。
|
||||
原来只发旧名 → Thunderbird 收到 0 个数据块、**通讯录永远是空的**
|
||||
（用户报「thunderbird 提示没有任何联系人」的真根因）。
|
||||
⚠️ 别两个都无脑发：正文会重复一份，客户端侧"卡数"翻倍
|
||||
（e2e 第22条 `BEGIN:VCARD` 计数当场抓到）。
|
||||
⚠️ 客户端没点名时（如 `addressbook-query` 空 body）默认发**正式名**。 */
|
||||
$cardDataXml = static function (array $card) use ($bodyRaw): string {
|
||||
$wantAB = stripos($bodyRaw, 'addressbook-data') !== false;
|
||||
$wantAD = stripos($bodyRaw, 'address-data') !== false;
|
||||
if (!$wantAB && !$wantAD) {
|
||||
$wantAB = true; // 没点名 → RFC 正式名
|
||||
}
|
||||
$body = CardDav::x($card['body'] . "\r\n"); // 结尾 CRLF 只加在正文，不参与 ETag
|
||||
$xml = '';
|
||||
if ($wantAB) {
|
||||
$xml .= '<card:addressbook-data>' . $body . '</card:addressbook-data>';
|
||||
}
|
||||
if ($wantAD) {
|
||||
$xml .= '<card:address-data>' . $body . '</card:address-data>';
|
||||
}
|
||||
return $xml;
|
||||
};
|
||||
|
||||
// 单张卡请求（/carddav/1.vcf/12.vcf）
|
||||
if ($cardId > 0) {
|
||||
$one = CardDav::one($familyId, $me, $cardId);
|
||||
@@ -336,15 +380,17 @@ if ($method === 'REPORT') {
|
||||
/* ⚠️ 正文与 ETag 都从 cardOf 取（**同一处**算），别在这里另算一次 ——
|
||||
v24 第 22 批实测：PROPFIND 用不带结尾 CRLF 的正文算 ETag、这里用带 CRLF 的算，
|
||||
同一张卡两个 ETag → 客户端认为它一直在变、反复重拉（同步不收敛）。
|
||||
结尾那条 CRLF 只加在**正文**（address-data 要有完整的 vCard 文本），
|
||||
不参与 ETag。 */
|
||||
结尾那条 CRLF 只加在**正文**（数据属性要有完整的 vCard 文本），
|
||||
不参与 ETag。
|
||||
|
||||
🔴 v33：属性名由 `$cardDataXml` 按客户端点名决定（见上面的闭包）。 */
|
||||
$card = CardDav::cardOf($one);
|
||||
$emit([
|
||||
'<d:response><d:href>'
|
||||
. CardDav::x(CardDav::bookHref($bookId) . $cardId . '.vcf') . '</d:href>'
|
||||
. '<d:propstat><d:prop>'
|
||||
. '<d:getetag>' . CardDav::x($card['etag']) . '</d:getetag>'
|
||||
. '<card:address-data>' . CardDav::x($card['body'] . "\r\n") . '</card:address-data>'
|
||||
. $cardDataXml($card)
|
||||
. '</d:prop></d:propstat></d:response>',
|
||||
]);
|
||||
exit;
|
||||
@@ -381,7 +427,7 @@ if ($method === 'REPORT') {
|
||||
. CardDav::x(CardDav::bookHref($bookId) . $rid . '.vcf') . '</d:href>'
|
||||
. '<d:propstat><d:prop>'
|
||||
. '<d:getetag>' . CardDav::x($card['etag']) . '</d:getetag>'
|
||||
. '<card:address-data>' . CardDav::x($card['body'] . "\r\n") . '</card:address-data>'
|
||||
. $cardDataXml($card)
|
||||
. '</d:prop></d:propstat></d:response>';
|
||||
}
|
||||
unset($bodyRaw);
|
||||
|
||||
Reference in New Issue
Block a user