Skylm808 commented on PR #125: URL: https://github.com/apache/dubbo-go-pixiu-samples/pull/125#issuecomment-4146432874
已处理，说明如下： 1. import 这块我按仓库 CI 实际使用的 `imports-formatter` 校验过，如果仅仅按照gofmt，CI会报错，所以我是用 formatter 输出为准 2. `json.Encode` 错误已补充处理，编码失败时会记录日志 3. 全局 `http.Client` 已改为通过 helper 创建独立 client，避免共享状态 4. 测试里的 `5s` 超时已提取为统一常量 5. 关于 `invalid SAML response` / `expired assertion` / `bad signature` / `missing required attributes`，我暂时没有放在 sample 测试里补：前 3 项更偏 `dubbo-go-pixiu` 主仓库里 SAML filter 本体的协议级测试，如果需要，我在主仓库进行添加对应的测试；当前 sample 这里使用的是 `forward_attributes`，并没有定义 `required attributes` 语义 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
