Changes in / [09fc5fb:f549cf5] in Klonkt

Files:

• src/routes/feed.js (modified) (4 diffs)
• src/services/guardianship/queues.js (modified) (1 diff)
• test/feed-tracks-only.test.js (added)
• test/help-c2s.test.js (modified) (2 diffs)

src/routes/feed.js

@@ -44 +44 @@
 
 // ==================== RSS 2.0 ====================
-router.get('/feed.xml', (req, res) => {
+/**
+ * Twee feeds, een bouwer.
+ *
+ *   /feed.xml    de site: posts EN open tracks door elkaar, chronologisch
+ *   /tracks.xml  alleen de open tracks -- de hele site als muziekkanaal
+ *
+ * WAAROM DIE TWEEDE (Robins vraag, 10-8). Hij vroeg naar een "hele site, open
+ * tracks"-library voor Funkwhale. Als AS2-object helpt zo'n ding daar niet: een
+ * Funkwhale-KANAAL wijst nergens naar een library, en hun library is een eigen
+ * actor met inbox en sleutel -- de optie die op 7 augustus is afgewezen. Maar de
+ * vorm waarin Funkwhale een kanaal WEL uitgeeft is RSS met iTunes en een
+ * enclosure per item, en dat is precies wat Klonkt hier al doet. Naast een echte
+ * kanaalfeed gelegd (audio.pepemoss.com/api/v1/channels/tnd/rss): dezelfde
+ * namespaces, dezelfde enclosure.
+ *
+ * Het enige verschil was dat onze feed ook gewone posts draagt. Een muziekkanaal
+ * met blogberichten ertussen is er geen, dus die feed staat er nu apart -- een
+ * URL om te plakken, zonder dat er iets naar buiten geduwd wordt.
+ */
+function stuurFeed(req, res, { alleenTracks }) {
   const site = res.locals.site;
   if (!site) return res.status(404).send('No site');
@@ -50 +69 @@
   const origin = siteOrigin(req);
   const base = origin + (res.locals.siteUrlBase || '');
-  const posts = postsForFeed(site.id);
+  const posts = alleenTracks ? [] : postsForFeed(site.id);
 
   // De tracks die deze site aan de federatie heeft opengezet, elk als eigen
@@ -65 +84 @@
   // poortregel en die hoort op een plek te staan.
   const tracks = siteOpenTracks(site.id);
-  const lastBuild = posts[0]?.published_at || new Date().toISOString();
+  const lastBuild = posts[0]?.published_at || tracks[0]?.created_at || new Date().toISOString();
 
   // De itunes-velden waar een podcast-app een kanaal aan herkent. Funkwhale
@@ -117 +136 @@
 <rss version="2.0" xmlns:atom="http://www.w3.org/2005/Atom" xmlns:content="http://purl.org/rss/1.0/modules/content/" xmlns:itunes="http://www.itunes.com/dtds/podcast-1.0.dtd">
   <channel>
-    <title>${escapeXml(site.title)}</title>
+    <title>${escapeXml(alleenTracks ? `${site.title} \u2014 muziek` : site.title)}</title>
     <link>${escapeXml(base + '/')}</link>
     <description>${escapeXml(site.description || site.tagline || '')}</description>
     <language>${escapeXml(site.language || 'nl')}</language>
     <lastBuildDate>${new Date(lastBuild).toUTCString()}</lastBuildDate>
-    <atom:link href="${escapeXml(base + '/feed.xml')}" rel="self" type="application/rss+xml" />
+    <atom:link href="${escapeXml(base + (alleenTracks ? '/tracks.xml' : '/feed.xml'))}" rel="self" type="application/rss+xml" />
 ${kanaalTags}
 ${items}
   </channel>
 </rss>`);
-});
+}
+
+router.get('/feed.xml', (req, res) => stuurFeed(req, res, { alleenTracks: false }));
+// De hele site als muziekkanaal: alleen wat op de federatie openstaat.
+router.get('/tracks.xml', (req, res) => stuurFeed(req, res, { alleenTracks: true }));
 
 // ==================== Atom 1.0 ====================

src/services/guardianship/queues.js

@@ -294 +294 @@
     'shaer:handledAt': h.state.handled ? h.state.handled.at : undefined,
     'shaer:pickedUpBy': h.state.pickedUpBy.map((p) => p.handle || p.uri),
+    // HOE OUD het oudste oppakken is (shaer-lgo). Barts besluit was dat
+    // "opgepikt" niet vervalt maar zichtbaar VEROUDERT -- het verschil tussen
+    // "er is iemand mee bezig" en "er was ooit iemand mee bezig". Het paneel
+    // toonde dat al; de apps konden het niet, want dit veld bleef hier liggen.
+    // Een oppak van vijf minuten zag er daar uit als een van vijf dagen, en de
+    // faalstand is hier nou juist "iedereen denkt dat het geregeld is".
+    //
+    // Als TIJDSTIP en niet als leeftijd: een leeftijd maakt elk antwoord anders
+    // en dan kan de ETag nooit gelijk zijn. De client rekent zelf terug, precies
+    // zoals guardian.js het doet.
+    'shaer:oldestPickupAt': h.state.oldestPickupAt || undefined,
     'shaer:formerWard': h.state.formerWard || undefined,
   }));

test/help-c2s.test.js

@@ -60 +60 @@
 });
 
+test('en de apps krijgen te horen HOE OUD dat oppakken is', () => {
+  // Barts besluit: opgepikt vervalt niet maar veroudert zichtbaar -- het
+  // verschil tussen "er is iemand mee bezig" en "er was ooit iemand mee bezig".
+  // Het paneel toonde dat al uit dezelfde staat; de apps kregen het veld niet,
+  // dus daar zag een oppak van vijf minuten eruit als een van vijf dagen.
+  const coll = queues.helpCollection('https://x/queues/help', 'oma');
+  const item = coll.orderedItems.find((i) => i.id === BOEI);
+  assert.ok(item['shaer:oldestPickupAt'], 'het veld staat er');
+  assert.ok(!Number.isNaN(Date.parse(item['shaer:oldestPickupAt'])), 'en is een leesbaar tijdstip');
+  // Een TIJDSTIP en geen leeftijd: een leeftijd maakt elk antwoord anders en
+  // dan kan de ETag nooit gelijk zijn. De client rekent zelf terug.
+  assert.equal(typeof item['shaer:oldestPickupAt'], 'string');
+});
+
 test('afsluiten vanuit de app sluit hem ook echt', async () => {
   await AP.ingestOutboxActivity(site(), user, marker('handled'));
@@ -75 +89 @@
   assert.ok(item['shaer:handledBy']);
   assert.equal(item['shaer:helpRequest'], true);
+  // En de ouderdom is WEG zodra hij is afgehandeld: hoe lang er iemand naar
+  // keek is dan geen informatie meer, en zou als "er wacht nog iets" lezen.
+  assert.equal(item['shaer:oldestPickupAt'], undefined);
 });