Skip to content

feat(common): get signed location api url - #489

Merged
Penty-d merged 13 commits into
west2-online:mainfrom
Sna-ken:main
Jul 29, 2026
Merged

feat(common): get signed location api url#489
Penty-d merged 13 commits into
west2-online:mainfrom
Sna-ken:main

Conversation

@Sna-ken

@Sna-ken Sna-ken commented Jun 7, 2026

Copy link
Copy Markdown
Contributor

自查 PR 结构

  • PR 标题符合这个格式: <type>(optional scope): <description>

  • 此 PR 标题的描述以用户为导向,足够清晰,其他人可以理解。

  • 我已经对所有 commit 提供了签名(GPG 密钥签名、SSH 密钥签名)

  • 这个 PR 属于强制变更/破坏性更改

如果是,请在 PR 标题中添加 BREAKING CHANGE 前缀,并在 PR 描述中详细说明。

这个 PR 的类型是什么?

feat:增加获取signed location api url的接口

这个 PR 做了什么 / 我们为什么需要这个 PR?

发送经纬度信息返回签名好的api url和headers

(可选)这个 PR 解决了哪个/些 issue?

对 Reviewer 预留的一些提醒

Signed-off-by: Sna-ken <1339440756@qq.com>
@Sna-ken
Sna-ken requested review from a team, jiuxia211, ozline and renbaoshuo as code owners June 7, 2026 08:37
@Sna-ken
Sna-ken requested a review from mutezebra as a code owner July 22, 2026 08:10
Comment thread internal/common/service/get_sigined_location_api_url.go Outdated
Comment thread internal/common/service/get_sigined_location_api_url.go Outdated
Comment thread config/config.example.yaml Outdated
Comment thread internal/common/service/get_sigined_location_api_url.go Outdated
Comment thread pkg/constants/token.go Outdated
Comment thread api/model/api/api.go
Comment thread internal/common/service/get_signed_location_api_url_test.go
@renbaoshuo renbaoshuo changed the title [WIP]feat:get signed location api url feat: get signed location api url Jul 28, 2026
@codecov

codecov Bot commented Jul 28, 2026

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 96.87500% with 2 lines in your changes missing coverage. Please review.

Files with missing lines Patch % Lines
...rnal/common/service/get_signed_location_api_url.go 95.34% 1 Missing and 1 partial ⚠️
@@            Coverage Diff             @@
##             main     #489      +/-   ##
==========================================
- Coverage   64.29%   63.48%   -0.81%     
==========================================
  Files         233      234       +1     
  Lines        6108     6168      +60     
==========================================
- Hits         3927     3916      -11     
- Misses       2055     2128      +73     
+ Partials      126      124       -2     
Flag Coverage Δ
unittest 63.48% <96.87%> (-0.81%) ⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

Files with missing lines Coverage Δ
api/handler/api/common_service.go 100.00% <100.00%> (ø)
internal/common/service/service.go 100.00% <100.00%> (ø)
...rnal/common/service/get_signed_location_api_url.go 95.34% <95.34%> (ø)

... and 1 file with indirect coverage changes

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@Sna-ken Sna-ken changed the title feat: get signed location api url feat(common): get signed location api url Jul 28, 2026
Comment thread internal/common/service/get_signed_location_api_url.go Outdated
Comment thread internal/common/service/get_signed_location_api_url_test.go
Comment thread internal/common/service/get_signed_location_api_url.go Outdated
Comment thread internal/common/service/get_signed_location_api_url.go Outdated
@Penty-d
Penty-d merged commit ed34331 into west2-online:main Jul 29, 2026
6 of 7 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants