tyler.durden
/
icinga2
mirror of https://github.com/Icinga/icinga2.git
1
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

Allow to disable brackets for the Url class and Format() 

This commit also adds unit tests.

refs #5706

refs #6207
This commit is contained in:
Michael Friedrich 2018-04-06 14:49:15 +02:00
parent 836103ef57
commit 9f418f5356
3 changed files with 20 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

8
lib/remote/url.cpp
View File

@ -208,6 +208,11 @@ void Url::SetQuery(const std::map<String, std::vector<String> >& query)
m_Query = query;
}
void Url::SetArrayFormatUseBrackets(bool useBrackets)
{
m_ArrayFormatUseBrackets = useBrackets;
}
void Url::AddQueryElement(const String& name, const String& value)
{
auto it = m_Query.find(name);
@ -276,8 +281,11 @@ String Url::Format(bool onlyPathAndQuery, bool printCredentials) const
temp += "&";
temp += key;
if (m_ArrayFormatUseBrackets) {
if (kv.second.size() > 1)
temp += "[]";
}
if (!s.IsEmpty())
temp += "=" + Utility::EscapeString(s, ACQUERY_ENCODE, false);

2
lib/remote/url.hpp
View File

@ -65,6 +65,7 @@ public:
void SetPort(const String& port);
void SetPath(const std::vector<String>& path);
void SetQuery(const std::map<String, std::vector<String> >& query);
void SetArrayFormatUseBrackets(bool useBrackets = true);
void AddQueryElement(const String& name, const String& query);
void SetQueryElements(const String& name, const std::vector<String>& query);
@ -78,6 +79,7 @@ private:
String m_Port;
std::vector<String> m_Path;
std::map<String, std::vector<String> > m_Query;
bool m_ArrayFormatUseBrackets;
String m_Fragment;
bool ParseScheme(const String& scheme);

8
test/remote-url.cpp
View File

@ -96,6 +96,14 @@ BOOST_AUTO_TEST_CASE(format)
url = new Url("/");
BOOST_CHECK(url->Format(false, false) == "/");
url = new Url("https://nsclient:8443/query/check_cpu?time%5B%5D=1m&time=5m&time%5B%5D=15m");
url->SetArrayFormatUseBrackets(false);
BOOST_CHECK(url2 = new Url(url->Format(false, false)));
url = new Url("https://icinga2/query?a[]=1&a[]=2&a[]=3");
url->SetArrayFormatUseBrackets(true);
BOOST_CHECK(url2 = new Url(url->Format(false, false)));
}
BOOST_AUTO_TEST_CASE(illegal_legal_strings)