Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Added option to remove all CRs from input stream #1606

Closed
wants to merge 8 commits into from
Closed

Added option to remove all CRs from input stream #1606

wants to merge 8 commits into from

Commits on Aug 23, 2016

  1. Added option to remove all CRs from input stream 

    Added the option removecr to inputs.exec to remove all carraige returns
(CR, ASCII 0x0D, Unicode codepoint \u0D, ^M). The option is boolean and
not enabled if not present in the config file.
    @butitsnotme
    butitsnotme committed Aug 23, 2016
    Configuration menu
    Copy the full SHA
    9df13cd View commit details
    Browse the repository at this point in the history

  2. Updated CHANGELOG.md with information about removecr

    @butitsnotme
    butitsnotme committed Aug 23, 2016
    Configuration menu
    Copy the full SHA
    8760c45 View commit details
    Browse the repository at this point in the history

  3. Ran go fmt ./...

    @butitsnotme
    butitsnotme committed Aug 23, 2016
    Configuration menu
    Copy the full SHA
    b708553 View commit details
    Browse the repository at this point in the history

  4. Moved removal of CRs to internal/internal.go 

    Moved the code to remove carriage returns from
plugins/inputs/exec/exec.go to internal/internal.go. Additionally
changed the conditional on which it gets applied from using a
configuration file option to checking if it is running on Windows.
    @butitsnotme
    butitsnotme committed Aug 23, 2016
    Configuration menu
    Copy the full SHA
    9b18f1d View commit details
    Browse the repository at this point in the history

  5. Moved Carriage Return check to correct place 

    Moved the carriage return removal back to the exec plugin. Added unit
testing for it. Fixed a bug (removing too many characters).
    @butitsnotme
    butitsnotme committed Aug 23, 2016
    Configuration menu
    Copy the full SHA
    6349b01 View commit details
    Browse the repository at this point in the history

  6. Ran go fmt ./...

    @butitsnotme
    butitsnotme committed Aug 23, 2016
    Configuration menu
    Copy the full SHA
    1e85870 View commit details
    Browse the repository at this point in the history

  7. Reverted CHANGELOG to master

    @butitsnotme
    butitsnotme committed Aug 23, 2016
    Configuration menu
    Copy the full SHA
    92f3862 View commit details
    Browse the repository at this point in the history

  8. Updated Changelog

    @butitsnotme
    butitsnotme committed Aug 23, 2016
    Configuration menu
    Copy the full SHA
    3b6e9dd View commit details
    Browse the repository at this point in the history