Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix: JAR files signed with the MD5 algorithm as unsigned (godot 2) #18626

Merged
merged 2 commits into from
May 8, 2018
Merged

Fix: JAR files signed with the MD5 algorithm as unsigned (godot 2) #18626

merged 2 commits into from
May 8, 2018

Conversation

RameshRavone
Copy link
Contributor

Fix: JAR files signed with the MD5 algorithm as unsigned

@RameshRavone RameshRavone changed the title Andorid export Fix: JAR files signed with the MD5 algorithm as unsigned (godot 2) May 5, 2018
@mhilbrunner mhilbrunner added this to the 2.1 milestone May 5, 2018
@mhilbrunner
Copy link
Member

mhilbrunner commented May 5, 2018

See notes in #18627 (same PR for 3.1)

@mhilbrunner
Copy link
Member

We merged it for 3.x. Not sure what the Android version requirements of Godot 2.1 are? This would need Android API 18 (4.3+)

@volzhs
Copy link
Contributor

volzhs commented May 8, 2018

https://github.com/godotengine/godot/blob/2.1/platform/android/SCsub#L105
the default min sdk for godot 2.1 is Android API 14.

I'm using min sdk api 15 for custom template.
I tested this PR with api 15 and it works fine.
I guess api 14 would fine too.

@mhilbrunner
Copy link
Member

@volzhs Did you test on device? Some devices <= 18 (4.3) apparently reject APKs with a SHA1/SHA-256 signature.

@mhilbrunner
Copy link
Member

Also, Google Play seems to reject APKs signed this way for API level < 18:
screen

@volzhs
Copy link
Contributor

volzhs commented May 8, 2018

@mhilbrunner I did it on a device but it has API 26. I didn't think it could be a problem with under API 18 devices.
it seems to let it as is for 2.1 then.

@akien-mga
Copy link
Member

If MD5-signed APKs are also rejected anyway, shouldn't we update the API level to 18 and use SHA-256 on 2.1 too?

@mhilbrunner
Copy link
Member

mhilbrunner commented May 8, 2018

Probably best, yeah. According to https://developer.android.com/about/dashboards/, less than 5% of users are below API 18. So merge this and raise min. API level to 18?

@akien-mga
Copy link
Member

So merge this and raise min. API level to 18?

I would prefer this PR to do the API level change (or another one, but merged at the same time).

@mhilbrunner
Copy link
Member

targetSdkVersion ist still 23 in master I believe. However, seeing as this will no longer be supported for new apps on Google Play as of November 2018 raising this is fine.

We should increase it to 27 in master too after merging this.

@akien-mga akien-mga merged commit 326fd35 into godotengine:2.1 May 8, 2018
akien-mga added a commit that referenced this pull request May 8, 2018
Matches the change for 2.1 in #18626, and the new requirements from Google
for new apps starting with August 2018 (targetSdkVersion 26 or higher):
https://android-developers.googleblog.com/2017/12/improving-app-security-and-performance.html
akien-mga added a commit that referenced this pull request May 8, 2018
Matches the change for 2.1 in #18626, and the new requirements from Google
for new apps starting with August 2018 (targetSdkVersion 26 or higher):
https://android-developers.googleblog.com/2017/12/improving-app-security-and-performance.html

(cherry picked from commit a655281)
@akien-mga
Copy link
Member

Thanks!

We should increase it to 27 in master too after merging this.

Done with a655281, and in 3.0 with 1905e1c.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants