Opened 11 years ago

Closed 11 years ago

#4546 closed defect (fixed)

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: Mateusz Łoskot

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 (3)

comment:1 by Even Rouault, 11 years ago

Cc: Mateusz Łoskot added

Suggested change seems to be reasonable. Mateusz ?

comment:2 by Mateusz Łoskot, 11 years ago

Yes, it does seem correct to me.

comment:3 by Even Rouault, 11 years ago

Milestone: 2.0.0
Resolution: fixed
Status: newclosed

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.