UPnP support

sullrich

Provide a patch in diff -rub format for all of your recent posts. At this point its becoming hard to follow you.

rsw686

Provide a patch in diff -rub format for all of your recent posts. At this point its becoming hard to follow you.

I think this is what you want.

miniupnpd.inc


36a37
> 	
40c41,43
< 			$start .= "killall miniupnpd; sleep 3; /usr/local/sbin/miniupnpd -p 2869{$ifaces_final}";
---
> 			$start .= "/usr/bin/killall miniupnpd\n";
> 			$start .= "while [ `ps -A | grep /usr/local/sbin/miniupnpd | grep -v grep | wc -l` != 0 ]; do sleep 1; done\n";
> 			$start .= "/usr/local/sbin/miniupnpd -p 2869{$ifaces_final}";
57c60,66
< 		start_service("miniupnpd");
---
> 
> 		/* if not ONE instance running lets start */
> 		/* or if $_POST data as user is changing settings */
> 		if((int)exec("ps -A | grep /usr/local/sbin/miniupnpd | grep -v grep | wc -l") != 1 || $_POST['interface_array']) {
> 			start_service("miniupnpd");
> 		} 
> 		
61c70
< ?>
\ No newline at end of file
---
> ?>

sullrich

diff -rub

rsw686

@sullrich:

diff -rub

Sorry here it is again.


--- miniupnpd.inc	2006-10-07 16:23:47.000000000 -0400
+++ miniupnpdnew.inc	2006-10-07 16:28:13.000000000 -0400
@@ -37,7 +37,9 @@
 			$start = "# Clear existing rules and rdr entries \n";
 			$start .= "/sbin/pfctl -aminiupnpd -Fr 2>&1 >/dev/null\n";
 			$start .= "/sbin/pfctl -aminiupnpd -Fn 2>&1 >/dev/null\n";
-			$start .= "killall miniupnpd; sleep 3; /usr/local/sbin/miniupnpd -p 2869{$ifaces_final}";
+			$start .= "/usr/bin/killall miniupnpd\n";
+			$start .= "while [ `ps -A | grep /usr/local/sbin/miniupnpd | grep -v grep | wc -l` != 0 ]; do sleep 1; done\n";
+			$start .= "/usr/local/sbin/miniupnpd -p 2869{$ifaces_final}";

 			/* override wan ip address, common for carp, etc */
 			if($overridewanip)
@@ -54,7 +56,13 @@
 			    )
 			);
 		}
+
+		/* if not ONE instance running lets start */
+		/* or if $_POST data as user is changing settings */
+		if((int)exec("ps -A | grep /usr/local/sbin/miniupnpd | grep -v grep | wc -l") != 1 || $_POST['interface_array']) {
 		start_service("miniupnpd");
+		} 
+		
 		config_unlock();
 		conf_mount_ro();
 	}

sullrich

Commited. Thanks.

Skud

Excellent work everybody.

I've paid my $50 through bradenmcg. It was sent to the general project with the instructions to distribute it as seen fit.

I've yet to try the latest miniupnpd builds or even RC3 for that matter. At the moment my main PC and server/domain controller are in peices all over my living room floor being upgraded. Hopefully this weekend it'll be up and running and I can give the latest code a try.

Keep up the good work!!

UPnP support is going to be a huge feature for the DIY crowd.

Thanks!!
Riley

ZPrime

I also donated another $50 as promised in the initial post… I sent both mine and Skud's to the project since I know there have been several people working on this now after databeestje did the initial stuff and I wanted to let you guys distribute it as you see fit. :)

It's still a pain in the ass to install on embedded though - that's the last thing I'd like to see resolved but I understand if it doesn't make it to 1.0 final. As long as there is some way (even unsupported) to make it work on embedded I stay happy. :D

jeroen234

i just used this script to update my miniupnp on my soekris to the 13 oct 18:30 uhr version
it will get the files from the pfsense packages server and move them to the correct maps

/etc/rc.conf_mount_rw
cd /tmp/
mkdir upnp
cd upnp
fetch http://pfsense.org/packages/config/miniupnpd/miniupnpd.inc 
mv -f miniupnpd.inc /usr/local/pkg/miniupnpd.inc
chmod 755 /usr/local/pkg/miniupnpd.inc
fetch http://pfsense.org/packages/config/miniupnpd/miniupnpd.xml 
mv -f miniupnpd.xml /usr/local/pkg/miniupnpd.xml
chmod 755 /usr/local/pkg/miniupnpd.xml
fetch http://pfsense.org/packages/config/miniupnpd/status_upnp.php 
mv -f status_upnp.php /usr/local/www/status_upnp.php
chmod 755 /usr/local/www/status_upnp.php
fetch http://pfsense.org/packages/config/miniupnpd/sbin/miniupnpd 
mv -f miniupnpd /usr/local/sbin/miniupnpd
chmod 755 /usr/local/sbin/miniupnpd
cd ..
rmdir upnp
/etc/rc.conf_mount_ro

rsw686

@bradenmcg:

It's still a pain in the ass to install on embedded though - that's the last thing I'd like to see resolved but I understand if it doesn't make it to 1.0 final. As long as there is some way (even unsupported) to make it work on embedded I stay happy. :D

I added the miniupnpd files to the 1.0-RELEASE image.

http://wgnrs.dynalias.com:81/pfsense/pfSense-Embedded-1.0-RELEASE-Miniupnpd.img.gz

I also updated the script sh-custom-image.sh for 1.0-RELEASE see

http://forum.pfsense.org/index.php/topic,1788.msg13972.html#msg13972

ZPrime

Very cool, thanks rsw686!

I'm guessing that from RC3 I can't upgrade to 1.0 final, right? Need to pull the CF out of the box again and flash?

rsw686

@bradenmcg:

Very cool, thanks rsw686!

I'm guessing that from RC3 I can't upgrade to 1.0 final, right? Need to pull the CF out of the box again and flash?

I thought I saw an upgrade, but honestly since your going from prerelease to an actual release I would just flash it.

hoba

Upgrades are available at the mirrors too: http://pfsense.com/mirror.php?section=updates

ZPrime

I thought I saw an upgrade, but honestly since your going from prerelease to an actual release I would just flash it.

On your recommendation I reflashed with your pre-built embedded miniupnp image…

It's working, but I'm getting a couple odd behaviors on startup.

During boot, the synching packages/startup scripts runs miniupnp.sh twice in a row. Is this by design?

The other thing I'm seeing is that on the console, the menu system comes up and sits for a while, and then I see the message "Terminated" and the menu redisplays itself.

I imagine there's something going on behind the scenes but i'm not seeing anything in the syslog, at least not through the web.

rsw686

@bradenmcg:

I thought I saw an upgrade, but honestly since your going from prerelease to an actual release I would just flash it.

On your recommendation I reflashed with your pre-built embedded miniupnp image…

It's working, but I'm getting a couple odd behaviors on startup.

During boot, the synching packages/startup scripts runs miniupnp.sh twice in a row. Is this by design?

The other thing I'm seeing is that on the console, the menu system comes up and sits for a while, and then I see the message "Terminated" and the menu redisplays itself.

I imagine there's something going on behind the scenes but i'm not seeing anything in the syslog, at least not through the web.

The two miniupnp.sh scripts starting is a bug in 1.0-RELEASE. See http://cvstrac.pfsense.com/tktview?tn=1124 It should say stopping miniupnpd.sh and then starting miniupnpd.sh. The stop / start code is right but the message displayed is wrong.

Looks like this right

Executing rc.d items…
Starting /usr/local/etc/rc.d/miniupnpd.sh...
done.
Starting /usr/local/etc/rc.d/miniupnpd.sh...
done.

Not sure on the menu issue, but thats not from miniupnpd. Everything is the same as the 1.0-RELEASE image except for the miniupnpd files and the miniupnpd installed packages section in the config. The files don't modify the system files, they are just added to the proper directorys just as if you installed it from the package manager.

How long does it take to say terminated. I'm assuming this is on the serial console as its the embedded image. You using hyperterminal in windows or some other terminal program. I am actually curious if mine does that as well. I just never leave it open.

EDIT:

I just looked at status.php in DMESG. I am getting the terminated. Must be something changed in 1.0-RELEASE as I don't remember seeing that before.

Enter an option:
Terminated

The Terminated is the /etc/rc.initial script being terminated and then restarted since it is in a loop. Something is killing the process.

rsw686

You know the terminated on the console issue isn't a bug. Mine shows terminated 4 times in dmesg on the status.php page. However it looks to be linked to the dhcp renewal on the wan interface. Which makes sense as the menu shows the current ip address for the WAN interface so it gets termianted and restarted to reflect the current WAN IP.

ZPrime

Actually, it looks like this…

Executing rc.d items...
Starting /usr/local/etc/rc.d/miniupnpd.sh...
Starting /usr/local/etc/rc.d/miniupnpd.sh...
done.

There's no "done" between each invocation of the script, which I found a little odd (and why I considered it might be a problem at all).

The "Terminated" thing makes sense though, I didn't really investigate it too hard as I've been playing Xbox instead (yay UPnP). ;D

billm

@bradenmcg:

Actually, it looks like this…

Executing rc.d items...
Starting /usr/local/etc/rc.d/miniupnpd.sh...
Starting /usr/local/etc/rc.d/miniupnpd.sh...
done.

That's a typo. It's purely cosmetic and is already fixed in CVS. I wouldn't waste your time on chasing down a bug there ;) Thanks

–Bill

rsw686

@bradenmcg:

Actually, it looks like this…

Executing rc.d items...
Starting /usr/local/etc/rc.d/miniupnpd.sh...
Starting /usr/local/etc/rc.d/miniupnpd.sh...
done.
There's no "done" between each invocation of the script, which I found a little odd (and why I considered it might be a problem at all).

The "Terminated" thing makes sense though, I didn't really investigate it too hard as I've been playing Xbox instead (yay UPnP). ;D

Are you sure you just didn't miss the done. Check status.php at the bottom in the DMESG section. Its not really possible for it not to show done. Look at the code.

for FILE in /usr/local/etc/rc.d/.sh; do
echo -n " Starting ${FILE}…"
sh $FILE stop >>/tmp/bootup_messages 2>&1 &
echo "done."
done
for FILE in /usr/local/etc/rc.d/.sh; do
echo -n " Starting ${FILE}..."
sh $FILE start >>/tmp/bootup_messages 2>&1 &
echo "done."
done

Reguardless your install seems to be working just fine as your playing xbox. ;)

rsw686

On miniupnpd settings page if I select two interfaces say LAN and OPT1 when it writes that setting to the config.xml file it stores it as lan,opt1 instead of lan opt1. Due to this the miniupnpd.inc script fails. Also when returning to the miniupnpd settings page it will not show any interfaces selected.

A temp fix I am using is putting the following code in miniupnpd.inc above if ($interface_array)

/* for some reason interface array is stored as lan,opt1 not lan opt1 */
$interface_array = str_replace(","," ",$interface_array);

sullrich

Are you sure you are on 1.0-RELEASE? I fixed that bug recently…