如何解决URLEncoder线上失效问题？品牌名含空格引发500错误排查

从报错信息看，核心问题是下游请求URL中的brand参数直接包含空格（Huawei Spa），未被正确编码为Huawei%20Spa，触发了URISyntaxException。你的代码逻辑存在两处关键错误，导致编码操作失效：

代码错误点 #

1. 编码后错误拆分字符串
   在ServiceImpl中，你对编码后的品牌名执行了split("[^a-zA-Z0-9\-_']+")，这个正则会把URL编码后的%20（空格的标准编码）中的%、2、0全部识别为非允许字符，直接将Huawei%20Spa拆分为Huawei和Spa两个独立字符串，完全破坏了编码结果。

2. 无意义的解码操作
   控制器中对已经被拆分破坏的字符串执行URLDecoder.decode，此时字符串中已无任何编码字符（%xx格式），解码操作完全无效，反而增加不必要的开销。

修复方案 #

1. 修正业务逻辑中的字符串处理 #

根据你的业务场景，分两种情况处理：

• 场景1：单个品牌查询
  直接保留原始品牌名，移除错误的split操作，编码逻辑应该放在生成下游请求URL的环节，而非业务逻辑中：

  @Override
  public Collection<String> getDistinctBrands(Filter1 productFilter) {
      List<String> distinctBrands = new ArrayList<>();
  
      if (productFilter != null && productFilter.getBrand() != null && !productFilter.getBrand().isEmpty()) {
          // 直接添加原始品牌名，编码在请求构建时处理
          distinctBrands.add(productFilter.getBrand());
      } else {
          distinctBrands = dealRepository.findDistinctBrands();
      }
      return distinctBrands;
  }
  

• 场景2：多个品牌用分隔符（如逗号）传递
  先拆分原始品牌字符串，再处理每个品牌，同样将编码逻辑放在请求构建环节：

  @Override
  public Collection<String> getDistinctBrands(Filter1 productFilter) {
      List<String> distinctBrands = new ArrayList<>();
  
      if (productFilter != null && productFilter.getBrand() != null && !productFilter.getBrand().isEmpty()) {
          // 先按分隔符拆分，再过滤空值
          String[] rawBrands = productFilter.getBrand().split(",");
          for (String brand : rawBrands) {
              String trimmedBrand = brand.trim();
              if (!trimmedBrand.isEmpty()) {
                  distinctBrands.add(trimmedBrand);
              }
          }
      } else {
          distinctBrands = dealRepository.findDistinctBrands();
      }
      return distinctBrands;
  }
  

2. 正确生成编码后的下游请求URL #

使用Spring的UriComponentsBuilder构建请求URL，它会自动按照URI规范对参数进行编码，彻底避免手动编码的错误：

// 示例：调用下游服务时的URL构建逻辑
UriComponentsBuilder urlBuilder = UriComponentsBuilder.fromHttpUrl("http://gateway-service/deal-solution-service/api/deals")
        .queryParam("categories", false)
        .queryParam("phonesAndSim", false)
        .queryParam("lteAndSim", false)
        .queryParam("tabletsAndLaptops", false)
        .queryParam("connectivity", false)
        .queryParam("priceMin", 0)
        .queryParam("priceMax", 50000)
        .queryParam("duration36", false)
        .queryParam("duration24", false)
        .queryParam("brand", productFilter.getBrand()); // 自动编码空格为%20

// 生成最终URL并发送请求
String encodedUrl = urlBuilder.toUriString();
// 后续发送请求逻辑...

3. 移除控制器中的无效解码 #

如果接口返回的是原始品牌名，无需解码，直接返回即可：

@Override
public ResponseEntity<List<String>> getDistinctBrands(Filter1 filter) {
    Collection<String> distinctBrands = dealService.getDistinctBrands(filter);
    return distinctBrands.isEmpty() 
            ? ResponseEntity.notFound().build() 
            : ResponseEntity.ok(new ArrayList<>(distinctBrands));
}

环境差异原因 #

本地/测试环境可能因为网关或服务器配置（如Tomcat的宽松模式）自动将空格替换为%20，掩盖了编码错误；而线上环境遵循严格的URI规范，直接抛出异常。

内容的提问来源于stack exchange，提问作者Esphi01