jonas-schievink/xml-rpc-rs

Support non-UTF-8 encodings

Open

#43 opened on Jul 7, 2018

View on GitHub
 (0 comments) (0 reactions) (0 assignees)Rust (21 forks)github user discovery
enhancementhelp wanted

Repository metrics

Stars
 (27 stars)
PR merge metrics
 (No merged PRs in 30d)

Description

We're currently using xml-rs to parse XML, which only supports UTF-8 (https://github.com/netvl/xml-rs/issues/10).

To do this, either xml-rs needs to get support for other encodings or we have to switch to quick-xml which does support non-UTF-8 documents (but needs some work - it needs to be better tested and doesn't support encodings that aren't ASCII-compatible).

Note that the XML-spec mandates UTF-8 and UTF-16 support, so xml-rs is technically non-compliant (as is this crate).

Contributor guide

Research direction
Investigate quick xml's support for non UTF 8 encodings and test it in the context of xml rpc rs. Alternatively, contribute to xml rs to add encoding support. Focus on understanding the XML encoding requirements and the library's current limitations.
Tech stack
rust
Domain
backendapi
Issue type
Feature
DifficultyEstimated implementation difficulty for a new contributor, from 1 for very small changes to 5 for expert-level work.
3
Estimated timeA rough time range for an experienced contributor to investigate, implement, test, and prepare a pull request.
Half day
Activity statusHow available the issue appears right now: fresh, active, stale, blocked, or waiting on maintainer input.
Fresh
ClarityHow clearly the issue explains the expected change, acceptance criteria, and next step.
Mostly clear
Prerequisites
RustXML basics
Newbie friendlinessA 1-100 score estimating how approachable this issue is for first-time contributors.
30