Code

Opened 4 months ago

Closed 4 months ago

Last modified 4 months ago

#21618 closed Cleanup/optimization (fixed)

GeoDjango OpenLayers.js admin include does not work over HTTPS

Reported by: kz26 Owned by: nobody
Component: GIS Version: 1.6
Severity: Normal Keywords: geodjango openlayers gis
Cc: Triage Stage: Ready for checkin
Has patch: yes Needs documentation: no
Needs tests: no Patch needs improvement: no
Easy pickings: yes UI/UX: no

Description

When attempting to access a GeoDjango-enabled admin page over HTTPS, the GeometryField widgets do not load because the page attempts to load http://openlayers.org/api/2.11/OpenLayers.js over plain HTTP, which is blocked by the Firefox security policy.

Attachments (2)

21618-1.diff (1.6 KB) - added by claudep 4 months ago.
Documentation complement
21618-2.diff (1.7 KB) - added by timo 4 months ago.
minor edits

Download all attachments as: .zip

Change History (8)

comment:1 Changed 4 months ago by claudep

  • Needs documentation unset
  • Needs tests unset
  • Patch needs improvement unset
  • Triage Stage changed from Unreviewed to Accepted
  • Type changed from Bug to Cleanup/optimization

See https://github.com/openlayers/openlayers/issues/1025

The fix for this ticket is to explain in the documentation that you should subclass the widget and specify your own URL (either self or cdn-hosted). Same for OpenStreetMap.js.

Changed 4 months ago by claudep

Documentation complement

comment:3 Changed 4 months ago by claudep

  • Has patch set
  • Patch needs improvement set

The above patch needs at least language review.

Changed 4 months ago by timo

minor edits

comment:4 Changed 4 months ago by timo

  • Patch needs improvement unset
  • Triage Stage changed from Accepted to Ready for checkin

comment:5 Changed 4 months ago by Claude Paroz <claude@…>

  • Resolution set to fixed
  • Status changed from new to closed

In b7a7baa55471e67b98d509cddf2f0994f7ce786f:

Fixed #21618 -- Added hints about openlayers.js hosting

Thanks kz26 for the report and Tim Graham for the excellent
review.

comment:6 Changed 4 months ago by Claude Paroz <claude@…>

In 2d08390d70007236e083183d78a4435d249c5aa8:

[1.6.x] Fixed #21618 -- Added hints about openlayers.js hosting

Thanks kz26 for the report and Tim Graham for the excellent
review.
Backport of b7a7baa554 from master.

Add Comment

Modify Ticket

Change Properties
<Author field>
Action
as closed
as The resolution will be set. Next status will be 'closed'
The resolution will be deleted. Next status will be 'new'
Author


E-mail address and user name can be saved in the Preferences.

 
Note: See TracTickets for help on using tickets.