Skip to content

Malformed data due to data.classe() return undefined #57 and fix Content-Type issue with utf-8#60

Merged
EddyVerbruggen merged 14 commits into
nativescript-community:masterfrom
kefahB:master
Feb 13, 2020
Merged

Malformed data due to data.classe() return undefined #57 and fix Content-Type issue with utf-8#60
EddyVerbruggen merged 14 commits into
nativescript-community:masterfrom
kefahB:master

Conversation

@kefahB

@kefahB kefahB commented Feb 6, 2020

Copy link
Copy Markdown
Contributor

Hi @EddyVerbruggen

I have fixed those issues that I facing on IOS only :

@kefahB

kefahB commented Feb 8, 2020

Copy link
Copy Markdown
Contributor Author

@EddyVerbruggen
During implementation with android I have facing the error javax.net.ssl.SSLPeerUnverifiedException .. my certificate is Domain Control Validated, after a long debugging, it seems the problem come from the Wildcard support .. I have edited https.android.ts and I have testing with my certificate and all test was done.

In order to dealing with a sub-domaine we must set commonName to *.domaine.com other way it will return SSLPeerUnverifiedException

Https.request error javax.net.ssl.SSLPeerUnverifiedException: Hostname demo.lunch-digi-pay.lu not verified:
certificate: sha256/i5fW5TLEfJtKsLBatgI2QctF2EPmxuVrGtVGW418CDM=
JS:     DN: CN=*.lunch-digi-pay.lu,OU=Domain Control Validated
JS:     subjectAltNames: [*.lunch-digi-pay.lu, lunch-digi-pay.lu]

Comment thread src/https.android.ts Outdated
Comment thread src/package.json
"name": "Robert Laverty",
"email": "roblav96@gmail.com",
"url": "https://github.com/roblav96"
},

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

👍

@EddyVerbruggen

Copy link
Copy Markdown
Member

Thanks! I'll merge this asap.

@EddyVerbruggen
EddyVerbruggen merged commit ea463e3 into nativescript-community:master Feb 13, 2020
@EddyVerbruggen EddyVerbruggen added this to the 1.3.0 milestone Feb 13, 2020
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.

2 participants