Commit message (Collapse) | Author | Age | Files | Lines | ||
---|---|---|---|---|---|---|
... | ||||||
* | | | Merge pull request #1744 from phil-davis/patch-5 | Renato Botelho | 2015-07-15 | 1 | -1/+1 | |
|\ \ \ | ||||||
| * | | | Avoid error loading rules for numeric host name in alias | Phil Davis | 2015-07-12 | 1 | -1/+1 | |
| |/ / | | | | | | | | | | | | | | | | | | | | | | | | | | | | Create a host-type alias. Put just a number in "IP or FQDN" - e.g. I made alias name "Zqw" and a single host "23". The webGUI reports: There were error(s) loading the rules: /tmp/rules.debug:44: syntax error - The line in question reads [44]: table { 23 } and /tmp/rules.debug has: table <Zqw> { 23 } Zqw = "<Zqw>" which pf does not cope with. It is possible to have a host name that is a number, and end up with a domain name like 23.mycompany.com - unfortunately some Wally allowed such things in standards many years ago, so it can be rather difficult to tell the difference between a number and a host name. This change improves the check when looking through alias entries and deciding if they are meant to be a name or a "bottom-level" value (address, subnet, port, port range). Anything that ends up looking like a host name gets given to filterdns to sort out. "Names" like "23" now get given to filterdns instead of being put directly into the table in pf. This makes things happier. Even if filterdns cannot resolve "23", at least it tries and nothing barfs. | |||||
* | | | Merge pull request #1745 from phil-davis/patch-6 | Renato Botelho | 2015-07-15 | 1 | -2/+2 | |
|\ \ \ | ||||||
| * | | | Interfaces GIF Edit fix do_input_validation | Phil Davis | 2015-07-13 | 1 | -2/+2 | |
| |/ / | | | | | | | Make the required fields be correct and match thier text names, which should each have their own gettext() cal so as to build a proper array at line 81. Basically it was all broken and the errors displayed when field/s were left empty were rubbish. | |||||
* | | | Merge pull request #1746 from phil-davis/patch-7 | Renato Botelho | 2015-07-15 | 1 | -2/+2 | |
|\ \ \ | ||||||
| * | | | Interfaces GRE Edit fix required fields text | Phil Davis | 2015-07-13 | 1 | -2/+2 | |
| |/ / | | | | | | | | | | The reqdfields had only 4 entries but reqdfieldsn has 5 entries and the field names to text descriptions did not match up. Fixed it. | |||||
* | | | Merge pull request #1747 from phil-davis/patch-8 | Renato Botelho | 2015-07-15 | 1 | -15/+17 | |
|\ \ \ | ||||||
| * | | | Interfaces PPPs edit avoid foreach() warning | Phil Davis | 2015-07-13 | 1 | -15/+17 | |
| |/ / | | | | | | | | | | If you go to Interfaces, assign, PPPs, press "+" to add an entry, then press Save without entering anything then you get a warning about the foreach() here. The is_array() check fixes it. | |||||
* | | | Merge pull request #1749 from phil-davis/cancel-referer | Renato Botelho | 2015-07-15 | 35 | -35/+210 | |
|\ \ \ | ||||||
| * | | | Cancel button after input error | Phil Davis | 2015-07-13 | 35 | -35/+210 | |
| |/ / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | If there is an input error then the edit page is redrawn showing the input errors. The HTTP_REFERER becomes the current page, rather than the true original referer. Then if you click Cancel the current page is just redrawn. This change makes the code remember the original referer, so if the user enters some invalid data, presses Save, reads the input error messages then presses Cancel they are taken back out to the original page - the same as if Cancel was used before Save. | |||||
* | | | Merge pull request #1752 from phil-davis/patch-9 | Chris Buechler | 2015-07-14 | 1 | -1/+6 | |
|\ \ \ | ||||||
| * | | | Fix glob for serial device names | Phil Davis | 2015-07-14 | 1 | -1/+6 | |
| | | | | | | | | | | | | | | | | Removing the "." that was in {,.[0-9]} allows it to match /dev/cuau10 and onward. I added lots of comments on the glob expression, because the format of the glob expression is not the same as an ordinary regex. | |||||
* | | | | Fix adding of VoIP rules from traffic shaper wizard where IP/alias is not | Chris Buechler | 2015-07-14 | 1 | -0/+6 | |
| | | | | | | | | | | | | | | | | specified. | |||||
* | | | | Merge pull request #1753 from mortencombat/patch-1 | Chris Buechler | 2015-07-14 | 1 | -2/+2 | |
|\ \ \ \ | |/ / / |/| | | | ||||||
| * | | | Fix GratisDNS support | mortencombat | 2015-07-14 | 1 | -2/+2 | |
|/ / / | | | | | | | The current implementation is not working for me, maybe the interface was changed by GratisDNS? I tested the update URL manually with the IP added ("&i=<ip>") which fixed the issue for me. Code adjusted to include IP. | |||||
* | | | Merge pull request #1748 from phil-davis/patch-9 | Chris Buechler | 2015-07-13 | 1 | -4/+4 | |
|\ \ \ | |/ / |/| | | ||||||
| * | | Fix references to Load Balancer Virtual Server redirect_mode | Phil Davis | 2015-07-13 | 1 | -4/+4 | |
|/ / | | | | | | | When adding a Virtual Server, if you press Save with blank fields, the validation does not show. That was because the switch statement selecting the fields to validate had wrong case values. Actually the "mode" here is locked to "redirect_mode" (later code to allow the mode to be selected by the user is commented out - not implemented. I fixed the reqdfieldsn array also for relay_mode case, even though it is never used. It looked dodgy the way it was. | |||||
* | | fix fsync, thanks Phil Davis for noticing | Chris Buechler | 2015-07-10 | 1 | -1/+1 | |
| | | ||||||
* | | fix fsync | Chris Buechler | 2015-07-10 | 1 | -1/+1 | |
| | | ||||||
* | | fsync after fclose here, clean up some white space while here. | Chris Buechler | 2015-07-10 | 1 | -2/+9 | |
| | | | | | | | | | | Conflicts: etc/inc/config.lib.inc | |||||
* | | fsync conf_path here too | Chris Buechler | 2015-07-10 | 1 | -0/+1 | |
| | | ||||||
* | | fix typo | Chris Buechler | 2015-07-10 | 1 | -1/+1 | |
| | | ||||||
* | | Merge pull request #1739 from yakar/patch-6 | Chris Buechler | 2015-07-08 | 1 | -2/+4 | |
|\ \ | ||||||
| * | | Update index.php | Aydın Yakar | 2015-07-07 | 1 | -2/+3 | |
| | | | ||||||
| * | | Update index.php | Aydın Yakar | 2015-07-07 | 1 | -2/+3 | |
|/ / | ||||||
* | | Fix dashboard hardware crypto display where AES-NI is enabled. Ticket | Chris Buechler | 2015-07-06 | 1 | -1/+1 | |
| | | ||||||
* | | Don't check whether the QinQ interface exists when deleting. Unnecessarily | Chris Buechler | 2015-07-06 | 1 | -2/+0 | |
| | | | | | | | | | | makes QinQ un-deletable where the parent interface no longer exists (removed, config restored from diff hardware, etc.). | |||||
* | | Make sure config.xml is safe on disk when restoring a backup, ticket #4803 | Renato Botelho | 2015-07-06 | 1 | -0/+1 | |
| | | ||||||
* | | Make sure temporary config file is safe on disk before rename, ticket #4803 | Renato Botelho | 2015-07-06 | 1 | -1/+1 | |
| | | ||||||
* | | Remove reference to vfs.forcesync | Renato Botelho | 2015-07-06 | 1 | -1/+0 | |
| | | ||||||
* | | Use right function pfSense_fsync to make sure config file is safe on disk, ↵ | Renato Botelho | 2015-07-06 | 1 | -4/+2 | |
| | | | | | | | | ticket #4803 | |||||
* | | fix includes so shellsession restartipsec works. | Chris Buechler | 2015-07-05 | 1 | -0/+2 | |
|/ | ||||||
* | remove debug.pfftpproxy, it no longer exists. | Chris Buechler | 2015-07-04 | 1 | -1/+0 | |
| | ||||||
* | de-activate sync on upgrade where it's enabled now that the root ↵ | Chris Buechler | 2015-07-03 | 1 | -3/+3 | |
| | | | | passwd/group problem is fixed. Ticket #4523 | |||||
* | Fix keyid identifers, and go back to using %any in ipsec.secrets as in ↵ | Chris Buechler | 2015-07-03 | 1 | -2/+4 | |
| | | | | previous versions, fixing a variety of other ID issues. Latter will break some mobile IPsec circumstances, fix for that to come after more testing. Ticket #4811 | |||||
* | Merge pull request #1736 from phil-davis/patch-1 | Renato Botelho | 2015-07-03 | 1 | -2/+2 | |
|\ | ||||||
| * | Fix put static route destination in config change description | Phil Davis | 2015-07-04 | 1 | -2/+2 | |
|/ | | | | When enabling or disabling a route by using the enable/disable button on the Routes page, the destination network was not being put in the config change description, although the code intended it to be there. This fixes it. | |||||
* | Improve handling of port ranges in relayd, fixes #4810 | jim-p | 2015-07-01 | 1 | -1/+5 | |
| | ||||||
* | Remove the unnecessary deletion of rc.conf. Add an empty rc.conf with a | Chris Buechler | 2015-07-01 | 3 | -6/+1 | |
| | | | | note so people don't think they should be using it. | |||||
* | Remove $array_keys references and use $pkg_info. Spotted-by: phil-davis | Renato Botelho | 2015-07-01 | 1 | -5/+2 | |
| | ||||||
* | Encode ca/cert descr in system_certmanager.php | jim-p | 2015-07-01 | 1 | -3/+7 | |
| | ||||||
* | Encode ca/cert descr in system_certmanager.php | jim-p | 2015-07-01 | 1 | -6/+14 | |
| | ||||||
* | Encode ca/cert/crl descr in system_crlmanager.php | jim-p | 2015-07-01 | 1 | -11/+16 | |
| | ||||||
* | Encode ca/cert descr in vpn_openvpn_server.php | jim-p | 2015-07-01 | 1 | -5/+5 | |
| | ||||||
* | Encode ca/cert descr in vpn_openvpn_client.php | jim-p | 2015-07-01 | 1 | -3/+3 | |
| | ||||||
* | Encode ca/cert descr in vpn_ipsec_phase1.php | jim-p | 2015-07-01 | 1 | -2/+2 | |
| | ||||||
* | Encode ca/cert descr in system_authservers.php | jim-p | 2015-07-01 | 1 | -1/+1 | |
| | ||||||
* | Encode ca/cert descr in system_usermanager.php | jim-p | 2015-07-01 | 1 | -1/+1 | |
| | ||||||
* | Encode cert/ca descr in system_advanced_admin.php | jim-p | 2015-07-01 | 1 | -1/+1 | |
| | ||||||
* | Encode cert/ca descr in services_captiveportal.php | jim-p | 2015-07-01 | 1 | -1/+1 | |
| |