badges/shields

DOMParser for dynamic xml badge spews log results in production

Open

#2.658 geöffnet am 6. Jan. 2019

Auf GitHub ansehen
 (0 Kommentare) (0 Reaktionen) (0 zugewiesene Personen)JavaScript (22.337 Stars) (5.423 Forks)batch import
good first issueservice-badge

Beschreibung

e.g.

@#[line:21,col:1]
[xmldom warning]        unclosed xml attribute
@#[line:22,col:1]
[xmldom warning]        unclosed xml attribute
@#[line:23,col:1]
[xmldom warning]        unclosed xml attribute
@#[line:24,col:1]
[xmldom error]  element parse error: Error: attribute value must after "="
@#[line:47,col:85]
[xmldom error]  element parse error: Error: invalid tagName:\
@#[line:47,col:232]
[xmldom warning]        unclosed xml attribute
@#[line:154,col:7]
[xmldom error]  entity not found: 
@#[line:212,col:143]
[xmldom error]  entity not found: 
@#[line:212,col:143]
[xmldom warning]        unclosed xml attribute
@#[line:299,col:31]
[xmldom warning]        attribute "selected" missed value!! "selected" instead!!
@#[line:342,col:14]
[xmldom error]  entity not found: 
@#[line:379,col:57]
[xmldom error]  entity not found: 

I imagine DOMParser is printing warnings by default. They have a way to provide our own error handler. Maybe we can use that to suppress these messages.

https://github.com/jindw/xmldom#api-reference

Contributor Guide

Tech Stack
javascriptnodejs
Domain
backend
Issue Type
bug
SchwierigkeitGeschätzte Implementierungsschwierigkeit für neue Contributors, von 1 für sehr kleine Änderungen bis 5 für Expertenarbeit.
2
Geschätzte ZeitEin grober Zeitrahmen, um zu recherchieren, zu implementieren, zu testen und einen Pull Request vorzubereiten.
1-3 hours
AktivitätsstatusWie verfügbar das Issue gerade wirkt: frisch, aktiv, stale, blockiert oder wartet auf Maintainer-Input.
stale
KlarheitWie klar das Issue die erwartete Änderung, Akzeptanzkriterien und den nächsten Schritt erklärt.
clear
Voraussetzungen
basic JavaScriptfamiliarity with xmldom library
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
70
Research-Richtung
Look for the DOMParser usage in the dynamic badge generation code in the badges/shields repository. The issue provides a reference to the xmldom API for custom error handlers. Implement a custom error handler that suppresses or ignore the warnings and errors shown in the issue, ensuring production logs are clean. Test that badges still generate correctly.