Maniphest T146157

Valid geojson is being rejected by sanitizer
Closed, ResolvedPublic
Actions

Description

This produces an error, but it works well at http://geojson.io (this is a line near Gabon)

<mapframe width=600 height=600>
{
  "type": "Feature",
  "id": "Q1000",
  "properties": {},
  "geometry": {
    "type": "GeometryCollection",
    "geometries": [
      {
        "type": "LineString",
        "coordinates": [          
          [
            -20.78146426141457,
            -20.24797277602244
          ],
          [
            20.80974825712424,
            30.094641604394508
          ]
        ]
      }
    ]
  }
}
</mapframe>

Details

	Subject	Repo	Branch	Lines +/-
	GeometryCollection is a subset of Geometry	mediawiki/extensions/Kartographer	wmf/1.28.0-wmf.18	+70 -2
	GeometryCollection is a subset of Geometry	mediawiki/extensions/Kartographer	wmf/1.28.0-wmf.20	+70 -2
	GeometryCollection is a subset of Geometry	mediawiki/extensions/Kartographer	wmf/1.28.0-wmf.19	+70 -2
	GeometryCollection is a subset of Geometry	mediawiki/extensions/Kartographer	master	+70 -2

Customize query in gerrit

Related Objects
Search...

		Status	Subtype	Assigned	Task
		Resolved		Yurik	T146159 Geoshape service should allow lines
		Resolved		MaxSem	T146157 Valid geojson is being rejected by sanitizer

Event Timeline

Yurik created this task.Sep 20 2016, 3:38 PM

Restricted Application added a project: Discovery-ARCHIVED. · View Herald TranscriptSep 20 2016, 3:38 PM

Restricted Application added a subscriber: Aklapper. · View Herald Transcript

Yurik added a parent task: T146159: Geoshape service should allow lines.Sep 20 2016, 4:07 PM

Yurik updated the task description. (Show Details)Sep 20 2016, 4:13 PM

Yurik updated the task description. (Show Details)

Added a test patch (not passing)

MaxSem claimed this task.Sep 20 2016, 5:58 PM

Change 311735 had a related patch set uploaded (by Yurik):
Added a GeometryCollection schema test

https://gerrit.wikimedia.org/r/311735

gerritbot added a project: Patch-For-Review.Sep 20 2016, 5:58 PM

Change 311735 merged by jenkins-bot:
GeometryCollection is a subset of Geometry

https://gerrit.wikimedia.org/r/311735

ReleaseTaggerBot added a project: MW-1.28-release (WMF-deploy-2016-10-04_(1.28.0-wmf.21)).Sep 21 2016, 2:00 AM

Change 311974 had a related patch set uploaded (by Yurik):
GeometryCollection is a subset of Geometry

https://gerrit.wikimedia.org/r/311974

Change 311975 had a related patch set uploaded (by Yurik):
GeometryCollection is a subset of Geometry

https://gerrit.wikimedia.org/r/311975

Change 311980 had a related patch set uploaded (by Yurik):
GeometryCollection is a subset of Geometry

https://gerrit.wikimedia.org/r/311980

Change 311975 merged by jenkins-bot:
GeometryCollection is a subset of Geometry

https://gerrit.wikimedia.org/r/311975

Change 311974 merged by jenkins-bot:
GeometryCollection is a subset of Geometry

https://gerrit.wikimedia.org/r/311974

Change 311980 merged by jenkins-bot:
GeometryCollection is a subset of Geometry

https://gerrit.wikimedia.org/r/311980

Yurik closed this task as Resolved.Sep 21 2016, 1:46 PM

ReleaseTaggerBot added projects: MW-1.28-release (WMF-deploy-2016-09-06_(1.28.0-wmf.18)), MW-1.28-release (WMF-deploy-2016-09-20_(1.28.0-wmf.20)), MW-1.28-release (WMF-deploy-2016-09-13_(1.28.0-wmf.19)).Sep 21 2016, 2:00 PM