getsentry / getsentry/sentry-java

Normalize `release` option

未關閉
#3,700 1 則留言 0 個 reaction 已指派 0 人 在 GitHub 檢視
Android Improvement
主要語言
Kotlin
星號
1.4k
分支
478
平均合併
3 天 4 小時
30 天內合併 PR
72

描述

### Description

Currently we read out the `versionName` as-is, but the customer can set any random string in there, which will break our release health feature (and everything around releases in general). So we should try to normalize the `versionName` to comply with SemVer (e.g. relax the semver regex by excluding a prepending `v` for example, if the version was set to `v1.2.3`).

If the version is still not a valid semver after normalization, we should fail/print a warning and let the customer know that the release feature won't be working correctly in some cases.

Related to:
* https://github.com/getsentry/sentry/issues/76502

貢獻指南

開啟貢獻指南

研究方向

先找出讀取 release 選項或 versionName 的位置,以及目前進行 SemVer 驗證的位置。檢查 SDK 如何發出警告,然後確認常見的帶前綴版本會被正規化,並確認仍然無效的值會產生清楚的警告,說明 release 功能將無法正常運作。

由索引模型根據 Issue 內容生成。

評估

技術堆疊
java, kotlin
領域
release
Issue 類型
缺陷
難度
3/5
預估耗時
1-2 天
活躍度
停滯
描述清晰度
基本清楚
新手友好度
35/100

把新 issue 寄到你的電子郵件信箱

精選適合新手參與的 GitHub issue 摘要。