Page MenuHomeFreeBSD

Minor fixes to VPN section

Authored by sevan on Jun 11 2017, 12:39 AM.



There is a double prompt in the IPsec vpn article at the moment.
e.g. # corp-net#
This patch attempts to address that by moving the hostname to the start of the line before using the root.prompt macro.
Fix spelling mistake picked up by igor.

Diff Detail

rD FreeBSD doc repository - subversion
Automatic diff as part of commit; lint not applicable.
Automatic diff as part of commit; unit tests not applicable.

Event Timeline

Clarify that FreeBSD 11 comes with IPsec enabled by default, a custom kernel is only required on older releases.

A few comments.

374 ↗(On Diff #29453)

That is output from the utility. Is that fixed in there and we did not pick it up yet or did they fix it and have not updated this section?

2130 ↗(On Diff #29453)

A   between &os; and 11 will be good to have.

374 ↗(On Diff #29453)

Ah, good catch, I didn't check the context. It was flagged by igor.

Drop corrections to command output.
Add non-breaking space.

sevan marked 2 inline comments as done.

We had encountered the same problem at BSDCan in another review. Seemed like a low-hanging fruit, but it isn't. ;-)

Indeed, need to check, check and check again :)

This revision was automatically updated to reflect the committed changes.

I think we generally say "later" rather than "newer".


s/older releases/previous versions/

Rewriting to avoid the if/pause/then structure (and leaving out some markup for clarity):

Add these options to the custom kernel configuration file to add IPsec support to previous versions of &os;.

Please eliminate "the following" whenever it is used incorrectly, which is almost always. The only time to use it is when it can't be replaced with a simpler "this" or "these".

Hi Warren,
Apologies for the late reply to this, just wanted to follow up to say I've not forgotten about this review, will get the points you raised addressed next week if not this Sunday.