Don't insert the address option if not specified.

[jcpunk]

The previous default made rsyncd an IPv4 only service by default.
This change instead uses the rsyncd default when unset. This
easily allows dual stack services to provide rsyncd.

[CLAassistant]

All committers have signed the CLA.

[puppet-community-rangefinder]

rsync::server is a class

that may have no external impact to Forge modules.

This module is declared in 30 of 578 indexed public Puppetfiles.

---

These results were generated with Rangefinder, a tool that helps predict the downstream impact of breaking changes to elements used in Puppet modules. You can run this on the command line to get a full report.

Exact matches are those that we can positively identify via namespace and the declaring modules' metadata. Non-namespaced items, such as Puppet 3.x functions, will always be reported as near matches only.

[kajinamit]

Thanks for submitting the change. Please see my inline comments.

Also, it seems you have not yet signed CLA, which is required to approve this change later.

[jcpunk]

Should be fixed up now.

[kajinamit]

Sorry for the delay in merge. Had some trouble with my account permission.
Do you mind rebasing this on the current master to resolve conflict ?

[kajinamit]

changing my vote because rebase is needed

[jcpunk]

Rebased