Skip to content

Commit fec22a6

Browse files
authored
Merge pull request #524 from Philipp91/utf-warning
Add warning about non-UTF characters also to phpDoc
2 parents ce956ba + 06bee24 commit fec22a6

File tree

2 files changed

+2
-0
lines changed

2 files changed

+2
-0
lines changed

lib/Fhp/BaseAction.php

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -77,6 +77,7 @@ public function serialize(): string
7777
* If a sub-class overrides this, it should call the parent function and include it in its result.
7878
*
7979
* @return array The serialized action, e.g. for storage in a database. This will not contain sensitive user data.
80+
* Note that this is not necessarily valid UTF-8, so you should store it as a BLOB column or raw bytes.
8081
*/
8182
public function __serialize(): array
8283
{

lib/Fhp/FinTs.php

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -159,6 +159,7 @@ public function __destruct()
159159
* carefully (not written to log files, only to a database or other storage system that would normally be used
160160
* for user data). The returned string never contains highly sensitive information (not the user's password or
161161
* PIN), so it probably does not need to be encrypted. Treat it like a session cookie of the same bank.
162+
* Note that this is not necessarily valid UTF-8, so you should store it as a BLOB column or raw bytes.
162163
*/
163164
public function persist(bool $minimal = false): string
164165
{

0 commit comments

Comments
 (0)