View Issue Details

IDProjectCategoryView StatusLast Update
0007451Talermerchant-pos-terminal (Android App)public2023-01-26 21:53
ReporterChristian Grothoff Assigned Togrote 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
PrioritynormalSeverityblockReproducibilityalways
Status closedResolutionfixed 
Platformi7OSDebian GNU/LinuxOS Versionsqueeze
Product Versiongit (master) 
Target Version0.9.1Fixed in Version0.9.1 
Summary0007451: PoS app does not work 'out of the box'
DescriptionThe app has a hard-coded link to a configuration resource which right now 404s. Not sure why it's gone (or where it went, for that matter), but we should modify something (web page or the App) so that when the user uses the 'default' configuration link, it works with some kind of simplistic shop content (as it used to, years ago...).
TagsNo tags attached.

Required notifications

Address or linkRemarkAdded byNotifiedActions

Activities

MS

2022-11-14 12:52

reporter   ~0019418

What are the configuration URL and format?

grote

2022-11-14 13:19

reporter   ~0019419

modify the demo/sandbox setup to
ensure that the backend instance in the Android Point-of-Sale app
exists? The file is
       https://docs.taler.net/_static/sample-pos-config.json
and it requires an instance 'pos' with api_key 'secret-token:sandbox'.

MS

2022-11-15 14:18

reporter   ~0019422

'pos' instance added on demo.

Christian Grothoff

2022-11-15 15:19

manager   ~0019423

Last edited: 2022-11-15 15:20

https://docs.taler.net/_static/sample-pos-config.json is correct now, as far as I can tell. I checked with the Kotlin code, and all fields seem to be fine. jq also thinks it is well-formed JSON, and the server returns the correct Mime-type. However, the POS app tells me "Error: The configuration JSON is malformed". :-(

Torsten, any ideas? (Might be good to show the position of the syntax error if the JSON failed to parse, and/or to give details about the broken/malformed field/path if the issue is deeper down in the configuration...). Syntax issues might need to be debugged by people other than us in the future, so good error messages here would seem important ;-).

grote

2022-11-16 19:12

reporter   ~0019431

When using this now with current POS app, it actually tells me App too old, please upgrade!

I am getting version=3:0:1 while the POS app is at 1:0:0

grote

2022-11-16 19:14

reporter   ~0019432

When simply bumping the version, I do get that malformed error when trying to pay something. The hint is about the `refund_delay`, so I guess more than just authentication has changed in the last 2 versions. Is there an overview over all changes somewhere? Also strange, that there's no authentication error. I thought this had changed?

Christian Grothoff

2022-11-16 19:53

manager   ~0019433

Last edited: 2022-11-16 19:54

There was a change in the encoding of time (absolute and relative). That is most likely the problem. See:

https://docs.taler.net/core/api-common.html#timestamps

As for the authentication change, if you are already using "Bearer", that's all, the secret-token: change is part of the configuration, which I already fixed in the docs.git.

grote

2023-01-03 13:48

reporter   ~0019555

Resolved with https://git.taler.net/taler-android.git/commit/?id=02ad9bde3e671a3566d68a6b968efe288e67d910 and https://git.taler.net/taler-android.git/commit/?id=3a12b1cc53f832e943b4825e132db6fe8ee083d6

Also made a new release of the PoS app.

Issue History

Date ModifiedUsernameFieldChange
2022-11-13 17:07Christian GrothoffNew Issue
2022-11-13 17:07Christian GrothoffStatusnew => assigned
2022-11-13 17:07Christian GrothoffAssigned To => grote
2022-11-14 05:58Christian GrothoffAssigned Togrote => MS
2022-11-14 12:52MSNote Added: 0019418
2022-11-14 13:19groteNote Added: 0019419
2022-11-15 14:18MSNote Added: 0019422
2022-11-15 14:18MSAssigned ToMS => Christian Grothoff
2022-11-15 15:19Christian GrothoffNote Added: 0019423
2022-11-15 15:20Christian GrothoffNote Edited: 0019423
2022-11-15 15:20Christian GrothoffAssigned ToChristian Grothoff => grote
2022-11-16 19:12groteNote Added: 0019431
2022-11-16 19:14groteNote Added: 0019432
2022-11-16 19:15groteAssigned Togrote => Christian Grothoff
2022-11-16 19:15groteStatusassigned => feedback
2022-11-16 19:52Christian GrothoffAssigned ToChristian Grothoff => grote
2022-11-16 19:53Christian GrothoffNote Added: 0019433
2022-11-16 19:53Christian GrothoffStatusfeedback => assigned
2022-11-16 19:54Christian GrothoffNote Edited: 0019433
2022-12-20 17:26Christian GrothoffTarget Version0.9.1 => 0.9.2
2022-12-20 18:28Christian GrothoffTarget Version0.9.2 => 0.9.4
2023-01-03 13:48groteStatusassigned => informing
2023-01-03 13:48groteResolutionopen => fixed
2023-01-03 13:48groteNote Added: 0019555
2023-01-06 14:52Christian GrothoffFixed in Version => 0.9.1
2023-01-06 14:52Christian GrothoffTarget Version0.9.4 => 0.9.1
2023-01-26 21:53Christian GrothoffStatusinforming => closed