Project

Profile

Help

Feature #4748

Option to supply Accept header in SaxonJS.getResource()

Added by Martynas Jusevicius 12 months ago. Updated about 1 month ago.

Status:
Closed
Priority:
Normal
Category:
-
Sprint/Milestone:
-
Start date:
2020-09-23
Due date:
% Done:

100%

Estimated time:
Applies to JS Branch:
2
Fix Committed on JS Branch:
2
Fixed in JS Release:
SEF Generated with:
Company:
-
Contact person:
-
Additional contact persons:
-

Description

I am trying to load an application/rdf+xml resource using SaxonJS.getResource() yet I get text/html, because Saxon-JS sends an Accept: */* header.

I think there should be an option to supply the Accept header.

History

#1 Updated by Norm Tovey-Walsh 8 months ago

  • Status changed from New to Resolved

I believe this is now implemented.

#2 Updated by Community Admin 7 months ago

  • Applies to JS Branch 2 added
  • Applies to JS Branch deleted (2.0)

#3 Updated by Debbie Lockett 7 months ago

  • Fix Committed on JS Branch 2 added

#4 Updated by Debbie Lockett 7 months ago

  • Status changed from Resolved to Closed
  • % Done changed from 0 to 100
  • Fixed in JS Release set to Saxon-JS 2.1

Bug fix applied in the Saxon-JS 2.1 maintenance release.

#5 Updated by Debbie Lockett 3 months ago

  • Status changed from Closed to In Progress
  • Assignee set to Debbie Lockett

Reopening, because the fix looks to be incomplete. SaxonJS.getResource calls platform.resourcePromise; in NodeJSPlatform this method processes headers if supplied, but not in BrowserPlatform.

The browser test iss4748 is actually testing the use of headers in ixsl:schedule-action/@http-request; which is a separate mechanism.

#6 Updated by Debbie Lockett 3 months ago

New test iss4748b added. Committed code changes to implement headers option for SaxonJS.getResource, for browser as well as Node platforms.

Also need to update the documentation.

#7 Updated by Debbie Lockett about 1 month ago

  • Status changed from In Progress to Resolved

Documentation now updated to include the new headers option for SaxonJS.getResource.

We should really have more tests for this new feature, and for testing SaxonJS.getResource in general. But I'm going to mark this as resolved anyway.

#8 Updated by Debbie Lockett about 1 month ago

  • Fixed in JS Release deleted (Saxon-JS 2.1)

#9 Updated by Debbie Lockett about 1 month ago

  • Fixed in JS Release set to Saxon-JS 2.3

Bug fix applied in the Saxon-JS 2.3 maintenance release.

#10 Updated by Debbie Lockett about 1 month ago

  • Status changed from Resolved to Closed

Please register to edit this issue

Also available in: Atom PDF Tracking page