View Issue Details

IDProjectCategoryView StatusLast Update
0009479Talermerchant portal webuipublic2025-05-10 02:11
ReporterChristian Grothoff Assigned Tosebasjm 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
PrioritynormalSeverityblockReproducibilityN/A
Status closedResolutionfixed 
Platformi7OSDebian GNU/LinuxOS Versionsqueeze
Product Versiongit (master) 
Target Version1.0Fixed in Version1.0 
Summary0009479: cannot add Taler Revenue API for latest libeufin-bank (regional currency setup/Netzbon)
DescriptionThe merchant backend SPA correctly (!) supports giving a username/password for auth type "with password", but then tries to access the taler-revenue API directly using that password. However (unless in compatibility mode), libeufin *requires* token authentication for that endpoint.

Thus, what the SPA *must* do is use the /token endpoint to convert the username/password into a token and it should then internally also only persist the token (without expiration). Naturally, the type of token requested should just be a "revenue" token.

Now, the endpoint URL for the /token API isn't part of the revenue API, but part of the core bank API (https://docs.taler.net/core/api-corebank.html#authentication), so I guess we should *first* ask about the authentication type, and then ask for the core bank API endpoint *if* the user picks "with password" and ask for the revenue API endpoint if there is token.
TagsNo tags attached.

Required notifications

Address or linkRemarkAdded byNotifiedActions

Relationships

related to 0009523 implementing Allow connecting the merchant backend to the bank revenue API via token in URI 

Activities

sebasjm

2025-04-07 16:24

developer   ~0024430

it's not clear if we want to support this flow

> entering the login credentials of service A in service B is not a good pattern that we want to teach to users

Christian Grothoff

2025-04-28 05:34

manager   ~0024694

Well, then we should remove that option in the UI! I can live with that, too. But we shouldn't keep a mode where we store the full-access login credentials in the merchant backend...

sebasjm

2025-04-30 17:46

developer   ~0024719

ACK, hidding the auth with token option under dev-view in merchant ui since there is no way to get that from the bank ui... unless you have dev knowledge.

sebasjm

2025-04-30 18:48

developer   ~0024729

4bb0784c0..bb7a207f8

Issue History

Date ModifiedUsernameFieldChange
2025-01-27 08:19Christian GrothoffNew Issue
2025-01-27 08:19Christian GrothoffStatusnew => assigned
2025-01-27 08:19Christian GrothoffAssigned To => sebasjm
2025-01-27 08:20Christian GrothoffPriorityurgent => normal
2025-01-27 08:20Christian GrothoffTarget Version1.0 => 1.0 stretch goals
2025-04-07 16:24sebasjmAssigned Tosebasjm => Florian Dold
2025-04-07 16:24sebasjmStatusassigned => feedback
2025-04-07 16:24sebasjmNote Added: 0024430
2025-04-26 00:15Florian DoldAssigned ToFlorian Dold =>
2025-04-28 05:34Christian GrothoffNote Added: 0024694
2025-04-28 05:34Christian GrothoffStatusfeedback => new
2025-04-28 05:34Christian GrothoffAssigned To => sebasjm
2025-04-28 05:34Christian GrothoffStatusnew => assigned
2025-04-28 06:47Christian GrothoffRelationship addedrelated to 0009523
2025-04-30 17:46sebasjmNote Added: 0024719
2025-04-30 18:48sebasjmNote Added: 0024729
2025-04-30 18:48sebasjmStatusassigned => informing
2025-04-30 18:48sebasjmResolutionopen => fixed
2025-04-30 18:51Christian GrothoffFixed in Version => 1.0
2025-04-30 18:51Christian GrothoffTarget Version1.0 stretch goals => 1.0
2025-05-10 02:11Christian GrothoffStatusinforming => closed
2026-08-23 14:18Florian DoldCategorymerchant backoffice SPA => merchant portal webui