Opened 7 years ago

Closed 7 years ago

#6744 closed defect (wontfix)

cpl_aws.cpp xml headers

Reported by: Kurt Schwehr Owned by: warmerdam
Priority: normal Milestone:
Component: default Version: unspecified
Severity: normal Keywords: aws xml
Cc:

Description

In VSIS3HandleHelper::CanRestartOnError():

if( !STARTS_WITH(pszErrorMsg, "<?xml") )

From a discussion with a Google engineer:

Strictly speaking, there is well-formed XML this will not match, which either has characters preceding <?xml or has no <?xml at all.

It really ought to be matching the Content-Type header first before guessing the contents are XML based on content.

Change History (2)

comment:1 by warmerdam, 7 years ago

While this may be true in theory, does it happen in practice with the XML that is intended to be processed?

comment:2 by Even Rouault, 7 years ago

Resolution: wontfix
Status: newclosed

In an attempt at lowering the number of opened tickets, closing that one. We might revisit the current approach if/when Amazon does a breaking change.

Note: See TracTickets for help on using tickets.