Tryton - Issues

 

Issue7375

Title Allow to limit database listed
Priority feature Status resolved
Superseder Nosy List ced, reviewbot, roundup-bot
Type feature request Components trytond
Assigned To ced Keywords review
Reviews 44991002
View: 44991002

Created on 2018-04-19.12:21:26 by ced, last changed by roundup-bot.

Messages
New changeset 30466d786155 by C├ędric Krier in branch 'default':
Add hostname configuration to list database
https://hg.tryton.org/trytond/rev/30466d786155
review44991002 updated at https://codereview.tryton.org/44991002/#ps60001
review44991002 updated at https://codereview.tryton.org/44991002/#ps40001
review44991002 updated at https://codereview.tryton.org/44991002/#ps20001
review44991002 updated at https://codereview.tryton.org/44991002/#ps1
msg40255 (view) Author: [hidden] (ced) (Tryton committer) (Tryton translator) Date: 2018-04-19.12:21:25
For now, we list all databases to which the server can connect and which looks like a Tryton database.
But on shared environment, we may want to use a single trytond instance but not list all the databases. Often the hostname is used to distinct the user prior to any authentication. So the hostname could be a parameter to list the databases.

I propose to add to ir.configuration an optional hostname. If it is filled, the database will only be listed for request matching the hostname.
I do not think we need to support many hostnames as this could be managed by using a redirect or setup a proxy which set the right HTTP_X_FORWARDED_HOST.
History
Date User Action Args
2018-05-22 23:42:40roundup-botsetstatus: testing -> resolved
nosy: + roundup-bot
messages: + msg40945
2018-05-08 14:13:38reviewbotsetmessages: + msg40573
2018-04-30 16:25:54reviewbotsetmessages: + msg40482
2018-04-30 15:58:49reviewbotsetmessages: + msg40477
2018-04-19 12:56:36reviewbotsetnosy: + reviewbot
messages: + msg40256
2018-04-19 12:36:58cedsetstatus: in-progress -> testing
reviews: 44991002
keyword: + review
2018-04-19 12:21:26cedcreate

Showing 10 items. Show all history (warning: this could be VERY long)