Skip to content

(#568) New domain wth ssl https://kuroba.io:8443#599

Merged
Adamantcheese merged 3 commits into
multi-featurefrom
new-domain-wth-ssl
Feb 9, 2020
Merged

(#568) New domain wth ssl https://kuroba.io:8443#599
Adamantcheese merged 3 commits into
multi-featurefrom
new-domain-wth-ssl

Conversation

@K1rakishou

@K1rakishou K1rakishou commented Feb 8, 2020

Copy link
Copy Markdown

Closes #568

Change DEV_API_ENDPOINT to https://kuroba.io:8443 but leave usesCleartextTraffic flag as is because Android won't allow to connect to it even though it redirects to https url. So we will have to wait couple of months before everyone has updated so we can safely remove that flag.

Release builds should be able to update as they had already done - via GitHub, should be no issues here.
Old dev clients should be able to update via the http flag.
New clients will be updated via new https url.

Also checked reporting/crash reporting on dev - works fine.
Release builds will be able to send reports/crash report with the next update. For now they can't.

…textTraffic flag as is because Android won't allow to connect to it even though it redirects to https url. So we will have to wait couple of months before everyone has updated so we can safely remove that flag.

Release builds should be able to update as they had already done - via GitHub, should be no issues here.
Old dev clients should be able to update via the http flag.
New client will be updated via new https url.

Also checked reporting/crash reporting on dev - works fine. Release build will be able to send reports/crash report with the next update. For now they can't.
@K1rakishou K1rakishou changed the title New domain wth ssl https://kuroba.io:8443 (#568) New domain wth ssl https://kuroba.io:8443 Feb 8, 2020
@K1rakishou

K1rakishou commented Feb 8, 2020

Copy link
Copy Markdown
Author

Nevermind, shit is fucked because neither Volley nor OkHttp does not work. Apparently Volley doesn't follow redirects and in case of OkHttp some I get some strange response code (405) which I have no idea where it comes from.

@K1rakishou

K1rakishou commented Feb 8, 2020

Copy link
Copy Markdown
Author

Welp, apparently people using dev build will have to update manually. I don't think we can do something else here. Release clients should be fine, though.

@K1rakishou

K1rakishou commented Feb 9, 2020

Copy link
Copy Markdown
Author

Nevermind x2, I managed to implement support for both http and https and the same time. Based fucking Vert.x it was a really great idea to use it for backend.

@Adamantcheese Adamantcheese merged commit 54656b9 into multi-feature Feb 9, 2020
@K1rakishou K1rakishou deleted the new-domain-wth-ssl branch February 9, 2020 07:27
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.

Automatic reporting of logs does not work on release builds

2 participants