WEBKT

Salesforce Apex安全必杀技 - 何时以及如何使用`Security.stripInaccessible()`加固字段级安全

53 0 0 0

老办法的“痛” - 手动检查 isAccessible()

现代解决方案 - Security.stripInaccessible()

stripInaccessible() vs 手动检查 - 深度对决

聚焦写操作安全 (CREATE, UPDATE)

性能考量 - 需要担心吗?

处理结果和潜在问题

最佳实践小结

结语

搞Salesforce开发的兄弟们,字段级安全(FLS)肯定不陌生吧?这玩意儿是咱们权限体系里的基石,确保张三看不到李四的工资,王五改不了赵六的客户状态。在Apex里强制执行FLS,尤其是处理DML操作(insert, update)时,一直是个不大不小的痛点。过去,我们可能得写一堆 Schema 方法调用,搞得代码又臭又长,还容易漏掉检查。谢天谢地,Salesforce后来给我们带来了 Security.stripInaccessible() 这个大杀器。

但这新工具也不是万能药,啥时候用?怎么用才最对味?它跟老办法比,优劣在哪?性能影响大不大?特别是,当你的Apex需要处理那些从外部系统怼过来的,或者来源不太可信的数据时,怎么用 stripInaccessible() 守住数据安全的最后一道防线?今天咱们就来把这些问题掰扯清楚。

老办法的“痛” - 手动检查 isAccessible()

stripInaccessible() 出现之前,或者说,在某些特定场景下,我们依赖 Schema 命名空间里的方法来检查字段权限。主要是这几个哥们儿:

  • Schema.sObjectType.YourObject__c.fields.YourField__c.getDescribe().isAccessible(): 检查当前用户是否有读取权限。
  • Schema.sObjectType.YourObject__c.fields.YourField__c.getDescribe().isCreateable(): 检查当前用户是否有创建权限。
  • Schema.sObjectType.YourObject__c.fields.YourField__c.getDescribe().isUpdateable(): 检查当前用户是否有更新权限。

想象一下,你要更新一个 Opportunity 对象,里面有十几个字段,数据可能来自一个LWC表单或者外部API。在执行 update 之前,理论上,你得对 每一个 你打算更新的字段,都调用 isUpdateable() 检查一遍。代码大概长这样:

List<Opportunity> oppsToUpdate = new List<Opportunity>();
// 假设 oppsToUpdate 已经填充了从外部来源获取的数据

List<Opportunity> safeOppsToUpdate = new List<Opportunity>();

// 获取字段描述结果,避免在循环中重复获取
Map<String, Schema.DescribeFieldResult> fieldDescribeMap = Schema.SObjectType.Opportunity.fields.getMap();

for (Opportunity opp : oppsToUpdate) {
    Opportunity safeOpp = new Opportunity(Id = opp.Id);

    // 检查 Name 字段
    if (fieldDescribeMap.get('Name').getDescribe().isUpdateable()) {
        safeOpp.Name = opp.Name;
    }
    // 检查 Amount 字段
    if (fieldDescribeMap.get('Amount').getDescribe().isUpdateable()) {
        safeOpp.Amount = opp.Amount;
    }
    // 检查 StageName 字段
    if (fieldDescribeMap.get('StageName').getDescribe().isUpdateable()) {
        safeOpp.StageName = opp.StageName;
    }
    // ... 对其他所有可能更新的字段重复这个过程 ...

    // 只有当至少有一个字段可以更新时才添加到列表
    // (或者根据业务逻辑决定,这里简化处理)
    if (safeOpp.getPopulatedFieldsAsMap().size() > 1) { // Id 总是存在的
      safeOppsToUpdate.add(safeOpp);
    }
}

if (!safeOppsToUpdate.isEmpty()) {
    try {
        update safeOppsToUpdate;
    } catch (DmlException e) {
        // 处理异常
        System.debug('Error updating opportunities: ' + e.getMessage());
    }
}

看到没?这代码简直是灾难:

  1. 啰嗦:每个字段都要写一个 if 语句,对象字段一多,代码体积蹭蹭涨。
  2. 易错:万一你新增了一个字段,忘了在Apex里加上对应的检查逻辑,安全漏洞就来了。或者手滑写错了字段API名,编译时还发现不了。
  3. 维护困难:每次对象模型变更,都可能要回来修改这堆检查代码。

这种方式在只有一两个字段需要检查的简单场景下还凑合,一旦涉及多个字段或者动态字段集,简直让人头大。

现代解决方案 - Security.stripInaccessible()

Security.stripInaccessible() 方法就是为了解决上面的痛点而生的。它的核心作用是:根据当前运行用户的字段级安全设置,从 SObject 记录列表中移除用户无权访问的字段。

它接受两个参数:

  1. accessCheckType: 一个 System.AccessType 枚举值,指定你要检查哪种权限。常用的是:
    • AccessType.READABLE: 检查读权限(虽然 stripInaccessible 主要用于写操作前的清理,但理论上也可用于读后清理,不过 WITH SECURITY_ENFORCED 在SOQL层面更常用)。
    • AccessType.CREATABLE: 检查创建权限,用于 insert 操作前。
    • AccessType.UPDATABLE: 检查更新权限,用于 update 操作前。
  2. sourceRecords: 一个 List<SObject>,包含你想要进行字段清理的记录。

它返回一个 SObjectAccessDecision 对象,你可以通过这个对象获取清理后的记录列表以及被移除的字段信息。

看个例子,还是更新 Opportunity 的场景,用 stripInaccessible() 就简洁多了:

List<Opportunity> oppsToUpdate = new List<Opportunity>();
// 假设 oppsToUpdate 已经填充了从外部来源获取的数据,可能包含用户无权更新的字段

// 关键步骤:移除当前用户无权更新的字段
SObjectAccessDecision decision = Security.stripInaccessible(
    AccessType.UPDATABLE,
    oppsToUpdate
);

// 获取清理后的记录列表
List<Opportunity> safeOppsToUpdate = decision.getRecords();

// 可选:查看哪些字段被移除了(主要用于调试或日志)
Map<String, Set<String>> removedFields = decision.getRemovedFields();
if (!removedFields.isEmpty()) {
    System.debug('Removed inaccessible fields: ' + removedFields);
}

// 执行 DML 操作
if (!safeOppsToUpdate.isEmpty()) {
    try {
        // 注意:这里更新的是清理后的列表 safeOppsToUpdate
        update safeOppsToUpdate;
    } catch (DmlException e) {
        // 处理异常
        System.debug('Error updating opportunities: ' + e.getMessage());
    }
}

对比一下,是不是清爽多了?

  • 简洁:无论多少个字段,一行 stripInaccessible() 调用搞定。
  • 安全:自动处理所有字段,不容易遗漏,减少了人为错误。
  • 易维护:对象模型变化时,只要你的代码逻辑是处理 SObject 列表,通常不需要修改这部分安全检查代码。

stripInaccessible() vs 手动检查 - 深度对决

那么,是不是以后无脑用 stripInaccessible() 就行了?大部分情况下是的,但了解它们的适用场景和细微差别很重要。

何时优先选择 Security.stripInaccessible()

  1. 处理外部数据/不可信来源:这是 stripInaccessible() 最闪光的场景。当你的Apex接收来自API、集成、LWC(用户可能通过浏览器开发者工具修改了传递的数据)等来源的数据时,你不能假设这些数据只包含用户有权操作的字段。在执行 insertupdate 前,用 stripInaccessible(AccessType.CREATABLE, ...)stripInaccessible(AccessType.UPDATABLE, ...) 清理一遍,是防止非法字段写入的最佳实践。

    • 场景举例:一个集成服务,使用一个拥有较多权限的集成用户账号,向 Salesforce 推送客户更新数据。但某个具体的业务流程要求,即便是这个集成用户,也不能修改 Account 对象的 Credit_Status__c 字段。在处理推送来的 Account 记录列表前,调用 Security.stripInaccessible(AccessType.UPDATABLE, accountList) 就能自动移除 Credit_Status__c(如果当前运行上下文的用户确实无权更新它),避免了非法更新。
  2. 批量 DML 操作:当你要处理大量记录时,手动为每条记录的每个字段做检查,性能和代码复杂度都堪忧。stripInaccessible() 可以一次性处理整个列表,代码更优雅,意图更清晰。

  3. 简化复杂逻辑:如果你的代码需要根据不同条件动态地构建 SObject 实例,或者处理的字段集不固定,stripInaccessible() 可以极大地简化你的 FLS 检查逻辑。

  4. 代码健壮性和安全性优先:在大多数业务场景下,代码的健壮性和安全性比那一点点可能的性能差异更重要。stripInaccessible() 提供了一种更可靠、更不容易出错的方式来强制执行 FLS。

何时可能还需要考虑手动检查 (isAccessible(), isCreateable(), isUpdateable())?

  1. 需要对特定字段的不可访问性进行精细化处理stripInaccessible() 是“沉默”的,它只是移除了字段,并不会抛出异常或者给你明确的信号说“嘿,这个字段不能写”。如果你需要根据某个字段是否可写,来执行不同的业务逻辑,或者向用户返回非常具体的错误信息(例如,“您无权修改‘合同金额’字段”),那么你可能需要先用 isUpdateable() 等方法检查,然后决定下一步操作。

    • 场景举例:一个页面允许用户修改订单的多个属性。如果用户试图修改“折扣率”字段,但他们没有权限,系统需要弹出一个明确的提示信息,而不是仅仅忽略这个修改。这种情况下,在尝试赋值前,先用 isUpdateable() 检查“折扣率”字段更合适。
  2. 极端性能敏感且字段明确的场景(谨慎使用!)stripInaccessible() 会检查传入 SObject 列表中 所有 存在的字段(populated fields)。如果你在一个非常非常紧凑的循环中,只需要更新一个或两个 固定 的字段,并且 已经验证 性能瓶颈确实在此,理论上 手动检查这几个特定字段可能比调用 stripInaccessible() 开销小一点点。但请注意:这通常是过早优化! stripInaccessible() 的性能通常足够好,为了这点微不足道的性能提升而牺牲代码简洁性和安全性,往往得不偿失。只有在性能分析工具(Profiler)明确指出这里是瓶颈时,才值得考虑。

  3. 读操作前的检查(但通常有更好的选择):虽然你可以用 stripInaccessible(AccessType.READABLE, ...) 来移除查询结果中用户不可读的字段,但这通常不是最佳实践。对于读操作,在 SOQL 查询层面使用 WITH SECURITY_ENFORCED 子句是更推荐的方式,它会在查询时就直接过滤掉用户无权访问的字段,如果查询中包含不可读字段还会直接抛出异常,更符合“失败优先”的安全原则。

聚焦写操作安全 (CREATE, UPDATE)

对于 insertupdate 操作,stripInaccessible() 的价值尤为突出。想象一下,没有它,你得多么小心翼翼地处理那些包含敏感字段的数据?

  • insert 操作:使用 stripInaccessible(AccessType.CREATABLE, recordsToInsert)。这能确保在创建新记录时,任何用户无权创建的字段(例如,系统生成的 Id,或者某些特定 Profile 无权设置初始值的字段)都会被移除,防止 DML 失败或产生不符合预期的默认值。

  • update 操作:使用 stripInaccessible(AccessType.UPDATABLE, recordsToUpdate)。这是最常见的用途,确保用户不能修改他们没有更新权限的字段。这对于保护数据完整性至关重要,特别是当数据来源复杂或用户权限模型精细时。

代码示例:安全地处理来自 LWC 的更新请求

假设你的 LWC 允许用户编辑联系人信息,并将修改后的 Contact 对象发送给 Apex 控制器进行更新。

public with sharing class ContactController {

    @AuraEnabled
    public static void updateContacts(List<Contact> contactsFromLWC) {
        // 防御性编程:检查输入是否为空
        if (contactsFromLWC == null || contactsFromLWC.isEmpty()) {
            System.debug('No contacts provided for update.');
            return;
        }

        // 关键:移除当前用户无权更新的字段
        // 即使 LWC 发送了用户界面上看不到或无权编辑的字段,这里也能处理
        SObjectAccessDecision decision = Security.stripInaccessible(
            AccessType.UPDATABLE,
            contactsFromLWC
        );

        // 获取清理后的记录列表
        List<Contact> safeContactsToUpdate = decision.getRecords();

        // 记录被移除的字段,用于审计或调试
        Map<String, Set<String>> removedFields = decision.getRemovedFields();
        if (!removedFields.isEmpty()) {
            System.debug('StripInaccessible removed fields during contact update: ' + JSON.serialize(removedFields));
            // 在实际应用中,可能需要更复杂的日志记录
        }

        // 执行更新操作
        if (!safeContactsToUpdate.isEmpty()) {
            try {
                update safeContactsToUpdate;
                System.debug('Successfully updated ' + safeContactsToUpdate.size() + ' contacts.');
            } catch (DmlException e) {
                // 更健壮的错误处理
                System.debug('Error updating contacts: ' + e.getMessage());
                // 可以考虑向上抛出自定义异常,或返回错误信息给 LWC
                throw new AuraHandledException('Failed to update contacts. Reason: ' + e.getMessage());
            }
        } else {
            System.debug('No contacts left to update after FLS check.');
            // 可能需要通知 LWC,没有进行任何更新
        }
    }
}

这段代码展示了如何在处理来自前端(可能被篡改)的数据时,利用 stripInaccessible() 作为一道安全屏障。

性能考量 - 需要担心吗?

stripInaccessible() 确实需要执行一些底层工作,包括检查每个 SObject 实例中每个字段的权限。这自然会带来一些性能开销。

  • 开销来源:主要是进行 Schema Describe 调用来获取字段的访问权限信息。这些调用是有成本的。
  • 影响因素
    • 记录数量:处理的记录越多,总开销越大。
    • 字段数量:每条记录中包含的字段越多,需要检查的字段也越多。
  • 与手动检查对比:如果只需要检查极少数(比如一两个)固定字段,手动检查的开销可能略低。但只要涉及多个字段,或者字段不固定,stripInaccessible() 的整体开销和代码复杂度优势就体现出来了。手动检查的代码本身执行也需要时间,而且更容易出错。
  • 结论:对于绝大多数应用场景,stripInaccessible() 的性能是可以接受的。它的设计目标是易用性和安全性。不要过早优化! 优先保证代码的正确性和安全性。只有当你通过性能分析(Apex Profiler)发现 stripInaccessible() 确实成为了系统瓶颈时,才需要考虑是否有替代方案(比如检查是否能减少传入的字段数量,或者在极特殊情况下回退到手动检查特定字段)。通常情况下,瓶颈更可能出现在 SOQL 查询、复杂计算或其他地方。

处理结果和潜在问题

stripInaccessible() 返回的 SObjectAccessDecision 对象不仅仅包含清理后的记录列表 (getRecords()),还提供了 getRemovedFields() 方法。这个方法返回一个 Map<String, Set<String>>,其中键是 SObject 的类型(如 'Account'),值是一个包含被移除字段 API 名称的集合。

Map<String, Set<String>> removed = decision.getRemovedFields();
if (removed.containsKey('Opportunity')) {
    Set<String> removedOppFields = removed.get('Opportunity');
    System.debug('Following Opportunity fields were removed: ' + String.join(new List<String>(removedOppFields), ', '));
    // 可以将这些信息记录到日志系统
}

这对于调试和审计非常有用,你可以知道哪些字段因为权限问题被默默地“干掉”了。但要注意,这不适合直接用来给终端用户反馈,因为它太技术化了。如果需要用户友好的反馈,还是得用手动检查。

另外,请记住 stripInaccessible() 是基于当前运行用户的权限来判断的。确保你的 Apex 代码运行在正确的用户上下文中(with sharing, without sharing, inherited sharing),这会直接影响 FLS 的检查结果。

最佳实践小结

  1. 优先使用 stripInaccessible():对于 Apex 中的 insertupdate 操作,尤其是在处理来自外部系统、API 或用户界面的数据时,将 stripInaccessible() 作为默认的 FLS 强制执行手段。
  2. 选对 AccessTypeinsert 前用 AccessType.CREATABLEupdate 前用 AccessType.UPDATABLE
  3. 手动检查用于特殊场景:当你需要基于字段可访问性进行精细的逻辑分支或提供明确的用户错误提示时,才考虑使用 isAccessible(), isCreateable(), isUpdateable()
  4. 别担心性能(除非真的需要):优先考虑代码的简洁、安全和可维护性。只有在性能分析证明 stripInaccessible() 是瓶颈时,才去优化它。
  5. 关注运行上下文stripInaccessible() 的行为取决于代码的共享模式 (with sharing 等)。
  6. 结合 WITH SECURITY_ENFORCED:对于读操作,优先在 SOQL 查询中使用 WITH SECURITY_ENFORCED 来保证数据读取的安全性。

结语

在 Salesforce Apex 开发中,正确执行字段级安全是构建可信赖应用的基础。Security.stripInaccessible() 提供了一个强大、简洁且相对安全的方式来应对这一挑战,特别是对于保护数据库的写操作。虽然它不是万能的,了解它的工作原理、适用场景以及与传统手动检查的区别,能让你在编写 Apex 代码时更加自信,构建出既健壮又安全的应用。下次当你需要处理 DML 操作和 FLS 时,记得优先考虑这位“安全卫士”!

Apex安全老司机 SalesforceApex字段级安全Security.stripInaccessible最佳实践

评论点评

打赏赞助
sponsor

感谢您的支持让我们更好的前行

分享

QRcode

https://www.webkt.com/article/8966