Skip to content
This repository

HTTPS clone URL

Subversion checkout URL

You can clone with HTTPS or Subversion.

Download ZIP

[fixes #43254575] - Null value #4

Merged
merged 1 commit into from over 1 year ago

2 participants

Chase Brammer Greg Allan
Chase Brammer
Owner

Added in a better regex to pull out any non printable characters from the xml

Chase Brammer cbrammer [fixes #43254575] - Null value
Added in a better regex to pull out any non printable characters from the xml
f4f01a3
Chase Brammer
Owner

@grobot review plz

Greg Allan grobot merged commit 64581cb into from
Greg Allan grobot closed this
Greg Allan grobot deleted the branch
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Showing 1 unique commit by 1 author.

Mar 15, 2013
Chase Brammer cbrammer [fixes #43254575] - Null value
Added in a better regex to pull out any non printable characters from the xml
f4f01a3
This page is out of date. Refresh to see the latest.

Showing 1 changed file with 1 addition and 1 deletion. Show diff stats Hide diff stats

  1. +1 1  lib/server.js
2  lib/server.js
@@ -47,7 +47,7 @@ var Server = function(server, path, services, wsdl) {
47 47 })
48 48 req.on('end', function() {
49 49 var xml = chunks.join(''), result;
50   - xml = xml.replace(/\0/g,'');
  50 + xml = xml.replace(/[\x00-\x1F]/g,'');
51 51 if (gunzip) {
52 52 gunzip.end();
53 53 gunzip = null

Tip: You can add notes to lines in a file. Hover to the left of a line to make a note

Something went wrong with that request. Please try again.