diff --git a/aws_ros1_common/include/aws_ros1_common/sdk_utils/logging/aws_ros_logger.h b/aws_ros1_common/include/aws_ros1_common/sdk_utils/logging/aws_ros_logger.h index f116792..a71c58a 100644 --- a/aws_ros1_common/include/aws_ros1_common/sdk_utils/logging/aws_ros_logger.h +++ b/aws_ros1_common/include/aws_ros1_common/sdk_utils/logging/aws_ros_logger.h @@ -36,6 +36,7 @@ class AWSROSLogger : public AWSLogSystem AWSROSLogger(AWSROSLogger const &) = delete; // Do not allow copy constructor AWSROSLogger & operator=(AWSROSLogger const &) = delete; // Do not allow assignment operator ~AWSROSLogger() override = default; + virtual void Flush(); protected: void LogTrace(const char * tag, const std::string & message) override; diff --git a/aws_ros1_common/src/sdk_utils/logging/aws_ros_logger.cpp b/aws_ros1_common/src/sdk_utils/logging/aws_ros_logger.cpp index eae200c..c58cb67 100644 --- a/aws_ros1_common/src/sdk_utils/logging/aws_ros_logger.cpp +++ b/aws_ros1_common/src/sdk_utils/logging/aws_ros_logger.cpp @@ -23,6 +23,8 @@ namespace Logging { AWSROSLogger::AWSROSLogger(Aws::Utils::Logging::LogLevel log_level) : AWSLogSystem(log_level) {} +void AWSROSLogger::Flush() {} + void AWSROSLogger::LogTrace(const char * tag, const std::string & message) { ROS_DEBUG("[%s] %s", tag, message.c_str()); diff --git a/aws_ros1_common/test/client_configuration_provider_test.cpp b/aws_ros1_common/test/client_configuration_provider_test.cpp index 81c409c..bac6442 100644 --- a/aws_ros1_common/test/client_configuration_provider_test.cpp +++ b/aws_ros1_common/test/client_configuration_provider_test.cpp @@ -37,7 +37,7 @@ void InitializeNodeAndConfig(ros::NodeHandle & node, Aws::Client::ClientConfigur node.setParam(CLIENT_CONFIG_PREFIX "/verify_SSL", true); config.verifySSL = true; node.setParam(CLIENT_CONFIG_PREFIX "/follow_redirects", true); - config.followRedirects = true; + config.followRedirects = Aws::Client::FollowRedirectsPolicy::ALWAYS; } /** @@ -56,10 +56,10 @@ TEST(DefaultClientConfigurationProvider, getClientConfiguration) generated_config .userAgent; /* Set the user agent to w/e was generated. Will be tested separately */ - prepared_config.followRedirects = false; + prepared_config.followRedirects = Aws::Client::FollowRedirectsPolicy::NEVER; ASSERT_NE(prepared_config, generated_config); - prepared_config.followRedirects = true; + prepared_config.followRedirects = Aws::Client::FollowRedirectsPolicy::ALWAYS; ASSERT_EQ(prepared_config, generated_config); }