Closed 18202781743 closed 4 months ago
Doc Check bot :green_circle: Document Coverage Check Passed!
[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by: 18202781743, kegechen
The full list of commands accepted by this bot can be found here.
关键摘要:
isValidAppId
函数中直接使用了 contains(' ')
来检查是否包含空格,这种方法不够健壮,建议使用正则表达式来匹配非法字符。isValidAppId
函数的实现依赖于 isValidFilename
函数,如果 appId
仅包含空格但被认为有效,那么 isValidFilename
可能不会被正确调用。ut_DConfigFileCheckAppId
测试用例中,guard
变量被声明但未使用,应该移除未使用的变量以提高代码清晰度。FILE_NAME
和 noAppidMetaPath
变量应该是常量或配置项,而不是硬编码的字符串,以提高代码的可维护性。std::tuple{QString("org foo"), false}
中的 false
应该是一个变量或者常量,而不是硬编码的值。是否建议立即修改: 是
user can open non-root controlled files when
appId
is relative path.Issue: https://bugzilla.suse.com/show_bug.cgi?id=1211374