View Issue Details

IDProjectCategoryView StatusLast Update
0007200Talerexchangepublic2022-11-04 19:52
ReporterttnAssigned 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
PrioritynormalSeverityminorReproducibilityhave not tried
Status closedResolutionfixed 
PlatformLinuxOSUbuntuOS Version20.04
Product Versiongit (master) 
Target Version0.9 
Summary0007200: configure script accepts old MHD
DescriptionAs seen in 0007151, an old MHD can be problematic.
The documented minimum version is currently 0.9.71.
However, the configure script does not error out if an old MHD is found.
It should.

Same can be said for merchant.

Question: What is the feature in 0.9.71 that is required by exchange (merchant)?
TagsNo tags attached.

Required notifications

Address or linkRemarkAdded byNotifiedActions

Relationships

related to 0007151 closedChristian Grothoff "make check" fails on test-merchant-walletharness.sh 

Activities

Christian Grothoff

2022-03-19 07:59

manager   ~0018811

Last edited: 2022-03-19 08:02

Oh, it's quite possible that it wasn't any particular feature that is required, but that there were bugfixes I deemed relevant enough to make 0.9.71 formally a requirement, just to avoid those bugs. Which is probably also why we didn't make it a hard error (yet).

ttn

2022-03-19 11:43

reporter   ~0018812

I think if the documented minimum version is X.Y.Z, we would be doing ourselves a favor by making it a hard requirement.
I've reassigned the issue to myself and will add some autoconfigury shortly.

ttn

2022-03-19 11:59

reporter   ~0018813

I've just pushed commits in both exchange and merchant to add a minimum version check (for libmicrohttpd).
As for the actual version required, i kept those at 0.9.39 (the previous value). Now that the mechanism is in place,
it's a small matter to bump the value whenever you deem best.

(At that time, we can close this issue.)

ttn

2022-03-23 06:35

reporter   ~0018833

I've just now bumped the version to 0.9.71:

- https://git.taler.net/exchange.git/commit/?id=7d2a009e1b7d68a
- https://git.taler.net/merchant.git/commit/?id=da177df9c8865c1

ttn

2022-03-23 06:37

reporter   ~0018834

I see that GNUnet configure script uses pkg-config for libmicrohttpd.
Something to consider for Taler in the future....

Issue History

Date ModifiedUsernameFieldChange
2022-03-19 05:24ttnNew Issue
2022-03-19 05:24ttnStatusnew => assigned
2022-03-19 05:24ttnAssigned To => Christian Grothoff
2022-03-19 05:24ttnRelationship addedrelated to 0007151
2022-03-19 07:59Christian GrothoffNote Added: 0018811
2022-03-19 08:02Christian GrothoffNote Edited: 0018811
2022-03-19 11:40ttnAssigned ToChristian Grothoff => ttn
2022-03-19 11:43ttnNote Added: 0018812
2022-03-19 11:59ttnNote Added: 0018813
2022-03-19 12:15ttnAssigned Tottn => Christian Grothoff
2022-03-23 06:35ttnNote Added: 0018833
2022-03-23 06:37ttnStatusassigned => informing
2022-03-23 06:37ttnResolutionopen => fixed
2022-03-23 06:37ttnNote Added: 0018834
2022-10-20 08:48Christian GrothoffTarget Version => 0.9
2022-11-04 19:52Christian GrothoffStatusinforming => closed