teintinu / dyuproject

Automatically exported from code.google.com/p/dyuproject
Apache License 2.0
1 stars 0 forks source link

HTTPS on local server broken with SDK 1.4.3 #30

Open GoogleCodeExporter opened 9 years ago

GoogleCodeExporter commented 9 years ago
Hi David,

It seems there's an issue/regression introduced with the GAE SDK 1.4.3:

- http://code.google.com/p/googleappengine/issues/detail?id=4823

- 
http://groups.google.com/group/google-appengine-java/browse_thread/thread/e55a0a
7ac294c418/d5a3bf57b69a7e18?show_docid=d5a3bf57b69a7e18

In a comment to the first entry, someone reports that it has to do with a 
missing call to 'doNotValidateCertificate()'.

Note that someone else reports it has to do with the header 'content-length'....

Just in case, here is the how I updated the code to use the Google Maps API:

String serviceAddress = 
"https://maps-api-ssl.google.com/maps/api/geocode/json?v=3&sensor=false&address=
" + address;
// return new URL(serviceAddress).openStream();
URL url = new URL(serviceAddress);
HTTPRequest request = new HTTPRequest(url);
request.getFetchOptions().doNotValidateCertificate();
HTTPResponse response = 
URLFetchServiceFactory.getURLFetchService().fetch(request);
return new ByteArrayInputStream(response.getContent());

A+, Dom

Original issue reported on code.google.com by dominiqu...@gmail.com on 4 Apr 2011 at 7:02

GoogleCodeExporter commented 9 years ago
Thanks Dom.
I'm not sure if I can work on the pending issues in the very near future. 
(Probably in 2 months)
If you'd like to have a crack at it, you can have commit access.
Ideally, I think it would be better if the openid/oauth projects would be 
re-built as a community project.  (See the latest post in the discussion group)

Original comment by david.yu...@gmail.com on 4 Apr 2011 at 7:59

GoogleCodeExporter commented 9 years ago
Hi David,

The defect has been accepted Google side: 
http://code.google.com/p/googleappengine/issues/detail?id=4823.

I appreciate your proposal for me to fix it. I'm going to fix it locally first. 
As my spare time is rare, I'll do my best on this.

A+, Dom

Original comment by dominiqu...@gmail.com on 6 Apr 2011 at 8:45

GoogleCodeExporter commented 9 years ago
Hi David,

Even if I tried to update your code, as the Google team recognized the issue, I 
preferred waiting for their fix. It's announced in the SDK 1.5.0 pre-release:

- Fixed an issue where HTTP POST was broken in the SDK if the Content-Length 
header was specified. 
http://code.google.com/p/googleappengine/issues/detail?id=4836 

A+, Dom

Original comment by dominiqu...@gmail.com on 9 May 2011 at 1:07