Compare commits
6
Commits
| Author | SHA1 | Date | |
|---|---|---|---|
|
|
8f12bf32bc | ||
|
|
b9ae128842 | ||
|
|
a86dca50b8 | ||
|
|
fc49eae512 | ||
|
|
ffbae9edfd | ||
|
|
2268c64250 |
@@ -9,3 +9,5 @@ keystore.properties
|
|||||||
*.jks
|
*.jks
|
||||||
release-artifacts/
|
release-artifacts/
|
||||||
app/src/main/res_bak/
|
app/src/main/res_bak/
|
||||||
|
scripts/*.json
|
||||||
|
scripts/*.json:Zone.Identifier
|
||||||
|
|||||||
@@ -31,9 +31,51 @@ Przed wydaniem zapisz wszystkie zmiany źródłowe w commicie. Następnie urucho
|
|||||||
```
|
```
|
||||||
|
|
||||||
Skrypt automatycznie zwiększa `versionCode` i ostatni człon `versionName`, uruchamia testy,
|
Skrypt automatycznie zwiększa `versionCode` i ostatni człon `versionName`, uruchamia testy,
|
||||||
buduje podpisany APK dla Gitea oraz AAB dla Google Play Console, tworzy commit wersji,
|
buduje podpisany APK oraz AAB, a po poprawnym buildzie pyta, czy wysłać aktualizację do
|
||||||
tag, push i Gitea Release. Artefakty zostają również zapisane lokalnie w
|
Google Play. Po odpowiedzi „tak” publikuje AAB w skonfigurowanym teście zamkniętym i
|
||||||
`release-artifacts/<tag>/`.
|
wysyła zmiany do weryfikacji. Następnie tworzy commit wersji, tag, push i Gitea Release.
|
||||||
|
Artefakty zostają również zapisane lokalnie w `release-artifacts/<tag>/`.
|
||||||
|
|
||||||
|
Do publikacji Google Play konto serwisowe musi mieć dla aplikacji
|
||||||
|
`pl.firmatpp.kierowca` uprawnienia „Wyświetlanie informacji o aplikacji (tylko do odczytu)”
|
||||||
|
oraz „Publikowanie aplikacji na ścieżkach testowych”. Pobrany klucz JSON umieść
|
||||||
|
bezpośrednio w folderze `scripts`, na przykład:
|
||||||
|
|
||||||
|
```text
|
||||||
|
scripts/google-play-service-account.json
|
||||||
|
```
|
||||||
|
|
||||||
|
Skrypt sam znajduje jedyny plik `scripts/*.json`, sprawdza, czy jest prawidłowym kluczem
|
||||||
|
konta serwisowego, i używa go do autoryzacji. Pliki `scripts/*.json` są ignorowane przez
|
||||||
|
Git i nie mogą zostać przypadkowo dodane do repozytorium. Jeśli w folderze znajduje się
|
||||||
|
więcej niż jeden prawidłowy klucz, skrypt przerwie działanie i poprosi o pozostawienie
|
||||||
|
jednego.
|
||||||
|
|
||||||
|
`GOOGLE_PLAY_TRACK` musi być identyfikatorem istniejącego testu zamkniętego. Domyślna
|
||||||
|
wartość to `alpha`. Opcjonalnie można ustawić `GOOGLE_PLAY_PACKAGE_NAME` i
|
||||||
|
`GOOGLE_PLAY_RELEASE_LANGUAGE` (domyślnie `pl-PL`). Zmienna
|
||||||
|
`GOOGLE_PLAY_SERVICE_ACCOUNT_CREDENTIALS` pozostaje opcjonalnym sposobem wskazania klucza
|
||||||
|
spoza folderu `scripts`. Skrypt nie zapisuje tokenu ani klucza w repozytorium; prywatny
|
||||||
|
klucz tymczasowy jest usuwany po zakończeniu.
|
||||||
|
|
||||||
|
Przed zatwierdzeniem skrypt waliduje edycję Google Play. Commit edycji jawnie wysyła
|
||||||
|
zmiany do weryfikacji. Jeżeli inne zmiany są już sprawdzane, skrypt nie anuluje ich —
|
||||||
|
kończy się błędem i można go wznowić po zakończeniu poprzedniej weryfikacji.
|
||||||
|
Jeżeli na koncie jest włączone publikowanie zarządzane, po akceptacji Google nadal trzeba
|
||||||
|
wybrać „Opublikuj zmiany” w Play Console; bez publikowania zarządzanego zaakceptowane
|
||||||
|
wydanie zostanie udostępnione testerom automatycznie.
|
||||||
|
|
||||||
|
Awaryjne wydanie wyłącznie do Gitea, bez Google Play:
|
||||||
|
|
||||||
|
```bash
|
||||||
|
./scripts/release-android.sh --skip-google-play
|
||||||
|
```
|
||||||
|
|
||||||
|
W środowisku bez interaktywnego terminala decyzję trzeba podać jawnie:
|
||||||
|
|
||||||
|
```bash
|
||||||
|
./scripts/release-android.sh --publish-google-play
|
||||||
|
```
|
||||||
|
|
||||||
Do autoryzacji Gitea używany jest `GITEA_TOKEN` albo dane zapisane w Git credential helper.
|
Do autoryzacji Gitea używany jest `GITEA_TOKEN` albo dane zapisane w Git credential helper.
|
||||||
Jeśli publikacja została przerwana już po utworzeniu commita wersji, można ją wznowić:
|
Jeśli publikacja została przerwana już po utworzeniu commita wersji, można ją wznowić:
|
||||||
|
|||||||
@@ -34,8 +34,8 @@ android {
|
|||||||
applicationId = "pl.firmatpp.kierowca"
|
applicationId = "pl.firmatpp.kierowca"
|
||||||
minSdk = 26
|
minSdk = 26
|
||||||
targetSdk = 35
|
targetSdk = 35
|
||||||
versionCode = 114
|
versionCode = 117
|
||||||
versionName = "1.0.61"
|
versionName = "1.0.64"
|
||||||
setProperty("archivesBaseName", "pl.firmatpp.kierowca")
|
setProperty("archivesBaseName", "pl.firmatpp.kierowca")
|
||||||
testInstrumentationRunner = "androidx.test.runner.AndroidJUnitRunner"
|
testInstrumentationRunner = "androidx.test.runner.AndroidJUnitRunner"
|
||||||
|
|
||||||
|
|||||||
@@ -50,7 +50,10 @@ object ApiErrorMapper {
|
|||||||
map(throwable).kind != ApiErrorKind.Network
|
map(throwable).kind != ApiErrorKind.Network
|
||||||
|
|
||||||
fun mapHttpStatus(statusCode: Int, body: String?): ApiError {
|
fun mapHttpStatus(statusCode: Int, body: String?): ApiError {
|
||||||
val code = body?.let { """"code"\s*:\s*"([^"]+)"""".toRegex().find(it)?.groupValues?.getOrNull(1) }
|
val code = body?.let {
|
||||||
|
""""code"\s*:\s*"([^"]+)"""".toRegex().find(it)?.groupValues?.getOrNull(1)
|
||||||
|
?: """"reason"\s*:\s*"([^"]+)"""".toRegex().find(it)?.groupValues?.getOrNull(1)
|
||||||
|
}
|
||||||
val message = body?.let { """"message"\s*:\s*"([^"]+)"""".toRegex().find(it)?.groupValues?.getOrNull(1) }
|
val message = body?.let { """"message"\s*:\s*"([^"]+)"""".toRegex().find(it)?.groupValues?.getOrNull(1) }
|
||||||
val retryable = body?.let { """"retryable"\s*:\s*(true|false)""".toRegex().find(it)?.groupValues?.getOrNull(1)?.toBooleanStrictOrNull() }
|
val retryable = body?.let { """"retryable"\s*:\s*(true|false)""".toRegex().find(it)?.groupValues?.getOrNull(1)?.toBooleanStrictOrNull() }
|
||||||
|
|
||||||
@@ -81,16 +84,21 @@ object ApiErrorMapper {
|
|||||||
}
|
}
|
||||||
val defaultRetryable = statusCode == 429 || statusCode in 500..599
|
val defaultRetryable = statusCode == 429 || statusCode in 500..599
|
||||||
val resolvedRetryable = retryable ?: defaultRetryable
|
val resolvedRetryable = retryable ?: defaultRetryable
|
||||||
val resolvedMessage = message?.takeIf { it.isNotBlank() } ?: when (kind) {
|
val resolvedMessage = when {
|
||||||
|
code == "INVALID_OTP" -> "Kod jest nieprawidłowy lub wygasł. Użyj kodu z najnowszego SMS-a."
|
||||||
|
kind == ApiErrorKind.RateLimited -> "Za dużo prób. Odczekaj chwilę i spróbuj ponownie."
|
||||||
|
!message.isNullOrBlank() -> message
|
||||||
|
else -> when (kind) {
|
||||||
ApiErrorKind.Auth -> "Sesja wygasła. Zaloguj się ponownie."
|
ApiErrorKind.Auth -> "Sesja wygasła. Zaloguj się ponownie."
|
||||||
ApiErrorKind.Forbidden -> "Brak uprawnień do tej operacji."
|
ApiErrorKind.Forbidden -> "Brak uprawnień do tej operacji."
|
||||||
ApiErrorKind.Validation -> "Serwer odrzucił operację. Nie została zapisana."
|
ApiErrorKind.Validation -> "Serwer odrzucił operację. Nie została zapisana."
|
||||||
ApiErrorKind.Conflict -> "Operacja jest w konflikcie z aktualnym stanem danych."
|
ApiErrorKind.Conflict -> "Operacja jest w konflikcie z aktualnym stanem danych."
|
||||||
ApiErrorKind.RateLimited -> "Za dużo prób. Aplikacja spróbuje ponownie później."
|
ApiErrorKind.RateLimited -> "Za dużo prób. Odczekaj chwilę i spróbuj ponownie."
|
||||||
ApiErrorKind.Server -> "Serwer nie potwierdził operacji. Aplikacja spróbuje ponownie."
|
ApiErrorKind.Server -> "Serwer nie potwierdził operacji. Aplikacja spróbuje ponownie."
|
||||||
ApiErrorKind.Network -> "Nie udało się połączyć z serwerem. Operacja nie została potwierdzona."
|
ApiErrorKind.Network -> "Nie udało się połączyć z serwerem. Operacja nie została potwierdzona."
|
||||||
ApiErrorKind.Unknown -> "Wystąpił błąd. Operacja nie została potwierdzona."
|
ApiErrorKind.Unknown -> "Wystąpił błąd. Operacja nie została potwierdzona."
|
||||||
}
|
}
|
||||||
|
}
|
||||||
|
|
||||||
return ApiError(
|
return ApiError(
|
||||||
kind = kind,
|
kind = kind,
|
||||||
@@ -103,7 +111,11 @@ object ApiErrorMapper {
|
|||||||
}
|
}
|
||||||
|
|
||||||
private fun mapHttpException(exception: HttpException): ApiError {
|
private fun mapHttpException(exception: HttpException): ApiError {
|
||||||
val body = runCatching { exception.response()?.errorBody()?.string() }.getOrNull()
|
val body = runCatching {
|
||||||
|
val source = exception.response()?.errorBody()?.source() ?: return@runCatching null
|
||||||
|
source.request(Long.MAX_VALUE)
|
||||||
|
source.buffer.clone().readUtf8()
|
||||||
|
}.getOrNull()
|
||||||
return mapHttpStatus(exception.code(), body)
|
return mapHttpStatus(exception.code(), body)
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|||||||
@@ -67,9 +67,11 @@ class RoutePointWorker(
|
|||||||
"retryable" to error.retryable,
|
"retryable" to error.retryable,
|
||||||
),
|
),
|
||||||
)
|
)
|
||||||
|
if (shouldStopRouteTrackingAfterPointError(error.kind, error.code)) {
|
||||||
|
ActiveRouteTrackingService.stop(applicationContext, routeId)
|
||||||
|
}
|
||||||
if (isInactiveRoutePointError(error.code)) {
|
if (isInactiveRoutePointError(error.code)) {
|
||||||
dao.delete(pointIds)
|
dao.delete(pointIds)
|
||||||
ActiveRouteTrackingService.stop(applicationContext, routeId)
|
|
||||||
continue
|
continue
|
||||||
}
|
}
|
||||||
|
|
||||||
@@ -105,3 +107,6 @@ class RoutePointWorker(
|
|||||||
}
|
}
|
||||||
|
|
||||||
internal fun isInactiveRoutePointError(code: String?): Boolean = code == "ROUTE_POINTS_INVALID_STATUS"
|
internal fun isInactiveRoutePointError(code: String?): Boolean = code == "ROUTE_POINTS_INVALID_STATUS"
|
||||||
|
|
||||||
|
internal fun shouldStopRouteTrackingAfterPointError(kind: ApiErrorKind, code: String?): Boolean =
|
||||||
|
kind == ApiErrorKind.Auth || isInactiveRoutePointError(code)
|
||||||
|
|||||||
@@ -136,6 +136,13 @@ class ActiveRouteTrackingService : Service(), LocationListener {
|
|||||||
|
|
||||||
override fun onDestroy() {
|
override fun onDestroy() {
|
||||||
runCatching { locationManager.removeUpdates(this) }
|
runCatching { locationManager.removeUpdates(this) }
|
||||||
|
if (Build.VERSION.SDK_INT >= Build.VERSION_CODES.N) {
|
||||||
|
stopForeground(STOP_FOREGROUND_REMOVE)
|
||||||
|
} else {
|
||||||
|
@Suppress("DEPRECATION")
|
||||||
|
stopForeground(true)
|
||||||
|
}
|
||||||
|
getSystemService(NotificationManager::class.java).cancel(NOTIFICATION_ID)
|
||||||
getSharedPreferences(TRACKING_PREFERENCES, Context.MODE_PRIVATE)
|
getSharedPreferences(TRACKING_PREFERENCES, Context.MODE_PRIVATE)
|
||||||
.edit()
|
.edit()
|
||||||
.remove(ACTIVE_ROUTE_ID)
|
.remove(ACTIVE_ROUTE_ID)
|
||||||
|
|||||||
@@ -247,8 +247,10 @@ fun DriverApp(
|
|||||||
}
|
}
|
||||||
}
|
}
|
||||||
SmsUserConsentEffect(
|
SmsUserConsentEffect(
|
||||||
enabled = state.screen == DriverScreen.Phone || state.screen == DriverScreen.Otp,
|
enabled = state.otpRequestPending ||
|
||||||
onCode = viewModel::updateOtpCode,
|
(state.screen == DriverScreen.Otp && !state.loading),
|
||||||
|
requestId = state.otpConsentRequestId,
|
||||||
|
onCode = viewModel::updateOtpCodeFromSms,
|
||||||
)
|
)
|
||||||
LaunchedEffect(initialRouteId, initialRouteTarget) {
|
LaunchedEffect(initialRouteId, initialRouteTarget) {
|
||||||
viewModel.openRouteFromNotification(initialRouteId, initialRouteTarget)
|
viewModel.openRouteFromNotification(initialRouteId, initialRouteTarget)
|
||||||
@@ -282,7 +284,13 @@ fun DriverApp(
|
|||||||
onUseOfflineNow = viewModel::useOfflineStartupDataNow,
|
onUseOfflineNow = viewModel::useOfflineStartupDataNow,
|
||||||
)
|
)
|
||||||
DriverScreen.Phone -> PhoneScreen(state, viewModel::requestOtp)
|
DriverScreen.Phone -> PhoneScreen(state, viewModel::requestOtp)
|
||||||
DriverScreen.Otp -> OtpScreen(state, viewModel::updateOtpCode, viewModel::verifyOtp, viewModel::back)
|
DriverScreen.Otp -> OtpScreen(
|
||||||
|
state = state,
|
||||||
|
onCodeChange = viewModel::updateOtpCode,
|
||||||
|
onSubmit = viewModel::verifyOtp,
|
||||||
|
onResend = { viewModel.requestOtp(state.phone) },
|
||||||
|
onBack = viewModel::back,
|
||||||
|
)
|
||||||
DriverScreen.Routes -> RoutesScreen(
|
DriverScreen.Routes -> RoutesScreen(
|
||||||
state = state,
|
state = state,
|
||||||
onRefresh = viewModel::refreshRoutesSilently,
|
onRefresh = viewModel::refreshRoutesSilently,
|
||||||
@@ -840,8 +848,19 @@ private fun OtpScreen(
|
|||||||
state: DriverUiState,
|
state: DriverUiState,
|
||||||
onCodeChange: (String) -> Unit,
|
onCodeChange: (String) -> Unit,
|
||||||
onSubmit: (String) -> Unit,
|
onSubmit: (String) -> Unit,
|
||||||
|
onResend: () -> Unit,
|
||||||
onBack: () -> Unit,
|
onBack: () -> Unit,
|
||||||
) {
|
) {
|
||||||
|
var nowEpochMillis by remember(state.otpResendAvailableAtEpochMillis) { mutableStateOf(System.currentTimeMillis()) }
|
||||||
|
val resendDelaySeconds = otpResendDelaySeconds(state.otpResendAvailableAtEpochMillis, nowEpochMillis)
|
||||||
|
|
||||||
|
LaunchedEffect(state.otpResendAvailableAtEpochMillis) {
|
||||||
|
while (nowEpochMillis < state.otpResendAvailableAtEpochMillis) {
|
||||||
|
delay(1_000L)
|
||||||
|
nowEpochMillis = System.currentTimeMillis()
|
||||||
|
}
|
||||||
|
}
|
||||||
|
|
||||||
AuthShell(title = "Kod SMS", subtitle = "Wpisz kod wyslany na ${state.maskedPhone}.", onBack = onBack) {
|
AuthShell(title = "Kod SMS", subtitle = "Wpisz kod wyslany na ${state.maskedPhone}.", onBack = onBack) {
|
||||||
OutlinedTextField(
|
OutlinedTextField(
|
||||||
value = state.otpCode,
|
value = state.otpCode,
|
||||||
@@ -853,7 +872,20 @@ private fun OtpScreen(
|
|||||||
)
|
)
|
||||||
ErrorText(state.error)
|
ErrorText(state.error)
|
||||||
Spacer(Modifier.height(if (state.error.isNullOrBlank()) 18.dp else 8.dp))
|
Spacer(Modifier.height(if (state.error.isNullOrBlank()) 18.dp else 8.dp))
|
||||||
PrimaryButton("Zaloguj") { onSubmit(state.otpCode) }
|
PrimaryButton(
|
||||||
|
label = "Zaloguj",
|
||||||
|
enabled = isRetrievedOtpCode(state.otpCode) && !state.loading,
|
||||||
|
) { onSubmit(state.otpCode) }
|
||||||
|
TextButton(
|
||||||
|
onClick = onResend,
|
||||||
|
enabled = resendDelaySeconds == 0 && !state.loading,
|
||||||
|
modifier = Modifier.align(Alignment.CenterHorizontally),
|
||||||
|
) {
|
||||||
|
Text(
|
||||||
|
if (resendDelaySeconds > 0) "Wyślij kod ponownie za $resendDelaySeconds s"
|
||||||
|
else "Wyślij kod ponownie",
|
||||||
|
)
|
||||||
|
}
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|
||||||
@@ -4390,9 +4422,10 @@ private fun PhotoPreviewErrorOverlay() {
|
|||||||
}
|
}
|
||||||
|
|
||||||
@Composable
|
@Composable
|
||||||
private fun PrimaryButton(label: String, onClick: () -> Unit) {
|
private fun PrimaryButton(label: String, enabled: Boolean = true, onClick: () -> Unit) {
|
||||||
Button(
|
Button(
|
||||||
onClick = onClick,
|
onClick = onClick,
|
||||||
|
enabled = enabled,
|
||||||
modifier = Modifier.fillMaxWidth().height(56.dp),
|
modifier = Modifier.fillMaxWidth().height(56.dp),
|
||||||
colors = ButtonDefaults.buttonColors(containerColor = TppTheme.colors.forest),
|
colors = ButtonDefaults.buttonColors(containerColor = TppTheme.colors.forest),
|
||||||
shape = MaterialTheme.shapes.small,
|
shape = MaterialTheme.shapes.small,
|
||||||
@@ -4544,28 +4577,48 @@ private fun bestLastKnownLocation(context: Context): Location? {
|
|||||||
}
|
}
|
||||||
|
|
||||||
@Composable
|
@Composable
|
||||||
private fun SmsUserConsentEffect(enabled: Boolean, onCode: (String) -> Unit) {
|
private fun SmsUserConsentEffect(enabled: Boolean, requestId: Long, onCode: (Long, String) -> Unit) {
|
||||||
val context = LocalContext.current
|
val context = LocalContext.current
|
||||||
|
var listenGeneration by remember(requestId) { mutableStateOf(0) }
|
||||||
|
var launchedRequestId by remember { mutableStateOf<Long?>(null) }
|
||||||
val launcher = rememberLauncherForActivityResult(ActivityResultContracts.StartActivityForResult()) { result ->
|
val launcher = rememberLauncherForActivityResult(ActivityResultContracts.StartActivityForResult()) { result ->
|
||||||
|
val completedRequestId = launchedRequestId
|
||||||
|
var codeDelivered = false
|
||||||
if (result.resultCode == Activity.RESULT_OK) {
|
if (result.resultCode == Activity.RESULT_OK) {
|
||||||
val message = result.data?.getStringExtra(SmsRetriever.EXTRA_SMS_MESSAGE).orEmpty()
|
val message = result.data?.getStringExtra(SmsRetriever.EXTRA_SMS_MESSAGE).orEmpty()
|
||||||
OtpCodeExtractor.extract(message)?.let(onCode)
|
if (completedRequestId != null) {
|
||||||
|
OtpCodeExtractor.extract(message)?.let { code ->
|
||||||
|
codeDelivered = true
|
||||||
|
onCode(completedRequestId, code)
|
||||||
|
}
|
||||||
|
}
|
||||||
|
}
|
||||||
|
if (!codeDelivered && enabled && completedRequestId == requestId) {
|
||||||
|
listenGeneration += 1
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|
||||||
DisposableEffect(enabled, context) {
|
LaunchedEffect(enabled, requestId, listenGeneration) {
|
||||||
if (!enabled) return@DisposableEffect onDispose { }
|
if (enabled && requestId > 0L) {
|
||||||
|
|
||||||
SmsRetriever.getClient(context).startSmsUserConsent(null)
|
SmsRetriever.getClient(context).startSmsUserConsent(null)
|
||||||
|
}
|
||||||
|
}
|
||||||
|
|
||||||
|
DisposableEffect(enabled, context, requestId) {
|
||||||
|
if (!enabled) return@DisposableEffect onDispose { }
|
||||||
|
|
||||||
val receiver = object : BroadcastReceiver() {
|
val receiver = object : BroadcastReceiver() {
|
||||||
override fun onReceive(receiverContext: Context?, intent: Intent?) {
|
override fun onReceive(receiverContext: Context?, intent: Intent?) {
|
||||||
if (intent?.action != SmsRetriever.SMS_RETRIEVED_ACTION) return
|
if (intent?.action != SmsRetriever.SMS_RETRIEVED_ACTION) return
|
||||||
|
|
||||||
val status = intent.smsRetrieverStatus() ?: return
|
val status = intent.smsRetrieverStatus() ?: return
|
||||||
if (status.statusCode != CommonStatusCodes.SUCCESS) return
|
when (status.statusCode) {
|
||||||
|
CommonStatusCodes.SUCCESS -> intent.smsConsentIntent()?.let { consentIntent ->
|
||||||
intent.smsConsentIntent()?.let(launcher::launch)
|
launchedRequestId = requestId
|
||||||
|
launcher.launch(consentIntent)
|
||||||
|
}
|
||||||
|
CommonStatusCodes.TIMEOUT -> listenGeneration += 1
|
||||||
|
}
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|
||||||
|
|||||||
@@ -68,6 +68,9 @@ data class DriverUiState(
|
|||||||
val startupOfflineAvailable: Boolean = false,
|
val startupOfflineAvailable: Boolean = false,
|
||||||
val phone: String = "",
|
val phone: String = "",
|
||||||
val otpCode: String = "",
|
val otpCode: String = "",
|
||||||
|
val otpConsentRequestId: Long = 0L,
|
||||||
|
val otpRequestPending: Boolean = false,
|
||||||
|
val otpResendAvailableAtEpochMillis: Long = 0L,
|
||||||
val maskedPhone: String = "",
|
val maskedPhone: String = "",
|
||||||
val driver: DriverDto? = null,
|
val driver: DriverDto? = null,
|
||||||
val routes: List<DriverRouteDto> = emptyList(),
|
val routes: List<DriverRouteDto> = emptyList(),
|
||||||
@@ -203,6 +206,7 @@ class DriverViewModel(application: Application) : AndroidViewModel(application)
|
|||||||
private val diagnosticSnapshotProvider = DiagnosticSnapshotProvider(application)
|
private val diagnosticSnapshotProvider = DiagnosticSnapshotProvider(application)
|
||||||
private val _state = MutableStateFlow(DriverUiState(loading = true))
|
private val _state = MutableStateFlow(DriverUiState(loading = true))
|
||||||
private val otpAutoSubmitPolicy = OtpAutoSubmitPolicy()
|
private val otpAutoSubmitPolicy = OtpAutoSubmitPolicy()
|
||||||
|
private val otpAttemptCoordinator = OtpAttemptCoordinator()
|
||||||
private val liveSyncClient = DriverLiveSyncClient(
|
private val liveSyncClient = DriverLiveSyncClient(
|
||||||
repository = repository,
|
repository = repository,
|
||||||
onConnected = { viewModelScope.launch { checkRemoteSyncState() } },
|
onConnected = { viewModelScope.launch { checkRemoteSyncState() } },
|
||||||
@@ -312,19 +316,64 @@ class DriverViewModel(application: Application) : AndroidViewModel(application)
|
|||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|
||||||
fun requestOtp(phone: String) = runLoading("request_otp") {
|
fun requestOtp(phone: String) {
|
||||||
if (!_state.value.isOnline) throw IOException("Logowanie wymaga połączenia z internetem.")
|
val requestSnapshot = _state.value
|
||||||
val response = repository.requestOtp(phone)
|
val now = System.currentTimeMillis()
|
||||||
|
val resendDelay = if (requestSnapshot.phone == phone) {
|
||||||
|
otpResendDelaySeconds(requestSnapshot.otpResendAvailableAtEpochMillis, now)
|
||||||
|
} else {
|
||||||
|
0
|
||||||
|
}
|
||||||
|
if (resendDelay > 0) {
|
||||||
|
_state.update { it.copy(error = "Nowy SMS możesz wysłać za $resendDelay s.") }
|
||||||
|
return
|
||||||
|
}
|
||||||
|
|
||||||
|
val isResend = requestSnapshot.screen == DriverScreen.Otp && requestSnapshot.phone == phone
|
||||||
|
val retainedCode = requestSnapshot.otpCode.takeIf { isResend }.orEmpty()
|
||||||
|
val attemptId = otpAttemptCoordinator.beginAttempt()
|
||||||
otpAutoSubmitPolicy.reset()
|
otpAutoSubmitPolicy.reset()
|
||||||
|
|
||||||
|
viewModelScope.launch {
|
||||||
|
_state.update {
|
||||||
|
it.copy(
|
||||||
|
loading = true,
|
||||||
|
phone = phone,
|
||||||
|
otpCode = retainedCode,
|
||||||
|
otpConsentRequestId = attemptId,
|
||||||
|
otpRequestPending = true,
|
||||||
|
error = null,
|
||||||
|
)
|
||||||
|
}
|
||||||
|
kotlinx.coroutines.yield()
|
||||||
|
|
||||||
|
val result = runCatching {
|
||||||
|
if (!_state.value.isOnline) throw IOException("Logowanie wymaga połączenia z internetem.")
|
||||||
|
repository.requestOtp(phone)
|
||||||
|
}
|
||||||
|
|
||||||
|
if (!otpAttemptCoordinator.isCurrent(attemptId)) return@launch
|
||||||
|
|
||||||
|
result.onSuccess { response ->
|
||||||
|
val bufferedCode = otpAttemptCoordinator.takeBufferedCode(attemptId)
|
||||||
_state.update {
|
_state.update {
|
||||||
it.copy(
|
it.copy(
|
||||||
screen = DriverScreen.Otp,
|
screen = DriverScreen.Otp,
|
||||||
phone = phone,
|
loading = false,
|
||||||
otpCode = "",
|
otpCode = bufferedCode ?: retainedCode,
|
||||||
maskedPhone = response.phoneMasked.orEmpty(),
|
maskedPhone = response.phoneMasked.orEmpty(),
|
||||||
|
otpRequestPending = false,
|
||||||
|
otpResendAvailableAtEpochMillis = System.currentTimeMillis() + OTP_RESEND_COOLDOWN_MILLIS,
|
||||||
error = null,
|
error = null,
|
||||||
)
|
)
|
||||||
}
|
}
|
||||||
|
bufferedCode?.let { applyOtpCode(it, retrievedFromSms = true) }
|
||||||
|
}.onFailure { throwable ->
|
||||||
|
otpAttemptCoordinator.discard(attemptId)
|
||||||
|
reportHandledException("request_otp", throwable)
|
||||||
|
_state.update { it.withApiError(throwable).copy(loading = false, otpRequestPending = false) }
|
||||||
|
}
|
||||||
|
}
|
||||||
}
|
}
|
||||||
|
|
||||||
fun verifyOtp(code: String) = runLoading("verify_otp") {
|
fun verifyOtp(code: String) = runLoading("verify_otp") {
|
||||||
@@ -336,11 +385,35 @@ class DriverViewModel(application: Application) : AndroidViewModel(application)
|
|||||||
}
|
}
|
||||||
|
|
||||||
fun updateOtpCode(code: String) {
|
fun updateOtpCode(code: String) {
|
||||||
val sanitized = code.filter(Char::isDigit).take(6)
|
applyOtpCode(sanitizeOtpCode(code), retrievedFromSms = false)
|
||||||
_state.update { it.copy(otpCode = sanitized) }
|
}
|
||||||
|
|
||||||
if (otpAutoSubmitPolicy.shouldSubmit(sanitized, _state.value.loading)) {
|
fun updateOtpCodeFromSms(attemptId: Long, code: String) {
|
||||||
verifyOtp(sanitized)
|
val sanitized = sanitizeOtpCode(code)
|
||||||
|
if (!isRetrievedOtpCode(sanitized)) return
|
||||||
|
|
||||||
|
val snapshot = _state.value
|
||||||
|
if (attemptId != snapshot.otpConsentRequestId || !otpAttemptCoordinator.isCurrent(attemptId)) return
|
||||||
|
|
||||||
|
if (snapshot.otpRequestPending) {
|
||||||
|
otpAttemptCoordinator.bufferRetrievedCode(attemptId, sanitized)
|
||||||
|
return
|
||||||
|
}
|
||||||
|
if (snapshot.screen != DriverScreen.Otp) return
|
||||||
|
|
||||||
|
applyOtpCode(sanitized, retrievedFromSms = true)
|
||||||
|
}
|
||||||
|
|
||||||
|
private fun applyOtpCode(code: String, retrievedFromSms: Boolean) {
|
||||||
|
_state.update { it.copy(otpCode = code) }
|
||||||
|
|
||||||
|
val shouldSubmit = if (retrievedFromSms) {
|
||||||
|
otpAutoSubmitPolicy.shouldSubmitRetrieved(code, _state.value.loading)
|
||||||
|
} else {
|
||||||
|
otpAutoSubmitPolicy.shouldSubmit(code, _state.value.loading)
|
||||||
|
}
|
||||||
|
if (shouldSubmit) {
|
||||||
|
verifyOtp(code)
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|
||||||
@@ -463,17 +536,16 @@ class DriverViewModel(application: Application) : AndroidViewModel(application)
|
|||||||
}.onFailure { throwable ->
|
}.onFailure { throwable ->
|
||||||
if (generation != routesRequestGeneration) return@onFailure
|
if (generation != routesRequestGeneration) return@onFailure
|
||||||
reportHandledException("load_routes", throwable, mapOf("date" to date))
|
reportHandledException("load_routes", throwable, mapOf("date" to date))
|
||||||
|
if (ApiErrorMapper.map(throwable).kind == ApiErrorKind.Auth) {
|
||||||
|
handleExpiredSession()
|
||||||
|
} else {
|
||||||
_state.update {
|
_state.update {
|
||||||
val apiError = ApiErrorMapper.map(throwable)
|
|
||||||
it.withApiError(throwable).copy(
|
it.withApiError(throwable).copy(
|
||||||
screen = when {
|
screen = if (it.screen == DriverScreen.Initializing) DriverScreen.Routes else it.screen,
|
||||||
apiError.kind == ApiErrorKind.Auth -> DriverScreen.Phone
|
|
||||||
it.screen == DriverScreen.Initializing -> DriverScreen.Routes
|
|
||||||
else -> it.screen
|
|
||||||
},
|
|
||||||
)
|
)
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
}
|
||||||
|
|
||||||
if (generation == routesRequestGeneration) _state.update {
|
if (generation == routesRequestGeneration) _state.update {
|
||||||
it.copy(loading = false, refreshing = false)
|
it.copy(loading = false, refreshing = false)
|
||||||
@@ -548,7 +620,12 @@ class DriverViewModel(application: Application) : AndroidViewModel(application)
|
|||||||
}
|
}
|
||||||
}
|
}
|
||||||
.onFailure { throwable ->
|
.onFailure { throwable ->
|
||||||
if (generation == routeDetailRequestGeneration) _state.update { it.withApiError(throwable) }
|
if (generation != routeDetailRequestGeneration) return@onFailure
|
||||||
|
if (ApiErrorMapper.map(throwable).kind == ApiErrorKind.Auth) {
|
||||||
|
handleExpiredSession()
|
||||||
|
} else {
|
||||||
|
_state.update { it.withApiError(throwable) }
|
||||||
|
}
|
||||||
}
|
}
|
||||||
|
|
||||||
if (generation == routeDetailRequestGeneration) _state.update { it.copy(refreshing = false) }
|
if (generation == routeDetailRequestGeneration) _state.update { it.copy(refreshing = false) }
|
||||||
@@ -1455,6 +1532,7 @@ class DriverViewModel(application: Application) : AndroidViewModel(application)
|
|||||||
dispatchSheetUploadsJob?.cancel()
|
dispatchSheetUploadsJob?.cancel()
|
||||||
realtimeBannerJob?.cancel()
|
realtimeBannerJob?.cancel()
|
||||||
liveSyncClient.stop()
|
liveSyncClient.stop()
|
||||||
|
ActiveRouteTrackingService.stop(getApplication())
|
||||||
offlineOutboxManager.clearAll()
|
offlineOutboxManager.clearAll()
|
||||||
repository.logout(notifyServer = _state.value.isOnline)
|
repository.logout(notifyServer = _state.value.isOnline)
|
||||||
syncRepository.clearCache()
|
syncRepository.clearCache()
|
||||||
@@ -1665,6 +1743,7 @@ class DriverViewModel(application: Application) : AndroidViewModel(application)
|
|||||||
|
|
||||||
private suspend fun handleExpiredSession() {
|
private suspend fun handleExpiredSession() {
|
||||||
liveSyncClient.stop()
|
liveSyncClient.stop()
|
||||||
|
ActiveRouteTrackingService.stop(getApplication())
|
||||||
offlineOutboxManager.clearAll()
|
offlineOutboxManager.clearAll()
|
||||||
repository.logout(notifyServer = false)
|
repository.logout(notifyServer = false)
|
||||||
syncRepository.clearCache()
|
syncRepository.clearCache()
|
||||||
|
|||||||
@@ -0,0 +1,45 @@
|
|||||||
|
package pl.firmatpp.kierowca.ui
|
||||||
|
|
||||||
|
class OtpAttemptCoordinator {
|
||||||
|
private var currentAttemptId: Long = 0L
|
||||||
|
private var bufferedCode: String? = null
|
||||||
|
|
||||||
|
fun beginAttempt(): Long {
|
||||||
|
currentAttemptId += 1L
|
||||||
|
bufferedCode = null
|
||||||
|
return currentAttemptId
|
||||||
|
}
|
||||||
|
|
||||||
|
fun isCurrent(attemptId: Long): Boolean = attemptId == currentAttemptId
|
||||||
|
|
||||||
|
fun bufferRetrievedCode(attemptId: Long, code: String): Boolean {
|
||||||
|
if (!isCurrent(attemptId) || !isRetrievedOtpCode(code)) return false
|
||||||
|
bufferedCode = code
|
||||||
|
return true
|
||||||
|
}
|
||||||
|
|
||||||
|
fun takeBufferedCode(attemptId: Long): String? {
|
||||||
|
if (!isCurrent(attemptId)) return null
|
||||||
|
return bufferedCode.also { bufferedCode = null }
|
||||||
|
}
|
||||||
|
|
||||||
|
fun discard(attemptId: Long) {
|
||||||
|
if (isCurrent(attemptId)) bufferedCode = null
|
||||||
|
}
|
||||||
|
}
|
||||||
|
|
||||||
|
internal fun sanitizeOtpCode(code: String): String =
|
||||||
|
code.filter(Char::isDigit).take(OTP_MAX_LENGTH)
|
||||||
|
|
||||||
|
internal fun isRetrievedOtpCode(code: String): Boolean =
|
||||||
|
code.length in OTP_MIN_LENGTH..OTP_MAX_LENGTH && code.all(Char::isDigit)
|
||||||
|
|
||||||
|
internal fun otpResendDelaySeconds(availableAtEpochMillis: Long, nowEpochMillis: Long): Int {
|
||||||
|
val remainingMillis = (availableAtEpochMillis - nowEpochMillis).coerceAtLeast(0L)
|
||||||
|
return ((remainingMillis + 999L) / 1_000L).toInt()
|
||||||
|
}
|
||||||
|
|
||||||
|
internal const val OTP_MIN_LENGTH = 4
|
||||||
|
internal const val OTP_DEFAULT_LENGTH = 6
|
||||||
|
internal const val OTP_MAX_LENGTH = 10
|
||||||
|
internal const val OTP_RESEND_COOLDOWN_MILLIS = 30_000L
|
||||||
@@ -4,14 +4,20 @@ class OtpAutoSubmitPolicy {
|
|||||||
private var lastSubmittedCode: String? = null
|
private var lastSubmittedCode: String? = null
|
||||||
|
|
||||||
fun shouldSubmit(code: String, loading: Boolean): Boolean {
|
fun shouldSubmit(code: String, loading: Boolean): Boolean {
|
||||||
if (code.length < OTP_LENGTH) {
|
return shouldSubmitCompleteCode(code, loading, code.length == OTP_DEFAULT_LENGTH)
|
||||||
|
}
|
||||||
|
|
||||||
|
fun shouldSubmitRetrieved(code: String, loading: Boolean): Boolean {
|
||||||
|
return shouldSubmitCompleteCode(code, loading, isRetrievedOtpCode(code))
|
||||||
|
}
|
||||||
|
|
||||||
|
private fun shouldSubmitCompleteCode(code: String, loading: Boolean, complete: Boolean): Boolean {
|
||||||
|
if (!complete) {
|
||||||
lastSubmittedCode = null
|
lastSubmittedCode = null
|
||||||
return false
|
return false
|
||||||
}
|
}
|
||||||
|
|
||||||
if (loading || code.length != OTP_LENGTH || code == lastSubmittedCode) {
|
if (loading || code == lastSubmittedCode) return false
|
||||||
return false
|
|
||||||
}
|
|
||||||
|
|
||||||
lastSubmittedCode = code
|
lastSubmittedCode = code
|
||||||
return true
|
return true
|
||||||
@@ -20,8 +26,4 @@ class OtpAutoSubmitPolicy {
|
|||||||
fun reset() {
|
fun reset() {
|
||||||
lastSubmittedCode = null
|
lastSubmittedCode = null
|
||||||
}
|
}
|
||||||
|
|
||||||
private companion object {
|
|
||||||
const val OTP_LENGTH = 6
|
|
||||||
}
|
|
||||||
}
|
}
|
||||||
|
|||||||
@@ -3,12 +3,16 @@ package pl.firmatpp.kierowca.data
|
|||||||
import java.io.IOException
|
import java.io.IOException
|
||||||
import java.net.UnknownHostException
|
import java.net.UnknownHostException
|
||||||
import kotlin.coroutines.cancellation.CancellationException
|
import kotlin.coroutines.cancellation.CancellationException
|
||||||
|
import okhttp3.MediaType.Companion.toMediaType
|
||||||
|
import okhttp3.ResponseBody.Companion.toResponseBody
|
||||||
import org.junit.Assert.assertEquals
|
import org.junit.Assert.assertEquals
|
||||||
import org.junit.Assert.assertFalse
|
import org.junit.Assert.assertFalse
|
||||||
import org.junit.Assert.assertSame
|
import org.junit.Assert.assertSame
|
||||||
import org.junit.Assert.assertTrue
|
import org.junit.Assert.assertTrue
|
||||||
import org.junit.Assert.fail
|
import org.junit.Assert.fail
|
||||||
import org.junit.Test
|
import org.junit.Test
|
||||||
|
import retrofit2.HttpException
|
||||||
|
import retrofit2.Response
|
||||||
|
|
||||||
class ApiErrorMapperTest {
|
class ApiErrorMapperTest {
|
||||||
@Test
|
@Test
|
||||||
@@ -84,4 +88,30 @@ class ApiErrorMapperTest {
|
|||||||
assertEquals("Storage timeout", error.message)
|
assertEquals("Storage timeout", error.message)
|
||||||
assertEquals(body, error.responseBody)
|
assertEquals(body, error.responseBody)
|
||||||
}
|
}
|
||||||
|
|
||||||
|
@Test
|
||||||
|
fun mapsInvalidOtpReasonToActionableMessage() {
|
||||||
|
val error = ApiErrorMapper.mapHttpStatus(422, """{"ok":false,"reason":"INVALID_OTP"}""")
|
||||||
|
|
||||||
|
assertEquals("INVALID_OTP", error.code)
|
||||||
|
assertEquals("Kod jest nieprawidłowy lub wygasł. Użyj kodu z najnowszego SMS-a.", error.message)
|
||||||
|
}
|
||||||
|
|
||||||
|
@Test
|
||||||
|
fun translatesRateLimitResponses() {
|
||||||
|
val error = ApiErrorMapper.mapHttpStatus(429, """{"message":"Too Many Attempts."}""")
|
||||||
|
|
||||||
|
assertEquals(ApiErrorKind.RateLimited, error.kind)
|
||||||
|
assertEquals("Za dużo prób. Odczekaj chwilę i spróbuj ponownie.", error.message)
|
||||||
|
}
|
||||||
|
|
||||||
|
@Test
|
||||||
|
fun preservesApiProblemWhenSameExceptionIsMappedMoreThanOnce() {
|
||||||
|
val body = """{"ok":false,"reason":"INVALID_OTP"}"""
|
||||||
|
.toResponseBody("application/json".toMediaType())
|
||||||
|
val exception = HttpException(Response.error<Any>(422, body))
|
||||||
|
|
||||||
|
assertEquals("INVALID_OTP", ApiErrorMapper.map(exception).code)
|
||||||
|
assertEquals("INVALID_OTP", ApiErrorMapper.map(exception).code)
|
||||||
|
}
|
||||||
}
|
}
|
||||||
|
|||||||
@@ -3,6 +3,7 @@ package pl.firmatpp.kierowca.data.upload
|
|||||||
import org.junit.Assert.assertFalse
|
import org.junit.Assert.assertFalse
|
||||||
import org.junit.Assert.assertTrue
|
import org.junit.Assert.assertTrue
|
||||||
import org.junit.Test
|
import org.junit.Test
|
||||||
|
import pl.firmatpp.kierowca.data.ApiErrorKind
|
||||||
|
|
||||||
class RoutePointWorkerRulesTest {
|
class RoutePointWorkerRulesTest {
|
||||||
@Test
|
@Test
|
||||||
@@ -15,4 +16,14 @@ class RoutePointWorkerRulesTest {
|
|||||||
assertFalse(isInactiveRoutePointError("SERVER_UNAVAILABLE"))
|
assertFalse(isInactiveRoutePointError("SERVER_UNAVAILABLE"))
|
||||||
assertFalse(isInactiveRoutePointError(null))
|
assertFalse(isInactiveRoutePointError(null))
|
||||||
}
|
}
|
||||||
|
|
||||||
|
@Test
|
||||||
|
fun revokedSessionStopsRouteTracking() {
|
||||||
|
assertTrue(shouldStopRouteTrackingAfterPointError(ApiErrorKind.Auth, null))
|
||||||
|
}
|
||||||
|
|
||||||
|
@Test
|
||||||
|
fun temporaryFailureKeepsRouteTrackingActive() {
|
||||||
|
assertFalse(shouldStopRouteTrackingAfterPointError(ApiErrorKind.Server, "SERVER_UNAVAILABLE"))
|
||||||
|
}
|
||||||
}
|
}
|
||||||
|
|||||||
@@ -0,0 +1,48 @@
|
|||||||
|
package pl.firmatpp.kierowca.ui
|
||||||
|
|
||||||
|
import org.junit.Assert.assertEquals
|
||||||
|
import org.junit.Assert.assertFalse
|
||||||
|
import org.junit.Assert.assertNull
|
||||||
|
import org.junit.Assert.assertTrue
|
||||||
|
import org.junit.Test
|
||||||
|
|
||||||
|
class OtpAttemptCoordinatorTest {
|
||||||
|
@Test
|
||||||
|
fun buffersSmsReceivedBeforeOtpRequestCompletes() {
|
||||||
|
val coordinator = OtpAttemptCoordinator()
|
||||||
|
val attemptId = coordinator.beginAttempt()
|
||||||
|
|
||||||
|
assertTrue(coordinator.bufferRetrievedCode(attemptId, "123456"))
|
||||||
|
assertEquals("123456", coordinator.takeBufferedCode(attemptId))
|
||||||
|
assertNull(coordinator.takeBufferedCode(attemptId))
|
||||||
|
}
|
||||||
|
|
||||||
|
@Test
|
||||||
|
fun ignoresConsentResultFromPreviousRequest() {
|
||||||
|
val coordinator = OtpAttemptCoordinator()
|
||||||
|
val previousAttempt = coordinator.beginAttempt()
|
||||||
|
val currentAttempt = coordinator.beginAttempt()
|
||||||
|
|
||||||
|
assertFalse(coordinator.bufferRetrievedCode(previousAttempt, "111111"))
|
||||||
|
assertTrue(coordinator.bufferRetrievedCode(currentAttempt, "222222"))
|
||||||
|
assertEquals("222222", coordinator.takeBufferedCode(currentAttempt))
|
||||||
|
}
|
||||||
|
|
||||||
|
@Test
|
||||||
|
fun acceptsOnlyOtpLengthsSupportedByApi() {
|
||||||
|
val coordinator = OtpAttemptCoordinator()
|
||||||
|
val attemptId = coordinator.beginAttempt()
|
||||||
|
|
||||||
|
assertFalse(coordinator.bufferRetrievedCode(attemptId, "123"))
|
||||||
|
assertTrue(coordinator.bufferRetrievedCode(attemptId, "1234"))
|
||||||
|
assertTrue(coordinator.bufferRetrievedCode(attemptId, "1234567890"))
|
||||||
|
assertFalse(coordinator.bufferRetrievedCode(attemptId, "12345678901"))
|
||||||
|
}
|
||||||
|
|
||||||
|
@Test
|
||||||
|
fun roundsResendCooldownUpToFullSeconds() {
|
||||||
|
assertEquals(30, otpResendDelaySeconds(40_000L, 10_000L))
|
||||||
|
assertEquals(1, otpResendDelaySeconds(10_001L, 10_000L))
|
||||||
|
assertEquals(0, otpResendDelaySeconds(10_000L, 10_001L))
|
||||||
|
}
|
||||||
|
}
|
||||||
@@ -38,4 +38,13 @@ class OtpAutoSubmitPolicyTest {
|
|||||||
assertFalse(policy.shouldSubmit("123456", loading = true))
|
assertFalse(policy.shouldSubmit("123456", loading = true))
|
||||||
assertFalse(policy.shouldSubmit("1234567", loading = false))
|
assertFalse(policy.shouldSubmit("1234567", loading = false))
|
||||||
}
|
}
|
||||||
|
|
||||||
|
@Test
|
||||||
|
fun submitsCompleteRetrievedCodesAcceptedByApi() {
|
||||||
|
val policy = OtpAutoSubmitPolicy()
|
||||||
|
|
||||||
|
assertTrue(policy.shouldSubmitRetrieved("1234", loading = false))
|
||||||
|
assertTrue(policy.shouldSubmitRetrieved("1234567890", loading = false))
|
||||||
|
assertFalse(policy.shouldSubmitRetrieved("123", loading = false))
|
||||||
|
}
|
||||||
}
|
}
|
||||||
|
|||||||
+456
-9
@@ -7,11 +7,17 @@ VERSION_FILE="$ROOT_DIR/app/build.gradle.kts"
|
|||||||
REMOTE_NAME="${GIT_REMOTE:-origin}"
|
REMOTE_NAME="${GIT_REMOTE:-origin}"
|
||||||
NOTES_FILE=""
|
NOTES_FILE=""
|
||||||
SKIP_TESTS=0
|
SKIP_TESTS=0
|
||||||
|
GOOGLE_PLAY_MODE="ask"
|
||||||
REUSE_CURRENT=0
|
REUSE_CURRENT=0
|
||||||
VERSION_FILE_CHANGED=0
|
VERSION_FILE_CHANGED=0
|
||||||
VERSION_COMMITTED=0
|
VERSION_COMMITTED=0
|
||||||
VERSION_BACKUP=""
|
VERSION_BACKUP=""
|
||||||
RELEASE_SUCCEEDED=0
|
RELEASE_SUCCEEDED=0
|
||||||
|
GOOGLE_PLAY_ACCESS_TOKEN=""
|
||||||
|
GOOGLE_PLAY_EDIT_ID=""
|
||||||
|
GOOGLE_PLAY_TEMP_DIR=""
|
||||||
|
GOOGLE_PLAY_CURRENT_VERSION_STATE=""
|
||||||
|
GOOGLE_PLAY_SUMMARY="pominięto"
|
||||||
|
|
||||||
usage() {
|
usage() {
|
||||||
cat <<'EOF'
|
cat <<'EOF'
|
||||||
@@ -21,13 +27,17 @@ Buduje i publikuje kolejną wersję aplikacji kierowcy:
|
|||||||
- automatycznie zwiększa versionCode o 1 i patch versionName o 1,
|
- automatycznie zwiększa versionCode o 1 i patch versionName o 1,
|
||||||
- uruchamia testy jednostkowe,
|
- uruchamia testy jednostkowe,
|
||||||
- buduje podpisany APK dla Gitea Release,
|
- buduje podpisany APK dla Gitea Release,
|
||||||
- buduje podpisany AAB dla Google Play Console,
|
- buduje podpisany AAB i publikuje go w teście zamkniętym Google Play,
|
||||||
|
- wysyła zmiany Google Play do weryfikacji,
|
||||||
- zapisuje zmianę wersji w commicie, tworzy tag i wykonuje push,
|
- zapisuje zmianę wersji w commicie, tworzy tag i wykonuje push,
|
||||||
- tworzy lub aktualizuje Gitea Release i wysyła APK oraz SHA256SUMS.
|
- tworzy lub aktualizuje Gitea Release i wysyła APK oraz SHA256SUMS.
|
||||||
|
|
||||||
Opcje:
|
Opcje:
|
||||||
--notes-file PLIK Treść release notes z podanego pliku.
|
--notes-file PLIK Treść release notes z podanego pliku.
|
||||||
--skip-tests Pomiń testDebugUnitTest.
|
--skip-tests Pomiń testDebugUnitTest.
|
||||||
|
--skip-google-play Nie pytaj i pomiń wysyłanie AAB do Google Play.
|
||||||
|
--publish-google-play
|
||||||
|
Nie pytaj i wyślij AAB do Google Play.
|
||||||
--reuse-current Nie zwiększaj wersji. Wznów publikację wersji zapisanej
|
--reuse-current Nie zwiększaj wersji. Wznów publikację wersji zapisanej
|
||||||
obecnie w app/build.gradle.kts.
|
obecnie w app/build.gradle.kts.
|
||||||
-h, --help Pokaż pomoc.
|
-h, --help Pokaż pomoc.
|
||||||
@@ -37,9 +47,19 @@ Zmienne środowiskowe:
|
|||||||
GITEA_BASE_URL Nadpisuje adres Gitea, np. https://gitea.example.com.
|
GITEA_BASE_URL Nadpisuje adres Gitea, np. https://gitea.example.com.
|
||||||
GITEA_REPOSITORY Nadpisuje owner/repository.
|
GITEA_REPOSITORY Nadpisuje owner/repository.
|
||||||
GIT_REMOTE Remote Git, domyślnie origin.
|
GIT_REMOTE Remote Git, domyślnie origin.
|
||||||
|
GOOGLE_PLAY_SERVICE_ACCOUNT_CREDENTIALS
|
||||||
|
Opcjonalne nadpisanie automatycznie znalezionego klucza JSON.
|
||||||
|
Domyślnie skrypt znajduje konto serwisowe w scripts/*.json.
|
||||||
|
Alternatywnie: GOOGLE_APPLICATION_CREDENTIALS.
|
||||||
|
GOOGLE_PLAY_PACKAGE_NAME
|
||||||
|
Identyfikator aplikacji, domyślnie pl.firmatpp.kierowca.
|
||||||
|
GOOGLE_PLAY_TRACK Nazwa testu zamkniętego, domyślnie alpha.
|
||||||
|
GOOGLE_PLAY_RELEASE_LANGUAGE
|
||||||
|
Język informacji o wersji, domyślnie pl-PL.
|
||||||
|
|
||||||
Wersja początkowa jest zawsze odczytywana z app/build.gradle.kts.
|
Wersja początkowa jest zawsze odczytywana z app/build.gradle.kts.
|
||||||
Google Play Console otrzymuje plik AAB, ponieważ jest to właściwy format publikacyjny.
|
Google Play otrzymuje AAB przez Android Publisher API. Konto serwisowe musi mieć
|
||||||
|
uprawnienie „Publikowanie aplikacji na ścieżkach testowych”.
|
||||||
EOF
|
EOF
|
||||||
}
|
}
|
||||||
|
|
||||||
@@ -54,6 +74,14 @@ while (($# > 0)); do
|
|||||||
SKIP_TESTS=1
|
SKIP_TESTS=1
|
||||||
shift
|
shift
|
||||||
;;
|
;;
|
||||||
|
--skip-google-play)
|
||||||
|
GOOGLE_PLAY_MODE="skip"
|
||||||
|
shift
|
||||||
|
;;
|
||||||
|
--publish-google-play)
|
||||||
|
GOOGLE_PLAY_MODE="publish"
|
||||||
|
shift
|
||||||
|
;;
|
||||||
--reuse-current)
|
--reuse-current)
|
||||||
REUSE_CURRENT=1
|
REUSE_CURRENT=1
|
||||||
shift
|
shift
|
||||||
@@ -82,6 +110,10 @@ done
|
|||||||
[[ -x ./gradlew ]] || { echo "Brak wykonywalnego ./gradlew." >&2; exit 1; }
|
[[ -x ./gradlew ]] || { echo "Brak wykonywalnego ./gradlew." >&2; exit 1; }
|
||||||
[[ -f keystore.properties ]] || { echo "Brak keystore.properties wymaganego do podpisania release." >&2; exit 1; }
|
[[ -f keystore.properties ]] || { echo "Brak keystore.properties wymaganego do podpisania release." >&2; exit 1; }
|
||||||
[[ -f "$VERSION_FILE" ]] || { echo "Brak $VERSION_FILE." >&2; exit 1; }
|
[[ -f "$VERSION_FILE" ]] || { echo "Brak $VERSION_FILE." >&2; exit 1; }
|
||||||
|
if [[ -n "$NOTES_FILE" && ! -f "$NOTES_FILE" ]]; then
|
||||||
|
echo "Nie istnieje plik release notes: $NOTES_FILE" >&2
|
||||||
|
exit 1
|
||||||
|
fi
|
||||||
|
|
||||||
branch="$(git branch --show-current)"
|
branch="$(git branch --show-current)"
|
||||||
[[ -n "$branch" ]] || { echo "Release nie może być wykonany z detached HEAD." >&2; exit 1; }
|
[[ -n "$branch" ]] || { echo "Release nie może być wykonany z detached HEAD." >&2; exit 1; }
|
||||||
@@ -165,11 +197,24 @@ play_aab_name="TPP-Kierowca-${next_name}-${next_code}.aab"
|
|||||||
|
|
||||||
cleanup_on_exit() {
|
cleanup_on_exit() {
|
||||||
local exit_code=$?
|
local exit_code=$?
|
||||||
|
if ((RELEASE_SUCCEEDED == 0)) && [[ -n "$GOOGLE_PLAY_EDIT_ID" && -n "$GOOGLE_PLAY_ACCESS_TOKEN" ]]; then
|
||||||
|
package_encoded="$(urlencode "$google_play_package")"
|
||||||
|
edit_encoded="$(urlencode "$GOOGLE_PLAY_EDIT_ID")"
|
||||||
|
curl -sS -o /dev/null \
|
||||||
|
--connect-timeout 20 \
|
||||||
|
--max-time 60 \
|
||||||
|
-X DELETE \
|
||||||
|
-H "@$GOOGLE_PLAY_TEMP_DIR/auth-header.txt" \
|
||||||
|
"https://androidpublisher.googleapis.com/androidpublisher/v3/applications/$package_encoded/edits/$edit_encoded" || true
|
||||||
|
fi
|
||||||
if ((RELEASE_SUCCEEDED == 0 && VERSION_FILE_CHANGED == 1 && VERSION_COMMITTED == 0)) && [[ -n "$VERSION_BACKUP" && -f "$VERSION_BACKUP" ]]; then
|
if ((RELEASE_SUCCEEDED == 0 && VERSION_FILE_CHANGED == 1 && VERSION_COMMITTED == 0)) && [[ -n "$VERSION_BACKUP" && -f "$VERSION_BACKUP" ]]; then
|
||||||
cp "$VERSION_BACKUP" "$VERSION_FILE"
|
cp "$VERSION_BACKUP" "$VERSION_FILE"
|
||||||
echo "Przywrócono poprzednią wersję w app/build.gradle.kts." >&2
|
echo "Przywrócono poprzednią wersję w app/build.gradle.kts." >&2
|
||||||
fi
|
fi
|
||||||
[[ -n "$VERSION_BACKUP" && -f "$VERSION_BACKUP" ]] && rm -f "$VERSION_BACKUP"
|
[[ -n "$VERSION_BACKUP" && -f "$VERSION_BACKUP" ]] && rm -f "$VERSION_BACKUP"
|
||||||
|
if [[ -n "$GOOGLE_PLAY_TEMP_DIR" && -d "$GOOGLE_PLAY_TEMP_DIR" ]]; then
|
||||||
|
rm -rf -- "$GOOGLE_PLAY_TEMP_DIR"
|
||||||
|
fi
|
||||||
if ((RELEASE_SUCCEEDED == 0)); then
|
if ((RELEASE_SUCCEEDED == 0)); then
|
||||||
echo "Release nie został ukończony. Po usunięciu przyczyny użyj --reuse-current, jeśli commit wersji już powstał." >&2
|
echo "Release nie został ukończony. Po usunięciu przyczyny użyj --reuse-current, jeśli commit wersji już powstał." >&2
|
||||||
fi
|
fi
|
||||||
@@ -180,6 +225,391 @@ trap cleanup_on_exit EXIT
|
|||||||
trap 'exit 130' INT
|
trap 'exit 130' INT
|
||||||
trap 'exit 143' TERM
|
trap 'exit 143' TERM
|
||||||
|
|
||||||
|
urlencode() {
|
||||||
|
jq -rn --arg value "$1" '$value | @uri'
|
||||||
|
}
|
||||||
|
|
||||||
|
base64url() {
|
||||||
|
openssl base64 -A | tr '+/' '-_' | tr -d '='
|
||||||
|
}
|
||||||
|
|
||||||
|
show_google_play_error() {
|
||||||
|
local response_file="$1"
|
||||||
|
jq '.error | {code, message, status, details}' "$response_file" >&2 2>/dev/null || true
|
||||||
|
}
|
||||||
|
|
||||||
|
decide_google_play_upload() {
|
||||||
|
local answer
|
||||||
|
|
||||||
|
case "$GOOGLE_PLAY_MODE" in
|
||||||
|
publish)
|
||||||
|
return 0
|
||||||
|
;;
|
||||||
|
skip)
|
||||||
|
GOOGLE_PLAY_SUMMARY="pominięto (--skip-google-play)"
|
||||||
|
return 0
|
||||||
|
;;
|
||||||
|
ask)
|
||||||
|
if [[ ! -t 0 ]]; then
|
||||||
|
echo "Nie można zapytać o publikację Google Play bez interaktywnego terminala." >&2
|
||||||
|
echo "Użyj --publish-google-play albo --skip-google-play." >&2
|
||||||
|
return 1
|
||||||
|
fi
|
||||||
|
while true; do
|
||||||
|
printf "\nBuild %s (%s) zakończony poprawnie. Czy wysłać aktualizację do Google Play? [t/N] " "$next_name" "$next_code"
|
||||||
|
if ! IFS= read -r answer; then
|
||||||
|
echo "Nie udało się odczytać odpowiedzi." >&2
|
||||||
|
return 1
|
||||||
|
fi
|
||||||
|
case "${answer,,}" in
|
||||||
|
t|tak|y|yes)
|
||||||
|
GOOGLE_PLAY_MODE="publish"
|
||||||
|
break
|
||||||
|
;;
|
||||||
|
""|n|nie|no)
|
||||||
|
GOOGLE_PLAY_MODE="skip"
|
||||||
|
GOOGLE_PLAY_SUMMARY="pominięto przez użytkownika"
|
||||||
|
break
|
||||||
|
;;
|
||||||
|
*)
|
||||||
|
echo "Nie rozpoznano odpowiedzi. Wpisz tak albo nie." >&2
|
||||||
|
;;
|
||||||
|
esac
|
||||||
|
done
|
||||||
|
;;
|
||||||
|
*)
|
||||||
|
echo "Niepoprawny tryb publikacji Google Play: $GOOGLE_PLAY_MODE" >&2
|
||||||
|
return 1
|
||||||
|
;;
|
||||||
|
esac
|
||||||
|
}
|
||||||
|
|
||||||
|
prepare_google_play() {
|
||||||
|
local credentials_path client_email now expires header payload unsigned_jwt signature assertion previous_umask
|
||||||
|
local token_response token_status preflight_response preflight_status package_encoded track_encoded
|
||||||
|
local candidate
|
||||||
|
local -a credentials_candidates=()
|
||||||
|
|
||||||
|
google_play_package="${GOOGLE_PLAY_PACKAGE_NAME:-pl.firmatpp.kierowca}"
|
||||||
|
google_play_track="${GOOGLE_PLAY_TRACK:-alpha}"
|
||||||
|
google_play_release_language="${GOOGLE_PLAY_RELEASE_LANGUAGE:-pl-PL}"
|
||||||
|
credentials_path="${GOOGLE_PLAY_SERVICE_ACCOUNT_CREDENTIALS:-${GOOGLE_APPLICATION_CREDENTIALS:-}}"
|
||||||
|
|
||||||
|
if [[ -z "$credentials_path" ]]; then
|
||||||
|
while IFS= read -r -d '' candidate; do
|
||||||
|
if jq -e '
|
||||||
|
.type == "service_account"
|
||||||
|
and (.client_email | type == "string" and length > 0)
|
||||||
|
and (.private_key | type == "string" and length > 0)
|
||||||
|
' "$candidate" >/dev/null 2>&1; then
|
||||||
|
credentials_candidates+=("$candidate")
|
||||||
|
fi
|
||||||
|
done < <(find "$ROOT_DIR/scripts" -maxdepth 1 -type f -name '*.json' -print0)
|
||||||
|
|
||||||
|
case "${#credentials_candidates[@]}" in
|
||||||
|
0)
|
||||||
|
echo "Nie znaleziono klucza konta serwisowego w $ROOT_DIR/scripts/*.json." >&2
|
||||||
|
echo "Umieść dokładnie jeden poprawny plik JSON konta serwisowego w folderze scripts." >&2
|
||||||
|
return 1
|
||||||
|
;;
|
||||||
|
1)
|
||||||
|
credentials_path="${credentials_candidates[0]}"
|
||||||
|
;;
|
||||||
|
*)
|
||||||
|
echo "Znaleziono więcej niż jeden klucz konta serwisowego w folderze scripts:" >&2
|
||||||
|
for candidate in "${credentials_candidates[@]}"; do
|
||||||
|
printf ' - %s\n' "$(basename "$candidate")" >&2
|
||||||
|
done
|
||||||
|
echo "Pozostaw jeden plik albo ustaw GOOGLE_PLAY_SERVICE_ACCOUNT_CREDENTIALS." >&2
|
||||||
|
return 1
|
||||||
|
;;
|
||||||
|
esac
|
||||||
|
fi
|
||||||
|
|
||||||
|
command -v openssl >/dev/null 2>&1 || {
|
||||||
|
echo "Brak wymaganego polecenia: openssl" >&2
|
||||||
|
return 1
|
||||||
|
}
|
||||||
|
|
||||||
|
[[ "$google_play_package" =~ ^[A-Za-z0-9._]+$ ]] || {
|
||||||
|
echo "Niepoprawne GOOGLE_PLAY_PACKAGE_NAME: $google_play_package" >&2
|
||||||
|
return 1
|
||||||
|
}
|
||||||
|
[[ -n "$google_play_track" ]] || { echo "GOOGLE_PLAY_TRACK nie może być puste." >&2; return 1; }
|
||||||
|
[[ "$google_play_release_language" =~ ^[A-Za-z0-9-]+$ ]] || {
|
||||||
|
echo "Niepoprawne GOOGLE_PLAY_RELEASE_LANGUAGE: $google_play_release_language" >&2
|
||||||
|
return 1
|
||||||
|
}
|
||||||
|
[[ -n "$credentials_path" ]] || {
|
||||||
|
echo "Nie wskazano ani nie znaleziono klucza konta serwisowego Google Play." >&2
|
||||||
|
return 1
|
||||||
|
}
|
||||||
|
[[ -r "$credentials_path" ]] || {
|
||||||
|
echo "Nie można odczytać klucza konta serwisowego: $credentials_path" >&2
|
||||||
|
return 1
|
||||||
|
}
|
||||||
|
|
||||||
|
GOOGLE_PLAY_TEMP_DIR="$(mktemp -d "${TMPDIR:-/tmp}/tpp-google-play.XXXXXX")"
|
||||||
|
chmod 700 "$GOOGLE_PLAY_TEMP_DIR"
|
||||||
|
client_email="$(jq -er '.client_email | select(type == "string" and length > 0)' "$credentials_path")" || {
|
||||||
|
echo "Klucz Google Play nie zawiera client_email." >&2
|
||||||
|
return 1
|
||||||
|
}
|
||||||
|
previous_umask="$(umask)"
|
||||||
|
umask 077
|
||||||
|
if ! jq -er '.private_key | select(type == "string" and length > 0)' "$credentials_path" > "$GOOGLE_PLAY_TEMP_DIR/private-key.pem"; then
|
||||||
|
umask "$previous_umask"
|
||||||
|
echo "Klucz Google Play nie zawiera private_key." >&2
|
||||||
|
return 1
|
||||||
|
fi
|
||||||
|
umask "$previous_umask"
|
||||||
|
|
||||||
|
now="$(date +%s)"
|
||||||
|
expires=$((now + 3600))
|
||||||
|
header="$(printf '%s' '{"alg":"RS256","typ":"JWT"}' | base64url)"
|
||||||
|
payload="$(jq -cn \
|
||||||
|
--arg iss "$client_email" \
|
||||||
|
--arg scope 'https://www.googleapis.com/auth/androidpublisher' \
|
||||||
|
--arg aud 'https://oauth2.googleapis.com/token' \
|
||||||
|
--argjson iat "$now" \
|
||||||
|
--argjson exp "$expires" \
|
||||||
|
'{iss:$iss,scope:$scope,aud:$aud,iat:$iat,exp:$exp}' | base64url)"
|
||||||
|
unsigned_jwt="$header.$payload"
|
||||||
|
signature="$(printf '%s' "$unsigned_jwt" \
|
||||||
|
| openssl dgst -sha256 -sign "$GOOGLE_PLAY_TEMP_DIR/private-key.pem" \
|
||||||
|
| base64url)"
|
||||||
|
assertion="$unsigned_jwt.$signature"
|
||||||
|
printf '%s' "$assertion" > "$GOOGLE_PLAY_TEMP_DIR/assertion.txt"
|
||||||
|
chmod 600 "$GOOGLE_PLAY_TEMP_DIR/assertion.txt"
|
||||||
|
|
||||||
|
token_response="$GOOGLE_PLAY_TEMP_DIR/token.json"
|
||||||
|
token_status="$(curl -sS \
|
||||||
|
--connect-timeout 20 \
|
||||||
|
--max-time 60 \
|
||||||
|
-o "$token_response" \
|
||||||
|
-w '%{http_code}' \
|
||||||
|
-X POST \
|
||||||
|
-H 'Content-Type: application/x-www-form-urlencoded' \
|
||||||
|
--data-urlencode 'grant_type=urn:ietf:params:oauth:grant-type:jwt-bearer' \
|
||||||
|
--data-urlencode "assertion@$GOOGLE_PLAY_TEMP_DIR/assertion.txt" \
|
||||||
|
'https://oauth2.googleapis.com/token')"
|
||||||
|
[[ "$token_status" == "200" ]] || {
|
||||||
|
echo "Nie udało się pobrać tokenu Google Play (HTTP $token_status)." >&2
|
||||||
|
jq '{error, error_description}' "$token_response" >&2 2>/dev/null || true
|
||||||
|
return 1
|
||||||
|
}
|
||||||
|
GOOGLE_PLAY_ACCESS_TOKEN="$(jq -er '.access_token | select(type == "string" and length > 0)' "$token_response")" || {
|
||||||
|
echo "Google OAuth nie zwrócił access_token." >&2
|
||||||
|
return 1
|
||||||
|
}
|
||||||
|
printf 'Authorization: Bearer %s\n' "$GOOGLE_PLAY_ACCESS_TOKEN" > "$GOOGLE_PLAY_TEMP_DIR/auth-header.txt"
|
||||||
|
chmod 600 "$GOOGLE_PLAY_TEMP_DIR/auth-header.txt"
|
||||||
|
rm -f "$GOOGLE_PLAY_TEMP_DIR/private-key.pem" "$GOOGLE_PLAY_TEMP_DIR/assertion.txt" "$token_response"
|
||||||
|
|
||||||
|
package_encoded="$(urlencode "$google_play_package")"
|
||||||
|
track_encoded="$(urlencode "$google_play_track")"
|
||||||
|
preflight_response="$GOOGLE_PLAY_TEMP_DIR/preflight.json"
|
||||||
|
preflight_status="$(curl -sS \
|
||||||
|
--connect-timeout 20 \
|
||||||
|
--max-time 60 \
|
||||||
|
-o "$preflight_response" \
|
||||||
|
-w '%{http_code}' \
|
||||||
|
-H "@$GOOGLE_PLAY_TEMP_DIR/auth-header.txt" \
|
||||||
|
"https://androidpublisher.googleapis.com/androidpublisher/v3/applications/$package_encoded/tracks/$track_encoded/releases")"
|
||||||
|
[[ "$preflight_status" == "200" ]] || {
|
||||||
|
echo "Brak dostępu do ścieżki Google Play $google_play_track dla $google_play_package (HTTP $preflight_status)." >&2
|
||||||
|
show_google_play_error "$preflight_response"
|
||||||
|
return 1
|
||||||
|
}
|
||||||
|
|
||||||
|
GOOGLE_PLAY_CURRENT_VERSION_STATE="$(jq -r --argjson code "$next_code" '
|
||||||
|
.releases[]?
|
||||||
|
| select(any(.activeArtifacts[]?; .versionCode == $code))
|
||||||
|
| .releaseLifecycleState // empty
|
||||||
|
' "$preflight_response" | sed -n '1p')"
|
||||||
|
if [[ -n "$GOOGLE_PLAY_CURRENT_VERSION_STATE" && "$REUSE_CURRENT" == "0" ]]; then
|
||||||
|
echo "Google Play ma już versionCode $next_code na ścieżce $google_play_track ($GOOGLE_PLAY_CURRENT_VERSION_STATE)." >&2
|
||||||
|
echo "Zwiększ wersję lokalną albo świadomie użyj --reuse-current." >&2
|
||||||
|
return 1
|
||||||
|
fi
|
||||||
|
if [[ "$GOOGLE_PLAY_CURRENT_VERSION_STATE" == "RELEASE_LIFECYCLE_STATE_NOT_APPROVED" ]]; then
|
||||||
|
echo "Wersja $next_code została odrzucona w Google Play. Zwiększ versionCode i utwórz nowe wydanie." >&2
|
||||||
|
return 1
|
||||||
|
fi
|
||||||
|
|
||||||
|
echo "Google Play: używam klucza $(basename "$credentials_path")."
|
||||||
|
echo "Google Play: zweryfikowano dostęp do $google_play_package, ścieżka $google_play_track."
|
||||||
|
}
|
||||||
|
|
||||||
|
publish_to_google_play() {
|
||||||
|
local release_notes="$1"
|
||||||
|
local package_encoded track_encoded edit_encoded api_base
|
||||||
|
local response status bundle_version_code existing_sha256 local_sha256
|
||||||
|
local play_notes track_payload release_name
|
||||||
|
|
||||||
|
case "$GOOGLE_PLAY_CURRENT_VERSION_STATE" in
|
||||||
|
RELEASE_LIFECYCLE_STATE_IN_REVIEW|RELEASE_LIFECYCLE_STATE_APPROVED_NOT_PUBLISHED|RELEASE_LIFECYCLE_STATE_PUBLISHED)
|
||||||
|
GOOGLE_PLAY_SUMMARY="już istnieje: $GOOGLE_PLAY_CURRENT_VERSION_STATE"
|
||||||
|
echo "Google Play: wersja $next_name ($next_code) ma już stan $GOOGLE_PLAY_CURRENT_VERSION_STATE; pomijam ponowną publikację."
|
||||||
|
return 0
|
||||||
|
;;
|
||||||
|
esac
|
||||||
|
|
||||||
|
package_encoded="$(urlencode "$google_play_package")"
|
||||||
|
track_encoded="$(urlencode "$google_play_track")"
|
||||||
|
api_base="https://androidpublisher.googleapis.com/androidpublisher/v3/applications/$package_encoded"
|
||||||
|
|
||||||
|
response="$GOOGLE_PLAY_TEMP_DIR/edit-create.json"
|
||||||
|
status="$(curl -sS \
|
||||||
|
--connect-timeout 20 \
|
||||||
|
--max-time 60 \
|
||||||
|
-o "$response" \
|
||||||
|
-w '%{http_code}' \
|
||||||
|
-X POST \
|
||||||
|
-H "@$GOOGLE_PLAY_TEMP_DIR/auth-header.txt" \
|
||||||
|
-H 'Content-Type: application/json' \
|
||||||
|
-d '{}' \
|
||||||
|
"$api_base/edits")"
|
||||||
|
[[ "$status" == "200" ]] || {
|
||||||
|
echo "Nie udało się utworzyć edycji Google Play (HTTP $status)." >&2
|
||||||
|
show_google_play_error "$response"
|
||||||
|
return 1
|
||||||
|
}
|
||||||
|
GOOGLE_PLAY_EDIT_ID="$(jq -er '.id | select(type == "string" and length > 0)' "$response")" || {
|
||||||
|
echo "Google Play nie zwrócił identyfikatora edycji." >&2
|
||||||
|
return 1
|
||||||
|
}
|
||||||
|
edit_encoded="$(urlencode "$GOOGLE_PLAY_EDIT_ID")"
|
||||||
|
|
||||||
|
response="$GOOGLE_PLAY_TEMP_DIR/track-get.json"
|
||||||
|
status="$(curl -sS \
|
||||||
|
--connect-timeout 20 \
|
||||||
|
--max-time 60 \
|
||||||
|
-o "$response" \
|
||||||
|
-w '%{http_code}' \
|
||||||
|
-H "@$GOOGLE_PLAY_TEMP_DIR/auth-header.txt" \
|
||||||
|
"$api_base/edits/$edit_encoded/tracks/$track_encoded")"
|
||||||
|
[[ "$status" == "200" ]] || {
|
||||||
|
echo "Nie znaleziono ścieżki Google Play $google_play_track (HTTP $status)." >&2
|
||||||
|
show_google_play_error "$response"
|
||||||
|
return 1
|
||||||
|
}
|
||||||
|
|
||||||
|
response="$GOOGLE_PLAY_TEMP_DIR/bundles.json"
|
||||||
|
status="$(curl -sS \
|
||||||
|
--connect-timeout 20 \
|
||||||
|
--max-time 60 \
|
||||||
|
-o "$response" \
|
||||||
|
-w '%{http_code}' \
|
||||||
|
-H "@$GOOGLE_PLAY_TEMP_DIR/auth-header.txt" \
|
||||||
|
"$api_base/edits/$edit_encoded/bundles")"
|
||||||
|
[[ "$status" == "200" ]] || {
|
||||||
|
echo "Nie udało się sprawdzić pakietów Google Play (HTTP $status)." >&2
|
||||||
|
show_google_play_error "$response"
|
||||||
|
return 1
|
||||||
|
}
|
||||||
|
|
||||||
|
existing_sha256="$(jq -r --argjson code "$next_code" '.bundles[]? | select(.versionCode == $code) | .sha256 // empty' "$response" | sed -n '1p')"
|
||||||
|
local_sha256="$(sha256sum "$artifact_dir/google-play/$play_aab_name" | awk '{print $1}')"
|
||||||
|
if [[ -n "$existing_sha256" ]]; then
|
||||||
|
if [[ "${existing_sha256,,}" != "${local_sha256,,}" && "$REUSE_CURRENT" == "0" ]]; then
|
||||||
|
echo "Google Play ma już versionCode $next_code z inną sumą SHA-256." >&2
|
||||||
|
return 1
|
||||||
|
fi
|
||||||
|
if [[ "${existing_sha256,,}" != "${local_sha256,,}" ]]; then
|
||||||
|
echo "Uwaga: lokalny AAB ma inną sumę niż istniejący versionCode $next_code; --reuse-current zachowuje pakiet już wysłany do Google Play." >&2
|
||||||
|
fi
|
||||||
|
bundle_version_code="$next_code"
|
||||||
|
echo "Google Play ma już AAB $next_name ($next_code); pomijam ponowny upload."
|
||||||
|
else
|
||||||
|
response="$GOOGLE_PLAY_TEMP_DIR/bundle-upload.json"
|
||||||
|
status="$(curl -sS \
|
||||||
|
--connect-timeout 20 \
|
||||||
|
--max-time 600 \
|
||||||
|
-o "$response" \
|
||||||
|
-w '%{http_code}' \
|
||||||
|
-X POST \
|
||||||
|
-H "@$GOOGLE_PLAY_TEMP_DIR/auth-header.txt" \
|
||||||
|
-H 'Content-Type: application/octet-stream' \
|
||||||
|
--data-binary "@$artifact_dir/google-play/$play_aab_name" \
|
||||||
|
"https://androidpublisher.googleapis.com/upload/androidpublisher/v3/applications/$package_encoded/edits/$edit_encoded/bundles?uploadType=media")"
|
||||||
|
[[ "$status" == "200" ]] || {
|
||||||
|
echo "Nie udało się wysłać AAB do Google Play (HTTP $status)." >&2
|
||||||
|
show_google_play_error "$response"
|
||||||
|
return 1
|
||||||
|
}
|
||||||
|
bundle_version_code="$(jq -er '.versionCode' "$response")" || {
|
||||||
|
echo "Google Play nie zwrócił versionCode przesłanego AAB." >&2
|
||||||
|
return 1
|
||||||
|
}
|
||||||
|
[[ "$bundle_version_code" == "$next_code" ]] || {
|
||||||
|
echo "Google Play odczytał versionCode $bundle_version_code zamiast $next_code." >&2
|
||||||
|
return 1
|
||||||
|
}
|
||||||
|
fi
|
||||||
|
|
||||||
|
play_notes="$(jq -rn --arg notes "$release_notes" '$notes | if length > 500 then .[0:497] + "..." else . end')"
|
||||||
|
release_name="TPP Kierowca $next_name ($next_code)"
|
||||||
|
track_payload="$(jq -n \
|
||||||
|
--arg track "$google_play_track" \
|
||||||
|
--arg release_name "$release_name" \
|
||||||
|
--arg version_code "$bundle_version_code" \
|
||||||
|
--arg language "$google_play_release_language" \
|
||||||
|
--arg notes "$play_notes" \
|
||||||
|
'{track:$track,releases:[{name:$release_name,versionCodes:[$version_code],status:"completed",releaseNotes:[{language:$language,text:$notes}]}]}')"
|
||||||
|
|
||||||
|
response="$GOOGLE_PLAY_TEMP_DIR/track-update.json"
|
||||||
|
status="$(curl -sS \
|
||||||
|
--connect-timeout 20 \
|
||||||
|
--max-time 60 \
|
||||||
|
-o "$response" \
|
||||||
|
-w '%{http_code}' \
|
||||||
|
-X PUT \
|
||||||
|
-H "@$GOOGLE_PLAY_TEMP_DIR/auth-header.txt" \
|
||||||
|
-H 'Content-Type: application/json' \
|
||||||
|
-d "$track_payload" \
|
||||||
|
"$api_base/edits/$edit_encoded/tracks/$track_encoded")"
|
||||||
|
[[ "$status" == "200" ]] || {
|
||||||
|
echo "Nie udało się przypisać AAB do ścieżki $google_play_track (HTTP $status)." >&2
|
||||||
|
show_google_play_error "$response"
|
||||||
|
return 1
|
||||||
|
}
|
||||||
|
|
||||||
|
response="$GOOGLE_PLAY_TEMP_DIR/edit-validate.json"
|
||||||
|
status="$(curl -sS \
|
||||||
|
--connect-timeout 20 \
|
||||||
|
--max-time 120 \
|
||||||
|
-o "$response" \
|
||||||
|
-w '%{http_code}' \
|
||||||
|
-X POST \
|
||||||
|
-H "@$GOOGLE_PLAY_TEMP_DIR/auth-header.txt" \
|
||||||
|
"$api_base/edits/$edit_encoded:validate")"
|
||||||
|
[[ "$status" == "200" ]] || {
|
||||||
|
echo "Walidacja edycji Google Play nie powiodła się (HTTP $status)." >&2
|
||||||
|
show_google_play_error "$response"
|
||||||
|
return 1
|
||||||
|
}
|
||||||
|
|
||||||
|
response="$GOOGLE_PLAY_TEMP_DIR/edit-commit.json"
|
||||||
|
status="$(curl -sS \
|
||||||
|
--connect-timeout 20 \
|
||||||
|
--max-time 120 \
|
||||||
|
-o "$response" \
|
||||||
|
-w '%{http_code}' \
|
||||||
|
-X POST \
|
||||||
|
-H "@$GOOGLE_PLAY_TEMP_DIR/auth-header.txt" \
|
||||||
|
"$api_base/edits/$edit_encoded:commit?changesNotSentForReview=false&changesInReviewBehavior=ERROR_IF_IN_REVIEW")"
|
||||||
|
[[ "$status" == "200" ]] || {
|
||||||
|
echo "Nie udało się zatwierdzić i wysłać zmian Google Play do weryfikacji (HTTP $status)." >&2
|
||||||
|
show_google_play_error "$response"
|
||||||
|
return 1
|
||||||
|
}
|
||||||
|
|
||||||
|
GOOGLE_PLAY_EDIT_ID=""
|
||||||
|
GOOGLE_PLAY_SUMMARY="wysłano do weryfikacji"
|
||||||
|
echo "Google Play: $release_name wysłano na ścieżkę $google_play_track i przekazano do weryfikacji."
|
||||||
|
}
|
||||||
|
|
||||||
if ((REUSE_CURRENT == 0)); then
|
if ((REUSE_CURRENT == 0)); then
|
||||||
if git show-ref --verify --quiet "refs/tags/$tag_name" || git ls-remote --exit-code --tags "$REMOTE_NAME" "refs/tags/$tag_name" >/dev/null 2>&1; then
|
if git show-ref --verify --quiet "refs/tags/$tag_name" || git ls-remote --exit-code --tags "$REMOTE_NAME" "refs/tags/$tag_name" >/dev/null 2>&1; then
|
||||||
echo "Tag $tag_name już istnieje. Użyj --reuse-current albo zwiększ wersję ręcznie." >&2
|
echo "Tag $tag_name już istnieje. Użyj --reuse-current albo zwiększ wersję ręcznie." >&2
|
||||||
@@ -235,14 +665,18 @@ cp "$aab_source" "$artifact_dir/google-play/$play_aab_name"
|
|||||||
sha256sum "gitea/$gitea_apk_name" "google-play/$play_aab_name" > SHA256SUMS.txt
|
sha256sum "gitea/$gitea_apk_name" "google-play/$play_aab_name" > SHA256SUMS.txt
|
||||||
)
|
)
|
||||||
|
|
||||||
|
decide_google_play_upload
|
||||||
|
if [[ "$GOOGLE_PLAY_MODE" == "publish" ]]; then
|
||||||
|
prepare_google_play
|
||||||
|
fi
|
||||||
|
|
||||||
if ((REUSE_CURRENT == 0)); then
|
if ((REUSE_CURRENT == 0)); then
|
||||||
git add "$VERSION_FILE"
|
git add "$VERSION_FILE"
|
||||||
git diff --cached --check
|
git diff --cached --check
|
||||||
git commit -m "chore: release android driver $next_name ($next_code)"
|
git commit -m "Wydaj aplikację kierowcy $next_name ($next_code)"
|
||||||
VERSION_COMMITTED=1
|
VERSION_COMMITTED=1
|
||||||
fi
|
fi
|
||||||
|
|
||||||
git push "$REMOTE_NAME" "$branch"
|
|
||||||
if ! git show-ref --verify --quiet "refs/tags/$tag_name"; then
|
if ! git show-ref --verify --quiet "refs/tags/$tag_name"; then
|
||||||
git tag -a "$tag_name" -m "TPP Kierowca $next_name ($next_code)"
|
git tag -a "$tag_name" -m "TPP Kierowca $next_name ($next_code)"
|
||||||
fi
|
fi
|
||||||
@@ -252,10 +686,8 @@ head_commit="$(git rev-parse HEAD)"
|
|||||||
echo "Tag $tag_name nie wskazuje aktualnego commita ($head_commit)." >&2
|
echo "Tag $tag_name nie wskazuje aktualnego commita ($head_commit)." >&2
|
||||||
exit 1
|
exit 1
|
||||||
}
|
}
|
||||||
git push "$REMOTE_NAME" "$tag_name"
|
|
||||||
|
|
||||||
if [[ -n "$NOTES_FILE" ]]; then
|
if [[ -n "$NOTES_FILE" ]]; then
|
||||||
[[ -f "$NOTES_FILE" ]] || { echo "Nie istnieje plik release notes: $NOTES_FILE" >&2; exit 1; }
|
|
||||||
release_notes="$(<"$NOTES_FILE")"
|
release_notes="$(<"$NOTES_FILE")"
|
||||||
else
|
else
|
||||||
previous_tag="$(git tag --sort=-version:refname | grep -Fxv "$tag_name" | sed -n '1p' || true)"
|
previous_tag="$(git tag --sort=-version:refname | grep -Fxv "$tag_name" | sed -n '1p' || true)"
|
||||||
@@ -264,7 +696,7 @@ else
|
|||||||
else
|
else
|
||||||
changes="$(git log -20 --format='- %s (%h)')"
|
changes="$(git log -20 --format='- %s (%h)')"
|
||||||
fi
|
fi
|
||||||
release_notes="TPP Kierowca $next_name ($next_code)
|
release_notes="TPP Kierowca $next_name ($next_code)
|
||||||
|
|
||||||
Zmiany:
|
Zmiany:
|
||||||
${changes:-'- Wydanie techniczne.'}
|
${changes:-'- Wydanie techniczne.'}
|
||||||
@@ -273,6 +705,15 @@ Artefakt Google Play Console (AAB) znajduje się lokalnie w:
|
|||||||
release-artifacts/$tag_name/google-play/$play_aab_name"
|
release-artifacts/$tag_name/google-play/$play_aab_name"
|
||||||
fi
|
fi
|
||||||
|
|
||||||
|
if [[ "$GOOGLE_PLAY_MODE" == "publish" ]]; then
|
||||||
|
publish_to_google_play "$release_notes"
|
||||||
|
GOOGLE_PLAY_ACCESS_TOKEN=""
|
||||||
|
rm -f "$GOOGLE_PLAY_TEMP_DIR/auth-header.txt"
|
||||||
|
fi
|
||||||
|
|
||||||
|
git push "$REMOTE_NAME" "$branch"
|
||||||
|
git push "$REMOTE_NAME" "$tag_name"
|
||||||
|
|
||||||
release_response="$(mktemp)"
|
release_response="$(mktemp)"
|
||||||
release_status="$(curl -sS "${auth_args[@]}" -o "$release_response" -w '%{http_code}' "$api_base/releases/tags/$tag_name")"
|
release_status="$(curl -sS "${auth_args[@]}" -o "$release_response" -w '%{http_code}' "$api_base/releases/tags/$tag_name")"
|
||||||
if [[ "$release_status" == "200" ]]; then
|
if [[ "$release_status" == "200" ]]; then
|
||||||
@@ -348,8 +789,12 @@ published_apk_hash="$(sha256sum "$downloaded_apk" | awk '{print $1}')"
|
|||||||
|
|
||||||
release_url="$gitea_base_url/$gitea_repository/releases/tag/$tag_name"
|
release_url="$gitea_base_url/$gitea_repository/releases/tag/$tag_name"
|
||||||
RELEASE_SUCCEEDED=1
|
RELEASE_SUCCEEDED=1
|
||||||
trap - EXIT INT TERM
|
|
||||||
[[ -n "$VERSION_BACKUP" ]] && rm -f "$VERSION_BACKUP"
|
[[ -n "$VERSION_BACKUP" ]] && rm -f "$VERSION_BACKUP"
|
||||||
|
if [[ -n "$GOOGLE_PLAY_TEMP_DIR" && -d "$GOOGLE_PLAY_TEMP_DIR" ]]; then
|
||||||
|
rm -rf -- "$GOOGLE_PLAY_TEMP_DIR"
|
||||||
|
GOOGLE_PLAY_TEMP_DIR=""
|
||||||
|
fi
|
||||||
|
trap - EXIT INT TERM
|
||||||
|
|
||||||
cat <<EOF
|
cat <<EOF
|
||||||
|
|
||||||
@@ -358,6 +803,8 @@ Release zakończony:
|
|||||||
Tag: $tag_name
|
Tag: $tag_name
|
||||||
Gitea: $release_url
|
Gitea: $release_url
|
||||||
APK: $artifact_dir/gitea/$gitea_apk_name
|
APK: $artifact_dir/gitea/$gitea_apk_name
|
||||||
Google Play: $artifact_dir/google-play/$play_aab_name
|
AAB: $artifact_dir/google-play/$play_aab_name
|
||||||
|
Google Play: $GOOGLE_PLAY_SUMMARY
|
||||||
|
Play track: ${google_play_track:-pominięto}
|
||||||
Checksumy: $artifact_dir/SHA256SUMS.txt
|
Checksumy: $artifact_dir/SHA256SUMS.txt
|
||||||
EOF
|
EOF
|
||||||
|
|||||||
Reference in New Issue
Block a user