Skip to content

Commit 3413a56

Browse files
author
Lauris Kaplinski
committed
Added code comments about utf8/xml requirement
Signed-off-by: Lauris Kaplinski <[email protected]>
1 parent 09f2464 commit 3413a56

File tree

1 file changed

+3
-0
lines changed

1 file changed

+3
-0
lines changed

src/crypto/Signer.cpp

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -73,6 +73,7 @@ Signer::~Signer() = default;
7373
* @param stateOrProvince
7474
* @param postalCode
7575
* @param countryName
76+
* The strings have to be utf8 encoded and not contain any control values (any char < 0x20 except 0x9, 0xa and 0xd)
7677
*/
7778
void Signer::setSignatureProductionPlace(const string &city,
7879
const string &stateOrProvince, const string &postalCode, const string &countryName)
@@ -90,6 +91,7 @@ void Signer::setSignatureProductionPlace(const string &city,
9091
* @param stateOrProvince
9192
* @param postalCode
9293
* @param countryName
94+
* The strings have to be utf8 encoded and not contain any control values (any char < 0x20 except 0x9, 0xa and 0xd)
9395
*/
9496
void Signer::setSignatureProductionPlaceV2(const string &city, const string &streetAddress,
9597
const string &stateOrProvince, const string &postalCode, const string &countryName)
@@ -194,6 +196,7 @@ void Signer::setProfile(const string &profile)
194196
/**
195197
* Sets signature roles according XAdES standard. The parameter may contain the signer’s role and optionally the signer’s resolution. Note that only one signer role value (i.e. one &lt;ClaimedRole&gt; XML element) should be used.
196198
* If the signer role contains both role and resolution then they must be separated with a slash mark, e.g. “role / resolution”.
199+
* The strings have to be utf8 encoded and not contain any control values (any char < 0x20 except 0x9, 0xa and 0xd)
197200
*/
198201
void Signer::setSignerRoles(const vector<string> &signerRoles)
199202
{

0 commit comments

Comments
 (0)