Skip to content

Remove control characters from Atom feeds#573

Open
ungeskriptet wants to merge 2 commits into
Mic92:mainfrom
ungeskriptet:invalid-chars
Open

Remove control characters from Atom feeds#573
ungeskriptet wants to merge 2 commits into
Mic92:mainfrom
ungeskriptet:invalid-chars

Conversation

@ungeskriptet
Copy link
Copy Markdown
Contributor

Remove control characters from Atom feeds

Some commit messages, such as the one in JoseVarelaP/In-The-Groove2-SM5@75c266e, may contain invalid control characters, which can cause parsing errors in ElementTree.

Add a small function to remove such characters before passing the response to ElementTree.

Comment thread nix_update/utils.py
Some commit messages, such as the one in [1], may contain invalid
control characters, which can cause parsing errors in ElementTree.

Add a small function to remove such characters before passing the
response to ElementTree.

[1] JoseVarelaP/In-The-Groove2-SM5@75c266e

Signed-off-by: David Wronek <david.wronek@mainlining.org>
Let's make sure the previous commit works as expected.

Signed-off-by: David Wronek <david.wronek@mainlining.org>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants