Hey there folks,

A while back (May/June '15) I posted about a cool terminal app called Wego that gave you text based weather forecast info.

DM said it would be cool if someone made one for Sync, and nolageek had posted that he had started such a project and put it on GitHub.

Well, I decided to work on nola's script (syncWX) and create some new icons based on the Wego set of icons, and finally I came out with a finished product I call syncWXremix.

Here's some links:
http://bbs.kd3.us/kd3doors.ssjs
https://github.com/KenDB3/syncWXremix

Feel free to have a look and if you are adventurous maybe even set it up.

~KenDB3

---
þ Synchronet þ KD3net-Rhode Island's only BBS about nothing. http://bbs.kd3.us

Re: Sync Weather App - syncWXremix
By: KenDB3 to All on Wed Dec 23 2015 11:40 am

Hey there folks,

A while back (May/June '15) I posted about a cool terminal app called Wego that gave you text based weather forecast info.

DM said it would be cool if someone made one for Sync, and nolageek had posted that he had started such a project and put it on GitHub.

Well, I decided to work on nola's script (syncWX) and create some new icons based on the Wego set of icons, and finally I came out with a finished product I call syncWXremix.

Here's some links:
http://bbs.kd3.us/kd3doors.ssjs
https://github.com/KenDB3/syncWXremix

Feel free to have a look and if you are adventurous maybe even set it up.

Very cool! I set it up on Vertrauen.

Here's one quick change I made so that it pulls the wunderground API from the ctrl/modopts.ini, rather than having it hard-coded in the script:

Replacing this:
var wungrndAPIkey = "xxxxx";
with this:
opts=load({},"modopts.js","SyncWX");
var wungrndAPIkey = opts.wungrndAPIkey;

And then add the following to your ctrl/modopts.ini:
[SyncWX]
wungrndAPIkey = <your API key here>

You can change the section and key name in the modopts.ini file however you like, but just make sure that you keep your weather.js in-sync with those changes. This allows a sysop to update their weather.js in the future without losing their local change (the configuration).

If you have other configuration settings (e.g. weathericon_ext, local IP address, etc.), you can also move them to the modopts.ini (add more keys to the "SyncWX" section).

One other change I made, not because I really needed it, but because the hard-coded "8.8.8.8" IP address kind of bugged me:

var weather_ip_address = resolve_ip(system.inet_addr);

Granted, this won't work for some sysops who use their router/gateway as their DNS server and have it configured to "lie" about the actual IP address of their public hostname by giving them their local/private/LAN address instead.

Thanks! This is very cool. And if you want to have it committed maintained in cvs.synchro.net (instead-of or in-addition-to GitHub or whatever), let me know.

digital man

Synchronet "Real Fact" #56:
Synchronet introduced Telnet, FTP, SMTP and POP3 support w/v3.00a-Win32 in 2000.
Norco, CA WX: 53.2øF, 87.0% humidity, 8 mph W wind, 0.00 inches rain/24hrs

---
þ Synchronet þ Vertrauen þ Home of Synchronet þ telnet://vert.synchro.net

Re: Sync Weather App - syncWXremix
By: Digital Man to KenDB3 on Wed Dec 23 2015 09:54 pm

opts=load({},"modopts.js","SyncWX");
var wungrndAPIkey = opts.wungrndAPIkey;

That should be "var opts=load(...", to be more right.

digital man

Synchronet "Real Fact" #34:
The irc.synchro.net network has more servers than users.
Norco, CA WX: 53.2øF, 87.0% humidity, 8 mph W wind, 0.00 inches rain/24hrs

---
þ Synchronet þ Vertrauen þ Home of Synchronet þ telnet://vert.synchro.net

Re: Sync Weather App - syncWXremix
By: Digital Man to KenDB3 on Wed Dec 23 2015 09:54 pm

Well, I decided to work on nola's script (syncWX) and create some new
icons based on the Wego set of icons, and finally I came out with a
finished product I call syncWXremix.

Here's some links:
http://bbs.kd3.us/kd3doors.ssjs
https://github.com/KenDB3/syncWXremix

Feel free to have a look and if you are adventurous maybe even set it
up.

Very cool! I set it up on Vertrauen.

Here's one quick change I made so that it pulls the wunderground API from the ctrl/modopts.ini, rather than having it hard-coded in the script:

Replacing this:
var wungrndAPIkey = "xxxxx";
with this:
var opts=load({},"modopts.js","SyncWX"); (added fix from 2nd msg)
var wungrndAPIkey = opts.wungrndAPIkey;

And then add the following to your ctrl/modopts.ini:
[SyncWX]
wungrndAPIkey = <your API key here>

You can change the section and key name in the modopts.ini file however you like, but just make sure that you keep your weather.js in-sync with those changes. This allows a sysop to update their weather.js in the future without losing their local change (the configuration).

If you have other configuration settings (e.g. weathericon_ext, local IP address, etc.), you can also move them to the modopts.ini (add more keys to the "SyncWX" section).

Thanks for the feedback digital man! I'll work on those :-) I have learned a lot just working through this app, and any tips and tricks are definitely welcome.

One other change I made, not because I really needed it, but because the hard-coded "8.8.8.8" IP address kind of bugged me:

var weather_ip_address = resolve_ip(system.inet_addr);

Granted, this won't work for some sysops who use their router/gateway as their DNS server and have it configured to "lie" about the actual IP address of their public hostname by giving them their local/private/LAN address instead.

That's a neat trick! What I noticed is if the IP comes back as a private IP, the JSON query fails pretty hard. The hard coded was all I could think of.

Thanks! This is very cool. And if you want to have it committed maintained in cvs.synchro.net (instead-of or in-addition-to GitHub or whatever), let me know.

I think at some point, yes, that would be cool. Not quite yet... but I'll ping you when I feel a bit more confident, lol.

Thanks again for the feedback!

~KenDB3

---
þ Synchronet þ KD3net-Rhode Island's only BBS about nothing. http://bbs.kd3.us

Re: Sync Weather App - syncWXremix
By: KenDB3 to All on Wed Dec 23 2015 11:40:15

Well, I decided to work on nola's script (syncWX) and create some new icons based on the Wego set of icons, and finally I came out with a finished product I call syncWXremix.

Here's some links:
http://bbs.kd3.us/kd3doors.ssjs
https://github.com/KenDB3/syncWXremix

Feel free to have a look and if you are adventurous maybe even set it up.

Hi KenDB3 - I downloaded this and set it up on my BBS according to the documentation, but when it runs, it shows this error:
!JavaScript D:\BBS\sbbs\xtrn\syncWX\weather.js line 70: TypeError: cu.current_observation is undefined

Have you seen that error before?

Nightfox

---
þ Synchronet þ Digital Distortion: digitaldistortionbbs.com

Re: Sync Weather App - syncWXremix
By: Nightfox to KenDB3 on Fri Dec 25 2015 01:35 pm

Hi KenDB3 - I downloaded this and set it up on my BBS according to the documentation, but when it runs, it shows this error:
!JavaScript D:\BBS\sbbs\xtrn\syncWX\weather.js line 70: TypeError: cu.current_observation is undefined

Have you seen that error before?

I absolutely have. So, that pops up whenever the GeoIP lookup fails. It's not really a graceful error.

I encountered the error with two different scenarios in my own testing.

1) Using HTML5 ftelnet, the Web Socket Service running on my BBS reports back as the IP of the BBS... on my local LAN. In my case it was a 192.168.x.x IP, and that fails the GeoIP lookup.

2) Same problem, different way of manifesting. I was on a wireless device, using a Putty Client, and the device was connected to the WiFi on the same network as my BBS, so again, I came through as 192.168.x.x.

There is a 3rd scenario I can think of that is less likely, Weather Underground does not support IPv6 lookups.

So, the fix for my first description was the hard coded system IP that digital man was suggesting a better fix for. Listed as steps 5 & 6 in my sysop.txt
file for the external program setup.

Please let me know if this helps fix, or at least identify, the problem. I haven't seen it fail with that error for any other reason besides non-public IPs.

~KenDB3

---
þ Synchronet þ KD3net-Rhode Island's only BBS about nothing. http://bbs.kd3.us

Re: Sync Weather App - syncWXremix
By: KenDB3 to Nightfox on Fri Dec 25 2015 22:38:54

1) Using HTML5 ftelnet, the Web Socket Service running on my BBS reports back as the IP of the BBS... on my local LAN. In my case it was a 192.168.x.x IP, and that fails the GeoIP lookup.

2) Same problem, different way of manifesting. I was on a wireless device, using a Putty Client, and the device was connected to the WiFi on the same network as my BBS, so again, I came through as 192.168.x.x.

So, the fix for my first description was the hard coded system IP that digital man was suggesting a better fix for. Listed as steps 5 & 6 in my sysop.txt file for the external program setup.

Please let me know if this helps fix, or at least identify, the problem. I haven't seen it fail with that error for any other reason besides non-public IPs.

I saw that happening with me too - Since I was connecting from my own network, I was getting an IP address of 192.168.1.x (wired or wireless, wouldn't make a difference).

I ended up changing the script (near line 38) so that rather than checking for one specific IP address, it checks to see if the user's IP address begins with 192.168.1 (which is what my local network uses), and if that's the case, it resolves my BBS's public IP address, and otherwise just uses the user's IP address:

// If the user is on my home network, then use my BBS's public IP address.
// Otherwise, use the user's (remote) IP address.
if (user.ip_address.indexOf("192.168.1") == 0)
var weather_ip_address = resolve_ip(system.inet_addr);
else
var weather_ip_address = user.ip_address;

Nightfox

---
þ Synchronet þ Digital Distortion: digitaldistortionbbs.com

Re: Sync Weather App - syncWXremix
By: KenDB3 to All on Wed Dec 23 2015 11:40:15

Well, I decided to work on nola's script (syncWX) and create some new icons based on the Wego set of icons, and finally I came out with a finished product I call syncWXremix.

Here's some links:
http://bbs.kd3.us/kd3doors.ssjs
https://github.com/KenDB3/syncWXremix

I like it, and I've also put it in my Synchronet filebase for download. One thing I noticed is that since the FILE_ID.DIZ has pipe symbols (|), my BBS seems to be interpreting some of the text as color codes and is thus cutting off a few of the characters (I think Renegade uses color codes using the pipe character). This is a screenshot of what the FILE_ID.DIZ description looks like on my BBS:
http://bit.ly/22t1gvM

Nightfox

---
þ Synchronet þ Digital Distortion: digitaldistortionbbs.com

Re: Sync Weather App - syncWXremix
By: KenDB3 to Digital Man on Thu Dec 24 2015 02:38 am

Re: Sync Weather App - syncWXremix
By: Digital Man to KenDB3 on Wed Dec 23 2015 09:54 pm

Well, I decided to work on nola's script (syncWX) and create some new
icons based on the Wego set of icons, and finally I came out with a
finished product I call syncWXremix.

Here's some links:
http://bbs.kd3.us/kd3doors.ssjs
https://github.com/KenDB3/syncWXremix

Feel free to have a look and if you are adventurous maybe even set it
up.

Very cool! I set it up on Vertrauen.

Here's one quick change I made so that it pulls the wunderground API from the ctrl/modopts.ini, rather than having it hard-coded in the script:

Replacing this:
var wungrndAPIkey = "xxxxx";
with this:
var opts=load({},"modopts.js","SyncWX"); (added fix from 2nd msg)
var wungrndAPIkey = opts.wungrndAPIkey;

And then add the following to your ctrl/modopts.ini:
[SyncWX]
wungrndAPIkey = <your API key here>

You can change the section and key name in the modopts.ini file however you like, but just make sure that you keep your weather.js in-sync with those changes. This allows a sysop to update their weather.js in the future without losing their local change (the configuration).

If you have other configuration settings (e.g. weathericon_ext, local IP address, etc.), you can also move them to the modopts.ini (add more keys to the "SyncWX" section).

Thanks for the feedback digital man! I'll work on those :-) I have learned a lot just working through this app, and any tips and tricks are definitely welcome.

One other change I made, not because I really needed it, but because the hard-coded "8.8.8.8" IP address kind of bugged me:

var weather_ip_address = resolve_ip(system.inet_addr);

Granted, this won't work for some sysops who use their router/gateway as their DNS server and have it configured to "lie" about the actual IP address of their public hostname by giving them their local/private/LAN address instead.

That's a neat trick! What I noticed is if the IP comes back as a private IP, the JSON query fails pretty hard. The hard coded was all I could think of.

Thanks! This is very cool. And if you want to have it committed maintained in cvs.synchro.net (instead-of or in-addition-to GitHub or whatever), let me know.

I think at some point, yes, that would be cool. Not quite yet... but I'll ping you when I feel a bit more confident, lol.

Here's another change I made today when I started to getting local alerts:

Replaced this:
console.putmsg(rd + "\001iPress any key to read the full alert\001n");
console.gotoxy(1,22);
console.crlf();
console.pause();
console.clear();
console.putmsg(rd + cu.alerts[0].message);

with this:
if(console.yesno("Read the full alert"))
console.putmsg(rd + cu.alerts[0].message);

In the original code, the viewing of the alert details could be skipped by hitting 'N' or Ctrl-C at the pause prompt, but most users won't know that.

digital man

Synchronet "Real Fact" #55:
Synchronet Terminal Server introduced SecureShell (SSH) support w/v3.14a (2006).
Norco, CA WX: 50.9øF, 21.0% humidity, 18 mph SSW wind, 0.00 inches rain/24hrs

---
þ Synchronet þ Vertrauen þ Home of Synchronet þ telnet://vert.synchro.net

Re: Sync Weather App - syncWXremix
By: KenDB3 to Nightfox on Fri Dec 25 2015 22:38:54

1) Using HTML5 ftelnet, the Web Socket Service running on my BBS reports back as the IP of the BBS... on my local LAN. In my case it was a 192.168.x.x IP, and that fails the GeoIP lookup.

2) Same problem, different way of manifesting. I was on a wireless device, using a Putty Client, and the device was connected to the WiFi on the same network as my BBS, so again, I came through as 192.168.x.x.

So, the fix for my first description was the hard coded system IP that digital man was suggesting a better fix for. Listed as steps 5 & 6 in my sysop.txt file for the external program setup.

Please let me know if this helps fix, or at least identify, the problem. I haven't seen it fail with that error for any other reason besides non-public IPs.

I saw that happening with me too - Since I was connecting from my own network, I was getting an IP address of 192.168.1.x (wired or wireless, wouldn't make a difference).

I ended up changing the script (near line 38) so that rather than checking for one specific IP address, it checks to see if the user's IP address begins with 192.168.1 (which is what my local network uses), and if that's the case, it resolves my BBS's public IP address, and otherwise just uses the user's IP address:

// If the user is on my home network, then use my BBS's public IP address. // Otherwise, use the user's (remote) IP address.
if (user.ip_address.indexOf("192.168.1") == 0)
var weather_ip_address = resolve_ip(system.inet_addr);
else
var weather_ip_address = user.ip_address;

Nightfox

I like the fix; into the list of changes it goes, lol. I'm away from home for the holiday, but plan to work on it again when I get back.

-KenDB3

---
þ Synchronet þ KD3net-Rhode Island's only BBS about nothing. http://bbs.kd3.us

Re: Sync Weather App - syncWXremix
By: KenDB3 to All on Wed Dec 23 2015 11:40:15

Well, I decided to work on nola's script (syncWX) and create some new icons based on the Wego set of icons, and finally I came out with a finished product I call syncWXremix.

Here's some links:
http://bbs.kd3.us/kd3doors.ssjs
https://github.com/KenDB3/syncWXremix

I like it, and I've also put it in my Synchronet filebase for download. One thing I noticed is that since the FILE_ID.DIZ has pipe symbols (|), my BBS seems to be interpreting some of the text as color codes and is thus cutting off a few of the characters (I think Renegade uses color codes using the pipe character). This is a screenshot of what the FILE_ID.DIZ description looks like on my BBS:
http://bit.ly/22t1gvM

Nightfox

That was certainly unexpected (and rather funny). I'll design it differently, I didn't really think about the pipe codes.

-KenDB3

---
þ Synchronet þ KD3net-Rhode Island's only BBS about nothing. http://bbs.kd3.us

Very cool! I set it up on Vertrauen.

Here's one quick change I made so that it pulls the wunderground API from the ctrl/modopts.ini, rather than having it hard-coded in the script:

Replacing this:
var wungrndAPIkey = "xxxxx";
with this:
var opts=load({},"modopts.js","SyncWX"); (added fix from 2nd msg) var wungrndAPIkey = opts.wungrndAPIkey;

And then add the following to your ctrl/modopts.ini:
[SyncWX]
wungrndAPIkey = <your API key here>

You can change the section and key name in the modopts.ini file however you like, but just make sure that you keep your weather.js in-sync with those changes. This allows a sysop to update their weather.js in the future without losing their local change (the configuration).

If you have other configuration settings (e.g. weathericon_ext, local IP address, etc.), you can also move them to the modopts.ini (add more keys to the "SyncWX" section).

Thanks for the feedback digital man! I'll work on those :-) I have learned a lot just working through this app, and any tips and tricks are definitely welcome.

One other change I made, not because I really needed it, but because the hard-coded "8.8.8.8" IP address kind of bugged me:

var weather_ip_address = resolve_ip(system.inet_addr);

Granted, this won't work for some sysops who use their router/gateway as their DNS server and have it configured to "lie" about the actual IP address of their public hostname by giving them their local/private/LAN address instead.

That's a neat trick! What I noticed is if the IP comes back as a private IP, the JSON query fails pretty hard. The hard coded was all I could think of.

Thanks! This is very cool. And if you want to have it committed maintained in cvs.synchro.net (instead-of or in-addition-to GitHub or whatever), let me know.

I think at some point, yes, that would be cool. Not quite yet... but I'll ping you when I feel a bit more confident, lol.

Here's another change I made today when I started to getting local alerts:

Replaced this:
console.putmsg(rd + "\001iPress any key to read the full alert\001n");
console.gotoxy(1,22);
console.crlf();
console.pause();
console.clear();
console.putmsg(rd + cu.alerts[0].message);

with this:
if(console.yesno("Read the full alert"))
console.putmsg(rd + cu.alerts[0].message);

In the original code, the viewing of the alert details could be skipped by hitting 'N' or Ctrl-C at the pause prompt, but most users won't know that.

digital man

That's really awesome, I appreciate the feedback (from everyone that has given input), this is my first real programming. I had a lot of fun with it, and learned quite a bit. So all the feedback is just adding to the learning experience.

-KenDB3

---
þ Synchronet þ KD3net-Rhode Island's only BBS about nothing. http://bbs.kd3.us

Re: Re: Sync Weather App - syncWXremix
By: KenDB3 to Nightfox on Sat Dec 26 2015 10:14 pm

Re: Sync Weather App - syncWXremix
By: KenDB3 to Nightfox on Fri Dec 25 2015 22:38:54

I like the fix; into the list of changes it goes, lol. I'm away from home for the holiday, but plan to work on it again when I get back.

Regardubg tge weather app, I like it so far! Just saying... :)

Sincerely,
Jon Justvig

---
þ Synchronet þ vintagebbsing.com:81

Given the bad weather around my neck of the woods this weekend, I figured it was a perfect time to give it a try.

Nicely done, Ken. I really like it!

My main suggestion would be to find some way to trap the error if there's a problem with the api lookup, so that it tells the user something descriptive about what went wrong and then quits gracefully back to the BBS.

--Josh

////--------------------------------------------------
BiC -=- http://breakintochat.com -=- bbs wiki and blog

---
þ Synchronet

Hi KenDB3 - I downloaded this and set it up on my BBS according to the documentation, but when it runs, it shows this error:
!JavaScript D:\BBS\sbbs\xtrn\syncWX\weather.js line 70: TypeError: cu.current_observation is undefined

Ken,
This is the exact same error i'm getting. Have tried using my public
ip instead of the Github but still get same result. :(

... Canadian DOS prompt: C:\EH?>

---
þ Synchronet þ ´ TBIRDS BBS ´ telnet://tbirds.dyndns.org

Re: Re: Sync Weather App - syncWXremix
By: KenDB3 to Nightfox on Sat Dec 26 2015 10:14 pm

Re: Sync Weather App - syncWXremix
By: KenDB3 to Nightfox on Fri Dec 25 2015 22:38:54

I like the fix; into the list of changes it goes, lol. I'm away from home for the holiday, but plan to work on it again when I get back.

Regardubg tge weather app, I like it so far! Just saying... :)

Sincerely,
Jon Justvig

Thanks Jon! Glad you like it!

-KenDB3

---
þ Synchronet þ KD3net-Rhode Island's only BBS about nothing. http://bbs.kd3.us

Given the bad weather around my neck of the woods this weekend, I figured it was a perfect time to give it a try.

Nicely done, Ken. I really like it!

My main suggestion would be to find some way to trap the error if there's a problem with the api lookup, so that it tells the user something descriptive about what went wrong and then quits gracefully back to the BBS.

--Josh

Thanks Josh! I agree, I'd like to find a way to handle that better, but I might need some help with that (if anyone out there is interested... hint hint :-P ).

-KenDB3

---
þ Synchronet þ KD3net-Rhode Island's only BBS about nothing. http://bbs.kd3.us

KenDB3 wrote to Jon Justvig <=-

Re: Re: Sync Weather App - syncWXremix
By: KenDB3 to Nightfox on Sat Dec 26 2015 10:14 pm

Regardubg tge weather app, I like it so far! Just saying... :)

Sincerely,
Jon Justvig

Thanks Jon! Glad you like it!

Disregard my typos, and you're welcome. A nice installation process as well.

Sincerely,
Jon Justvig

... So easy, a child could do it. Child sold separately.
--- MultiMail/Win32 v0.49
þ Synchronet þ vintagebbsing.com:81

Hi KenDB3 - I downloaded this and set it up on my BBS according to the documentation, but when it runs, it shows this error: !JavaScript D:\BBS\sbbs\xtrn\syncWX\weather.js line 70: TypeError: cu.current_observation is undefined

Ken,
This is the exact same error i'm getting. Have tried using my public
ip instead of the Github but still get same result. :(

Hmmm.. are you by chance logging in from a device on the same network (router/wireless)? Nightfox saw this issue (and I did too). I'm probably going to use Nightfox's solution in my next update. For now, maybe see if you can test from Ree's page: http://my.ftelnet.ca. It will proxy it and will come from a public IP. If its working for others off of your local network, then hopefully you can see it working this way. If not, let me know (I might have a few questions to narrow down the issue).

-KenDB3

---
þ Synchronet þ KD3net-Rhode Island's only BBS about nothing. http://bbs.kd3.us

Hi KenDB3 - I downloaded this and set it up on my BBS according t the documentation, but when it runs, it shows this error: !JavaScript D:\BBS\sbbs\xtrn\syncWX\weather.js line 70: TypeError cu.current_observation is undefined

Ken,
This is the exact same error i'm getting. Have tried using my public
ip instead of the Github but still get same result. :(

to use Nightfox's solution in my next update. For now, maybe see if you can test from Ree's page: http://my.ftelnet.ca. It will proxy it and will come f a public IP. If its working for others off of your local network, then hopefully you can see it working this way. If not, let me know (I might have few questions to narrow down the issue).

Yes, i am logging in locally on the network via 192.168.*.*. I did try Nightfox's replacement but it had the same result. :(

Also tried the ftelnet embed going through the proxy in Georgia but
again had the same result. Bummed out it's not working for me but
i'm happy it's working for so many others. Hat's off, Ken. Great
work!

Any ideas or thoughts greatly appreciated.
Best wishes

... I'm up to my ASCII in files.

---
þ Synchronet þ ´ TBIRDS BBS ´ telnet://tbirds.dyndns.org

Yes, i am logging in locally on the network via 192.168.*.*. I did try Nightfox's replacement but it had the same result. :(

What IP addresses does your home network use? My home network IPs are 192.168.1.*, so if your 3rd digit is (or can be) something other than 1, you could change yours to look for 192.168 instead of 192.168.1.

Nightfox

---
þ Synchronet þ Digital Distortion: digitaldistortionbbs.com

Yes, i am logging in locally on the network via 192.168.*.*. I did try Nightfox's replacement but it had the same result. :(

Hi KenDB3 - I'm wondering if there could be an option loaded from a configuration file to specify a local network IP identification so that
sysops wouldn't have to modify the JavaScript source. A sysop could put something like 192.168 in that configuration option, and if the script finds that in the user's IP address (similar to the fix I used), then it would resolve the BBS's IP address.

Nightfox

---
þ Synchronet þ Digital Distortion: digitaldistortionbbs.com

Yes, i am logging in locally on the network via 192.168.*.*. I did try Nightfox's replacement but it had the same result. :(

What IP addresses does your home network use? My home network IPs are 192.168.1.*, so if your 3rd digit is (or can be) something other than 1, you

Ok, it's working!:) I wasn't grasping the whole local lan thing
correctly. Was trying to use the local lan ip of the actual bbs
instead of the connecting devices.<duh..smacks head> Once that
issue was resolved i SLOWLY retyped (correctly this time)
Nightfox's code and it now works anywhere in the house. YEAH! :)

Thanks guys. Freaking awesome!

... Honey, I think I just formatted the cat!

---
þ Synchronet þ ´ TBIRDS BBS ´ telnet://tbirds.dyndns.org

!JavaScript D:\BBS\sbbs\xtrn\syncWX\weather.js line 70: TypeError cu.current_observation is undefined

Ken,
This is the exact same error i'm getting. Have tried using my public

Hmmm.. are you by chance logging in from a device on the same network (router/wireless)? Nightfox saw this issue (and I did too). I'm probably goi

KenDB3, It's working now. My goofy ass mistake with the local lan ip.
Thank you!

... Ever meet a Sysop who would admit the problem was his?

---
þ Synchronet þ ´ TBIRDS BBS ´ telnet://tbirds.dyndns.org

Hi KenDB3 - I downloaded this and set it up on my BBS according t the documentation, but when it runs, it shows this error: !JavaScript D:\BBS\sbbs\xtrn\syncWX\weather.js line 70: TypeError cu.current_observation is undefined

Ken,
This is the exact same error i'm getting. Have tried using my public ip instead of the Github but still get same result. :(

to use Nightfox's solution in my next update. For now, maybe see if you can test from Ree's page: http://my.ftelnet.ca. It will proxy it and will come f a public IP. If its working for others off of your local network, then hopefully you can see it working this way. If not, let me know (I might have few questions to narrow down the issue).

Yes, i am logging in locally on the network via 192.168.*.*. I did try Nightfox's replacement but it had the same result. :(

Also tried the ftelnet embed going through the proxy in Georgia but
again had the same result. Bummed out it's not working for me but
i'm happy it's working for so many others. Hat's off, Ken. Great
work!

Any ideas or thoughts greatly appreciated.
Best wishes

Ok, thanks for doing the ftelnet test (and thanks for the encouragement). I don't think you are having a problem with a Private IP (I logged in as guest on your BBS and saw the problem as well). I'm wondering if it is an API Key issue.

Bad news: Right now I am not sure. Best advice is to make sure you have an API key from wunderground.com, and you have it entered correctly.

Good news (hopefully): I think I have figured out how to capture error data from the response coming from wunderground.com. So, when I make some adjustments and do a new release, we might be able to sort out what's causing your particular issue.

Try checking the API Key, because that ~feels~ like the culprit right now.

Check:
1) Did you sign up for one? (sysop.txt should help guide you)
2) Did you enter it correctly (no spaces before or after)
it should look something like this:
var wungrndAPIkey = "123456789ab01234";

Hang in there! I received a lot of great suggestions and code fixes from folks (this really is an awesome community by the by, you guys rock), so I'm going to work on a new version :-)

~KenDB3

---
þ Synchronet þ KD3net-Rhode Island's only BBS about nothing. http://bbs.kd3.us

Yes, i am logging in locally on the network via 192.168.*.*. I did try Nightfox's replacement but it had the same result. :(

Hi KenDB3 - I'm wondering if there could be an option loaded from a configuration file to specify a local network IP identification so that sysops wouldn't have to modify the JavaScript source. A sysop could put something like 192.168 in that configuration option, and if the script finds that in the user's IP address (similar to the fix I used), then it would resolve the BBS's IP address.

Nightfox

I think DM pointed me in the right direction on that, and I want to implement that change. Check out his suggestion:

Here's one quick change I made so that it pulls the wunderground API from the ctrl/modopts.ini, rather than having it hard-coded in the script:

Replacing this:
var wungrndAPIkey = "xxxxx";
with this:
var opts=load({},"modopts.js","SyncWX");
var wungrndAPIkey = opts.wungrndAPIkey;

And then add the following to your ctrl/modopts.ini:
[SyncWX]
wungrndAPIkey = <your API key here>

You can change the section and key name in the modopts.ini file however you like, but just make sure that you keep your weather.js in-sync with those changes. This allows a sysop to update their weather.js in the future without losing their local change (the configuration).

If you have other configuration settings (e.g. weathericon_ext, local IP address, etc.), you can also move them to the modopts.ini (add more keys to the "SyncWX" section).

I think this is exactly what you are talking about. I have some work to do on it, and I'll try to be quick about it, lol. I just got back home this evening so hopefully I can make the updates I want to implement soon.

~KenDB3

---
þ Synchronet þ KD3net-Rhode Island's only BBS about nothing. http://bbs.kd3.us

i'm happy it's working for so many others. Hat's off, Ken. Great
work!

Ok, thanks for doing the ftelnet test (and thanks for the encouragement). I don't think you are having a problem with a Private IP (I logged in as guest your BBS and saw the problem as well). I'm wondering if it is an API Key iss

Thanks, Ken. Everything works great when i login with my Sysop
credentials. The door is setup to display at login.

Like you've already mentioned, the GUEST account doesn't work
at login but i noticed when accessed from the external menu,
it works with the GUEST account.

Does that make any sense?

Thanks again for the help.

... Wanna giggle? Try (EDITOR=EDLIN !) in anything.

---
þ Synchronet þ ´ TBIRDS BBS ´ telnet://tbirds.dyndns.org

!JavaScript D:\BBS\sbbs\xtrn\syncWX\weather.js line 70: TypeError cu.current_observation is undefined

Ken,
This is the exact same error i'm getting. Have tried using my public

Hmmm.. are you by chance logging in from a device on the same network (router/wireless)? Nightfox saw this issue (and I did too). I'm probably goi

KenDB3, It's working now. My goofy ass mistake with the local lan ip.
Thank you!

... Ever meet a Sysop who would admit the problem was his?

Awesome! Glad it is working now! And Glad that you like it!

~KenDB3

---
þ Synchronet þ KD3net-Rhode Island's only BBS about nothing. http://bbs.kd3.us

Re: Re: Sync Weather App - syncWXremix
By: tbirdsradio to KenDB3 on Tue Dec 29 2015 08:36 am

i'm happy it's working for so many others. Hat's off, Ken. Great
work!

Ok, thanks for doing the ftelnet test (and thanks for the encouragement). I don't think you are having a problem with a Private IP (I logged in as guest your BBS and saw the problem as well). I'm wondering if it is an API Key iss

Thanks, Ken. Everything works great when i login with my Sysop
credentials. The door is setup to display at login.

Like you've already mentioned, the GUEST account doesn't work
at login but i noticed when accessed from the external menu,
it works with the GUEST account.

Does that make any sense?

Can you define "doesn't work"?

digital man

Synchronet "Real Fact" #42:
Synchronet added Baja/PCMS support with v2.00a (1994).
Norco, CA WX: 58.4øF, 32.0% humidity, 1 mph N wind, 0.02 inches rain/24hrs

---
þ Synchronet þ Vertrauen þ Home of Synchronet þ telnet://vert.synchro.net

Like you've already mentioned, the GUEST account doesn't work
at login but i noticed when accessed from the external menu,
it works with the GUEST account.

Does that make any sense?

Can you define "doesn't work"?

Sure. I meant that the following error continues to show up:
error: !JavaScript D:\BBS\sbbs\xtrn\syncWX\weather.js line 70:
TypeError cu.current_observation is undefined

I think i spoke to soon though when i said everything was ok because i
noticed when Ken logged back into my system as a registered user the
error showed up again in the logs.

When i login as Sysop everything works, which is why i thought all was
good to go.




... Oxymoron: Windows Expert

---
þ Synchronet þ ´ TBIRDS BBS ´ telnet://tbirds.dyndns.org

Like you've already mentioned, the GUEST account doesn't work
at login but i noticed when accessed from the external menu,
it works with the GUEST account.

Does that make any sense?

Can you define "doesn't work"?

Sure. I meant that the following error continues to show up:
error: !JavaScript D:\BBS\sbbs\xtrn\syncWX\weather.js line 70:
TypeError cu.current_observation is undefined

I think i spoke to soon though when i said everything was ok because i noticed when Ken logged back into my system as a registered user the
error showed up again in the logs.

When i login as Sysop everything works, which is why i thought all was
good to go.

I'm working on a way to capture more info. In the mean time, can you do a cut/paste of the code you changed based on what Nightfox suggested? (aka, where you made the fix so your private IP's were working correctly)

Also, what is your private IP scheme on your local network (if you don't mind posting here or maybe you could email it to me instead)? Do you have 192.168.1.x, 192.168.100.x, etc... or is it something like 10.x.x.x?

I'd love to see it fixed for you.

~KenDB3

---
þ Synchronet þ KD3net-Rhode Island's only BBS about nothing. http://bbs.kd3.us

Thanks Josh! I agree, I'd like to find a way to handle that better, but I might need some help with that (if anyone out there is interested... hint hint :-P ).

Okay, Ken, I went ahead and wrote up some code to test for error messages in the JSON response. I sent you a pull request on GitHub. Feel free to use my suggestions, or write your own better version!

Also, I noticed k5jat created an issue on GitHub about a non-geolocating method for getting the data. The API *does* support queries that specify either city/state or zip code. I wonder if you might consider a fallback for cases where IP-based queries fail that builds a new query URL using SBBS' system.location variable.

--Josh


////--------------------------------------------------
BiC -=- http://breakintochat.com -=- bbs wiki and blog

---
þ Synchronet

Re: Re: Sync Weather App - syncWXremix
By: tbirdsradio to Digital Man on Tue Dec 29 2015 05:44 pm

Like you've already mentioned, the GUEST account doesn't work
at login but i noticed when accessed from the external menu,
it works with the GUEST account.

Does that make any sense?

Can you define "doesn't work"?

Sure. I meant that the following error continues to show up:
error: !JavaScript D:\BBS\sbbs\xtrn\syncWX\weather.js line 70:
TypeError cu.current_observation is undefined

I think i spoke to soon though when i said everything was ok because i noticed when Ken logged back into my system as a registered user the
error showed up again in the logs.

When i login as Sysop everything works, which is why i thought all was
good to go.

In v3.16, the user's IP address (as exposed via JavaScript) during the logon process is the user's previously-used IP address. So if you logon as a non-sysop and then logoff and back on, does that work?

digital man

Synchronet "Real Fact" #9:
The name "DOVE-Net" was suggested by King Drafus (sysop of The Beast's Domain). Norco, CA WX: 47.2øF, 67.0% humidity, 0 mph W wind, 0.00 inches rain/24hrs

---
þ Synchronet þ Vertrauen þ Home of Synchronet þ telnet://vert.synchro.net

Re: Re: Sync Weather App - syncWXremix
By: Kirkman to KenDB3 on Tue Dec 29 2015 11:16 pm

Thanks Josh! I agree, I'd like to find a way to handle that better, but I might need some help with that (if anyone out there is interested... hint hint :-P ).

Okay, Ken, I went ahead and wrote up some code to test for error messages in the JSON response. I sent you a pull request on GitHub. Feel free to use my suggestions, or write your own better version!

Also, I noticed k5jat created an issue on GitHub about a non-geolocating method for getting the data. The API *does* support queries that specify either city/state or zip code. I wonder if you might consider a fallback for cases where IP-based queries fail that builds a new query URL using SBBS' system.location variable.

Or use the current user's location or zip/postal code if/when available?

digital man

Synchronet "Real Fact" #45:
Synchronet External X/Y/ZMODEM protocol driver (SEXYZ) was introduced in 2005. Norco, CA WX: 47.2øF, 67.0% humidity, 0 mph W wind, 0.00 inches rain/24hrs

---
þ Synchronet þ Vertrauen þ Home of Synchronet þ telnet://vert.synchro.net

Re: Re: Sync Weather App - syncWXremix
By: Kirkman to KenDB3 on Tue Dec 29 2015 11:16 pm

Thanks Josh! I agree, I'd like to find a way to handle that better,
but I might need some help with that (if anyone out there is
interested... hint hint :-P ).

Okay, Ken, I went ahead and wrote up some code to test for error messages in the JSON response. I sent you a pull request on GitHub. Feel free to use my suggestions, or write your own better version!

I actually had worked something out, but I like yours better, lol. I've pulled it into the code. I might add something else, but I'm not sure if its better as an on-screen alert or as part of the Log, but I was going to add a visual of the request string that was sent to wunderground.com as a way of further seeing what happened. Thoughts?

Also, I noticed k5jat created an issue on GitHub about a non-geolocating method for getting the data. The API *does* support queries that specify either city/state or zip code. I wonder if you might consider a fallback for cases where IP-based queries fail that builds a new query URL using SBBS' system.location variable.

Thanks for pointing it out, I responded there, but I will give you my thoughts here. I like the idea of a fall back, but my concern with system.location is it might not match up with the data needed to form a query. For example, mine is set to just "Rhode Island" with no city. But, maybe we can come up with something, I am definitely open to suggestions and am happy to have people throwing in some code.

~KenDB3

---
þ Synchronet þ KD3net-Rhode Island's only BBS about nothing. http://bbs.kd3.us

Re: Re: Sync Weather App - syncWXremix
By: KenDB3 to Kirkman on Wed Dec 30 2015 01:28 am

Re: Re: Sync Weather App - syncWXremix
By: Kirkman to KenDB3 on Tue Dec 29 2015 11:16 pm

Thanks Josh! I agree, I'd like to find a way to handle that better,
but I might need some help with that (if anyone out there is
interested... hint hint :-P ).

Okay, Ken, I went ahead and wrote up some code to test for error messages in the JSON response. I sent you a pull request on GitHub. Feel free to use my suggestions, or write your own better version!

I actually had worked something out, but I like yours better, lol. I've pulled it into the code. I might add something else, but I'm not sure if its better as an on-screen alert or as part of the Log, but I was going to add a visual of the request string that was sent to wunderground.com as a way of further seeing what happened. Thoughts?

You could log it for the sysop with the the log() function. Use log(LOG_DEBUG,...) if you want it to only be displayed when the sysop has chosen to view debug-level log output.

Also, I noticed k5jat created an issue on GitHub about a non-geolocating method for getting the data. The API *does* support queries that specify either city/state or zip code. I wonder if you might consider a fallback for cases where IP-based queries fail that builds a new query URL using SBBS' system.location variable.

Thanks for pointing it out, I responded there, but I will give you my thoughts here. I like the idea of a fall back, but my concern with system.location is it might not match up with the data needed to form a query. For example, mine is set to just "Rhode Island" with no city. But, maybe we can come up with something, I am definitely open to suggestions and am happy to have people throwing in some code.

The modopts.ini could also be used to provide a postal/zip code for the system (but I think the user's location/postal/zip would be better).

digital man

Synchronet "Real Fact" #36:
Synchronet's Windows Control Panel is built with Borland C++ Builder.
Norco, CA WX: 42.0øF, 70.0% humidity, 0 mph SSE wind, 0.00 inches rain/24hrs

---
þ Synchronet þ Vertrauen þ Home of Synchronet þ telnet://vert.synchro.net

Also, what is your private IP scheme on your local network (if you don't min posting here or maybe you could email it to me instead)? Do you have 192.168.1.x, 192.168.100.x, etc... or is it something like 10.x.x.x?

I'd love to see it fixed for you.

Ok, i'm not exactly sure where i was muffin' things up at but below is
the code that eventually worked for me. Had a friend dial in and he
claimed it looked great and loves the weather reports. Also no error's
were noted in the logs this time.

if (user.ip_address.indexOf("192.168.0") == 0) {
var weather_ip_address = resolve_ip(system.inet_addr);
} else {
var weather_ip_address = "my.public.ip";
}

Thanks guys for all your help and assistance. I really appreciate it
and love this gem of a program.

... DOS means never having to live hand-to-mouse.

---
þ Synchronet þ ´ TBIRDS BBS ´ telnet://tbirds.dyndns.org

Re: Re: Sync Weather App - syncWXremix
By: Digital Man to KenDB3 on Wed Dec 30 2015 12:39 am

You could log it for the sysop with the the log() function. Use log(LOG_DEBUG,...) if you want it to only be displayed when the sysop has chosen to view debug-level log output.

I totally used this. Thanks for the tip!

Also, I noticed k5jat created an issue on GitHub about a
non-geolocating method for getting the data. The API *does*
support queries that specify either city/state or zip code. I
wonder if you might consider a fallback for cases where IP-based
queries fail that builds a new query URL using SBBS'
system.location variable.

Thanks for pointing it out, I responded there, but I will give you my
thoughts here. I like the idea of a fall back, but my concern with
system.location is it might not match up with the data needed to form
a query. For example, mine is set to just "Rhode Island" with no city.
But, maybe we can come up with something, I am definitely open to
suggestions and am happy to have people throwing in some code.

The modopts.ini could also be used to provide a postal/zip code for the system (but I think the user's location/postal/zip would be better).

I implemented something for the Sysop to define either US Postal ZIP Code or an Aiport Code (ICAO or IATA, but ICAO seems better). This way it will work for US and Non-US.

I'd love a reliable way to get the user's location, but I'm stymied as to how. Quick question though... I remember reading that anything ran as a Logon event could not have interactive input. Is that still true. It is not mentioned in the Wiki for installing doors, but it is mentioned here: http://synchro.net/docs/external_programs.html

My thought is, there is no better way to get an answer than by asking someone. I have users that only put a partial location when creating a user, or type something incorrectly, or just make stuff up. So, I don't think using their user data is a good option. But asking them to provide a ZIP or Airport code within the app itself might work well.

~KenDB3

---
þ Synchronet þ KD3net-Rhode Island's only BBS about nothing. http://bbs.kd3.us

Re: Re: Sync Weather App - syncWXremix
By: tbirdsradio to KenDB3 on Wed Dec 30 2015 03:26 pm

Also, what is your private IP scheme on your local network (if you
don't min posting here or maybe you could email it to me instead)? Do
you have 192.168.1.x, 192.168.100.x, etc... or is it something like

Ok, i'm not exactly sure where i was muffin' things up at but below is
the code that eventually worked for me. Had a friend dial in and he claimed it looked great and loves the weather reports. Also no error's were noted in the logs this time.

if (user.ip_address.indexOf("192.168.0") == 0) {
var weather_ip_address = resolve_ip(system.inet_addr);
} else {
var weather_ip_address = "my.public.ip";
}

This all looks good. But for some reason when I log into your BBS, I see the weather report for your area (Evansville, IN). I'm not sure what... but, I just put together all the recent updates from GitHub into a new ZIP file for download if you feel like giving it a shot. New setup instructions thanks to digital man's suggestion of using ctrl/modopts.ini.

http://bbs.kd3.us/kd3doors.ssjs

ver. 1.01b
I slapped a "b" at the end of the version number, because no one else has tested it but me (that I know of).

Thanks guys for all your help and assistance. I really appreciate it
and love this gem of a program.

I'm really glad you like it!

~KenDB3

---
þ Synchronet þ KD3net-Rhode Island's only BBS about nothing. http://bbs.kd3.us

Re: Re: Sync Weather App - syncWXremix
By: KenDB3 to Digital Man on Thu Dec 31 2015 12:38 am

Re: Re: Sync Weather App - syncWXremix
By: Digital Man to KenDB3 on Wed Dec 30 2015 12:39 am

You could log it for the sysop with the the log() function. Use log(LOG_DEBUG,...) if you want it to only be displayed when the sysop has chosen to view debug-level log output.

I totally used this. Thanks for the tip!

Also, I noticed k5jat created an issue on GitHub about a
non-geolocating method for getting the data. The API *does*
support queries that specify either city/state or zip code. I
wonder if you might consider a fallback for cases where IP-based Ki>> queries fail that builds a new query URL using SBBS'
system.location variable.

Thanks for pointing it out, I responded there, but I will give you my
thoughts here. I like the idea of a fall back, but my concern with
system.location is it might not match up with the data needed to form
a query. For example, mine is set to just "Rhode Island" with no city.
But, maybe we can come up with something, I am definitely open to
suggestions and am happy to have people throwing in some code.

The modopts.ini could also be used to provide a postal/zip code for the system (but I think the user's location/postal/zip would be better).

I implemented something for the Sysop to define either US Postal ZIP Code or an Aiport Code (ICAO or IATA, but ICAO seems better). This way it will work for US and Non-US.

I'd love a reliable way to get the user's location, but I'm stymied as to how. Quick question though... I remember reading that anything ran as a Logon event could not have interactive input. Is that still true. It is not mentioned in the Wiki for installing doors, but it is mentioned here: http://synchro.net/docs/external_programs.html

The programs/scripts configured via SCFG->External Programs->Fixed Events cannot have user interaction. Programs/scripts configured to run via SCFG->External Programs->Online Programs *can* have user interaction.

My thought is, there is no better way to get an answer than by asking someone. I have users that only put a partial location when creating a user, or type something incorrectly, or just make stuff up. So, I don't think using their user data is a good option. But asking them to provide a ZIP or Airport code within the app itself might work well.

Sure. Though, I'd only ask them if you can't derive it from the zip/postal code in their user record or from their current IP address.

digital man

Synchronet "Real Fact" #74:
Rob's alias "digital man" was inspired by a song on Rush's 1982 "Signals" album.
Norco, CA WX: 47.8øF, 64.0% humidity, 4 mph W wind, 0.00 inches rain/24hrs

---
þ Synchronet þ Vertrauen þ Home of Synchronet þ telnet://vert.synchro.net

This all looks good. But for some reason when I log into your BBS, I see the weather report for your area (Evansville, IN). I'm not sure what... but, I j put together all the recent updates from GitHub into a new ZIP file for download if you feel like giving it a shot. New setup instructions thanks to digital man's suggestion of using ctrl/modopts.ini.

ver. 1.01b
I slapped a "b" at the end of the version number, because no one else has tested it but me (that I know of).

Just now got it working. Thanks. Was unable to get my nonip ICAO code
or zip to work but bbsip with no fallback did. Really like the option
to read the full report/or not. Awesome!

... Best file compressor around: DEL *.* (100% compression!)

---
þ Synchronet þ ´ TBIRDS BBS ´ telnet://tbirds.dyndns.org

ver. 1.01b

Just now got it working. Thanks. Was unable to get my nonip ICAO code

Once again i got ahead of myself. Dial up user phoned in and received
this error:

"Error in weather.js api.underground.com returned a 'querynotfound'
error with this description. 'No cities match your search query'."

As previously mentioned, it's working for me locally. Also as stated
in my last message, for "fallback_type", i'm using "bbsip" because
i wasn't able to get the ICAO or zip to work for me.

Never had a "modopts.ini" file in my Ctrl directory so i created one.
Here is the contents of that file:

[syncWX]
wungrndAPIkey = my api key
weathericon_ext = .asc
fallback_type = bbsip
fallback =

Do i have the above right?

... Boost your system's speed by 200% - DEL C:\WINDOWS\*.*

---
þ Synchronet þ ´ TBIRDS BBS ´ telnet://tbirds.dyndns.org

My thought is, there is no better way to get an answer than by asking someone. I have users that only put a partial location when creating a user, or type something incorrectly, or just make stuff up. So, I don't think using their user data is a good option. But asking them to provide a ZIP or Airport code within the app itself might work well.

There are services that you can query from JavaScript where you provide an IP address, and they give you (at least approximately) the location of that IP address. That way, you wouldn't have to ask the user (unless perhaps you wanted to be sure you got more precise information).

A couple search results I found: http://stackoverflow.com/questions/4937517/ip-to-location-using-javascript http://bit.ly/1MIRMRB

Nightfox

---
þ Synchronet þ Digital Distortion: digitaldistortionbbs.com

There are services that you can query from JavaScript where you provide an IP address, and they give you (at least approximately) the location of that IP address. That way, you wouldn't have to ask the user (unless perhaps you wanted to be sure you got more precise information).

A couple search results I found: http://stackoverflow.com/questions/4937517/ip-to-location-using-javascript http://bit.ly/1MIRMRB

We do have 'exec/load/geoip.js' for this, if it still works. Not sure if it or any alternative would be much better or worse than weather underground's existing geo-IP stuff, though.


---
echicken
electronic chicken bbs - bbs.electronicchicken.com - 416-273-7230
þ Synchronet þ electronic chicken bbs - bbs.electronicchicken.com

There are services that you can query from JavaScript where you provide an IP address, and they give you (at least approximately) the location of that IP address. That way, you wouldn't have to ask the user (unless perhaps you wanted to be sure you got more precise information).

A couple search results I found: http://stackoverflow.com/questions/4937517/ip-to-location-using-javascript http://bit.ly/1MIRMRB

We do have 'exec/load/geoip.js' for this, if it still works. Not sure if it or any alternative would be much better or worse than weather underground's existing geo-IP stuff, though.

Ah, interesting. I didn't realize that was there.

Nightfox

---
þ Synchronet þ Digital Distortion: digitaldistortionbbs.com

Re: Re: Sync Weather App - syncWXremix
By: Digital Man to KenDB3 on Wed Dec 30 2015 10:13 pm

The programs/scripts configured via SCFG->External Programs->Fixed Events cannot have user interaction. Programs/scripts configured to run via SCFG->External Programs->Online Programs *can* have user interaction.

Thanks for the clarification. That makes more sense to me now.

My thought is, there is no better way to get an answer than by asking
someone. I have users that only put a partial location when creating a
user, or type something incorrectly, or just make stuff up. So, I
don't think using their user data is a good option. But asking them to
provide a ZIP or Airport code within the app itself might work well.

Sure. Though, I'd only ask them if you can't derive it from the zip/postal code in their user record or from their current IP address.

I agree. Not that I am quite ready for that yet, I have more to learn to get to there. But, I am always open to help. It looks like people like the app, so maybe we can get it onto CVS now? :-)

~KenDB3

---
þ Synchronet þ KD3net-Rhode Island's only BBS about nothing. http://bbs.kd3.us

Works great.. Have it setup as a logon event and as a door. Great job!


Thanks
Mike

---
þ Synchronet þ RetroDigital BBS - rdnetbbs.com

Re: Re: Sync Weather App - syncWXremix
By: tbirdsradio to KenDB3 on Thu Dec 31 2015 09:34 am

Once again i got ahead of myself. Dial up user phoned in and received
this error:

"Error in weather.js api.underground.com returned a 'querynotfound'
error with this description. 'No cities match your search query'."

As previously mentioned, it's working for me locally. Also as stated
in my last message, for "fallback_type", i'm using "bbsip" because
i wasn't able to get the ICAO or zip to work for me.

Never had a "modopts.ini" file in my Ctrl directory so i created one.
Here is the contents of that file:

[syncWX]
wungrndAPIkey = my api key
weathericon_ext = .asc
fallback_type = bbsip
fallback =

Do i have the above right?

That's all correct. I just didn't account for dial-up users. Mainly because I don't have any dial-up access, it didn't really dawn on me.

Can anyone shed some light on how I might test for this scenario? I'm guessing the user's IP comes up as undefined in this case?

~KenDB3

---
þ Synchronet þ KD3net-Rhode Island's only BBS about nothing. http://bbs.kd3.us

Re: Sync Weather App - syncWXremix
By: Nightfox to echicken on Thu Dec 31 2015 12:00 pm

There are services that you can query from JavaScript where you
provide an IP address, and they give you (at least approximately)
the location of that IP address. That way, you wouldn't have to ask
the user (unless perhaps you wanted to be sure you got more precise

information).

A couple search results I found:
http://stackoverflow.com/questions/4937517/ip-to-location-using-java
script http://bit.ly/1MIRMRB

We do have 'exec/load/geoip.js' for this, if it still works. Not sure
if it or any alternative would be much better or worse than weather
underground's existing geo-IP stuff, though.

Ah, interesting. I didn't realize that was there.

Also, I feel like the Weather Underground GeoIP lookup is fairly accurate. It doesn't fail entirely unless it is not a public IP. The inaccuracies that you might see are a different story though. Example, my in-laws are on satellite Internet in a rural area of New York. Their IP always comes up as somewhere in Kansas because of how that network is managed. Nature of the beast, really. You see the same thing in IPs for mobile networks/cellular and other odd stuff (like someone on a VPN).

Kirkman hit on something in a thread on GitHub. Maybe doing a lookup based on what is available to us: Public IP of the User, failover to ZIP Code of the user in their BBS profile, failover to Public IP/ZIP/Airport code of the BBS. Ask the user if they want to change locations, and then store the preference in a JSON database.

In the meantime, I never really thought about Dial-up users and I need some way of sorting that out...

~KenDB3

---
þ Synchronet þ KD3net-Rhode Island's only BBS about nothing. http://bbs.kd3.us

Re: Re: Sync Weather App - syncWXremix
By: KenDB3 to tbirdsradio on Fri Jan 01 2016 08:11 am

[syncWX]
wungrndAPIkey = my api key
weathericon_ext = .asc
fallback_type = bbsip
fallback =

Do i have the above right?

That's all correct. I just didn't account for dial-up users. Mainly because I don't have any dial-up access, it didn't really dawn on me.

Can anyone shed some light on how I might test for this scenario? I'm guessing the user's IP comes up as undefined in this case?

Ken, i'm using "SEXPOTS" (Rob Swindell's brilliant program) to pass the dial-up session over to the bbs. What i can add is that SyncWX worked well in conjunction with SEXPOTS in the original release. I know this
because of the compliments the weather report received when they dialed in.

Unfortunately that takes us back to the time when my bbs would only display the weather report for my city and my city alone. Don't know what that was
about but the new version fixed that for me. When they call back, i'll look to see if the problem repeats itself and let you know.

Best wishes,
Happy New Year!

---
þ Synchronet þ ´ TBIRDS BBS ´ telnet://tbirds.dyndns.org

Can anyone shed some light on how I might test for this scenario? I'm guessing the user's IP comes up as undefined in this case?

Your best bet would be to do a pattern match against user.ip_address to see if it looks like a valid IP address or not. In the case of dial-up users (via SEXPOTS), user.ip_address may be set to their phone number (just a string of digits, sans any hyphens as far as I can tell), if the modem passed caller ID info along.

---
echicken
electronic chicken bbs - bbs.electronicchicken.com - 416-273-7230
þ Synchronet þ electronic chicken bbs - bbs.electronicchicken.com

Re: Sync Weather App - syncWXremix
By: KenDB3 to Nightfox on Fri Jan 01 2016 14:02:38

In the meantime, I never really thought about Dial-up users and I need some way of sorting that out...

That's a good point.. I haven't had dial-up users with my current BBS, so I'm not sure how that works. I wonder if it depends on what you're using to allow dialup users, or if Synchronet will assign some arbitrary IP address to a dialup user..

Nightfox

---
þ Synchronet þ Digital Distortion: digitaldistortionbbs.com

Re: Re: Sync Weather App - syncWXremix
By: KenDB3 to tbirdsradio on Fri Jan 01 2016 08:11 am

Re: Re: Sync Weather App - syncWXremix
By: tbirdsradio to KenDB3 on Thu Dec 31 2015 09:34 am

Once again i got ahead of myself. Dial up user phoned in and received this error:

"Error in weather.js api.underground.com returned a 'querynotfound' error with this description. 'No cities match your search query'."

Do i have the above right?

That's all correct. I just didn't account for dial-up users. Mainly because I don't have any dial-up access, it didn't really dawn on me.

Can anyone shed some light on how I might test for this scenario? I'm guessing the user's IP comes up as undefined in this case?

Assuming the BBS is using SEXPOTS, the user's IP address should be the IP address of the BBS computer, or possibly 127.0.0.1 (I forget which). You can programatically detect a SEXPOTS/dial-up connection by checking if user.connection is a number (e.g. "28800") rather than a protocol string (e.g. "Telnet").

Something like this:
var dialup = (parseInt(user.connection) > 0);

At least, I think would work (without actually trying it). Let me know how it works for you,

digital man

Synchronet "Real Fact" #10:
DOVE-Net was originally an exclusive ("elite") WWIVnet network in O.C., Calif. Norco, CA WX: 60.9øF, 20.0% humidity, 2 mph NW wind, 0.00 inches rain/24hrs

---
þ Synchronet þ Vertrauen þ Home of Synchronet þ telnet://vert.synchro.net

Re: Re: Sync Weather App - syncWXremix
By: KenDB3 to tbirdsradio on Fri Jan 01 2016 08:11 am

That's all correct. I just didn't account for dial-up users. Mainly because I don't have any dial-up access, it didn't really dawn on me.

Dial-up with SEXPOTS worked like a champ for me today! No error in the logs unlike yesterday. Wanted to let you know as soon as i found out. YES!!!

---
þ Synchronet þ ´ TBIRDS BBS ´ telnet://tbirds.dyndns.org

Re: Sync Weather App - syncWXremix
By: KenDB3 to All on Wed Dec 23 2015 11:40:15

Hi KenDB3,

I noticed something a little odd when running syncWXremix from my external programs menu while logged into my BBS. If you press Q any time while it's running (i.e., at the pause at the end of the weather report, or while you're reading the full alert), it quits back to the external programs menu (which is probably expected), but Synchronet doesn't output any of the items for the external programs menu. I can still type a number to run one of the doors though, so it only seems to be affecting the configuration. I'm not sure if this is due to something happening in syncWXremix or due to something with the way Synchronet handles JavaScript mods, but wanted to let you know.

I've captured some screenshots to illustrate what's going on. For instance, this is what my General/Misc. external programs menu normally looks like: http://bit.ly/1PC37JD
In syncWXremix, if I'm at the screen pause:
http://bit.ly/1R5pZ5J
If I press Q right there, it quits back to my General/Misc. doors menu, but none of the doors are listed:
http://bit.ly/1mVQwpr

Nightfox

---
þ Synchronet þ Digital Distortion: digitaldistortionbbs.com

Re: Sync Weather App - syncWXremix
By: Nightfox to KenDB3 on Fri Jan 01 2016 07:27 pm

Re: Sync Weather App - syncWXremix
By: KenDB3 to All on Wed Dec 23 2015 11:40:15

Hi KenDB3,

I noticed something a little odd when running syncWXremix from my external programs menu while logged into my BBS. If you press Q any time while it's running (i.e., at the pause at the end of the weather report, or while you're reading the full alert), it quits back to the external programs menu (which is probably expected), but Synchronet doesn't output any of the items for the external programs menu. I can still type a number to run one of the doors though, so it only seems to be affecting the configuration. I'm not sure if this is due to something happening in syncWXremix or due to something with the way Synchronet handles JavaScript mods, but wanted to let you know.

I've captured some screenshots to illustrate what's going on. For instance, this is what my General/Misc. external programs menu normally looks like: http://bit.ly/1PC37JD
In syncWXremix, if I'm at the screen pause:
http://bit.ly/1R5pZ5J
If I press Q right there, it quits back to my General/Misc. doors menu, but none of the doors are listed:
http://bit.ly/1mVQwpr

I tried to reproduce this, but could not. What revision of exec/xtrn_sec.js do you have? 1.15 is the latest. It sound like the console "abort" flag isn't being cleared when returning from the script (hitting 'Q' or Ctrl-C from the pause prompt will set this flag), but if I can't reproduce the problem it'll be hard to fix.

digital man

Synchronet "Real Fact" #85:
Stephen and Rob have a fledgling podcast at http://techdorks.net (also iTunes). Norco, CA WX: 53.6øF, 24.0% humidity, 12 mph W wind, 0.00 inches rain/24hrs

---
þ Synchronet þ Vertrauen þ Home of Synchronet þ telnet://vert.synchro.net

Re: Sync Weather App - syncWXremix
By: KenDB3 to All on Wed Dec 23 2015 11:40 am

DM said it would be cool if someone made one for Sync, and nolageek had posted that he had started such a project and put it on GitHub.
Well, I decided to work on nola's script (syncWX) and create some new icons based on the Wego set of icons, and finally I came out with a finished product I call syncWXremix.

|01-|03nolageek

---
þ Synchronet þ Capitol Shrill BBS - Washington, DC - capitolshrill.com

Re: Sync Weather App - syncWXremix
By: KenDB3 to All on Wed Dec 23 2015 11:40 am

DM said it would be cool if someone made one for Sync, and nolageek had posted that he had started such a project and put it on GitHub.
Well, I decided to work on nola's script (syncWX) and create some new icons based on the Wego set of icons, and finally I came out with a finished product I call syncWXremix.

I just came across this on github the other day and was checking it out. I love what you did with it, especially moving the config to the mod options file. Good stuff!

|01-|03nolageek

---
þ Synchronet þ Capitol Shrill BBS - Washington, DC - capitolshrill.com

Re: Sync Weather App - syncWXremix
By: tbirdsradio to KenDB3 on Sun Dec 27 2015 10:34 am

Hi KenDB3 - I downloaded this and set it up on my BBS according to
the documentation, but when it runs, it shows this error:
!JavaScript D:\BBS\sbbs\xtrn\syncWX\weather.js line 70: TypeError:
cu.current_observation is undefined

I wrote the original syncWX and I get this error too, but I've only noticed it when calling from the local network, not when I call in from an outside IP. Not sure if that's what you're doing to?

|01-|03nolageek

---
þ Synchronet þ Capitol Shrill BBS - Washington, DC - capitolshrill.com

Re: Sync Weather App - syncWXremix
By: Digital Man to Nightfox on Fri Jan 01 2016 20:37:55

I noticed something a little odd when running syncWXremix from my
external programs menu while logged into my BBS. If you press Q any
time while it's running (i.e., at the pause at the end of the weather
report, or while you're reading the full alert), it quits back to the
external programs menu (which is probably expected), but Synchronet
doesn't output any of the items for the external programs menu. I can

I tried to reproduce this, but could not. What revision of exec/xtrn_sec.js do you have? 1.15 is the latest. It sound like the console "abort" flag isn't being cleared when returning from the script (hitting 'Q' or Ctrl-C from the pause prompt will set this flag), but if I can't reproduce the problem it'll be hard to fix.

My xtrn_sec.js was version 1.14, so I replaced it with 1.15 but still saw the same issue.

I'm still using the Synchronet 1.16c official release and haven't updated to any of the 1.17 builds yet. Do you think that might have something to do with what I'm seeing?

Nightfox

---
þ Synchronet þ Digital Distortion: digitaldistortionbbs.com

Re: Sync Weather App - syncWXremix
By: Nightfox to Digital Man on Fri Jan 01 2016 10:32 pm

Re: Sync Weather App - syncWXremix
By: Digital Man to Nightfox on Fri Jan 01 2016 20:37:55

I noticed something a little odd when running syncWXremix from my
external programs menu while logged into my BBS. If you press Q any
time while it's running (i.e., at the pause at the end of the weather
report, or while you're reading the full alert), it quits back to the
external programs menu (which is probably expected), but Synchronet
doesn't output any of the items for the external programs menu. I can

I tried to reproduce this, but could not. What revision of exec/xtrn_sec.js do you have? 1.15 is the latest. It sound like the console "abort" flag isn't being cleared when returning from the script (hitting 'Q' or Ctrl-C from the pause prompt will set this flag), but if I can't reproduce the problem it'll be hard to fix.

My xtrn_sec.js was version 1.14, so I replaced it with 1.15 but still saw the same issue.

I'm still using the Synchronet 1.16c official release and haven't updated to any of the 1.17 builds yet. Do you think that might have something to do with what I'm seeing?

I reviewed the changes since v3.16 to the files I would think might have an impact on this behavior and couldn't find any changes which seemed related/relevant. It's possible I missed something somewhere.

Can you try the current dev build (3.17a) and see what you find? You can always go back to 3.16c if you have an issue.

digital man

Synchronet "Real Fact" #8:
The name "DOVE-Net" comes from: The Beast's DOmain / VErtrauen network.
Norco, CA WX: 52.2øF, 26.0% humidity, 8 mph WNW wind, 0.00 inches rain/24hrs

---
þ Synchronet þ Vertrauen þ Home of Synchronet þ telnet://vert.synchro.net

Re: Re: Sync Weather App - syncWXremix
By: KenDB3 to tbirdsradio on Fri Jan 01 2016 08:11 am

That's all correct. I just didn't account for dial-up users. Mainly because I don't have any dial-up access, it didn't really dawn on me.

Dial-up with SEXPOTS worked like a champ for me today! No error in the logs unlike yesterday. Wanted to let you know as soon as i found out. YES!!!

I think I know what is happening here. I requested an update from digital man on the way the Javascript variable for the user's IP works. Previously (before 2015-12-16 builds), the object ip_address would report the last IP seen from the previous session for that user (aka, the IP was stored from the last time they logged in), but only until you get through the Logon events.

So, during Logon, I would see my last IP, then when I ran it as a door I would see my current IP. DM changed this behavior recently. I wonder if the error message you saw was from the previous session.

After seeing DM's post about dialup with SEXPOTS, I am hoping that the user's IP would come through as something the script already catches and handles properly, but I probably need more testing.

~KenDB3

---
þ Synchronet þ KD3net-Rhode Island's only BBS about nothing. http://bbs.kd3.us

Re: Sync Weather App - syncWXremix
By: Digital Man to Nightfox on Fri Jan 01 2016 23:15:57

I noticed something a little odd when running syncWXremix from my
external programs menu while logged into my BBS. If you press Q
any time while it's running (i.e., at the pause at the end of the
weather report, or while you're reading the full alert), it quits
back to the external programs menu (which is probably expected),
but Synchronet doesn't output any of the items for the external
programs menu. I can

I'm still using the Synchronet 3.16c official release and haven't
updated to any of the 3.17 builds yet. Do you think that might have
something to do with what I'm seeing?

I reviewed the changes since v3.16 to the files I would think might have an impact on this behavior and couldn't find any changes which seemed related/relevant. It's possible I missed something somewhere.

Can you try the current dev build (3.17a) and see what you find? You can always go back to 3.16c if you have an issue.

I downloaded & copied the latest 3.17a build, but I still see the same issue with my external programs not being written after pressing Q to quit out of syncWXremix. It must be something else with my system, but I'm not sure what that would be. A configuration file perhaps?

Nightfox

---
þ Synchronet þ Digital Distortion: digitaldistortionbbs.com

Re: Sync Weather App - syncWXremix
By: nolageek to tbirdsradio on Fri Jan 01 2016 11:23 pm

!JavaScript D:\BBS\sbbs\xtrn\syncWX\weather.js line 70:
TypeError: cu.current_observation is undefined

I wrote the original syncWX and I get this error too, but I've only noticed it when calling from the local network, not when I call in from an outside IP. Not sure if that's what you're doing to?

Aloha, nolageek! New code was added to the latest version that eliminated the error listed above.

Best,
Happy New Year!

---
þ Synchronet þ ´ TBIRDS BBS ´ telnet://tbirds.dyndns.org

Re: Re: Sync Weather App - syncWXremix
By: tbirdsradio to KenDB3 on Fri Jan 01 2016 07:22 pm

Dial-up with SEXPOTS worked like a champ for me today! No error in the logs unlike yesterday. Wanted to let you know as soon as i found out. YES!!!

This morning with the dial-up user i noticed the error popped up again. 'querynotfound'. 'No cities match your search query'.

Below is how SEXPOTS passes the info into the Terminal Server Log:

SEXPOTS connection detected at 115200 bps
CID: XXXXXXXXXX LAST,FIRST // X = Phone number w/area-code, followed by name

At a lost to explain why all was good yesterday but today the error.

---
þ Synchronet þ ´ TBIRDS BBS ´ telnet://tbirds.dyndns.org

Re: Sync Weather App - syncWXremix
By: Nightfox to Digital Man on Sat Jan 02 2016 09:42 am

Re: Sync Weather App - syncWXremix
By: Digital Man to Nightfox on Fri Jan 01 2016 23:15:57

I noticed something a little odd when running syncWXremix from my
external programs menu while logged into my BBS. If you press Q
any time while it's running (i.e., at the pause at the end of the
weather report, or while you're reading the full alert), it quits
back to the external programs menu (which is probably expected),
but Synchronet doesn't output any of the items for the external
programs menu. I can

I'm still using the Synchronet 3.16c official release and haven't
updated to any of the 3.17 builds yet. Do you think that might have
something to do with what I'm seeing?

I reviewed the changes since v3.16 to the files I would think might have an impact on this behavior and couldn't find any changes which seemed related/relevant. It's possible I missed something somewhere.

Can you try the current dev build (3.17a) and see what you find? You can always go back to 3.16c if you have an issue.

I downloaded & copied the latest 3.17a build, but I still see the same issue with my external programs not being written after pressing Q to quit out of syncWXremix. It must be something else with my system, but I'm not sure what that would be. A configuration file perhaps?

What command-shell are you using? Does the command-shell actually execute the exec/xtrn_sec.js script or is it just using the old internal xtrn_sec function? I think that might explain the difference in behavior seen.

digital man

Synchronet "Real Fact" #64:
Synchronet can dynamically compress and uncompress message bases (using LZH). Norco, CA WX: 63.3øF, 23.0% humidity, 3 mph ESE wind, 0.00 inches rain/24hrs

---
þ Synchronet þ Vertrauen þ Home of Synchronet þ telnet://vert.synchro.net

Re: Sync Weather App - syncWXremix
By: Digital Man to Nightfox on Sat Jan 02 2016 01:35 pm

Re: Sync Weather App - syncWXremix
By: Nightfox to Digital Man on Sat Jan 02 2016 09:42 am

Re: Sync Weather App - syncWXremix
By: Digital Man to Nightfox on Fri Jan 01 2016 23:15:57

I noticed something a little odd when running syncWXremix from my >>> external programs menu while logged into my BBS. If you press Q >>> any time while it's running (i.e., at the pause at the end of the >>> weather report, or while you're reading the full alert), it quits >>> back to the external programs menu (which is probably expected), >>> but Synchronet doesn't output any of the items for the external >>> programs menu. I can

I'm still using the Synchronet 3.16c official release and haven't
updated to any of the 3.17 builds yet. Do you think that might have
something to do with what I'm seeing?

I reviewed the changes since v3.16 to the files I would think might have an impact on this behavior and couldn't find any changes which seemed related/relevant. It's possible I missed something somewhere.

Can you try the current dev build (3.17a) and see what you find? You can always go back to 3.16c if you have an issue.

I downloaded & copied the latest 3.17a build, but I still see the same issue with my external programs not being written after pressing Q to quit out of syncWXremix. It must be something else with my system, but I'm not sure what that would be. A configuration file perhaps?

What command-shell are you using? Does the command-shell actually execute the exec/xtrn_sec.js script or is it just using the old internal xtrn_sec function? I think that might explain the difference in behavior seen.

I think that's it. The internal xtrn_sec function would terminate a program listing when it detected that the console-abort flag was set and the xtrn_sec.js was not (an oversight when porting this logic from C++ to JS).

I fixed xtrn_sec.js to behave the same as the internal xtrn_sec function in this regard and also committed a change to sbbs itself to clear the console-abort flag after executing any JS or Baja modules. This might be overkill, but does fix the observed problem.

digital man

Synchronet "Real Fact" #67:
Robert D. Bouman, the author of SyncEdit, died in the mid to late 1990's. Norco, CA WX: 63.2øF, 23.0% humidity, 3 mph SE wind, 0.00 inches rain/24hrs

---
þ Synchronet þ Vertrauen þ Home of Synchronet þ telnet://vert.synchro.net

Re: Sync Weather App - syncWXremix
By: Digital Man to Nightfox on Sat Jan 02 2016 13:35:25

What command-shell are you using? Does the command-shell actually execute the exec/xtrn_sec.js script or is it just using the old internal xtrn_sec function? I think that might explain the difference in behavior seen.

I'm using my own custom command shell (written in JavaScript). It's calling the bbs.xtrn_sec() function. I updated my command shell to run xtrn_sec.js instead:
bbs.exec("?xtrn_sec.js");
With that change, the issue with not displaying the menu items after pressing 'Q' is no longer happening. And I'm still using version 1.15 of xtrn_sec.js (with the Synchronet 3.17 nightly build).

Nightfox

---
þ Synchronet þ Digital Distortion: digitaldistortionbbs.com

Re: Sync Weather App - syncWXremix
By: Nightfox to Digital Man on Sat Jan 02 2016 05:08 pm

Re: Sync Weather App - syncWXremix
By: Digital Man to Nightfox on Sat Jan 02 2016 13:35:25

What command-shell are you using? Does the command-shell actually execute the exec/xtrn_sec.js script or is it just using the old internal xtrn_sec function? I think that might explain the difference in behavior seen.

I'm using my own custom command shell (written in JavaScript). It's calling the bbs.xtrn_sec() function. I updated my command shell to run xtrn_sec.js instead:
bbs.exec("?xtrn_sec.js");
With that change, the issue with not displaying the menu items after pressing 'Q' is no longer happening. And I'm still using version 1.15 of xtrn_sec.js (with the Synchronet 3.17 nightly build).

Okay, so you actually found 2 problems:
1. The xtrn_sec.js was not accurately mimicking the behavior of the old hard-coded bbs.xtrn_sec() function - I consider this a bug.
2. The console-aborted flag was not being cleared after executing an external module (not exactly a bug, but something needs to be done to address this).

I committed fixes to both. xtrn_sec.js itself could have cleared the console-aborted flag after executing a program, but that wouldn't have provided the fix for the hard-coded xtrn_sec() and other means by which external modules are executed.

digital man

Synchronet "Real Fact" #17:
Rob Swindell first learned to program in C by hacking on WWIV BBS software. Norco, CA WX: 55.8øF, 49.0% humidity, 2 mph SE wind, 0.00 inches rain/24hrs

---
þ Synchronet þ Vertrauen þ Home of Synchronet þ telnet://vert.synchro.net

Re: Sync Weather App - syncWXremix
By: nolageek to KenDB3 on Fri Jan 01 2016 11:12 pm

I just came across this on github the other day and was checking it out. I love what you did with it, especially moving the config to the mod options file. Good stuff!

Well, I owe you a huge thanks for putting the code out there. I got inspired and started working on it. And the modopts bit was all Digital Man, he's made some great suggestions, along with other folks (huge thanks to you guys too).

~KenDB3

---
þ Synchronet þ KD3net-Rhode Island's only BBS about nothing. http://bbs.kd3.us

Re: Sync Weather App - syncWXremix
By: nolageek to tbirdsradio on Fri Jan 01 2016 11:23 pm

Hi KenDB3 - I downloaded this and set it up on my BBS according
to the documentation, but when it runs, it shows this error:
!JavaScript D:\BBS\sbbs\xtrn\syncWX\weather.js line 70:
TypeError: cu.current_observation is undefined

I wrote the original syncWX and I get this error too, but I've only noticed it when calling from the local network, not when I call in from an outside IP. Not sure if that's what you're doing to?

Hopefully we have the Private IP thing worked out between a few folks working on it. Grab the latest and greatest from GitHub and try connecting from your local network again. Let me know what errors you see, if any. There is a Debug level Log message too that can show you ultimately what query is being sent to wunderground, which might be able to help pinpoint the problem.

~KenDB3

---
þ Synchronet þ KD3net-Rhode Island's only BBS about nothing. http://bbs.kd3.us

Re: Re: Sync Weather App - syncWXremix
By: tbirdsradio to KenDB3 on Sat Jan 02 2016 11:40 am

Dial-up with SEXPOTS worked like a champ for me today! No error in
the logs unlike yesterday. Wanted to let you know as soon as i found
out. YES!!!

This morning with the dial-up user i noticed the error popped up again. 'querynotfound'. 'No cities match your search query'.

Below is how SEXPOTS passes the info into the Terminal Server Log:

SEXPOTS connection detected at 115200 bps
CID: XXXXXXXXXX LAST,FIRST // X = Phone number w/area-code, followed by name

At a lost to explain why all was good yesterday but today the error.

Hmmm, I am trying to wrap my brain around it too. Could you do me a favor? Could you change your LogLevel to be Debugging? This should also generate a Debug message in your Synchronet Control Panel that has the actual String sent to wunderground.com.

Changing the LogLevel is done in /sbbs/ctrl/sbbs.ini

More info about it is listed here: http://wiki.synchro.net/config:sbbs.ini#loglevel

And see if you can grab 1 or 2 examples. If you can actually dial into your own board (or if you can have a friend specifically test for you), could you grab the string generated when the Door is run during the Logon event, and then again later when the Door is run as an External from the External Programs menu. It is to help with what I described in my earlier message.

You can email me results or you can post them here. Email: kendb3 at bbs.kd3.us

~KenDB3

---
þ Synchronet þ KD3net-Rhode Island's only BBS about nothing. http://bbs.kd3.us

Re: Sync Weather App - syncWXremix
By: Digital Man to Nightfox on Sat Jan 02 2016 02:10 pm

I fixed xtrn_sec.js to behave the same as the internal xtrn_sec function in this regard and also committed a change to sbbs itself to clear the console-abort flag after executing any JS or Baja modules. This might be overkill, but does fix the observed problem.

Is there anything I should do/add that would help avoid this for anyone that hasn't updated? Just curious.

~KenDB3

---
þ Synchronet þ KD3net-Rhode Island's only BBS about nothing. http://bbs.kd3.us

Re: Sync Weather App - syncWXremix
By: KenDB3 to Digital Man on Sat Jan 02 2016 11:16 pm

Re: Sync Weather App - syncWXremix
By: Digital Man to Nightfox on Sat Jan 02 2016 02:10 pm

I fixed xtrn_sec.js to behave the same as the internal xtrn_sec function in this regard and also committed a change to sbbs itself to clear the console-abort flag after executing any JS or Baja modules. This might be overkill, but does fix the observed problem.

Is there anything I should do/add that would help avoid this for anyone that hasn't updated? Just curious.

The script could clear the abort flag before terminating. The code would be:
console.aborted = false;

digital man

Synchronet "Real Fact" #36:
Synchronet's Windows Control Panel is built with Borland C++ Builder.
Norco, CA WX: 54.5øF, 53.0% humidity, 0 mph WSW wind, 0.00 inches rain/24hrs

---
þ Synchronet þ Vertrauen þ Home of Synchronet þ telnet://vert.synchro.net

Re: Sync Weather App - syncWXremix
By: Digital Man to KenDB3 on Sat Jan 02 2016 08:27 pm

Re: Sync Weather App - syncWXremix
By: KenDB3 to Digital Man on Sat Jan 02 2016 11:16 pm

Re: Sync Weather App - syncWXremix
By: Digital Man to Nightfox on Sat Jan 02 2016 02:10 pm

I fixed xtrn_sec.js to behave the same as the internal xtrn_sec function in this regard and also committed a change to sbbs itself to clear the console-abort flag after executing any JS or Baja modules. This might be overkill, but does fix the observed problem.

Is there anything I should do/add that would help avoid this for anyone that hasn't updated? Just curious.

The script could clear the abort flag before terminating. The code would be:
console.aborted = false;

Something else that should be addressed: When using a non-ANSI terminal (or just disabling ANSI for the user) causes the door to not display very well:

.---. .--.
.-( ). )
(___.__)___)___)
`,`,`,`,`,`,`
,`,`,`,`,`,`,`Your Location: Irvine, CACurrent Conditions: RainTemp: 55.2ø F (
12.9ø C)Sunrise/Sunset: 6:56 / 16:56Lunar Phase: Waning CrescentWind: From the S
SW at 2.7 MPH UV Index: 0TuesdayRainLo / Hi49 / 57 øFWednesdayRainLo / Hi48 /
55 øFThursdayRainLo / Hi44 / 55 øFFridayClearLo / Hi44 / 57 øFAreal Flood Advis
ory, Flash Flood Watch: 2:34 PM PST on January 5, 2016Expires 4:00 AM PST on Jan
uary 07, 2016[û] Read the full alert? [Yes] No

digital man

Synchronet "Real Fact" #10:
DOVE-Net was originally an exclusive ("elite") WWIVnet network in O.C., Calif. Norco, CA WX: 52.7øF, 96.0% humidity, 7 mph ENE wind, 0.57 inches rain/24hrs

---
þ Synchronet þ Vertrauen þ Home of Synchronet þ telnet://vert.synchro.net

Re: Sync Weather App - syncWXremix
By: Digital Man to KenDB3 on Tue Jan 05 2016 04:42 pm

Something else that should be addressed: When using a non-ANSI terminal (or just disabling ANSI for the user) causes the door to not display very well:

.---. .--.
.-( ). )
(___.__)___)___)
`,`,`,`,`,`,`
,`,`,`,`,`,`,`Your Location: Irvine, CACurrent Conditions: RainTemp: 55.2ø F (
12.9ø C)Sunrise/Sunset: 6:56 / 16:56Lunar Phase: Waning CrescentWind: From the S
SW at 2.7 MPH UV Index: 0TuesdayRainLo / Hi49 / 57 øFWednesdayRainLo / Hi48 /
55 øFThursdayRainLo / Hi44 / 55 øFFridayClearLo / Hi44 / 57 øFAreal Flood Advis
ory, Flash Flood Watch: 2:34 PM PST on January 5, 2016Expires 4:00 AM PST on Jan
uary 07, 2016[û] Read the full alert? [Yes] No

Whoa, that's ugly. Anyone have some tips to keep the spacing and CRLFs?

~KenDB3

---
þ Synchronet þ KD3net-Rhode Island's only BBS about nothing. http://bbs.kd3.us

Whoa, that's ugly. Anyone have some tips to keep the spacing and CRLFs?

I usually just leave such users to wallow in the inadequacy of their terminals.

However, my guess would be that it's related to the copious use of console.gotoxy() in this script. You might want to throw in a 'simple' view for non-ANSI terminals, eg:

if (!(user.settings&USER_ANSI)) {
// print out weather info line-by-line
} else {
// your existing display-routine goes here
}

You can still achieve a certain amount of formatting (spaces between fields, padding numbers of unknown length) by using the format() function. This could help to retain the look that you're going for without using positioning sequences.
---
echicken
electronic chicken bbs - bbs.electronicchicken.com - 416-273-7230
þ Synchronet þ electronic chicken bbs - bbs.electronicchicken.com

Re: Sync Weather App - syncWXremix
By: echicken to KenDB3 on Tue Jan 05 2016 11:47 pm

Whoa, that's ugly. Anyone have some tips to keep the spacing and CRLFs?

I usually just leave such users to wallow in the inadequacy of their terminals.

However, my guess would be that it's related to the copious use of console.gotoxy() in this script. You might want to throw in a 'simple' view for non-ANSI terminals, eg:

if (!(user.settings&USER_ANSI)) {
// print out weather info line-by-line
} else {
// your existing display-routine goes here
}

echicken has good advice, but I just wanted to point out that the "more correct" method of checking the user's terminal capabilities would be: if(console.term_supports(USER_ANSI)) {
// send ANSI stuff
} else {
// send non-ANSI stuff
}

digital man

Synchronet "Real Fact" #4:
Synchronet version 3 for Win32 development began in 1999.
Norco, CA WX: 51.5øF, 84.0% humidity, 8 mph ENE wind, 0.57 inches rain/24hrs

---
þ Synchronet þ Vertrauen þ Home of Synchronet þ telnet://vert.synchro.net

Re: Sync Weather App - syncWXremix
By: Digital Man to echicken on Tue Jan 05 2016 10:24 pm

Whoa, that's ugly. Anyone have some tips to keep the spacing and
CRLFs?

I usually just leave such users to wallow in the inadequacy of their
terminals.

However, my guess would be that it's related to the copious use of
console.gotoxy() in this script. You might want to throw in a
'simple' view for non-ANSI terminals, eg:

if (!(user.settings&USER_ANSI)) {
// print out weather info line-by-line
} else {
// your existing display-routine goes here
}

echicken has good advice, but I just wanted to point out that the "more correct" method of checking the user's terminal capabilities would be: if(console.term_supports(USER_ANSI)) {
// send ANSI stuff
} else {
// send non-ANSI stuff
}

You guys rock!

Working on it right now.

GitHub page will have the fix shortly if you want to try it out. Just need to adjust for your extremely helpful input.

~KenDB3

---
þ Synchronet þ KD3net-Rhode Island's only BBS about nothing. http://bbs.kd3.us

Re: Sync Weather App - syncWXremix
By: KenDB3 to Digital Man on Wed Jan 06 2016 01:33 am

Whoa, that's ugly. Anyone have some tips to keep the spacing and
CRLFs?

I usually just leave such users to wallow in the inadequacy of their
terminals.

However, my guess would be that it's related to the copious use of
console.gotoxy() in this script. You might want to throw in a
'simple' view for non-ANSI terminals, eg:

if (!(user.settings&USER_ANSI)) {
// print out weather info line-by-line
} else {
// your existing display-routine goes here
}

echicken has good advice, but I just wanted to point out that the
"more correct" method of checking the user's terminal capabilities
would be: if(console.term_supports(USER_ANSI)) {
// send ANSI stuff
} else {
// send non-ANSI stuff
}

You guys rock!
Working on it right now.
GitHub page will have the fix shortly if you want to try it out. Just need to adjust for your extremely helpful input.

Fix is up: https://github.com/KenDB3/syncWXremix

I'll package this for a new release soon. I was hoping to hear back from Tbird if the code for detecting Dialup worked well.

Just FYI, he was seeing the Caller ID phone number in the user.ip_address, which was causing the issue.

~KenDB3

---
þ Synchronet þ KD3net-Rhode Island's only BBS about nothing. http://bbs.kd3.us

echicken has good advice, but I just wanted to point out that the "more correct" method of checking the user's terminal capabilities would be: if(console.term_supports(USER_ANSI)) {

Good to know. I often wondered what the 'terminal flags' were, for use with console.term_supports, but never bothered to ask.

---
echicken
electronic chicken bbs - bbs.electronicchicken.com - 416-273-7230
þ Synchronet þ electronic chicken bbs - bbs.electronicchicken.com

Re: Sync Weather App - syncWXremix
By: echicken to Digital Man on Wed Jan 06 2016 01:33 am

echicken has good advice, but I just wanted to point out that the "more correct" method of checking the user's terminal capabilities would be: if(console.term_supports(USER_ANSI)) {

Good to know. I often wondered what the 'terminal flags' were, for use with console.term_supports, but never bothered to ask.

Ah, here are the terminal flags:

USER_ANSI
USER_COLOR
USER_NO_EXASCII
USER_RIP
USER_WIP
USER_HTML

Those last likely never set.

If multiple flags are passed-in (OR'd together with '|'), then all of those terminal settings must be enabled for the function to return 'true'.

digital man

Synchronet "Real Fact" #61:
Name of Synchronet PCMS compiler/language "Baja" was coined by Michael Swindell.
Norco, CA WX: 49.9øF, 90.0% humidity, 5 mph WSW wind, 0.33 inches rain/24hrs

---
þ Synchronet þ Vertrauen þ Home of Synchronet þ telnet://vert.synchro.net

Re: Sync Weather App - syncWXremix
By: tbirdsradio to nolageek on Sat Jan 02 2016 11:31 am

Aloha, nolageek! New code was added to the latest version that eliminated the error listed above.

Awesome, I'm probably going to switch to this new one pretty soon. Has a few more features than the one I did. :)

|01-|03nolageek

---
þ Synchronet þ Capitol Shrill BBS - Washington, DC - capitolshrill.com

Re: Re: Sync Weather App - syncWXremix
By: KenDB3 to Digital Man on Thu Dec 31 2015 03:43 pm

I agree. Not that I am quite ready for that yet, I have more to learn to get to there. But, I am always open to help. It looks like people like the app, so maybe we can get it onto CVS now? :-)

It sounds like you're already using GitHub for revision control of this project. If you're happy with GitHub, then it might be best to just leave it there and make "releases" to Synchronet sysops the old fashioned way (e.g. uploading .zip files to BBSes, FTP servers, web-sites).

Though we tend to use the Synchronet CVS repository like a package distribution system, that's not really what it's intended use is for and often what's in CVS is broken in some way, sometimes intentionally, at least temporarily. Having "official" releases outside of CVS is preferred. I was mainly just offering CVS access to you for the revision control of this project, but if you already have it in GitHub, then you have that part covered (just remember to check-in early and often). :-)

digital man

Synchronet "Real Fact" #86:
Synchronet Message Base and its library, SMBLIB, was used by AXiS BBS Software. Norco, CA WX: 49.6øF, 95.0% humidity, 4 mph WNW wind, 0.51 inches rain/24hrs

---
þ Synchronet þ Vertrauen þ Home of Synchronet þ telnet://vert.synchro.net

Re: Sync Weather App - syncWXremix
By: nolageek to tbirdsradio on Wed Jan 06 2016 04:08 pm

Awesome, I'm probably going to switch to this new one pretty soon. Has a few more features than the one I did. :)

You have no idea how glad I am you like it! I really appreciate you putting it out there in the first place, or else I never would have been inspired to work on it. Thanks nolageek!

~KenDB3

---
þ Synchronet þ KD3net-Rhode Island's only BBS about nothing. http://bbs.kd3.us

Re: Re: Sync Weather App - syncWXremix
By: Digital Man to KenDB3 on Wed Jan 06 2016 01:54 pm

like the app, so maybe we can get it onto CVS now? :-)

It sounds like you're already using GitHub for revision control of this project. If you're happy with GitHub, then it might be best to just leave it there and make "releases" to Synchronet sysops the old fashioned way (e.g. uploading .zip files to BBSes, FTP servers, web-sites).

Though we tend to use the Synchronet CVS repository like a package distribution system, that's not really what it's intended use is for and often what's in CVS is broken in some way, sometimes intentionally, at least temporarily. Having "official" releases outside of CVS is preferred. I was mainly just offering CVS access to you for the revision control of this project, but if you already have it in GitHub, then you have that part covered (just remember to check-in early and often). :-)

I get it now, that makes sense. I really chose GitHub because I wanted to keep the history/lineage from nolageek to show how the script started, and since it was a Gist already I tracked down how to turn that into a full repository. There was suprisingly little on how to do it, but GitHub's tech support sent me great instructions.

However, I am getting used to it, and (dare I say) I like it. In my head, I really do think of Sync CVS as a distro system, it's where I get some great games!

I also wanted to say thank you, to you and the other great folks here that are helpful. I was rather scared to release this out to folks because I am not a programmer. I had one programming course that I actually finished, and I passed by the skin of my teeth (and probably because the professor liked me). I just got inspired to work on it, and realized other people might like to use it. It's been a challenge.. a really fun challenge, lol. And I am learning as I go. But, again, you are really helpful and incredibly patient, and that made this a lot of fun. So, if you see some rookie mistakes, it's because this really is my first rodeo, and I barely know what I am doing... but I try to make it look good anyway :-P

~KenDB3

---
þ Synchronet þ KD3net-Rhode Island's only BBS about nothing. http://bbs.kd3.us

Re: Re: Sync Weather App - syncWXremix
By: KenDB3 to Digital Man on Wed Jan 06 2016 10:41 pm

I also wanted to say thank you, to you and the other great folks here that are helpful.

You're very welcome. And of course, we all appreciate those who contribute stuff, including you!


digital man

Synchronet "Real Fact" #2:
Synchronet version 2 was written in C and 8086 assembly programming languages. Norco, CA WX: 47.9øF, 91.0% humidity, 3 mph WNW wind, 0.60 inches rain/24hrs

---
þ Synchronet þ Vertrauen þ Home of Synchronet þ telnet://vert.synchro.net

Re: Re: Sync Weather App - syncWXremix
By: KenDB3 to Digital Man on Wed Jan 06 2016 10:41 pm

However, I am getting used to it, and (dare I say) I like it. In my head, I really do think of Sync CVS as a distro system, it's where I get some great games!

apt-get upgrade-synchronet

:)

---
þ Synchronet þ realitycheckBBS -- http://realitycheckBBS.org

Re: Re: Sync Weather App - syncWXremix
By: Poindexter Fortran to KenDB3 on Thu Jan 07 2016 06:53 am

However, I am getting used to it, and (dare I say) I like it. In my
head, I really do think of Sync CVS as a distro system, it's where I
get some great games!

apt-get upgrade-synchronet

:)

Win32 :-P

But yes. While we are at it, SUDO make me a sandwich.

https://xkcd.com/149/

~KenDB3

---
þ Synchronet þ KD3net-Rhode Island's only BBS about nothing. http://bbs.kd3.us