Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Charset detection not default as utf-8 #306

Open
alexburley opened this issue Apr 8, 2020 · 10 comments
Open

Charset detection not default as utf-8 #306

alexburley opened this issue Apr 8, 2020 · 10 comments

Comments

@alexburley
Copy link
Contributor

#170

Basically a clone of this issue

https://www.w3.org/TR/encoding/ looks like utf-8 is now default?

Are you open to pull request?

@tomas
Copy link
Owner

tomas commented Apr 8, 2020

Sure, go ahead!

@alexburley
Copy link
Contributor Author

@tomas What's the testing situation. I've forked and set up tests but getting a lot of failures, is there any specific environment set up thats not in the Readme?

In particular

decoder.spec.js -> test B -> encodes to UTF-8

Fails off the bat.

@tomas
Copy link
Owner

tomas commented Apr 8, 2020

Yeah I just pushed a commit that breaks everything, haha. Give me five minutes.

@tomas
Copy link
Owner

tomas commented Apr 8, 2020

Ok, go ahead and pull.

@alexburley
Copy link
Contributor Author

@tomas wicked thanks

@alexburley
Copy link
Contributor Author

@tomas Would you be okay with installing nock as a dev dependancy? Useful for specifying test cases for decoding.

e.g.

const nock = require('nock');
const url = https://some.url.com
nock(url)
	.get('/')
	.reply(200, 'Magyarországi Fióktelepe', {
		'content-type': 'text/html; charset=EUC-JP',
	});

 needle.get(url, { decode: false }, function(err, resp) {
          ....
   })

@tomas
Copy link
Owner

tomas commented Apr 8, 2020

Yeah, no prob.

@alexburley
Copy link
Contributor Author

PR raised #308

tomas added a commit that referenced this issue Apr 9, 2020
Charset detection not default as utf-8 #306
@alexburley
Copy link
Contributor Author

alexburley commented Apr 9, 2020

@tomas Thanks :) How are releases managed? Is there a timeline for when this will be published?

@alexburley
Copy link
Contributor Author

@tomas any update?

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

No branches or pull requests

2 participants