Commit message (Collapse) | Author | Age | Files | Lines | |
---|---|---|---|---|---|
* | Skip any numeric-only aliases in the ruleset to prevent errors from those | Chris Buechler | 2015-03-04 | 2 | -0/+9 |
| | | | | who configured them on previous versions where that was allowed. Ticket | ||||
* | Add missing comma. Fixes #4485 | jim-p | 2015-03-04 | 1 | -1/+1 |
| | |||||
* | Enable UnicastOnly in radvd for ovpn* interfaces. Ticket #4455 | Chris Buechler | 2015-03-03 | 1 | -0/+3 |
| | |||||
* | Tweak the carp demotion factors slightly to avoid CARP transitions that are ↵ | jim-p | 2015-03-03 | 1 | -0/+2 |
| | | | | most likely unnecessary. | ||||
* | Merge pull request #1533 from phil-davis/Code-Style-etc-inc-u | Renato Botelho | 2015-03-03 | 4 | -772/+1126 |
|\ | |||||
| * | Code style etc inc u | Phil Davis | 2015-03-03 | 4 | -772/+1126 |
| | | |||||
* | | Merge pull request #1532 from phil-davis/Code-Style-etc-inc-r-s | Renato Botelho | 2015-03-03 | 8 | -1542/+2343 |
|\ \ | |||||
| * | | Code style etc in r s | Phil Davis | 2015-03-03 | 8 | -1542/+2343 |
| |/ | |||||
* | | Merge pull request #1531 from phil-davis/patch-2 | Renato Botelho | 2015-03-03 | 1 | -1/+1 |
|\ \ | |||||
| * | | Be safe use require_once in zeromq | Phil Davis | 2015-03-02 | 1 | -1/+1 |
| |/ | | | | | | | | | | | | | | | I was testing code and just doing stuff like: require_once("zeromq.inc"); in Diagnostics->Command Prompt, PHP Execute That brings an error because underneath that PHP Execute code it has already included auth.inc I guess zeromq.inc is used quite separately to the rest of the system, and must be OK just having a "require" here. But it seems safer to always use require_once, just in case it gets called in a new way/sequence. Comments welcome. | ||||
* | | Remove "Prefer old SA" option, and ignore it in all existing configurations. ↵ | Chris Buechler | 2015-03-03 | 1 | -11/+1 |
|/ | | | | Breaks things in many cases with strongSwan. For the very rare circumstances where this is actually desirable, it's just a sysctl that can be set in tunables. | ||||
* | Merge pull request #1529 from phil-davis/Code-Style-pkg-utils | Renato Botelho | 2015-03-02 | 1 | -296/+403 |
|\ | |||||
| * | Code style for pkg-utils | Phil Davis | 2015-02-28 | 1 | -296/+403 |
| | | |||||
* | | Merge pull request #1528 from phil-davis/Code-Style-putil | Renato Botelho | 2015-03-02 | 1 | -472/+627 |
|\ \ | |||||
| * | | Code style for pfsense-utils | Phil Davis | 2015-02-28 | 1 | -472/+627 |
| |/ | |||||
* | | Merge pull request #1526 from phil-davis/Code-Style-openvpn | Renato Botelho | 2015-03-02 | 1 | -165/+286 |
|\ \ | |||||
| * | | Code style openvpn.inc | Phil Davis | 2015-02-28 | 1 | -165/+286 |
| |/ | |||||
* | | Merge pull request #1525 from phil-davis/Code-Style-etc-inc-i-to-p | Renato Botelho | 2015-03-02 | 16 | -491/+654 |
|\ \ | |||||
| * | | Code style for etc inc i to p | Phil Davis | 2015-02-28 | 16 | -491/+654 |
| |/ | |||||
* | | Ancient bug on upgrade_014_to_015 | Phil Davis | 2015-03-01 | 1 | -1/+1 |
| | | | | | | | | | | | | | | | | | | | | This code looked silly the way it was, with the construct: $var = $var; unset($var); Seems it was accidentally changed to this way many years ago by https://github.com/pfsense/pfsense/commit/588a183b0e58f09932ffef35cc0003cca2313aba IMHO we want to do the conversion to $config['system']['gateway'] here so that later config conversion will then process it as expected. Note that in a current (2.2) config there is not $config['system']['gateway'] but that is switched over later in upgrade_021_to_022 - which switches back to $config['interfaces']['wan']['gateway'] and then later config conversions do their thing with that. I guess this will only effect people who upgrade from some really old config (or monowall?) | ||||
* | | Semi-colon went AWOL in dyndns.class | Phil Davis | 2015-02-28 | 1 | -1/+1 |
|/ | | | after putting it back the code runs much better :) | ||||
* | Merge pull request #1524 from phil-davis/Code-Style-Guide-Interfaces-inc | Renato Botelho | 2015-02-27 | 1 | -1052/+1488 |
|\ | |||||
| * | Code style guide interfaces.inc | Phil Davis | 2015-02-27 | 1 | -1052/+1488 |
| | | | | | | | | | | | | This is another big file that has many diffs so Github refuses to display them. I thought it best to keep doing individual commit-pull for ones like this. | ||||
* | | Merge pull request #1523 from phil-davis/patch-2 | Renato Botelho | 2015-02-27 | 1 | -760/+1098 |
|\ \ | |||||
| * | | Code style guide changes for filter.inc 2nd version | Phil Davis | 2015-02-26 | 1 | -760/+1098 |
| |/ | | | | | This is the changes to filter.inc as per the commits in https://github.com/pfsense/pfsense/pull/1521 but done in just 1 clean commit. | ||||
* | | Merge pull request #1522 from phil-davis/Code-Style-Guide-etc-inc-f-to-g | Renato Botelho | 2015-02-27 | 6 | -490/+644 |
|\ \ | |/ |/| | |||||
| * | More style guide changes | Phil Davis | 2015-02-26 | 2 | -7/+13 |
| | | | | | | | | | | | | | | | | In gwlb.inc at line 676 and 779 I added an extra set of brackets. In the "if" clause as a whole there were a mix of && and || used that were relying on the PHP standard that && has precedence over || In actual fact the original code should have been working fine, the extra brackets make sure that the intention is clear. | ||||
| * | Code Style Guide etc inc f to g | Phil Davis | 2015-02-26 | 6 | -486/+634 |
| | | |||||
* | | Fix type (trime->trim) | jim-p | 2015-02-26 | 1 | -1/+1 |
| | | |||||
* | | Fix indent and remove some unecessary () | Renato Botelho | 2015-02-26 | 5 | -24/+24 |
| | | |||||
* | | Merge pull request #1520 from phil-davis/Code-Style-Guide-etc-inc-a-to-e | Renato Botelho | 2015-02-26 | 12 | -1256/+1807 |
|\ \ | |||||
| * | | Code style guide etc in a to e | Phil Davis | 2015-02-26 | 12 | -1256/+1807 |
| | | | |||||
* | | | Merge pull request #1519 from phil-davis/Code-style-etc-misc | Renato Botelho | 2015-02-26 | 9 | -112/+149 |
|\ \ \ | |||||
| * | | | Code style guide changes for miscellaneous files | Phil Davis | 2015-02-26 | 9 | -112/+149 |
| |/ / | | | | | | | | | | under etc | ||||
* | | | Merge pull request #1516 from phil-davis/patch-2 | Renato Botelho | 2015-02-26 | 1 | -1/+2 |
|\ \ \ | |||||
| * | | | interface_netgraph_needed can miss setting found equals true | Phil Davis | 2015-02-25 | 1 | -1/+2 |
| | | | | | | | | | | | | | | | | | | | | | | | | This routine seems to go looking to see if the passed-in interface is PPP-style. At the end, if it is not PPP-style then it calls pfsense_ngctl_detach. This foreach loop in its current state will always exit after the first iteration that is not mode "server". But it looks like it should look through all the 'pppoe' entries until it finds the interface or gets to the end. In theory the code will sometimes miss setting $found = true when it should have. And thus pfsense_ngctl_detach would get called later for a PPP-style interface. I noticed this while reviewing for code style guide - it is an example where the indenting shows the intention but there are no curlies to implement it. | ||||
* | | | | Merge pull request #1515 from phil-davis/Style-updates | Renato Botelho | 2015-02-26 | 60 | -1145/+1294 |
|\ \ \ \ | |_|_|/ |/| | | | |||||
| * | | | Minor spacing edits to etc files | Phil Davis | 2015-02-25 | 2 | -4/+4 |
| | | | | |||||
| * | | | Code style for etc files | Phil Davis | 2015-02-25 | 60 | -1146/+1295 |
| | | | | |||||
* | | | | remove unused legacy code | Chris Buechler | 2015-02-26 | 1 | -6/+0 |
| | | | | |||||
* | | | | Log ifconfig commands used to setup wireless interfaces | Chris Buechler | 2015-02-26 | 1 | -0/+7 |
| |_|/ |/| | | |||||
* | | | Put the bits to use the new reset utility | Ermal LUÇI | 2015-02-25 | 1 | -0/+4 |
| |/ |/| | |||||
* | | Ticket #4418 Actually make each entry a clear token to strongswan parser for ↵ | Ermal LUÇI | 2015-02-25 | 1 | -2/+3 |
| | | | | | | | | dns_split | ||||
* | | Ticket #4418 make sure the dns_split is separated with spaces rather than ↵ | Ermal LUÇI | 2015-02-25 | 1 | -7/+0 |
| | | | | | | | | space or comma to comply with strongswan requirements. | ||||
* | | Ticket #4418 Make the DNS names attr 28675 space separated as identified by ↵ | Ermal LUÇI | 2015-02-25 | 1 | -1/+16 |
| | | | | | | | | Jeffrey Dvornek | ||||
* | | remove old, unused code | Chris Buechler | 2015-02-25 | 1 | -2/+0 |
| | | |||||
* | | Initialize var and move unset outside the loop | Renato Botelho | 2015-02-24 | 1 | -1/+3 |
| | | |||||
* | | Merge pull request #1495 from k-paulius/fix-dhcp6c-4436 | Renato Botelho | 2015-02-24 | 1 | -14/+17 |
|\ \ | |/ |/| | |||||
| * | Do not request prefix delegation if no tracking interfaces are setup to | k-paulius | 2015-02-16 | 1 | -14/+17 |
| | | | | | | use it. Ticket #4436 | ||||
* | | Merge pull request #1479 from surrural/master | Renato Botelho | 2015-02-24 | 2 | -12/+55 |
|\ \ |