|
Patch for the director "Bug" in virtual host port verifcation: msg#00023java.enhydra.general
-----BEGIN PGP SIGNED MESSAGE----- Hash: SHA1 Hi! Since 2 years ago, we are using enhydra director at the office. It's working great but there is a feature in director that never worked correctly: the port validtion in the virtualserver restriction. The port validation run fine for client and server restriction but not for virtualserver restriction. At the moment, we really need these feature working so i made a patch and now the port is verifed when specified in a virtualserver restriction. The conf entry <Application prefix="/admin"> <AppServer host="192.168.5.11" port="9001" /> <Restrict virtualserver="myhost.dyndns.org" port="8007" /> </Application> will now work properly and only allow connection made to the vituralhost "myhost.dyndns.org" using port 8007 be passed to the enhydra application. I've seen a very few post talking about this "bug". So i put here the patch i made that correct this behaviour. It's very small and simple. If someone from the development team want to give a try i will be happy. Regards. João Paulo Ribeiro [root@host common]# diff edir_load_choose.c edir_load_choose.c.old | more 201,205d200 < } else if ((rp->port != 0) && < (local_addr->sin_port != htons(rp->port))) { < EDIR_DEBUG_EVERYTHING(("Port %d doesn't match %d", \ < ntohs(local_addr->sin_port), \ < rp->port)); [root@host common]# -----BEGIN PGP SIGNATURE----- Version: GnuPG v1.2.1 (MingW32) Comment: Using GnuPG with Mozilla - http://enigmail.mozdev.org iD8DBQE/ZtoVbwM7R+C+9B0RArj8AJ9VFRXcOkrP2/IgdpSKWpT7EFSUSwCfSxEx h7JMy+IETgssR8dWyLlfGZE= =hM7L -----END PGP SIGNATURE-----
|
|
| <Prev in Thread] | Current Thread | [Next in Thread> |
|---|---|---|
| Previous by Date: | Re: Enhydra 5.1, Boris Klug |
|---|---|
| Next by Date: | [Ann] Jivan 1.0 RC 1, Arno Schatz |
| Previous by Thread: | Enhydra 5.1 RC2, Madl Alfred |
| Next by Thread: | [Ann] Jivan 1.0 RC 1, Arno Schatz |
| Indexes: | [Date] [Thread] [Top] [All Lists] |
| News | FAQ | advertise |