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(android): allow TiDownloadManager to use TiSocketFactory #10553

Merged
merged 5 commits into from Mar 7, 2019

Conversation

garymathews
Copy link
Contributor

@garymathews garymathews commented Dec 19, 2018

  • Allow TiDownloadManager to use TiSocketFactory for SSL compatibility on Android 4.4
  • Test on Android 4.4 (API 19)
NOTE: Can't say I'm particularly happy about using reflection here. Ideally, we should move the classes to prevent this issue.
TEST CASE
const win = Titanium.UI.createWindow({ backgroundColor: 'gray' });
const img = Ti.UI.createImageView({
	image: 'https://homepages.cae.wisc.edu/~ece533/images/mountain.png',
	height: Ti.UI.FILL,
	width: Ti.UI.FILL
});
win.add(img);
win.open();

JIRA Ticket

@build
Copy link
Contributor

build commented Dec 19, 2018

Messages
📖

💾 Here's the generated SDK zipfile.

📖

✅ All tests are passing
Nice one! All 2995 tests are passing.

Generated by 🚫 dangerJS against 132b396

Copy link
Contributor

@jquick-axway jquick-axway left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

CR: Pass

@sgtcoolguy sgtcoolguy modified the milestones: 8.0.0, 8.1.0 Jan 10, 2019
@keerthi1032
Copy link
Contributor

keerthi1032 commented Mar 7, 2019

FR passed. image loads as expected.
Test Enviornment:
Operating System
Name = Mac OS X
Version = 10.13.6
Architecture = 64bit
Memory = 17179869184
Node.js
Node.js Version = 8.12.0
npm Version = 6.4.1
Titanium CLI
CLI Version = 5.1.1
Titanium SDK
SDK Version = local sdk 8.1.0.v20190109140049
Device =Samsung s5 android 6,samung android 4.4.2
Emulator =NEXUS 4.4,pixel android 9.

@keerthi1032 keerthi1032 merged commit 33e34d5 into tidev:master Mar 7, 2019
@keerthi1032
Copy link
Contributor

PR Merged

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.

None yet

5 participants