getsentry / getsentry/sentry-java

Normalize `release` option

Aberta
#3,700 1 comentário 0 reações 0 responsáveis Ver no GitHub
Android Improvement
Linguagem predominante
Kotlin
Estrelas
1.4k
Forks
478
Merge médio
3d 4h
PRs com merge (30d)
72

Descrição

### 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

Guia de contribuição

Abrir o guia de contribuição

Direção de pesquisa

Comece localizando onde a opção de release ou versionName é lida e onde a validação de SemVer é feita atualmente. Verifique como os avisos são emitidos no SDK e, em seguida, confirme que versões comuns com prefixo são normalizadas e que valores que ainda forem inválidos geram um aviso claro de que os recursos de release não funcionarão corretamente.

Escrita pelo modelo de indexação a partir do texto da issue.

Avaliação

Stack de tecnologia
java, kotlin
Domínio
release
Tipo de issue
Bug
Dificuldade
3/5
Tempo estimado
1-2 dias
Status de atividade
Estagnada
Clareza
Razoavelmente clara
Facilidade para iniciantes
35/100

Receba novas issues na sua caixa de entrada

Um resumo curto de issues do GitHub para quem está começando.