GeezerVison

The World Wide Web, restored to 800 pixels of glory.

GeezerVison is showing github.com — src: set HAS_USERNAME/PASSWORD more strictly · nodejs/node@0e88f44

File tree

Expand file treeCollapse file tree

Open diff view settings

Filter options

src
- node_url.cc
test/parallel
- test-whatwg-url-custom-deepequal.js

Expand file treeCollapse file tree

Open diff view settings

Collapse file

`‎src/node_url.cc‎`

Copy file name to clipboardExpand all lines: src/node_url.cc

+20-8Lines changed: 20 additions & 8 deletions

Original file line number	Diff line number	Diff line change
`@@ -1209,21 +1209,33 @@ inline url_data HarvestBase(Environment* env, Local<Object> base_obj) {`
`1209`	`1209`	`base_obj->Get(env->context(), env->scheme_string()).ToLocalChecked();`
`1210`	`1210`	`base.scheme = Utf8Value(env->isolate(), scheme).out();`
`1211`	`1211`
`1212`		`- auto GetStr = [&](std::string url_data::* member,`
	`1212`	`+ auto GetStr = [&](std::string url_data::*member,`
`1213`	`1213`	`int flag,`
`1214`		`- Local<String> name) {`
	`1214`	`+ Local<String> name,`
	`1215`	`+ bool empty_as_present) {`
`1215`	`1216`	`Local<Value> value = base_obj->Get(env->context(), name).ToLocalChecked();`
`1216`	`1217`	`if (value->IsString()) {`
`1217`	`1218`	`Utf8Value utf8value(env->isolate(), value.As<String>());`
`1218`	`1219`	`(base.member).assign(utf8value, utf8value.length());`
`1219`		`- base.flags \|= flag;`
	`1220`	`+ if (empty_as_present \|\| value.As<String>()->Length() != 0) {`
	`1221`	`+ base.flags \|= flag;`
	`1222`	`+ }`
`1220`	`1223`	`}`
`1221`	`1224`	`};`
`1222`		`- GetStr(&url_data::username, URL_FLAGS_HAS_USERNAME, env->username_string());`
`1223`		`- GetStr(&url_data::password, URL_FLAGS_HAS_PASSWORD, env->password_string());`
`1224`		`- GetStr(&url_data::host, URL_FLAGS_HAS_HOST, env->host_string());`
`1225`		`- GetStr(&url_data::query, URL_FLAGS_HAS_QUERY, env->query_string());`
`1226`		`- GetStr(&url_data::fragment, URL_FLAGS_HAS_FRAGMENT, env->fragment_string());`
	`1225`	`+ GetStr(&url_data::username,`
	`1226`	`+ URL_FLAGS_HAS_USERNAME,`
	`1227`	`+ env->username_string(),`
	`1228`	`+ false);`
	`1229`	`+ GetStr(&url_data::password,`
	`1230`	`+ URL_FLAGS_HAS_PASSWORD,`
	`1231`	`+ env->password_string(),`
	`1232`	`+ false);`
	`1233`	`+ GetStr(&url_data::host, URL_FLAGS_HAS_HOST, env->host_string(), true);`
	`1234`	`+ GetStr(&url_data::query, URL_FLAGS_HAS_QUERY, env->query_string(), true);`
	`1235`	`+ GetStr(&url_data::fragment,`
	`1236`	`+ URL_FLAGS_HAS_FRAGMENT,`
	`1237`	`+ env->fragment_string(),`
	`1238`	`+ true);`
`1227`	`1239`
`1228`	`1240`	`Local<Value> port =`
`1229`	`1241`	`base_obj->Get(env->context(), env->port_string()).ToLocalChecked();`

Collapse file

`‎test/parallel/test-whatwg-url-custom-deepequal.js‎`

Copy file name to clipboard

+18Lines changed: 18 additions & 0 deletions

Original file line number	Diff line number	Diff line change
`@@ -0,0 +1,18 @@`
	`1`	`+'use strict';`
	`2`	`+// This tests that the internal flags in URL objects are consistent, as manifest`
	`3`	`+// through assert libraries.`
	`4`	`+// See https://github.com/nodejs/node/issues/24211`
	`5`	`+`
	`6`	`+// Tests below are not from WPT.`
	`7`	`+`
	`8`	`+require('../common');`
	`9`	`+const assert = require('assert');`
	`10`	`+`
	`11`	`+assert.deepStrictEqual(`
	`12`	`+ new URL('./foo', 'https://example.com/'),`
	`13`	`+ new URL('https://example.com/foo')`
	`14`	`+);`
	`15`	`+assert.deepStrictEqual(`
	`16`	`+ new URL('./foo', 'https://user:[email protected]/'),`
	`17`	`+ new URL('https://user:[email protected]/foo')`
	`18`	`+);`

0 commit comments

Comments

0 (0)

You are visitor 01402503 — GeezerVison does not store the pages you look up.
Best viewed with Netscape Navigator 4.7 or Internet Explorer 6 at 800×600. No cookies required.