Ticket #4546 (closed defect: fixed)

Opened 15 months ago

Last modified 15 months ago

Wrong header in GeoJSON driver

Reported by: sergeyastakhov Owned by: warmerdam
Priority: normal Milestone: 1.10.0
Component: default Version: svn-trunk
Severity: normal Keywords:
Cc: mloskot

Description

http://trac.osgeo.org/gdal/browser/trunk/gdal/ogr/ogrsf_frmts/geojson/ogrgeojsondatasource.cpp#L466

char* papsOptions[] = { (char*) "HEADERS=Accept: text/plain Accept: application/json", NULL };

This is a wrong format for "Accept" header. Right header should be like this:

char* papsOptions[] = { (char*) "HEADERS=Accept: text/plain, application/json", NULL };

Change History

Changed 15 months ago by rouault

  • cc mloskot added

Suggested change seems to be reasonable. Mateusz ?

Changed 15 months ago by mloskot

Yes, it does seem correct to me.

Changed 15 months ago by rouault

  • status changed from new to closed
  • resolution set to fixed
  • milestone set to 2.0.0

r24110 /trunk/gdal/ogr/ogrsf_frmts/geojson/ogrgeojsondatasource.cpp: GeoJSON: fix HEADERS field send to server (#4546)

Note: See TracTickets for help on using tickets.