View Issue Details

IDProjectCategoryView StatusLast Update
0006444Talermerchant backendpublic2024-01-12 13:04
ReporterChristian Grothoff Assigned ToChristian Grothoff Resolution deadline
ImplementerReviewer 
Release signed byDeployed by 
VerifierCommunicator 
Data protection authority notification requiredNo  
CSIRT notification requiredNo  
Estimated effort (hours)Actual effort (hours)0 
Fast-trackNo
Work packagenone
PrioritynormalSeveritymajorReproducibilityalways
Status closedResolutionnot fixable 
Platformi7OSDebian GNU/LinuxOS Versionsqueeze
Product Versiongit (master) 
Target Version0.8Fixed in Version0.8 
Summary0006444: merchant backend returns 100 continue before 404 on POST /private/orders
DescriptionInstead of failing immediately, the backend first 404s and then fails shortly thereafter with a 404.
Somehow we don't do the right thing (TM) and check early on for the 404 failure case.

Also odd: we still seem to fail before the actual upload succeeds, at least with my current WooCommerce test.

Steps To ReproducePost to backend where no instance is configured. Transcript from wireshark:

POST //private/orders HTTP/1.1
Host: backend.test.taler.net
Accept: */*
Authorization: ApiKey sandbox
Content-Type: application/json
Expect: 100-continue

HTTP/1.1 100 Continue

HTTP/1.1 404 Not Found
Server: nginx
Date: Sat, 25 Jul 2020 10:46:15 GMT
Content-Type: application/json
Content-Length: 57
Connection: keep-alive
Vary: Accept-Encoding
Access-Control-Allow-Origin: *

{
  "code": 2000,
  "hint": "merchant instance unknown"
}
TagsNo tags attached.

Required notifications

Address or linkRemarkAdded byNotifiedActions

Activities

Christian Grothoff

2020-07-25 18:19

manager   ~0016522

Turns out this is the nginx reverse proxy's 100 continue. It just doesn't wait for the backend to react :-(.

Christian Grothoff

2020-07-25 18:21

manager   ~0016523

And they wontfix'ed the bug 7 years ago: https://trac.nginx.org/nginx/ticket/493

Issue History

Date ModifiedUsernameFieldChange
2020-07-25 10:55Christian GrothoffNew Issue
2020-07-25 10:55Christian GrothoffStatusnew => assigned
2020-07-25 10:55Christian GrothoffAssigned To => Christian Grothoff
2020-07-25 18:19Christian GrothoffNote Added: 0016522
2020-07-25 18:21Christian GrothoffNote Added: 0016523
2020-07-25 18:23Christian GrothoffStatusassigned => informing
2020-07-25 18:23Christian GrothoffResolutionopen => not fixable
2020-07-25 18:23Christian GrothoffFixed in Version => 0.8
2021-08-24 14:23Christian GrothoffStatusinforming => closed
2024-01-12 13:04Christian GrothoffCategorymerchant backend API (C) => mechant backend
2025-03-22 13:09Christian GrothoffCategorymechant backend => merchant backend