Backend for auto-updating APK #83

rtsisyk · 2024-02-09T18:31:44Z

curl -vv -H 'x-om-appversion: 2022.08.01-1-Web' http://localhost:59830/releases

{
	"published_at": "2024-02-06T15:08:00Z",
	"code": 24020611,
	"news": {
		"en-US": "<TEXT>"
	},
	"apk": {
		"name": "OrganicMaps-24020611-web-release.apk",
		"size": 62334329,
		"url": "https://github.com/organicmaps/organicmaps/releases/download/2024.02.06-11-android/OrganicMaps-24020611-web-release.apk"
	}
}

```bash curl -vv -H 'x-om-appversion: 2022.08.01-1-Web' http://localhost:59830/releases ``` ```json { "published_at": "2024-02-06T15:08:00Z", "code": 24020611, "news": { "en-US": "<TEXT>" }, "apk": { "name": "OrganicMaps-24020611-web-release.apk", "size": 62334329, "url": "https://github.com/organicmaps/organicmaps/releases/download/2024.02.06-11-android/OrganicMaps-24020611-web-release.apk" } } ``` Signed-off-by: Roman Tsisyk <[email protected]>

biodranik

Введём новый app id или переименуем web? Переименование поломает текущих пользователей.

biodranik · 2024-02-09T22:21:03Z

src/releases.ts

+const GITHUB_RELEASES_URL: string = 'https://api.github.com/repos/organicmaps/organicmaps/releases';
+// https://docs.github.com/en/rest/authentication/authenticating-to-the-rest-api?apiVersion=2022-11-28#authenticating-with-a-personal-access-token
+const GITHUB_BEARER_TOKEN: string =
+  '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';


Может лучше его из секретов подтягивать? Мало ли опенсорснем потом? Тогда лучше перегенерить.

Я посмотрю как убрать в секреты. Вообще пока кажется, что работает даже без него.

Тогда убрать, если не нужно. Релизы же публичные, ключ тоже тогда публичный может быть или вообще ненужен.

biodranik · 2024-02-09T22:29:23Z

src/versions.ts

 // 2022.11.20 for iOS versions released before November 21 (without donate menu)
 // 2022.11.24-4-ios for newer iOS versions (with donate menu)
 // 2022.12.24-10-Google for Android
+// 2022.12.24-10-Google-beta for Android


Is Web-beta supported?

#83 (comment)

biodranik · 2024-02-09T22:31:24Z

test/releases.test.ts

+    '2022.08.23-5-Google-debug',
+    '2022.08.23-1-fdroid-beta',
+    '2022.08.23-1-fdroid-debug',
+    '2022.08.23-1-web-beta',


Этот надо поддерживать.

Или ввести новый флейвор, придумать хорошее название и сделать другой app id, чтобы ставился рядом с уже имеющимися fdroid и web и google и мог их заменить легко. Может даже импортнуть метки автоматом.

Текущий PR добавляет проверку релизов на GitHub. На GitHub выкладывается "web-release". Ничего другого туда не выкладывается.

Будет работать сейчас, после переименования?

rtsisyk · 2024-02-10T10:49:09Z

Введём новый app id или переименуем web? Переименование поломает текущих пользователей.

Это отдельная проблема, которая никак не затрагивается в этом PR.

biodranik · 2024-02-10T12:16:18Z

Это отдельная проблема, которая никак не затрагивается в этом PR.

Мы просто так тратим время, или решаем задачу, как дать людям замену версии в фдроиде? Удаление приложения (с удалением меток), чтобы установить его же версию из другого стора — не сработает, сам же понимаешь.

rtsisyk · 2024-02-11T07:56:25Z

Это отдельная проблема, которая никак не затрагивается в этом PR.

Мы просто так тратим время, или решаем задачу, как дать людям замену версии в фдроиде? Удаление приложения (с удалением меток), чтобы установить его же версию из другого стора — не сработает, сам же понимаешь.

Я пришлю PR в основную репу. Давай здесь закончим хотя бы с автообновлением web. Потом посмотрим стоит ли эту APK совать юзерам в F-Droid напрямую или надо еще что-нибудь там сделать. Может быть какую-нибудь магическую кнопку, которая сама всё поставит и мигрирует. Посмотри плз формат json в api - это будет трудно менять. Я добавил то, что технически нужно.

biodranik · 2024-02-11T16:26:02Z

src/releases.ts

+const GITHUB_RELEASES_URL: string = 'https://api.github.com/repos/organicmaps/organicmaps/releases';
+// https://docs.github.com/en/rest/authentication/authenticating-to-the-rest-api?apiVersion=2022-11-28#authenticating-with-a-personal-access-token
+const GITHUB_BEARER_TOKEN: string =
+  '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';


Тогда убрать, если не нужно. Релизы же публичные, ключ тоже тогда публичный может быть или вообще ненужен.

src/releases.ts

biodranik · 2024-02-11T17:57:25Z

src/releases.ts

+  type?: string;
+  apk: {
+    url: string;
+    name: string;


Что будет в имени?

Название файла. Пока не уверен что прям остро необходимо, может не понадобится.

filename тогда?

Что там ещё есть полезного?

src/releases.ts

biodranik · 2024-02-11T17:59:28Z

src/releases.ts

+  published_at: Date;
+  draft: boolean;
+  prerelease: boolean;
+  body: string;


Это описание воцнью на гитхабе? В каком формате? Ссылка на него есть?

Это текст из релиза на гитхабе в формате markdown текста.

Херово что маркдаун, у нас же нет рендерера на девайсе, верно?
Если простого решения нет, то придётся присылать ссылку на гитхаб страницу с этим текстом.

Рендерить маркдаун пока не будем, верно?

biodranik · 2024-02-11T18:02:02Z

src/releases.ts

+  const appVersion = parseAppVersion(request.headers.get('x-om-appversion'));
+  if (!appVersion) return new Response('Unknown app version', { status: 400 });
+
+  // The release version doesn't have `-release` suffix, thus type should be `undefined`.


Пояснишь, о каких версиях речь? У нас же много разных билдов и сборок, плюс не забывай, что все, кто нас форкает, будут тоже автоматом стучаться на мета сервер с их собственными версиями и названиями. Мы видели их уже в юзер агентах.

Релизные версии не имеют суффикса "-Release" в версии, поэтому type == undefined => "release".

А если в type мусор от всяких форков? Лучше явно проверять на наши типы.

biodranik · 2024-02-11T18:35:26Z

test/versions.test.ts

@@ -18,6 +18,22 @@ describe('parseDataVersion', () => {
  test('', () => expect(parseDataVersion(null)).toEqual(null));
 });

+describe('parseApkName', () => {
+  const tests: { [key: string]: object | null } = {


В моём PR было больше тестов на не поддерживаемые кейзы https://github.com/organicmaps/meta/pull/65/files#diff-4a1ee83ef3a98a79eeaeb2506070a86479a12bad67c28ba34045f71159794373R35

Разве там был парсинг имени APK?

Там тесты на кривые x-om-appversion.

biodranik · 2024-03-05T22:44:02Z

@rtsisyk давай проверим, что работает с новой веб апк и замержим?

rtsisyk force-pushed the github-releases-updater branch from f21156b to 1f49ae1 Compare February 9, 2024 18:33

biodranik reviewed Feb 9, 2024

View reviewed changes

biodranik reviewed Feb 11, 2024

View reviewed changes

biodranik force-pushed the master branch from 8a421ff to e41c085 Compare December 13, 2024 13:58

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Backend for auto-updating APK #83

Backend for auto-updating APK #83

rtsisyk commented Feb 9, 2024 •

edited

Loading

biodranik left a comment

biodranik Feb 9, 2024

rtsisyk Feb 10, 2024

biodranik Feb 11, 2024

biodranik Feb 9, 2024

rtsisyk Feb 10, 2024

biodranik Feb 9, 2024

rtsisyk Feb 10, 2024

biodranik Mar 5, 2024

rtsisyk commented Feb 10, 2024

biodranik commented Feb 10, 2024

rtsisyk commented Feb 11, 2024

biodranik Feb 11, 2024

biodranik Feb 11, 2024

rtsisyk Feb 11, 2024

biodranik Feb 11, 2024

biodranik Feb 11, 2024

rtsisyk Feb 11, 2024

biodranik Feb 11, 2024

biodranik Mar 5, 2024

biodranik Feb 11, 2024

rtsisyk Feb 11, 2024

biodranik Feb 11, 2024

biodranik Feb 11, 2024

rtsisyk Feb 11, 2024

biodranik Feb 11, 2024

biodranik commented Mar 5, 2024

Backend for auto-updating APK #83

Are you sure you want to change the base?

Backend for auto-updating APK #83

Conversation

rtsisyk commented Feb 9, 2024 • edited Loading

biodranik left a comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

rtsisyk commented Feb 10, 2024

biodranik commented Feb 10, 2024

rtsisyk commented Feb 11, 2024

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

biodranik commented Mar 5, 2024

rtsisyk commented Feb 9, 2024 •

edited

Loading