Download & Extend
OpenLayers » Issues

JS file URL (openlayers_source) should respect RFC 1808 Section 4

Posted by andrewsuth on July 20, 2012 at 8:35am
Project:OpenLayers
Version:6.x-2.x-dev
Component:OL UI
Category:feature request
Priority:normal
Assigned:Unassigned
Status:closed (fixed)

Issue Summary

At present, it is not possible to enter the URL source for OpenLayers.js using // before the domain name (RFC 1808, Section 4) which allows the browser to determine which protocol to use to retreive the file (HTTPS / HTTP). eg. //www.example.com/js/OpenLayers.js

While openlayers.org itself does not support HTTPS (eg. https://openlayers.org/dev/OpenLayers.js), we use a file from our CDN which does support HTTPS.

Patch attached.

AttachmentSizeStatusTest resultOperations
openlayers_rfc_1808.patch745 bytesTest request sentNoneView details
Login or register to post comments

Comments

#1

Posted by andrewsuth on July 20, 2012 at 8:54am

Added user explanation of the use of the double slash.

AttachmentSizeStatusTest resultOperations
openlayers_rfc_1808_2.patch1.67 KBTest request sentNoneView details

#2

Posted by Pol on July 20, 2012 at 12:18pm
Status:needs review» needs work

Hello @andrewsuth,

Thanks for your patch!
I'm willing to commit it but I need a patch for OL 7.x then I'll commit this patch to OL 6.x.

Thanks

#3

Posted by Pol on August 19, 2012 at 5:48pm

Patch for 7.x-2.x

AttachmentSizeStatusTest resultOperations
0001-Issue-1694048-JS-file-URL-openlayers_source-should-r.patch2.36 KBTest request sentNoneView details

#4

Posted by Pol on August 19, 2012 at 5:47pm
Status:needs work» needs review

#5

Posted by Pol on August 19, 2012 at 7:04pm
Status:needs review» fixed

Committed on 7.x & 6.x.

Thanks andrewsuth !

#6

Posted by System Message on September 2, 2012 at 7:11pm
Status:fixed» closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.

nobody click here