RT-N66U VPN Client BUG when changing LAN settings

[Tigzy]

Hey,
I just flashed my RT-N66U with the latest build, and I discovered some critical bug (I cannot use the VPN because of that).

After initial flash, and WAN configuration, the VPN Client form looks good.
I can choose the client instance, and choose to use or not traffic redirect <= this is important statement.
Config is like this:
WAN: 192.168.1.18 (static)
LAN: 192.168.1.1

When I change the LAN to fix the subnet conflict, to 192.168.2.1, the VPN Client form becomes broken:
"Select client instance" is empty, and so is "Redirect internet traffic".
So I cannot exclude IPs from VPN, and I cannot setup more than one VPN instance.

Please let me know if you can reproduce.
Thanks,

 

[RMerlin]

Manually type the IP rather than use the dropdown - this might take some time to be re-populated for the new subnet.

 

[Tigzy]

Manually type the IP rather than use the dropdown - this might take some time to be re-populated for the new subnet.

I'm sorry, which IP?

 

[Tigzy]

Better explained with images:
1/ Before LAN modification:



2/ After LAN modification

 

[RMerlin]

Can you open your browser console and see if there's any Javascript error? This looks like something is preventing your webpage from fully loading.

 

[Tigzy]

That's right, JS error:

 

[Tigzy]

I have this code:

function showLANIPList(){
if(clientList.length == 0){
setTimeout(function() {
genClientList();
showLANIPList();
}, 500);
return false;
}

However I don't find any declaration for clientList. Should it be clientlist_array ?

 

[Tigzy]

Also, genClientList isn't declared in any of the JS files I have.

 

[john9527]

Make sure any manually assigned client names don't contain any special characters. Merlin is working through escaping the special chars, but there may still be a few exposures, or possibly the 382 webui merge reverted something.

 

[Tigzy]

What is client names?
If it's the VPN instances names, I didn't touch anything they are basically the default values.

 

[Tigzy]

The js error is self explaining, it doesn't find the variable

 

[john9527]

What is client names?

Either the name you assigned when setting a manually assigned IP address under the DHCP server page, or a name you assigned a client from within the networkmap page.

It may not find the variable because it failed to correctly parse the names (which may not be reported as a separate javascript error).

Also, genClientList isn't declared in any of the JS files I have.

genClientList is in client_function.js

 

[Tigzy]

OK I'll take a look, but basically I just restored default settings and changed the lan ip.

 

[john9527]

You can also do a 'View page source' and look to see if there is a processed value of clientList in the source, or if it's corrupted.

 

[Tigzy]

When I looked into the debugger the variable was not existing. Can you tell me where it's declared? Which file?

 

[john9527]

When I looked into the debugger the variable was not existing. Can you tell me where it's declared? Which file?

also in client_function.js

var clientList = new Array(0);

But, do the right click on the page, and view source.....it should tell what is going wrong.

 

[Tigzy]

I found that login from WAN doesn't produce the same bug.

 

[Tigzy]

on LAN side, client_function.js is 85 lines, whereas on WAN side it's about 3000 lines.
There's something wrong on JS generation when connected from LAN. I cannot go further as I don't know the project much...

 

[Jack Yaz]

You have cleared your cache with Ctrl+F5 right? .68 had a major UI update and cache clear is a must

 

[Tigzy]

I did, but I didn't install any previous version.