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.

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

Add TLS support for file monitoring #88

Merged
merged 13 commits into from Oct 10, 2017

Merge branch 'master' into tls-support-filemonitoring

  • Loading branch information
Shwetajain148
Shwetajain148 committed May 29, 2017
commit 69a9a52554ab7d3f9c6f65a929c826a4e98eb49b
@@ -152,13 +152,13 @@ installLogglyConf()
if [ "$LINUX_ENV_VALIDATED" = "false" ]; then
checkLinuxLogglyCompatibility
fi

#create rsyslog dir if it doesn't exist, Modify the permission on rsyslog directory if exist on Ubuntu
createRsyslogDir

#if all the above check passes, write the 22-loggly.conf file
checkAuthTokenAndWriteContents

#create rsyslog dir if it doesn't exist, Modify the permission on rsyslog directory if exist on Ubuntu
createRsyslogDir

if [ "$LINUX_DO_VERIFICATION" = "true" ]; then
#check if the logs are going to loggly fro linux system now
checkIfLogsMadeToLoggly
@@ -852,18 +852,17 @@ checkIfTLS()
LOGGLY_SYSLOG_PORT=6514
break;;
[Nn]* )
LINUX_DO_VERIFICATION="false"
logMsgToConfigSysLog "INFO" "INFO: Skipping Linux verification."
break;;
* ) echo "Please answer yes or no.";;
esac
done
else
logMsgToConfigSysLog "WARN" "WARN: Your system logs are being send insecurely. We prefer to send system logs securely so switching to secure configuration."
logMsgToConfigSysLog "WARN" "WARN: Your system logs are being send insecurely. We prefer to send system logs securely so switching to secure configuration."

This comment has been minimized.

@mostlyjason

mostlyjason Aug 30, 2017
Contributor

sent...the secure

This comment has been minimized.

@Shwetajain148

Shwetajain148 Sep 1, 2017
Author Contributor

@mchaudhary @mostlyjason, Since this change, was in configure-linux script so I made the above changes in existing PR for configure-linux script here: #102 so that these changes will not depend on this file-monitoring PR to be merged.

LOGGLY_TLS_SENDING="true"
LOGGLY_SYSLOG_PORT=6514

fi
fi

fi
}

You are viewing a condensed version of this merge commit. You can view the full changes here.
ProTip! Use n and p to navigate between commits in a pull request.
You can’t perform that action at this time.