Productkaarten in Best Buy Guides - Development-iteratie #260

In sprint #260 hebben we de Best Buy Guides uitgebreid met productkaarten, waarmee de redactie aanraders en alternatieven beter kan uitlichten. Verder hebben we een nieuw Ceph-cluster in gebruik genomen.

Productkaarten in Best Buy Guides

Begin dit jaar hebben we door middel van een prototype onderzocht hoe we de Best Buy Guides kunnen verbeteren. Daaruit hebben we veel kwantitatieve inzichten verkregen, naast de kwalitatieve feedback die jullie hebben ingestuurd, waarvoor dank. Een van de inzichten die dit prototype heeft opgeleverd, is de wenselijkheid om aanraders en alternatieven in Best Buy Guides beter uit te lichten door middel van een productkaart. Na een aantal aanvullende a/b-tests is het nu mogelijk om productkaarten in Best Buy Guides te plaatsen. Dit is te zien in de nieuwste BBG: Laptop Best Buy Guide - Juni 2023.

De productkaart toont een beknopt overzicht van het uitgelichte product met onder andere specificaties, een toelichting van de redactie en een koppeling met de Pricewatch. Nieuw is de 'directe click-out' naar de webshop die het product voor de laagste (basis)prijs in de Pricewatch aanbiedt. Prijzen vergelijken en eventuele filters toepassen kunnen ook nog steeds, via de link 'Bekijk alle prijzen'.

Nieuw Ceph-cluster in gebruik genomen

Ook op het gebied van hosting is er het een en ander veranderd. Zo hebben we een nieuw Ceph-cluster in gebruik genomen, bestaande uit zes Dell R650-servers, met daarin vier ssd's van 3,5TB. Deze servers zijn daarnaast voorzien van een twaalfcore-Intel Silver Xeon en 128GB DDR4-geheugen. Ze zijn met 25Gbit/s verbonden aan een (nieuw) netwerk, dat bestaat uit Juniper EX4650-switches met 48x 10/25Gbit/s- en 8x 100Gbit/s-poorten.

We gebruiken dit Ceph-cluster voor de opslag van redactionele afbeeldingen, video's, plaatjes die gebruikers uploaden en VM-images. In totaal is er nu zo'n 13TiB aan data opgeslagen in 18 miljoen objects, wat met een replicatiefactor van 3 neer komt op een kleine 40TiB aan gebruikte ruimte, net iets minder dan de helft van de totale capaciteit.

En verder

• Onder bepaalde artikelen en op IT Pro worden vacatureplaatsingen (IT-banen) getoond. Deze worden vanaf nu verborgen voor abonnees, als onderdeel van hun advertentievrije ervaring op Tweakers.
• De weergave van 'Nu in de community' op de frontpage is verbeterd, zodat met alle schermbreedtes dezelfde blokken worden getoond.
• We hebben 250 euro uitgekeerd aan een beveiligingsonderzoeker die een XSS-probleem heeft gemeld.

Reacties (32)

[Robbierut4]

We hebben 250 euro uitgekeerd aan een beveiligingsonderzoeker die een XSS-probleem heeft gemeld.

Dit klinkt wel interessant. Gaat hier nog een uitgebreider artikel aan worden besteed? Vol met alle technische details die tweakers graag willen weten hopelijk

[DaFeliX]

Ik weet niet of het een volledig artikel waard is, maar het zit zo:

We hebben recentelijk wijzigingen doorgevoerd om beter te kunnen meten hoe onze zoekmachine wordt gebruikt. Dit meten doen we niet server-sided, waardoor we op de een-of-andere manier bepaalde data moeten kunnen doorgeven op het moment dat je een zoekopdracht uitvoert. Dat doen we simpelweg door wat extra inhoud aan de pagina toe te voegen, die we daarna kunnen uitlezen. Zo voegen we bijvoorbeeld het zoekwoord toe in de pagina.

Voor bijna alle pagina's op Tweakers maken we gebruik van Twig als de rendering engine. Twig zorgt er automatisch voor dat alle niet-vertrouwde data goed word geëscaped, zodat user-input niet als HTML wordt gezien.
Nu hebben we ook het forum, waar we geen Twig gebruiken maar een andere rendering engine gebruiken. Deze engine escaped niet automatisch, dat moet je handmatig doen. Je voelt 'm misschien al aankomen: Ook in een van deze oude templates die door deze engine word gerendered, werd het zoekwoord in de pagina geplaatst. Maar daar hadden we wel zelf (handmatig) escaping moeten toepassen, wat we gewoonweg zijn vergeten (waarschijnlijk doordat we gewend zijn aan Twig).

Op die manier werd er dus een XSS geïntroduceerd. Deze was overigens erg snel ontdekt door deze beveiligingsonderzoeker, waardoor we het ook snel hebben kunnen repareren.

[AW_Bos]

Mooie changelog! Ik ben benieuwd wanneer jullie het wysiwyg-project weer gaan afstoffen.

[Anonymoussaurus]

Ik zou echt super graag Markdown support zien. Ik vind deze editor met UBB-coding niet meer van deze tijd. Te weinig functionaliteit. Kan relatief makkelijk geïmplementeerd worden: https://github.com/markdown-it/markdown-it

[Reactie gewijzigd door Anonymoussaurus op 22 juli 2024 20:17]

[AW_Bos]

Met enkel markdown red je het niet als vervanging. Liever ondersteuning voor beiden dan mét wysiwyg-ondersteuning.

[Reactie gewijzigd door AW_Bos op 22 juli 2024 20:17]

[Anonymoussaurus]

Daarom zeg ik ook support en niet "ik zie graag Markdown het huidige UBB vervangen". Ik bedoel dat alleen UBB niet meer van deze tijd is.

[Verwijderd]

Dit draadje toch ?

forumtopic: Wil jij jouw ervaringen delen met wysiwyg-editors?

Ik ben er bij mijzelf ook nog niet mee bezig geweest...

[Gropah]

Wat je linkt is een parser. En die zou waarschijnlijk ook nog uitgebreid moeten worden met functionaliteit voor linken naar usernames en/of pagina's binnen twekers. Ook is een parser nog geen volledige editor, en al helemaal niet een editor die ook goed werkt op mobiel. Daarbij, je moet ook het oude omzetten, of een parser daarvoor beschikbaar houden. Het word al snel complexer dan je denkt.

Verder vraag ik mij af of er bij het schrijven van de markdown implementatie wel goed is nagedacht over parsebaarheid en gebruikersgemak. Een voorbeeld: links gaan vaak met deze syntax:

[link text](https://markdown-it.github.io/)

Een kale link zoals https://markdown-it.github.io/ word standaard niets mee gedaan. Dus om zo'n link te plaatsen moet je [https://markdown-it.github.io/](https://markdown-it.github.io/) doen. Wat niet echt handig is. Daarbij hebben links ook nog wel eens een ( of ) in de link zelf zitten. Vooral wikipedia heeft daar een handje van. Maar die moet je dan dus gaan escapen in de syntax, omdat het anders als het einde van een link word gezien. En genoeg mensen weten niet hoe escapen werkt of dat het nodig is of vergeten het gewoon, getuige het aantal kapotte wikipedia linkjes dat ik op reddit zie.

Ik snap dat markdown op een aantal vlakken simpeler schrijft, maar het is imo niet voor alles handiger.

[ACM]

Verder vraag ik mij af of er bij het schrijven van de markdown implementatie wel goed is nagedacht over parsebaarheid en gebruikersgemak.

Ik denk dat het grootste voordeel van Markdown is dat het ook zonder parsingslag, dus als 'markup' best goed leesbaar is. En dat geldt een stuk minder voor html of ubb.

Maar het heeft inderdaad als nadeel dat het daardoor beperkter is dan expliciete markuptalen en ook ambigue resultaten kan opleveren als je het probeert te parsen.

En genoeg mensen weten niet hoe escapen werkt of dat het nodig is of vergeten het gewoon, getuige het aantal kapotte wikipedia linkjes dat ik op reddit zie.

Moderne browsers laten het ook weg bij het copy&pasten Het is daardoor ook behoorlijk lastig om een goed escaped link te fabriceren.

[Jeoh]

Nee hoor, je kan ook <https://markdown-it.github.io/> doen.

[field33P]

Eens, Markdown is ook veel makkelijker en vlotter te tikken op apparaten met touchscreen dan UBB-code. En niet te vergeten: intuïtiever.

[Zeror]

Niet eens. Ik vind markdown echt verschrikkelijk. Ik druk liever op knopjes die mijn geselecteerde tekst omzetten iets. Dat gezegd hebbende: De huidige UBB systeem kan ook intuïtiever gemaakt worden als de ondersteuning voor touchscreen verder wordt verbeterd. Ik zou voor uitbreiding van UBB gaan ipv vervanging.

[Anonymoussaurus]

Is dat ook niet gewoon gewenning? Developers (en DevOps/SecOps etc.) zijn gewend om in Markdown te typen door het gebruik van GitLab en GitHub, waarbij documentatie vrijwel altijd in Markdown wordt getypt. En laten de meeste IT'ers (en dus Tweaker gebruikers) nou bekend zijn met Markdown.

[reefclaw]

Nou, de enige reden waarom ik als engineer markdown gebruik is omdat ik geforceerd word om markdown te gebruiken.
Veel te vaak loop ik me weer te frustreren dat mijn text ineens weer word omgezet in iets anders, omdat markdown normale tekens misbruikt voor styling.

[Verwijderd]

Kwestie van Markdown leren gebruiken. \ voor je speciale tekens gooien en klaar. een backslash voor een backtick of asterisk rendert dan gewoon als dat teken ipv een lijst of vette tekst. Overigens een escape character in een heleboel talen, dus dat is niets nieuws of raars voor menig engineer. Dan kun je net zo goed lopen zeiken op alle moderne programmeertalen.

[Reactie gewijzigd door Verwijderd op 22 juli 2024 20:17]

[reefclaw]

Behalve dat veel markdown editors/parsers die ik heb gebruikt de escape character niet ondersteun(d)en.
Dit is namelijk het eerste dat ik heb geprobeerd.

[Verwijderd]

Dan zijn deze niet compliant en dat ligt dan aan de renderer die de regels breekt, niet Markdown. Die van Github, Azure DevOps, VS Code en Obsidian ondersteunen dit gewoon.

Officiële site:
https://daringfireball.ne...markdown/syntax#backslash

[Reactie gewijzigd door Verwijderd op 22 juli 2024 20:17]

[Jerie]

Is ook zeer gebruikelijk om op *nix met \ te escapen. Dat werkt werkelijk overal en nergens en is daardoor intuïtief.

[Anonymoussaurus]

Veel te vaak loop ik me weer te frustreren dat mijn text ineens weer word omgezet in iets anders

Wat bedoel je hiermee?

[reefclaw]

Dat ik text hebt getyped met bepaalde leestekens o.i.d. en ineens word dit omgezet in iets anders dan ik heb getyped.
Dit is trouwens ook groter issue doordat veel markdown parsers die ik heb moeten gebruiken geen escape characters ondersteunen.

[Zeror]

Dit is voor mij ook een van de redenen dat mij frustreert aan markdown.. als developer.

[Verwijderd]

Die knopjes kunnen nog steeds met markdown. Dit is niet exclusief aan de huidige 2005-stijl taal die gebruikt wordt. Discord doet dit bvb ook. Selecteer je tekst, druk op de B en hop, je hele tekst is dikgedrukt en staat nu tussen de speciale leestekens.

[quote] wordt dan > zonder dat een sluitende tag nodig is. [b] -> * en de huidige abominatie die [list][i][i][i][i][i][i] is kan dan gewoon een streepje voor je tekst worden, net als in bvb Notes, MS Word. Zo'n beetje elke degelijke tekstverwerker.

[Reactie gewijzigd door Verwijderd op 22 juli 2024 20:17]

[Macron]

Ik kreeg laatst het verzoek om te stemmen op een van drie dark modes. Ben benieuwd hoe het daarmee staat.

[jelle.]

Zie .plan: Bètatest vernieuwd reviewformulier - Development-iteraties #258/259, daar vind je ook de resultaten van dat onderzoek

[Xm0ur3r]

Ben ik dan echt de enige hier die een product in V&A wilt aanbieden met een veiling minder dan 3 dagen, gewoon voor een dagje bijvoorbeeld?
Meerdere keren ingeschoten maar nooit antwoord op gekregen.

[Rmaxx]

Ik vind die cards wel ok, maar in de opsomming van specs mag van mij meteen het schermtype(oled/ips/va) en de gpu erbij. Nu moet je toch nog in de tekst gaan speuren, en het staat er soms wel en soms niet duidelijk bij. Ook mis ik in de BBG nu net weer de goedkopere game/hybrid laptops. Ik denk dat veel mensen toch een leuke prijs laptop zoeken waarmee je toch net een beetje kunt gamen (scholieren bijv.) Vaak vind je die ook wel onder de 1000, nu staat er wel 1 gaming alternatief, maar daar moet je de gpu weer uit de tekst halen. Just my 2cts.

[Anonymoussaurus]

Was deze iteratie-apply de reden dat Tweakers er net even kort uit lag (en traag was/is)? Zie forumtopic: Traagheid en downtime Tweakers.

Anyway, mooie changelog weer.

[Reactie gewijzigd door Anonymoussaurus op 22 juli 2024 20:17]

[jelle.]

We werken met continuous delivery, dus de aanpassingen die hier staan beschreven zijn de afgelopen 2 weken al uitgerold. De traagheid en downtime is ongerelateerd, mijn collega's zijn hiernaar aan het kijken

[ikloon]

Nee, we doen continuous delivery, dus alle verbeteringen vanuit deze iteratie stonden al live. Maar er wordt gekeken naar wat de traagheid/downtime veroorzaakt.

[Reactie gewijzigd door ikloon op 22 juli 2024 20:17]

[vinkjb]

Tweakers lag er heel even uit en onbereikbaar, iets later supertraag...

[Anonymoussaurus]

Ja, zie forumtopic: Traagheid en downtime Tweakers.

[vinkjb]

thnx