Opened 7 years ago

Closed 6 years ago

#26098 closed New feature (fixed)

Support Geodjango admin widgets on SSL

Reported by: F. Malina Owned by: nobody
Component: GIS Version: dev
Severity: Normal Keywords:
Cc: Triage Stage: Accepted
Has patch: yes Needs documentation: no
Needs tests: no Patch needs improvement: yes
Easy pickings: no UI/UX: no

Description

Not supporting SSL is documented here:
https://docs.djangoproject.com/en/1.9/ref/contrib/gis/forms-api/#django.contrib.gis.widgets.OpenLayersWidget

Linked openlayers.js file is not on https or a CDN of any kind. It's just hotlinked from openlayers server, which they discourage. https://github.com/openlayers/ol2/issues/1025

There is secure openlayers CDN available on cdnjs, but unsure if tiles would be available on https although I am aware there was some progress in this area lately.

Change History (6)

comment:1 Changed 7 years ago by Tim Graham

Triage Stage: UnreviewedSomeday/Maybe

Maybe asking on the geodjango mailing list would help form a concrete proposal.

comment:2 Changed 7 years ago by Tim Graham

#25436 is a duplicate that suggested to use protocol-relative schemes.

comment:3 Changed 7 years ago by Daniel Hahler

suggested to use protocol-relative schemes

It would be better to use https always by now.

comment:4 Changed 6 years ago by Daniel Hahler

Has patch: set

comment:5 Changed 6 years ago by Tim Graham

Patch needs improvement: set
Triage Stage: Someday/MaybeAccepted

comment:6 Changed 6 years ago by Tim Graham <timograham@…>

Resolution: fixed
Status: newclosed

In b1d6b0a7:

Fixed #26098 -- Used cdnjs.cloudflare for OpenLayers.js.

Note: See TracTickets for help on using tickets.
Back to Top