diff options
author | Philipp Kerling <pkerling@casix.org> | 2019-01-10 10:33:49 +0100 |
---|---|---|
committer | Philipp Kerling <pkerling@casix.org> | 2019-01-10 10:33:49 +0100 |
commit | d4d01d73dc8265a8c71c489226e9e58c4b83f365 (patch) | |
tree | 9ed118143f3a8feadbef86ed6d6406bdf688cde5 /configure.ac | |
parent | 8c9755a790a48fc614ac220f734e66d880203ee5 (diff) |
VideoInfoScanner: Correctly redact URLs
CURL::GetRedacted does not work on decoded URLs since e.g. the password
part may include an encoded @ (%40) that in decoded form will confuse
the redaction and expose part of the password. Also, there is no
particularly strong reason to decode URLs for log messages here.
When matching the regular expressions, however, the URL must be decoded,
but the username/password details are not important, so redact them
before matching. Otherwise, they might get exposed during further
logging done on the decoded URLs.
Diffstat (limited to 'configure.ac')
0 files changed, 0 insertions, 0 deletions